Skip to content
This repository has been archived by the owner on Apr 10, 2024. It is now read-only.

tests: save to temporary directory #115

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ckamm
Copy link

@ckamm ckamm commented Oct 4, 2018

There were two tests that wrote to the same file in tests/fixtures
which lead to the annoyance of manually having to revert the file
to avoid spurious commit changes. Probably the order of test
execution isn't stable.

With this change, the pytest 'tmpdir' fixture is used to provide a
unique temporary directory for each test.

There were two tests that wrote to the same file in tests/fixtures
which lead to the annoyance of manually having to revert the file
to avoid spurious commit changes. Probably the order of test
execution isn't stable.

With this change, the pytest 'tmpdir' fixture is used to provide a
unique temporary directory for each test.
@coveralls
Copy link

Pull Request Test Coverage Report for Build 360

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 57.994%

Totals Coverage Status
Change from base Build 349: 0.0%
Covered Lines: 1440
Relevant Lines: 2483

💛 - Coveralls

@ludwigschubert ludwigschubert self-assigned this Mar 12, 2021
@ludwigschubert ludwigschubert marked this pull request as draft March 12, 2021 15:59
@ludwigschubert ludwigschubert added bug Something isn't working lucid.misc labels Mar 12, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working lucid.misc
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants