Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Commit efab3d1

Browse files
ToumashToumash
committed
fix: move vite7 tests to tests folder and use injected dependency
- Move vite7-manifest-test from packages/ to tests/ - Add vite as peerDependency to vite-plugin - Use pnpm's injected dependency for the test package to ensure the plugin resolves vite from the test context (Vite 7) instead of the plugin's own node_modules (Vite 3) - Update CI workflow paths
1 parent fa30de1 commit efab3d1

8 files changed

Lines changed: 65 additions & 22 deletions

File tree

‎.github/workflows/vite-plugin.yml‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -7,14 +7,14 @@ on:
77
branches: [main]
88
paths:
99
- "packages/vite-plugin/**"
10-
- "packages/vite7-manifest-test/**"
10+
- "tests/vite7-manifest-test/**"
1111
- ".github/workflows/vite-plugin.yml"
1212
- "!**.md"
1313
pull_request:
1414
branches: [main]
1515
paths:
1616
- "packages/vite-plugin/**"
17-
- "packages/vite7-manifest-test/**"
17+
- "tests/vite7-manifest-test/**"
1818
- ".github/workflows/vite-plugin.yml"
1919
- "!**.md"
2020

@@ -216,7 +216,7 @@ jobs:
216216

217217
defaults:
218218
run:
219-
working-directory: "packages/vite7-manifest-test"
219+
working-directory: "tests/vite7-manifest-test"
220220

221221
# Steps represent a sequence of tasks that will be executed as part of the job
222222
steps:

‎packages/vite-plugin/package.json‎

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -81,6 +81,9 @@
8181
"rollup": "2.79.2",
8282
"rxjs": "7.5.7"
8383
},
84+
"peerDependencies": {
85+
"vite": "^3.0.0 || ^4.0.0 || ^5.0.0 || ^6.0.0 || ^7.0.0"
86+
},
8487
"devDependencies": {
8588
"@extend-chrome/messages": "1.2.2",
8689
"@extend-chrome/storage": "1.5.0",

‎pnpm-lock.yaml‎

Lines changed: 49 additions & 18 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎pnpm-workspace.yaml‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,5 +3,7 @@ packages:
33
- 'packages/**'
44
# all playground apps
55
- 'playgrounds/**'
6+
# integration tests
7+
- 'tests/**'
68
# exclude packages that are inside test directories
79
- '!**/test/**'

packages/vite7-manifest-test/manifest.test.ts renamed to tests/vite7-manifest-test/manifest.test.ts

File renamed without changes.

packages/vite7-manifest-test/package.json renamed to tests/vite7-manifest-test/package.json

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,10 +9,17 @@
99
"test:watch": "vitest"
1010
},
1111
"devDependencies": {
12-
"@crxjs/vite-plugin": "workspace:*",
1312
"@types/node": "^22.13.14",
1413
"typescript": "~5.7.2",
1514
"vite": "^7.0.0",
1615
"vitest": "^3.0.0"
16+
},
17+
"dependencies": {
18+
"@crxjs/vite-plugin": "workspace:*"
19+
},
20+
"dependenciesMeta": {
21+
"@crxjs/vite-plugin": {
22+
"injected": true
23+
}
1724
}
1825
}

packages/vite7-manifest-test/src/content.js renamed to tests/vite7-manifest-test/src/content.js

File renamed without changes.

packages/vite7-manifest-test/vitest.config.ts renamed to tests/vite7-manifest-test/vitest.config.ts

File renamed without changes.

0 commit comments

Comments
 (0)