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

db/seed: Adapt seeding scripts to new payment structure #605

Merged
merged 12 commits into from
Mar 10, 2024

Conversation

sashko9807
Copy link
Member

It appears I missed to adapt the seeding scripts to the new structure, which prevents the front-end CI/CD E2E job as it requires the seeding data.

Tests:
yarn seed - Runs successfully, with 33 payment records created, and 33 donations records created.

- Payment table will save the information regarding the payment
- Donation table will save information regarding the donation(e.g. campaign, wish etc..)
In future situations, we might need to decrement many vaults at once as well,(e.g. payment refund etc..)

- Added some unit tests to cover vaults updates
-Did some refactoring on commonly used types, as well as test mocks
As the name of the model is used as a type, the acceptednaming convention is to name that type in singular way
Copy link

github-actions bot commented Mar 10, 2024

✅ Tests will run for this PR. Once they succeed it can be merged.

@sashko9807 sashko9807 added area: seeds Prisma seed related activities run tests Allows running the tests workflows for forked repos labels Mar 10, 2024
@github-actions github-actions bot removed the run tests Allows running the tests workflows for forked repos label Mar 10, 2024
In production, the insertion is done within a single transaction, thus the dates should be the same
@sashko9807 sashko9807 added the run tests Allows running the tests workflows for forked repos label Mar 10, 2024
@github-actions github-actions bot removed the run tests Allows running the tests workflows for forked repos label Mar 10, 2024
@sashko9807 sashko9807 merged commit cf32825 into podkrepi-bg:master Mar 10, 2024
9 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: seeds Prisma seed related activities
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants