Skip to content

clean up pathfora tests #665

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.

Sign up for GitHub

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

Jump to bottom
Merged
ashyablok merged 4 commits into from
Dec 2, 2025
Merged

clean up pathfora tests #665

ashyablok merged 4 commits into from
Dec 2, 2025

Conversation

@ashyablok
Copy link
Contributor

@ashyablok ashyablok commented Nov 21, 2025 •
edited
Loading

Fix up pathfora tests:

  • Break up into multiple files and use more specific decscribes
  • utilize new helper functions
  • remove unused tests

@ashyablok ashyablok marked this pull request as ready for review November 24, 2025 21:44
@ashyablok ashyablok requested review from a team, humanchimp, markhayden and teijas and removed request for a team November 24, 2025 21:58
@ashyablok ashyablok merged commit fa365ab into develop Dec 2, 2025
1 check passed
@ashyablok ashyablok deleted the refactor-tests branch December 2, 2025 19:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@markhayden markhayden markhayden approved these changes

@prosdev prosdev prosdev approved these changes

@humanchimp humanchimp Awaiting requested review from humanchimp humanchimp was automatically assigned from lytics/frontend-engineering

@teijas teijas Awaiting requested review from teijas teijas was automatically assigned from lytics/frontend-engineering

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ashyablok @markhayden @prosdev