Skip to content
This repository

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Fix for GRAILS-9042 #185

Merged
merged 1 commit into from almost 2 years ago

2 participants

Erik Pragt graemerocher
Erik Pragt

I've created a plugin for Fitnesse, and Fitnesse can only give the number of executed tests after the tests have been run. In order to work around this, I supply 1 test to run in the doPrepare() method, to trigger the test run. However, this number is also used in the reporting, which is, in my case, incorrect.

Erik Pragt GRAILS-9042 Make _GrailsTest report the real number of tests, instead…
… of given by prepare. Fixed by recalculating the real number of tests which are executed.
835dede
graemerocher graemerocher merged commit 9c1ecb4 into from April 27, 2012
graemerocher graemerocher closed this April 27, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Showing 1 unique commit by 1 author.

Apr 20, 2012
Erik Pragt GRAILS-9042 Make _GrailsTest report the real number of tests, instead…
… of given by prepare. Fixed by recalculating the real number of tests which are executed.
835dede
This page is out of date. Refresh to see the latest.

Showing 1 changed file with 1 addition and 0 deletions. Show diff stats Hide diff stats

  1. 1  scripts/_GrailsTest.groovy
1  scripts/_GrailsTest.groovy
@@ -290,6 +290,7 @@ runTests = { GrailsTestType type, File compiledClassesDir ->
290 290
             def result = type.run(testEventPublisher)
291 291
             def end = new Date()
292 292
 
  293
+            testCount = result.passCount + result.failCount
293 294
             grailsConsole.addStatus "Completed $testCount $type.name test${testCount > 1 ? 's' : ''}, ${result.failCount} failed in ${end.time - start.time}ms"
294 295
             grailsConsole.lastMessage = ""
295 296
 
Commit_comment_tip

Tip: You can add notes to lines in a file. Hover to the left of a line to make a note

Something went wrong with that request. Please try again.