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

🧑‍💻 (org): Include test data for org service #166

Merged
merged 11 commits into from
Dec 16, 2024

Conversation

byte-sized-emi
Copy link
Contributor

@byte-sized-emi byte-sized-emi commented Nov 28, 2024

resolves #141

blocked by #145 / #151

Still need to make a plan/docs for how to include test data in prod

@byte-sized-emi byte-sized-emi self-assigned this Nov 28, 2024
@byte-sized-emi byte-sized-emi force-pushed the org-service-test-data-141 branch 2 times, most recently from 5593741 to 0af0883 Compare December 8, 2024 11:15
@byte-sized-emi byte-sized-emi marked this pull request as ready for review December 8, 2024 11:15
@byte-sized-emi
Copy link
Contributor Author

Test data is now available in DEV as well as TEST modes, using flyway migrations. Tests that want to use this test data can inject the Flyway object and re-run the migrations manually.

@d135-1r43
Copy link
Collaborator

Kannst du jeweils ein Beispiel machen für:

  • ein Test, der auf einer Datenbank ohne Daten arbeitet
  • ein Test, der auf den Testdaten arbeitet

@byte-sized-emi
Copy link
Contributor Author

Test der mit Testdaten arbeitet: BommelTest.simpleGetParentsTest()
Test der mit leerer Datenbank arbeitet: BommelTest.getParentsWithCycle()

@byte-sized-emi byte-sized-emi force-pushed the org-service-test-data-141 branch from 788d0a1 to e9e0b2e Compare December 12, 2024 09:41
@byte-sized-emi
Copy link
Contributor Author

Oh, und, weil du Flyway#clean und Flyway#migrate bei den BommelTests' @beforeeach hinzugefügt hast: Ich würde das nicht machen. Die Tests failen, weil sie eine leere Datenbank erwarten, und ich finde es sehr unnötig nur weil wir jetzt Testdaten haben jeden Test auf diese umzuschreiben. Momentan gibt es nur einen einzigen Test, der diese Testdaten tatsächlich nutzt, und dort kann man ohne Probleme die flyway migrations ausführen lassen und die Testdaten nutzen. Daher ist eine leere Datenbank der von mir aus sinnvollere Default.

@d135-1r43
Copy link
Collaborator

Oh, und, weil du Flyway#clean und Flyway#migrate bei den BommelTests' @beforeeach hinzugefügt hast: Ich würde das nicht machen. Die Tests failen, weil sie eine leere Datenbank erwarten, und ich finde es sehr unnötig nur weil wir jetzt Testdaten haben jeden Test auf diese umzuschreiben. Momentan gibt es nur einen einzigen Test, der diese Testdaten tatsächlich nutzt, und dort kann man ohne Probleme die flyway migrations ausführen lassen und die Testdaten nutzen. Daher ist eine leere Datenbank der von mir aus sinnvollere Default.

@byte-sized-emi Passt, kannst du gern wieder ändern und verbessern.

@byte-sized-emi byte-sized-emi changed the title WIP: 🧑‍💻 (org): Include test data for org service 🧑‍💻 (org): Include test data for org service Dec 16, 2024
@d135-1r43 d135-1r43 merged commit c042f53 into main Dec 16, 2024
9 of 10 checks passed
@d135-1r43 d135-1r43 deleted the org-service-test-data-141 branch December 16, 2024 20:05
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.

Testdaten für Org Service
2 participants