Add cross-referencing comments r.e. stripping ignores

stkent · stkent · commit b237ad32f184 · 2017-06-12T14:00:36.000-04:00
diff --git a/bin/journey-test.sh b/bin/journey-test.sh
@@ -236,6 +236,9 @@ solve_all_exercises() {
     "$EXECPATH"/gradlew compileTestJava
     # Ensure we run all the tests (as delivered, all but the first is @Ignore'd)
     for testfile in `find . -name "*Test.${TRACK_SRC_EXT}"`; do
+      # Strip @Ignore annotations to ensure we run the tests (as delivered, all but the first is @Ignore'd).
+      # Note that unit-test.sh also strips @Ignore annotations via the Gradle task copyTestsFilteringIgnores.
+      # The stripping implementations here and in copyTestsFilteringIgnores should be kept consistent.
       sed 's/@Ignore\(\(.*\)\)\{0,1\}//' ${testfile} > "${tempfile}" && mv "${tempfile}" "${testfile}"
     done
     "$EXECPATH"/gradlew test
diff --git a/exercises/build.gradle b/exercises/build.gradle
@@ -29,6 +29,9 @@ subprojects { project ->
     }
   }
 
+  // Note that journey-test.sh also strips @Ignore annotations using sed. The
+  // stripping implementations here and in journey-test.sh should be kept
+  // consistent.
   task copyTestsFilteringIgnores(type: Copy) {
     from "src/test/java"
     into "build/gen/test/java"

Original file line number	Diff line number	Diff line change
`@@ -29,6 +29,9 @@ subprojects { project ->`
`29`	`29`	`}`
`30`	`30`	`}`
`31`	`31`
	`32`	`+ // Note that journey-test.sh also strips @Ignore annotations using sed. The`
	`33`	`+ // stripping implementations here and in journey-test.sh should be kept`
	`34`	`+ // consistent.`
`32`	`35`	`task copyTestsFilteringIgnores(type: Copy) {`
`33`	`36`	`from "src/test/java"`
`34`	`37`	`into "build/gen/test/java"`