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

Add test decorators for cartesian product

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: utils
    • Labels:
      None
    • Team:
      External
    • Urgent?:
      No

      Description

      Some time ago, I added some decorators for use in tests (DM-22199). One thing that has annoyed me is that sometimes I want the cartesian product of the parameters I supply, but the current decorators require spelling out all the combinations. I've now implemented versions that will do the product, and would like to add these as well.

        Attachments

          Activity

          Show
          tjenness Tim Jenness added a comment - https://docs.pytest.org/en/6.2.x/usage.html#dropping-to-pdb-python-debugger-on-failures
          Hide
          tjenness Tim Jenness added a comment -

          Maybe we could relax the dev guide rules a little and say that whilst we prefer unittest that there are some scenarios where it might be simpler to use pytest?

          Show
          tjenness Tim Jenness added a comment - Maybe we could relax the dev guide rules a little and say that whilst we prefer unittest that there are some scenarios where it might be simpler to use pytest?
          Hide
          price Paul Price added a comment -

          May I merge this, and update the dev guide?

          Show
          price Paul Price added a comment - May I merge this, and update the dev guide?
          Hide
          ktl Kian-Tat Lim added a comment -

          I think we'd like to start moving to pytest.  I will file an RFC to that effect.  In the meantime, I think it's best for you to merge this but not update the Dev Guide.  You can use this, but we will not recommend it for others to use pending the pytest RFC.

          Show
          ktl Kian-Tat Lim added a comment - I think we'd like to start moving to pytest.  I will file an RFC to that effect.  In the meantime, I think it's best for you to merge this but not update the Dev Guide.  You can use this, but we will not recommend it for others to use pending the pytest RFC.
          Hide
          price Paul Price added a comment -

          Merged utils only.

          Thanks, both!

          Show
          price Paul Price added a comment - Merged utils only. Thanks, both!

            People

            Assignee:
            price Paul Price
            Reporter:
            price Paul Price
            Reviewers:
            Tim Jenness
            Watchers:
            Kian-Tat Lim, Paul Price, Tim Jenness
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.