Uploaded image for project: 'Request For Comments'
  1. Request For Comments
  2. RFC-44

Remove build system dependencies from tests

    Details

    • Type: RFC
    • Status: Implemented
    • Resolution: Done
    • Component/s: DM
    • Labels:
      None
    • Location:
      this ticket

      Description

      The discussion in DM-2527 demonstrated that test code uses EUPS to determine where things are located. This has two problems:

      1. Calling EUPS in a loop resulted in slow downs of the tests as the EUPS database was repeatedly read.
      2. This makes it impossible to run the tests standalone and burns-in a particular packaging implementation that is not relevant for the tests.

      This RFC proposes that EUPS be removed from tests and an alternative scheme be used for locating supporting infrastructure. Some tests still need to locate other packages (in particular afwdata) so any replacement scheme must support some form of package location.

      Some options for locating afwdata:

      1. Provide a test utility function for locating a package which can use an environment variable to find afwdata.
      2. Add afwdata as a git submodule in the test directory. This could result in multiple checkouts of the repository.

      This RFC invites further discussion on whether EUPS should be removed from the tests. Comments on replacement functionality are welcomed to demonstrate feasibility.

        Attachments

          Container Issues

            Issue Links

              Activity

                People

                • Assignee:
                  tjenness Tim Jenness
                  Reporter:
                  tjenness Tim Jenness
                  Watchers:
                  Frossie Economou, Jim Bosch, John Swinbank, Mario Juric, Robert Lupton, Russell Owen, Tim Jenness
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  7 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Planned End:

                    Summary Panel