test: abbreviate integration
to int
, rename vitest config files
#1018
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Motivation
integration-test
repeatedly, abbreviating toint-test
would be more convenient.vite.config.{type}.ts
file pattern doesn't realiably recognized by tooling because of the custom suffixes. Also, it may not be clear that the configuration is only for Vitest, not for Vite -vitest.config.ts
naming is another standardized config file name, which would be more appropriate in our case.Changes
*.integration.test.ts
files to*.int.test.ts
.vite.config.{unit|integration|e2e}.ts
files tovitest.{unit|int|e2e}.config.ts
.Follow-up
For running E2E tests, typing out
npx nx nxv-e2e <project>-e2e
is cumbersome. I'd like to rename thenxv-e2e
target toe2e-test
. Then all our testing targets would have intuitive names -unit-test
,int-test
,e2e-test
.I've submitted a PR to
@push-based/nx-verdaccio
which adds an option to rename the inferrednxv-e2e
target. Once that's released, we can rename the target on our end.