Uploaded image for project: 'Yetus'
  1. Yetus
  2. YETUS-693

Add warning about requirements for XML test-patch test

    Details

    • Type: Improvement
    • Status: Open
    • Priority: Minor
    • Resolution: Unresolved
    • Affects Version/s: 0.6.0, 0.7.0, 0.8.0
    • Fix Version/s: None
    • Component/s: Precommit
    • Labels:

      Description

      When running Test-Patch with some JVM's (currently seen in Azul JDK7 over in HBase) the "xml" test always votes -1

      ...
      
      15:51:42 ============================================================================
      15:51:42 ============================================================================
      15:51:42                           XML verification: patch
      15:51:42 ============================================================================
      15:51:42 ============================================================================
      15:51:42 
      15:51:42 
      15:51:42 
      15:51:42 
      15:51:42 ============================================================================
      ...
      17:24:45 |  -1  |           xml  |   0m  0s   | The patch has 1 ill-formed XML file(s). 
      ...
      17:24:45               Reason | Tests
      17:24:45                 XML  |  Parsing Error(s): 
      17:24:45                      |  pom.xml 
      ...
      

      The actual output indicates the problem is the JDK we have doesn't support javascript scripting:

      pom.xml:
      
      script engine for language js can not be found
      

      Maybe we could do a quick known-pass and known-fail check in setup for the plugin?

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                Unassigned
                Reporter:
                busbey Sean Busbey
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated: