Uploaded image for project: 'Lucene - Core'
  1. Lucene - Core
  2. LUCENE-4212

Tests should not use new Random() without args

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 4.0-ALPHA, 6.0
    • general/build
    • None
    • New

    Description

      They should be using random() etc, and if they create one, it should pass in a seed.

      Otherwise, they probably won't reproduce.

      Attachments

        1. LUCENE-4212.patch
          10 kB
          Robert Muir
        2. LUCENE-4212.patch
          6 kB
          Robert Muir
        3. LUCENE-4212.patch
          4 kB
          Robert Muir
        4. LUCENE-4212.patch
          4 kB
          Robert Muir

        Issue Links

          requires

          Bug - A problem which impairs or prevents the functions of the product. LUCENE-6752 include Math.random() into forbiddenAPI

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Closed

          Activity

            People

              Unassigned Unassigned
              rcmuir Robert Muir
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: