Merge pull request #42 from ember-cli/NullVoxPopuli-patch-4

Don't allow CI to run linting twice

Author: NullVoxPopuli

files/README.md

@@ -33,7 +33,7 @@ Make use of the many generators for code, try `ember help generate` for more det
 ### Running Tests
 
 - `<%= invokeScriptPrefix %> test`
-- `<%= invokeScriptPrefix %> test:ember <% if (npm) { %>-- <% } %>--server`
+- `<%= invokeScriptPrefix %> test <% if (npm) { %>-- <% } %>--server`
 
 ### Linting
 

files/package.json

@@ -24,8 +24,7 @@
     "lint:js:fix": "eslint . --fix<% if (typescript) { %>",
     "lint:types": "glint<% } %>",
     "start": "vite",
-    "test": "concurrently \"<%= packageManager %>:lint\" \"<%= packageManager %>:test:*\" --names \"lint,test:\" --prefixColors auto",
-    "test:ember": "vite build --mode development && testem ci"
+    "test": "vite build --mode development && testem ci"
   },
   "exports": {
     "./tests/*": "./tests/*",

tests/default.test.mjs

@@ -68,10 +68,10 @@ describe('basic functionality', function () {
         let result;
 
         try {
-          result = await project.execa('pnpm', ['test:ember']);
+          result = await project.execa('pnpm', ['test']);
         } catch (err) {
           console.log(err.stdout, err.stderr);
-          throw 'Failed to successfully run test:ember';
+          throw 'Failed to successfully run test';
         }
 
         // make sure that each of the tests that we expect actually show up