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

refactor(server): split api and jobs into separate e2e suites #6307

Merged
merged 2 commits into from
Jan 10, 2024

Conversation

jrasm91
Copy link
Contributor

@jrasm91 jrasm91 commented Jan 10, 2024

In this PR:

  • Split api and jobs e2e tests into two separate suites
  • Add back fsync=off (refactor(server): e2e tests #4536)
  • Correctly teardown the app in metadata e2e so the test exists promptly

Next steps:

  • Move all api based testing to e2e/api
  • Add more microservices tests, including exiftool
  • Clean up how the CLI e2e tests integrate with the code inside of server/
  • Clean up (separate) e2e and unit fixtures
  • Use the production build on immich-microservices for format testing

Copy link

cloudflare-pages bot commented Jan 10, 2024

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 5102273
Status: ✅  Deploy successful!
Preview URL: https://6d4e6885.immich.pages.dev
Branch Preview URL: https://refactor-e2e-2.immich.pages.dev

View logs

@jrasm91 jrasm91 force-pushed the refactor/e2e-2 branch 3 times, most recently from 75294c7 to 403a88f Compare January 10, 2024 03:25
@@ -33,8 +32,7 @@ describe(`${AssetController.name} (e2e)`, () => {
});

afterAll(async () => {
await db.disconnect();
await app.close();
await testApp.teardown();
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is what was causing the process to hang (not cleaning up exiftool-vendored)

@jrasm91 jrasm91 enabled auto-merge (squash) January 10, 2024 04:02
@jrasm91 jrasm91 merged commit bf1dd36 into main Jan 10, 2024
21 checks passed
@jrasm91 jrasm91 deleted the refactor/e2e-2 branch January 10, 2024 04:04
Copy link
Member

@danieldietzler danieldietzler left a comment

Choose a reason for hiding this comment

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

🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants