Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Clean up tests #2495

Merged
merged 4 commits into from
Apr 19, 2022
Merged

Clean up tests #2495

merged 4 commits into from
Apr 19, 2022

Commits on Apr 16, 2022

  1. clean up PathFinderTest.test_image_num()

    @janosh
    janosh committed Apr 16, 2022
    Configuration menu
    Copy the full SHA
    d40d342 View commit details
    Browse the repository at this point in the history

  2. fix E722 do not use bare except

    @janosh
    janosh committed Apr 16, 2022
    Configuration menu
    Copy the full SHA
    fe7d3e9 View commit details
    Browse the repository at this point in the history

  3. fix dulong-petit test case in ElasticTensorExpansionTest.test_thermal… 

    …_expansion_coeff()

plus lots of  F841 local variable 'foo' is assigned to but never used
    @janosh
    janosh committed Apr 16, 2022
    Configuration menu
    Copy the full SHA
    74f7eee View commit details
    Browse the repository at this point in the history

  4. add missing test case in IcohplistTest.test_values() (thanks @JaGeo) 

    fixes flake8 F841 unused variable
    @janosh
    janosh committed Apr 16, 2022
    Configuration menu
    Copy the full SHA
    576ea88 View commit details
    Browse the repository at this point in the history