Issue 38: Fix failed test due to CPython str/enum behavior change #42
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes a test failure that was due to the
CyclingStatus
end status of an experiment being printed as the enum name (i.e.CyclingStatus.LOW_CAPACITY
) instead of the enum's string value (i.e.capacity is less than 1% of initial CLS capacity
).This failure is due to a behavior change between versions
3.10
and3.11
of CPython, where the__format__
behavior of a class inheriting bothstr
andEnum
was changed (from returning the enum value to returning the enum name in3.11
+). See this CPython issue for further details.This PR also updates the GitHub actions to run the tests using multiple versions of Python. Previously, only
3.10
was tested which is why this failure was not encountered on previous PRs.This addresses #38, and relates to openjournals/joss-reviews#6537.