ember-testem 0.3.1 → 0.3.3

package/README.md

@@ -95,9 +95,9 @@ See [my-v1-app](../../tests/my-v1-app) for an example.
 
 <summary>V2 addons</summary>
 
-See [my-v2-addon](../../tests/my-v2-addon) for an example.
+In general, I recommend creating a test app as a separate package (using workspaces). This way, you can separate dependencies for source code and testing (avoid past issues with v1 addons). You would set up `ember-testem` by following the steps for v2 apps.
 
-For brevity, additional file changes aren't shown below. For example, you will need to split the Babel and TypeScript configurations into two files: one for production and another for testing.
+If you still want to create the test app in the same package, you will need to install and configure additional dependencies. See [my-v2-addon](../../tests/my-v2-addon) for an example.
 
 ```json
 {
@@ -108,32 +108,13 @@ For brevity, additional file changes aren't shown below. For example, you will n
 }
 ```
 
-```ts
-/* vite.test.config.mts */
-import { extensions, ember } from '@embroider/vite';
-import { babel } from '@rollup/plugin-babel';
-import { defineConfig } from 'vite';
+Files that need changes:
 
-export default defineConfig({
-  build: {
-    minify: false,
-    rollupOptions: {
-      input: {
-        tests: 'tests/index.html',
-      },
-    },
-    sourcemap: true,
-  },
-  plugins: [
-    ember(),
-    babel({
-      babelHelpers: 'inline',
-      extensions,
-      parallel: true,
-    }),
-  ],
-});
-```
+- Babel (`babel.config.mjs`, `babel.build.config.mjs`)
+- Dependencies (`package.json`)
+- Rollup (`rollup.config.mjs`)
+- TypeScript (`tsconfig.json`, `tsconfig.build.json`)
+- Vite (`vite.test.config.mts`)
 
 </details>
 
@@ -237,7 +218,7 @@ ember-test --port 0
 >
 > `ember-testem` provides a limited API on purpose.
 >
-> For `ember-cli` options like `--host`, `--launch`, `--reporter`, `--ssl`, `--ssl-cert`, `--ssl-key`, `--test-page`, `--testem-debug`), [configure `testem` in `testem.mjs` instead](https://github.com/ember-cli/ember-cli/blob/v7.3.0-ember-cli/lib/tasks/test.js#L54-L71).
+> For `ember-cli` options like `--host`, `--launch`, `--query`, `--reporter`, `--ssl`, `--ssl-cert`, `--ssl-key`, `--test-page`, `--testem-debug`), [configure `testem` in `testem.mjs` instead](https://github.com/ember-cli/ember-cli/blob/v7.3.0-ember-cli/lib/tasks/test.js#L54-L71).
 
 
 ## Compatibility

package/dist/bin/ember-test.js

@@ -43,12 +43,12 @@ const argv = yargs(hideBin(process.argv))
     .parseSync();
 const codemodOptions = {
     buildPath: argv['path'],
-    filters: {
-        filter: argv['filter'],
-        module: argv['module'],
-    },
     port: argv['test-port'] ?? argv['port'],
     projectRoot: argv['root'] ?? process.cwd(),
+    qunit: {
+        filter: argv['filter'],
+        testModule: argv['module'],
+    },
     server: argv['server'],
 };
 void run(codemodOptions);

package/dist/src/steps/create-options.js

@@ -1,10 +1,10 @@
 export function createOptions(codemodOptions) {
-    const { buildPath, filters, port, projectRoot, server } = codemodOptions;
+    const { buildPath, port, projectRoot, qunit, server } = codemodOptions;
     return {
         buildPath,
-        filters,
         launchBrowser: Boolean(server),
         port,
         projectRoot,
+        qunit,
     };
 }

package/dist/src/steps/run-tests/run-tests-in-ci.js

@@ -2,7 +2,7 @@ import { join } from 'node:path';
 import Testem from 'testem';
 import { getQueryParams } from '../../utils/run-tests/get-query-params.js';
 export async function runTestsInCI(options) {
-    const { buildPath, filters, port, projectRoot } = options;
+    const { buildPath, port, projectRoot } = options;
     const testemInstance = new Testem();
     const startOptions = {
         launch: false,
@@ -13,7 +13,7 @@ export async function runTestsInCI(options) {
         config_dir: projectRoot,
         cwd: join(projectRoot, buildPath),
         parallel: 1,
-        query_params: getQueryParams(filters),
+        query_params: getQueryParams(options),
     });
     return new Promise((resolve, reject) => {
         testemInstance.startCI(startOptions, (exitCode, error) => {

package/dist/src/steps/run-tests/run-tests-in-dev.js

@@ -3,7 +3,7 @@ import { watch } from 'chokidar';
 import Testem from 'testem';
 import { getQueryParams } from '../../utils/run-tests/get-query-params.js';
 export async function runTestsInDev(options) {
-    const { buildPath, filters, port, projectRoot } = options;
+    const { buildPath, port, projectRoot } = options;
     const testemInstance = new Testem();
     const startOptions = {
         launch: false,
@@ -13,7 +13,7 @@ export async function runTestsInDev(options) {
         ...startOptions,
         config_dir: projectRoot,
         cwd: join(projectRoot, buildPath),
-        query_params: getQueryParams(filters),
+        query_params: getQueryParams(options),
     });
     const watcher = watch([buildPath], {
         awaitWriteFinish: true,

package/dist/src/utils/run-tests/get-query-params.js

@@ -1,10 +1,11 @@
-export function getQueryParams(filters) {
+export function getQueryParams(options) {
+    const { qunit } = options;
     const params = [];
-    if (filters.module) {
-        params.push(`module=${filters.module}`);
+    if (qunit.testModule) {
+        params.push(`module=${qunit.testModule}`);
     }
-    if (filters.filter) {
-        params.push(`filter=${filters.filter}`);
+    if (qunit.filter) {
+        params.push(`filter=${qunit.filter}`);
     }
     return params.join('&');
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ember-testem",
-  "version": "0.3.1",
+  "version": "0.3.3",
   "description": "Backward- and forward-compatible testing for Ember projects",
   "keywords": [
     "ci",
@@ -30,15 +30,15 @@
     "yargs": "^18.2.0"
   },
   "devDependencies": {
-    "@codemod-utils/tests": "^3.3.0",
+    "@codemod-utils/tests": "^3.3.1",
     "@shared-configs/eslint-config-node": "0.0.0",
     "@shared-configs/prettier": "0.0.0",
     "@shared-configs/typescript": "0.0.0",
-    "@types/node": "^22.20.4",
+    "@types/node": "^22.20.5",
     "@types/yargs": "^17.0.35",
-    "concurrently": "^10.0.5",
-    "eslint": "^10.12.0",
-    "prettier": "^3.9.9",
+    "concurrently": "^10.0.6",
+    "eslint": "^10.13.0",
+    "prettier": "^3.9.10",
     "typescript": "^6.0.3"
   },
   "engines": {