Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MIJN-9980-BUG/geblokkeerde-passen-worden-eruit-gefiltert #1724

Open
wants to merge 12 commits into
base: main
Choose a base branch
from

Conversation

RoanPaulus
Copy link
Contributor

@RoanPaulus RoanPaulus commented Jan 23, 2025

  • Adjusted filter to only filter out replaced passes or expired passes.
  • Refactored tests so these are easier to manage. Now cleans up after every test and creating passes are more flexible/readable.
  • Changed a happy path test into testing the filter, effectively testing both things at once.

@RoanPaulus RoanPaulus changed the title MIJN-9980/BUG-geblokkeerde-passen-worden-eruit-gefiltert MIJN-9980-BUG/geblokkeerde-passen-worden-eruit-gefiltert Jan 24, 2025
@RoanPaulus RoanPaulus force-pushed the MIJN-9980/BUG-geblokkeerde-passen-worden-eruit-gefiltert branch from b9373e8 to 5bacb09 Compare January 24, 2025 12:38
@RoanPaulus RoanPaulus marked this pull request as ready for review January 24, 2025 13:37
@RoanPaulus RoanPaulus marked this pull request as draft January 24, 2025 16:56
@RoanPaulus RoanPaulus marked this pull request as ready for review January 27, 2025 12:16
@RoanPaulus RoanPaulus marked this pull request as draft January 27, 2025 12:16
@RoanPaulus RoanPaulus marked this pull request as ready for review January 27, 2025 12:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants