Skip to content

Commit

Permalink
Merge pull request #86 from lsst/tickets/DM-29139a
Browse files Browse the repository at this point in the history
DM-29139: Update ignore glob in pytest test collection to include *.tmp
  • Loading branch information
timj committed Mar 11, 2021
2 parents 7199c6c + 182cbb6 commit 8076a8c
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 1 deletion.
1 change: 0 additions & 1 deletion python/lsst/sconsUtils/scripts.py
Original file line number Diff line number Diff line change
Expand Up @@ -658,7 +658,6 @@ def s(ll):
state.env.Depends(pyTest, swigMods)
state.env.Depends(pyTest, state.targets["python"])
state.env.Depends(pyTest, state.targets["shebang"])
state.env.Depends(pyTest, state.targets["examples"])
result = ccList + pyList
state.targets["tests"].extend(result)
return result
Expand Down
4 changes: 4 additions & 0 deletions python/lsst/sconsUtils/tests.py
Original file line number Diff line number Diff line change
Expand Up @@ -341,6 +341,10 @@ def runPythonTests(self, pyList):
# test collection
interpreter += " --ignore=doc/html --ignore=doc/xml"

# Also include temporary files made by compilers.
# These can come from examples directories that include C++.
interpreter += " --ignore-glob='*.tmp'"

target = os.path.join(self._tmpDir, "pytest-{}.xml".format(self._env['eupsProduct']))

# Work out how many jobs scons has been configured to use
Expand Down

0 comments on commit 8076a8c

Please sign in to comment.