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
Do not skip tests in CI #9598
base: main
Are you sure you want to change the base?
Do not skip tests in CI #9598
Conversation
❌ @Jarred-Sumner 5 files with test failures on linux-x64: |
❌ @Jarred-Sumner 4 files with test failures on linux-x64-baseline: |
❌ @Jarred-Sumner 7 files with test failures on bun-darwin-aarch64:
|
❌🪟 @Jarred-Sumner, there are 13 test regressions on Windows x86_64
|
❌ @Jarred-Sumner 6 files with test failures on bun-darwin-x64:
|
import { Pool, Client } from "pg"; | ||
import { parse } from "pg-connection-string"; | ||
import postgres from "postgres"; | ||
|
||
const CONNECTION_STRING = process.env.TLS_POSTGRES_DATABASE_URL; | ||
|
||
const it = CONNECTION_STRING ? test : test.skip; | ||
const it = CONNECTION_STRING ? test : test.skipIf(!isBunCI); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i agree that ci should run this, but from outside contributors this test will fail as the environment doesnt load. has this been fixed?
import { MongoClient } from "mongodb"; | ||
|
||
const CONNECTION_STRING = process.env.TLS_MONGODB_DATABASE_URL; | ||
|
||
const it = CONNECTION_STRING ? test : test.skip; | ||
const it = CONNECTION_STRING ? test : test.skipIf(!isBunCI); | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Neat way to handle it
aliases as test
.
What does this PR do?
We should not be skipping tests in CI.
How did you verify your code works?
CI