Skip to content
This repository has been archived by the owner on Nov 3, 2023. It is now read-only.

[tests] Kill test_eval_fixed #3597

Merged
merged 2 commits into from
Apr 19, 2021
Merged

[tests] Kill test_eval_fixed #3597

merged 2 commits into from
Apr 19, 2021

Conversation

stephenroller
Copy link
Contributor

Patch description
This test sucks. It's our most common failure. Kill it with fire, it's not even worth saving.

Testing steps

$ pytest --seconds 600 tests/test_tra.py
418 passed, 133 skipped, 912 warnings in 636.21s (0:10:36)

@stephenroller stephenroller changed the title Kill test_eval_fixed [tests] Kill test_eval_fixed Apr 18, 2021
Copy link
Contributor

@EricMichaelSmith EricMichaelSmith left a comment

Choose a reason for hiding this comment

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

Yeah makes sense

@stephenroller stephenroller merged commit 2426d74 into master Apr 19, 2021
@stephenroller stephenroller deleted the test_eval_fixed branch April 19, 2021 22:49
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants