search

AFM-SPM / TopoStats

An AFM image analysis program to batch process data and obtain statistics from images
https://afm-spm.github.io/TopoStats/
GNU Lesser General Public License v3.0
60 stars 11 forks source link

Fix processing tests #939

Closed SylviaWhittle closed 1 month ago

SylviaWhittle commented 1 month ago

This PR (hopefully) fixes the tests in test_processing.py for the #932

SylviaWhittle commented 1 month ago

Ah heck, so this PR is just to address the processing tests, not the others... Any ideas on how to do a PR that fixes only some of the tests? @ns-rse

SylviaWhittle commented 1 month ago

The processing tests pass on my machine, you might want to verify though

image
ns-rse commented 1 month ago

Any ideas on how to do a PR that fixes only some of the tests? @ns-rse

The Continuous Integration tests will continue to fail until all tests pass, no way around that I can think of without switching to using pytest --testmon in CI if that is even possible as it would be problematic if say a newer package were installed in CI than on a local machine where the .testmondata.

Its useful to run the whole test suite in CI though and ensuring it runs on all pull requests was something I deliberately introduced the other day with #921 as had this been the case from the start (rather than just running on PRs that target main) we would have known about and addressed the failing tests earlier.