Uploaded image for project: 'Data Management'
  1. Data Management
  2. DM-7379

random seed not run in command line test

    Details

    • Type: Bug
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: ip_diffim
    • Labels:
      None

      Description

      In the conversion of ip_diffim to pytest a random.seed was moved to inside the setup_module function as opposed to the seUp method. When run from the command line (for example using Jenkins) this test can fail.

        Attachments

          Issue Links

            Activity

            Hide
            fred3m Fred Moolekamp added a comment -

            Changes made and tested on Jenkins.

            Show
            fred3m Fred Moolekamp added a comment - Changes made and tested on Jenkins.
            Hide
            vpk24 Vishal Kasliwal [X] (Inactive) added a comment -

            Looks fine to me... I presume that it passes with $py.test tests/*.py ?

            Show
            vpk24 Vishal Kasliwal [X] (Inactive) added a comment - Looks fine to me... I presume that it passes with $py.test tests/*.py ?
            Hide
            vpk24 Vishal Kasliwal [X] (Inactive) added a comment -

            Looks fine as long as it has been tested with $py.test tests/*.py

            Show
            vpk24 Vishal Kasliwal [X] (Inactive) added a comment - Looks fine as long as it has been tested with $py.test tests/*.py
            Hide
            fred3m Fred Moolekamp added a comment -

            Yup, it has. Thanks Vishal.

            Show
            fred3m Fred Moolekamp added a comment - Yup, it has. Thanks Vishal.

              People

              • Assignee:
                fred3m Fred Moolekamp
                Reporter:
                fred3m Fred Moolekamp
                Reviewers:
                Vishal Kasliwal [X] (Inactive)
                Watchers:
                Fred Moolekamp, Vishal Kasliwal [X] (Inactive)
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel