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

Add unit tests for task code in faro

    XMLWordPrintable

    Details

    • Type: Story
    • Status: In Progress
    • Resolution: Unresolved
    • Fix Version/s: None
    • Component/s: faro
    • Labels:
      None

      Description

      Currently we have unit tests for algorithmic code, but little coverage on the task code.

        Attachments

          Issue Links

          relates to

          RFC - An issue whose primary purpose is to advertise an intent and allow stakeholders to comment. RFC-793 Allow pytest-only features in Python unit tests

          • Implemented
          mentioned in

          Page Loading...

          Page Loading...

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            lguy Leanne Guy added a comment -

            Let's see what happens with RFC-793 before launching into this

            Show
            lguy Leanne Guy added a comment - Let's see what happens with RFC-793 before launching into this
            Hide
            Permalink
            Parejkoj John Parejko added a comment -

            Leanne Guy: that RFC is independent of writing some tests of faro. Even with that RFC, the tests would still be written as lsst.utils.tests.TestCase objects, which are derived from unittest.TestCase and have several useful added features. The additional pytest features would be useful, but you can start building tests before that.

            Show
            Parejkoj John Parejko added a comment - Leanne Guy : that RFC is independent of writing some tests of faro. Even with that RFC, the tests would still be written as lsst.utils.tests.TestCase objects, which are derived from unittest.TestCase and have several useful added features. The additional pytest features would be useful, but you can start building tests before that.
            Hide
            Permalink
            kbechtol Keith Bechtol added a comment - - edited

            This was partially address by addition of

            https://github.com/lsst/faro/blob/master/tests/test_tasks.py

            We haven't yet added tests for all analysis contexts.

            Show
            kbechtol Keith Bechtol added a comment - - edited This was partially address by addition of https://github.com/lsst/faro/blob/master/tests/test_tasks.py We haven't yet added tests for all analysis contexts.

              People

              Assignee:
              Unassigned Unassigned
              Reporter:
              kbechtol Keith Bechtol
              Watchers:
              John Parejko, Keith Bechtol, Leanne Guy
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:

                  Jenkins

                  No builds found.