joel/duplicated-tests - Branch - prisma/prisma

prisma

Blog Docs Changelog

test(db pull): remove duplicated tests

#19222Merged

Comparing

joel/duplicated-tests

(

9089c16

) with

main

(

589e093

)

Untouched: 4

Benchmarks

Passed

getPlatform

packages/get-platform/bench/get-platform.bench.ts::get-platform::getPlatform

+1%

13.9 ms13.7 ms

client generation ~50 Models

packages/client/src/__tests__/benchmarks/huge-schema/huge-schema.bench.ts::typescript::client generation ~50 Models

+1%

1.8 s1.8 s

typescript compilation ~50 Models

packages/client/src/__tests__/benchmarks/huge-schema/huge-schema.bench.ts::typescript::typescript compilation ~50 Models

20.1 ms20 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 s10.8 s

Commits

Click on a commit to change the comparison range

Base

main

589e093

-66.68%

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

2 years ago

by Jolg42

+67.15%

fix lint

89335a2

2 years ago

by Jolg42

-0.22%

remove 1 more test setup

9089c16

2 years ago

by Jolg42

Home Terms Privacy Docs