Skip to content

Commit

Permalink
tests: Update run_test.py to fix coverage
Browse files Browse the repository at this point in the history
  • Loading branch information
Fabian Neuschmidt committed Feb 6, 2015
1 parent 6999876 commit e0fdc2e
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion run_tests.py
View file
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@

ignore_list = [
os.path.join(tempfile.gettempdir(), "**"),
os.path.join(get_python_lib(), "**"),
os.path.join(os.path.dirname(get_python_lib()), "**"),

This comment has been minimized.

Sign in to view

Copy link
@sils

sils Feb 6, 2015

Member

joining with ".." is simpler for me. Any reason for using dirname? Makes not really sense contextually.
os.path.join("coalib", "tests", "**"),
os.path.join("bears", "tests", "**")
]
Expand Down

3 comments on commit e0fdc2e

@sils
Copy link
Member

@sils sils commented on e0fdc2e Feb 6, 2015

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

proposal for commit message:
run_tests: Ignore parent dir of stdlib

@fneu
Copy link
Contributor

@fneu fneu commented on e0fdc2e Feb 6, 2015

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i would still like to know why there is something in that directory. It should be empty resides the one folder really. But it is not only for this commit.

@sils
Copy link
Member

@sils sils commented on e0fdc2e Feb 6, 2015 via email

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.