Lucene - Core
  1. Lucene - Core
  2. LUCENE-3988

improve test output to be nicer to 80chars long terminals

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 4.0-ALPHA
    • Component/s: general/test
    • Labels:
      None
    • Lucene Fields:
      New

      Description

      these lines tend to always use 82 chars:

      [junit4] Tests run:   4, Failures:   0, Errors:   0, Skipped:   0, Time:  3.97s
      

      Can we remove some of the spaces so it fits? Maybe remove the word 'run' from "Tests run".

      occasionally (not always) long classnames wrap too 'Running org.apache.lucene.this.that.TestFoo' ... maybe
      just print the short classname?

        Issue Links

          Activity

          Uwe Schindler made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          Hide
          Dawid Weiss added a comment -

          So change it like I suggested – I can't please everybody. If it bothers you, change it:

                              useSimpleNames="false"
                              maxClassNameColumns="1000000" 
          

          or remove maxClassNameColumns entirely.

          Show
          Dawid Weiss added a comment - So change it like I suggested – I can't please everybody. If it bothers you, change it: useSimpleNames="false" maxClassNameColumns="1000000" or remove maxClassNameColumns entirely.
          Hide
          Uwe Schindler added a comment -

          I would like the have the short statistics, but still the full classname (no ellipsis) to make copy-paste work.

          Show
          Uwe Schindler added a comment - I would like the have the short statistics, but still the full classname (no ellipsis) to make copy-paste work.
          Dawid Weiss made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Fix Version/s 4.0 [ 12314025 ]
          Resolution Fixed [ 1 ]
          Hide
          Dawid Weiss added a comment -

          Will be fixed by the next release of randomizedtesting.

          Show
          Dawid Weiss added a comment - Will be fixed by the next release of randomizedtesting.
          Dawid Weiss made changes -
          Field Original Value New Value
          Link This issue is part of LUCENE-3808 [ LUCENE-3808 ]
          Hide
          Dawid Weiss added a comment - - edited

          I thought about this a bit. The previous output was a mirror of surefire. After some deliberation I don't think it makes sense to present the information so verbosely (0 errors, 0 failures, etc.). How about this:

             [junit4] Suite: TestReversedWildcardFilterFactory
             [junit4] Time:  3.00s, 4 tests
             [junit4]  
             [junit4] Suite: [...]r.update.processor.UniqFieldsUpdateProcessorFactoryTest
             [junit4] Time:  3.00s, 4 tests, 1 skipped
             [junit4]  
             [junit4] Suite: org.apache.solr.spelling.SpellPossibilityIteratorTest
             [junit4] Time:  3.00s, 4 tests, 1 error  <<< FAILURES!
             [junit4]  
             [junit4] Suite: org.buhu.update.processor.BlahBlag
             [junit4] Time:  3.00s, 4 tests, 1 error, 2 failures, 1 skipped
          

          Test name will be displayed in full or truncated (with an ellipsis) to fit into the desired number of columns (80 by default)?

          Show
          Dawid Weiss added a comment - - edited I thought about this a bit. The previous output was a mirror of surefire. After some deliberation I don't think it makes sense to present the information so verbosely (0 errors, 0 failures, etc.). How about this: [junit4] Suite: TestReversedWildcardFilterFactory [junit4] Time: 3.00s, 4 tests [junit4] [junit4] Suite: [...]r.update.processor.UniqFieldsUpdateProcessorFactoryTest [junit4] Time: 3.00s, 4 tests, 1 skipped [junit4] [junit4] Suite: org.apache.solr.spelling.SpellPossibilityIteratorTest [junit4] Time: 3.00s, 4 tests, 1 error <<< FAILURES! [junit4] [junit4] Suite: org.buhu.update.processor.BlahBlag [junit4] Time: 3.00s, 4 tests, 1 error, 2 failures, 1 skipped Test name will be displayed in full or truncated (with an ellipsis) to fit into the desired number of columns (80 by default)?
          Robert Muir created issue -

            People

            • Assignee:
              Unassigned
              Reporter:
              Robert Muir
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development