completed #66 - now all log messages should have appropriate level

aim42 · Dec 26, 2014 · e994d62 · e994d62
1 parent 4a797d9
commit e994d62
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 7 deletions.
diff --git a/core/src/main/groovy/org/aim42/htmlsanitycheck/AllChecksRunner.groovy b/core/src/main/groovy/org/aim42/htmlsanitycheck/AllChecksRunner.groovy
@@ -72,7 +72,7 @@ class AllChecksRunner {
         this.checkingResultsDir = checkingResultsDir
         this.checkExternalResources = checkExternalResources
 
-        logger.info("AlLChecksRunner created")
+        logger.debug("AlLChecksRunner created")
     }
 
     /**
@@ -96,7 +96,7 @@ class AllChecksRunner {
      */
     public PerRunResults performAllChecks() {
 
-        logger.info "entered performAllChecks"
+        logger.debug "entered performAllChecks"
 
         filesToCheck.each { file ->
 
@@ -177,7 +177,6 @@ class AllChecksRunner {
         )
 
         return missingImagesChecker.performCheck()
-        //logger.info imageCheckingResults.toString()
     }
 
     /**

diff --git a/core/src/main/groovy/org/aim42/htmlsanitycheck/check/MissingImageFilesChecker.groovy b/core/src/main/groovy/org/aim42/htmlsanitycheck/check/MissingImageFilesChecker.groovy
@@ -34,6 +34,8 @@ class MissingImageFilesChecker extends Checker {
         //get list of all image-tags "<img..." in html file
         images = pageToCheck.getAllImageTags()
 
+        logger.debug("images to check: ${images}")
+
         checkAllImages()
 
         return checkingResults
@@ -51,9 +53,6 @@ class MissingImageFilesChecker extends Checker {
     private void checkSingleLocalImage(HtmlElement image) {
         String relativePathToCurrentImage = image.getImageSrcAttribute()
 
-        //logger.info( "image: " + image)
-        //logger.info( "relPathToCurImage: \n$relativePathToCurrentImage")
-
         // check only "local" image references
         // (that is, NO remote URL)
         if (!URLUtil.isRemoteURL(relativePathToCurrentImage)) {

diff --git a/core/src/test/groovy/org/aim42/htmlsanitycheck/check/MissingImageFilesCheckerTest.groovy b/core/src/test/groovy/org/aim42/htmlsanitycheck/check/MissingImageFilesCheckerTest.groovy
@@ -39,7 +39,7 @@ class MissingImageFilesCheckerTest extends GroovyTestCase {
 
         checkingResults = new SingleCheckResults()
 
-        logger.info( "imagesDir: $imagesDir")
+        logger.debug( "imagesDir: $imagesDir")
     }
-Original file line number
+Diff line change
@@ Expand Up / @@ -39,7 +39,7 @@ class MissingImageFilesCheckerTest extends GroovyTestCase { @@
             checkingResults = new SingleCheckResults()
-            logger.info( "imagesDir: $imagesDir")
+            logger.debug( "imagesDir: $imagesDir")
         }
@@ Expand Down @@