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

Repro of Sentry failure with vitest #222

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

Conversation

dcramer
Copy link
Owner

@dcramer dcramer commented Sep 30, 2024

TypeError: eq is not a function
 ❯ getTableNames src/test/setup-test-env.ts:63:12
     61|     .select({ tablename: pgTables.tablename })
     62|     .from(pgTables)
     63|     .where(eq(pgTables.schemaname, schemaname));
       |            ^
     64|
     65|   return tnQuery
 ❯ clearTables src/test/setup-test-env.ts:83:28
 ❯ src/test/setup-test-env.ts:158:9

Disables Sentry in the test env as its causing issues (likely due to ESM) where dependencies report as undefined after import.

This also adjusts our server package to be ESM.
@dcramer dcramer changed the base branch from main to feat/upgrade-vitest September 30, 2024 21:02
Copy link

codecov bot commented Sep 30, 2024

❌ We are unable to process any of the uploaded JUnit XML files. Please ensure your files are in the right format.

Base automatically changed from feat/upgrade-vitest to main September 30, 2024 22:07
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.

1 participant