Skip to content

Friday test hacking/fix file handles error#7887

Merged
jmax-gsa merged 3 commits intomainfrom
friday-test-hacking/fix_file_handles_error
Mar 3, 2023
Merged

Friday test hacking/fix file handles error#7887
jmax-gsa merged 3 commits intomainfrom
friday-test-hacking/fix_file_handles_error

Conversation

@jmax-gsa
Copy link
Contributor

No description provided.

Why: spec/features/idv/hybrid_flow_test.rb was failing locally with "too many file handles" error.
With some experimenting and web searching we found this possible solution and it made the test
succeed five times in a row. (We hope it's fixed.)

Co-authored-by: John Maxwell <john.maxwell@gsa.gov>
Co-authored-by: Douglas Price <douglas.price@gsa.gov>
Co-authored-by: John Skinner <john.skinner@gsa.gov>
Co-authored-by: Alexander Bradley <alexander.bradley@gsa.gov>

[skip changelog]
…ng anything

We theorize that the setting is reset to false on a per-class basis. It persists across tests in this file.
Copy link
Contributor

@soniaconnolly soniaconnolly left a comment

Choose a reason for hiding this comment

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

This consistently fixes the problem on my machine. Let's do it!

@jmax-gsa jmax-gsa marked this pull request as ready for review March 3, 2023 19:46
@jmax-gsa jmax-gsa merged commit 6183c2e into main Mar 3, 2023
@jmax-gsa jmax-gsa deleted the friday-test-hacking/fix_file_handles_error branch March 3, 2023 19:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants