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

Remove build system dependencies from tests


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


      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.


          Issue Links



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


                • Created:
                  Planned End:

                  Summary Panel