Grails
  1. Grails
  2. GRAILS-5249

Remove full stop character after path to test report

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Trivial Trivial
    • Resolution: Fixed
    • Affects Version/s: 1.2-M2
    • Fix Version/s: 1.2-RC1
    • Component/s: Testing
    • Labels:
      None

      Description

      A fullstop character is appended to the path to the test reports which makes it annoying when selecting the path to copy and paste it. In contrast ALL the other paths in the test session do not have a full stop character after them.

      So:
      a) remove it for consistency
      b) remove it to make our lives that tiny bit easier

      Thanks!

      Here's how it appears:

      ...
      Tests PASSED - view reports in /Users/dclifton/Sites/managementconsole/test/reports.
      ...
      

      Here's a full test session, for reference, note that no other path has an extra full stop after it.

      Welcome to Grails 1.2-M2 - http://grails.org/
      Licensed under Apache Standard License 2.0
      Grails home is set to: /opt/grails
      
      Base Directory: /Users/dclifton/Sites/managementconsole
      Running script /opt/grails/scripts/TestApp.groovy
      Environment set to test
        [groovyc] Compiling 2 source files to /Users/dclifton/.grails/1.2-M2/projects/managementconsole/classes
          [mkdir] Created dir: /Users/dclifton/Sites/managementconsole/test/reports/html
          [mkdir] Created dir: /Users/dclifton/Sites/managementconsole/test/reports/plain
      Instrumenting classes for coverage ...
         [delete] Deleting: /Users/dclifton/Sites/managementconsole/cobertura.ser
      [cobertura-instrument] Cobertura 1.9.3 - GNU GPL License (NO WARRANTY) - See COPYRIGHT file
      [cobertura-instrument] Instrumenting 175 files
      [cobertura-instrument] Cobertura: Saved information on 175 classes.
      [cobertura-instrument] Instrument time: 1645ms
      
      Starting unit tests ...
      Running tests of type 'unit'
          [mkdir] Created dir: /Users/dclifton/.grails/1.2-M2/projects/managementconsole/test-classes/unit
        [groovyc] Compiling 7 source files to /Users/dclifton/.grails/1.2-M2/projects/managementconsole/test-classes/unit
      -------------------------------------------------------
      Running 45 unit tests...
      Running test cellcrypt.CompanyTests...PASSED
      Running test cellcrypt.DeviceTests...PASSED
      Running test cellcrypt.LicenseTests...PASSED
      Running test cellcrypt.LicenseTypeTests...PASSED
      Running test cellcrypt.ProductTests...PASSED
      Running test cellcrypt.ProductTypeTests...PASSED
      Running test PublicControllerTests...PASSED
      Tests Completed in 4373ms ...
      -------------------------------------------------------
      Tests passed: 45
      Tests failed: 0
      -------------------------------------------------------
      
      Starting integration tests ...
           [copy] Copying 1 file to /Users/dclifton/.grails/1.2-M2/projects/managementconsole/test-classes/integration
           [copy] Copying 1 file to /Users/dclifton/.grails/1.2-M2/projects/managementconsole/test-classes
           [copy] Copied 2 empty directories to 1 empty directory under /Users/dclifton/.grails/1.2-M2/projects/managementconsole/test-classes
           [copy] Copying 1 file to /Users/dclifton/.grails/1.2-M2/projects/managementconsole/classes
      Running tests of type 'integration'
      No tests found in test/integration to execute ...
      [junitreport] Processing /Users/dclifton/Sites/managementconsole/test/reports/TESTS-TestSuites.xml to /var/folders/aD/aD5iGSbtHyOvDPpsDchvE++++TI/-Tmp-/null2100351626
      [junitreport] Loading stylesheet jar:file:/opt/grails/lib/ant-junit-1.7.1.jar!/org/apache/tools/ant/taskdefs/optional/junit/xsl/junit-frames.xsl
      [junitreport] Transform time: 1309ms
      [junitreport] Deleting: /var/folders/aD/aD5iGSbtHyOvDPpsDchvE++++TI/-Tmp-/null2100351626
      
      Tests PASSED - view reports in /Users/dclifton/Sites/managementconsole/test/reports.
      Cobertura: Loaded information on 175 classes.
      Cobertura: Saved information on 175 classes.
          [mkdir] Created dir: /Users/dclifton/Sites/managementconsole/test/reports/cobertura
      [cobertura-report] Cobertura 1.9.3 - GNU GPL License (NO WARRANTY) - See COPYRIGHT file
      [cobertura-report] Cobertura: Loaded information on 175 classes.
      [cobertura-report] Report time: 690ms
      Done with post processing reports in 1699ms
         [delete] Deleting directory /Users/dclifton/Sites/managementconsole/web-app/WEB-INF/classes
         [delete] Deleting: /Users/dclifton/.grails/1.2-M2/projects/managementconsole/resources/web.xml
         [delete] Deleting directory /Users/dclifton/.grails/1.2-M2/projects/managementconsole/classes
         [delete] Deleting directory /Users/dclifton/.grails/1.2-M2/projects/managementconsole/resources
         [delete] Deleting directory /Users/dclifton/.grails/1.2-M2/projects/managementconsole/test-classes
      Cobertura Code Coverage Complete (view reports in: /Users/dclifton/Sites/managementconsole/test/reports/cobertura)
      Cobertura: Loaded information on 175 classes.
      Cobertura: Saved information on 175 classes.
      

        Activity

        Show
        Luke Daley added a comment - http://github.com/grails/grails/commit/43f7294653e5ecee835a8e309b4af1fcb95aa55c

          People

          • Assignee:
            Luke Daley
            Reporter:
            Dominic Clifton
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development