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

Update test data provider #768

Merged
merged 5 commits into from
Oct 13, 2024
Merged

Update test data provider #768

merged 5 commits into from
Oct 13, 2024

Conversation

PabloDinella
Copy link
Collaborator

I enriched TestDataProvider with some data, such as some assignments to 1 of the Friending arrangements, to ease the testing during development and have more examples.

I also added a one time recurrence monitoring policy called One Time Checkin, to the Friending policy, but there might be better real life example.

It's WIP still because I need to update the unit tests, might need some help to understand it first.

@PabloDinella PabloDinella force-pushed the update-test-data-provider branch 2 times, most recently from 042be7a to 96d5380 Compare September 2, 2024 20:29
@PabloDinella PabloDinella force-pushed the update-test-data-provider branch from 96d5380 to 7bb531b Compare September 12, 2024 17:51
@PabloDinella PabloDinella force-pushed the update-test-data-provider branch from 628dc7a to 8391fda Compare September 13, 2024 23:54
@PabloDinella PabloDinella force-pushed the update-test-data-provider branch from 8391fda to c7376bb Compare September 16, 2024 17:08
@PabloDinella PabloDinella marked this pull request as ready for review September 16, 2024 17:10
@PabloDinella
Copy link
Collaborator Author

I updated the branch according our last conversations. Created new families and volunteer, so existing authorization tests doesn't break. Also used variable names in a more intuitive way, like fisherFamilyGuid instead guid8, for the new family.

Copy link
Member

@LarsKemmann LarsKemmann left a comment

Choose a reason for hiding this comment

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

10/10. Humor quality meets or exceeds expectations. 😁

@LarsKemmann LarsKemmann merged commit c272ffe into main Oct 13, 2024
1 check passed
@LarsKemmann LarsKemmann deleted the update-test-data-provider branch October 13, 2024 23:52
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.

2 participants