Skip to content

Fix flaky spec on proposal filters#14474

Merged
alecslupu merged 2 commits intodevelopfrom
chore/reset-cabybara-session
Apr 10, 2025
Merged

Fix flaky spec on proposal filters#14474
alecslupu merged 2 commits intodevelopfrom
chore/reset-cabybara-session

Conversation

@alecslupu
Copy link
Copy Markdown
Contributor

@alecslupu alecslupu commented Apr 4, 2025

🎩 What? Why?

This PR fixes a flaky spec on proposal filters.

Testing

Describe the best way to test or validate your PR.

for i in $(seq 1 100); do echo $i ;  bin/rspec decidim-proposals/spec/system/filter_proposals_spec.rb:466 || break ; done

♥️ Thank you!

@github-actions github-actions bot added module: dev type: internal PRs that aren't necessary to add to the CHANGELOG for implementers labels Apr 4, 2025
github-actions[bot]
github-actions bot previously approved these changes Apr 4, 2025
github-actions[bot]
github-actions bot previously approved these changes Apr 4, 2025
github-actions[bot]
github-actions bot previously approved these changes Apr 4, 2025
github-actions[bot]
github-actions bot previously approved these changes Apr 4, 2025
github-actions[bot]
github-actions bot previously approved these changes Apr 4, 2025
github-actions[bot]
github-actions bot previously approved these changes Apr 4, 2025
github-actions[bot]
github-actions bot previously approved these changes Apr 4, 2025
github-actions[bot]
github-actions bot previously approved these changes Apr 4, 2025
github-actions[bot]
github-actions bot previously approved these changes Apr 4, 2025
github-actions[bot]
github-actions bot previously approved these changes Apr 4, 2025
github-actions[bot]
github-actions bot previously approved these changes Apr 4, 2025
github-actions[bot]
github-actions bot previously approved these changes Apr 4, 2025
github-actions[bot]
github-actions bot previously approved these changes Apr 4, 2025
github-actions[bot]
github-actions bot previously approved these changes Apr 5, 2025
github-actions[bot]
github-actions bot previously approved these changes Apr 5, 2025
github-actions[bot]
github-actions bot previously approved these changes Apr 5, 2025
github-actions[bot]
github-actions bot previously approved these changes Apr 5, 2025
github-actions[bot]
github-actions bot previously approved these changes Apr 5, 2025
github-actions[bot]
github-actions bot previously approved these changes Apr 5, 2025
github-actions[bot]
github-actions bot previously approved these changes Apr 5, 2025
@alecslupu alecslupu changed the title Reset Capybara session if a spec fails Fix flaky spec on proposal filters Apr 9, 2025
@alecslupu alecslupu marked this pull request as ready for review April 9, 2025 08:43
Copy link
Copy Markdown
Member

@andreslucena andreslucena left a comment

Choose a reason for hiding this comment

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

I can confirm that this is working locally with the provided script for reproducing it.

My intuition is that this failure is related to the complexity of this particular context. Is the one with more code (185) and has lots of before blocks. As I don't time to provide an alternative for now, this is good enough for fixing the failure that we were seeing lately.

@alecslupu
Copy link
Copy Markdown
Contributor Author

merging with codecov failing pipeline.
merging based on @andreslucena 's approval.

@alecslupu alecslupu merged commit 51d803f into develop Apr 10, 2025
57 of 58 checks passed
@alecslupu alecslupu deleted the chore/reset-cabybara-session branch April 10, 2025 13:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

flaky spec module: proposals type: internal PRs that aren't necessary to add to the CHANGELOG for implementers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants