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.

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

Alerce database experiment #7

Merged
merged 1 commit into from
Jul 21, 2024
Merged

Alerce database experiment #7

merged 1 commit into from
Jul 21, 2024

Conversation

dirodriguezm
Copy link
Collaborator

Big refactor !!

  • Added alercedb package to simulate the ALeRCE PSQL Database (for ZTF).
  • Added tests for all output types of sync queries with each database table

Notes:

  • tests use testcontainers when running with the env variable ENV empty or ENV=DEV
  • otherwise they use an external instance that should be running when the test starts
    • this instance is a dagger service
  • two databases are created (inside the same instance) one for the pkg/alercedb package and one for the internal/tapsync package, to avoid conflicts between the two packages during tests

@dirodriguezm dirodriguezm force-pushed the alerce-database-experiment branch from a8b3421 to c17250d Compare July 21, 2024 01:32
@dirodriguezm
tests: add sync tests against alerce database
808f5aa 
test: add more queries to the csv tests for different tables

test: add tests against alerce database

add tests for alercehtml, alercetext and alercevotable
@dirodriguezm dirodriguezm force-pushed the alerce-database-experiment branch from c17250d to 808f5aa Compare July 21, 2024 01:33
@dirodriguezm dirodriguezm merged commit ded40ca into main Jul 21, 2024
1 check passed
@dirodriguezm dirodriguezm deleted the alerce-database-experiment branch July 21, 2024 01:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Sync
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@dirodriguezm