BlogDocs

test(db pull): remove duplicated tests(#19222)

Merged
Merging
joel/duplicated-tests
(
9089c16
) into
main
(
589e093
)
0%
IMPROVEMENTS
0
REGRESSIONS
0
UNTOUCHED
4
NEW
0
DROPPED
0
IGNORED
0

Benchmarks

client generation ~50 Models
packages/client/src/__tests__/benchmarks/huge-schema/huge-schema.bench.ts::typescript::client generation ~50 Models
+1%
1.8 s
1.8 s
typescript compilation ~50 Models
packages/client/src/__tests__/benchmarks/huge-schema/huge-schema.bench.ts::typescript::typescript compilation ~50 Models
0%
20.1 ms
20 ms
client generation 100 models with relations
packages/client/src/__tests__/benchmarks/lots-of-relations/lots-of-relations.bench.ts::typescript::client generation 100 models with relations
-1%
10.7 s
10.8 s
getPlatform
packages/get-platform/bench/get-platform.bench.ts::get-platform::getPlatform
+1%
13.9 ms
13.7 ms

Commits

Click on a commit to change the comparison range
base
main
589e093
-67%
test(db pull): remove duplicated tests Note that tests are much faster since the split, local run: ``` Test Suites: 10 passed, 10 total Tests: 1 skipped, 82 passed, 83 total Snapshots: 470 passed, 470 total Time: 26.46 s ``` Since https://github.com/prisma/prisma/issues/17582 we do not need to test the rendering of the introspection warnings in the CLI, since they are already tested in prisma-engines. We only need an integration test. Removed tests were added in prisma-engines from the following PRs: https://github.com/prisma/prisma-engines/pull/3622 https://github.com/prisma/prisma-engines/pull/3657 https://github.com/prisma/prisma-engines/pull/3849 https://github.com/prisma/prisma-engines/pull/3859 https://github.com/prisma/prisma-engines/pull/3871 https://github.com/prisma/prisma-engines/pull/3877 https://github.com/prisma/prisma-engines/pull/3873
2170b52
1 year ago
by Jolg42
+67%
fix lint
89335a2
1 year ago
by Jolg42
0%
remove 1 more test setup
9089c16
1 year ago
by Jolg42
ResourcesHomePricingDocsBlogGitHub
Copyright © 2024 CodSpeed Technology SAS. All rights reserved.