fix(core): test executor should fail properly (#411)

Author: AgentEnder

.github/workflows/main.yml

@@ -5,10 +5,6 @@ on:
   push:
     branches: [master, dev]
 
-env:
-  NX_BRANCH: ${{ github.event.number }}
-  NX_RUN_GROUP: ${{ github.run_id }}
-
 jobs:
   build:
     runs-on: ubuntu-latest

.github/workflows/pr.yml

@@ -1,11 +1,6 @@
 name: Run PR checks
 
 on: pull_request
-    
-
-env:
-  NX_BRANCH: ${{ github.event.number }}
-  NX_RUN_GROUP: ${{ github.run_id }}
 
 jobs:
   build:

.github/workflows/release.yml

@@ -2,10 +2,6 @@ name: Release
 
 on: workflow_dispatch
 
-env:
-  NX_BRANCH: ${{ github.event.number }}
-  NX_RUN_GROUP: ${{ github.run_id }}
-
 jobs:
   build:
     runs-on: ubuntu-latest

.github/workflows/smoke.yml

@@ -5,11 +5,6 @@ on:
   workflow_dispatch:
   schedule:
     - cron: '0 0 * * *'
-  
-
-env:
-  NX_BRANCH: main
-  NX_RUN_GROUP: smoke
 
 jobs:
   smoke:

.releaserc.js

@@ -37,7 +37,7 @@ module.exports = {
       '@semantic-release/exec',
       {
         prepareCmd:
-          'npx ts-node tools/scripts/patch-package-versions ${nextRelease.version}',
+          'npx ts-node tools/scripts/patch-package-versions --version ${nextRelease.version} --project all',
         publishCmd: [
           'npx ts-node tools/scripts/publish-all ${nextRelease.version} ${nextRelease.channel}',
           'nx deploy docs-site',

e2e/core-e2e/tests/nx-dotnet.spec.ts

@@ -60,7 +60,7 @@ describe('nx-dotnet e2e', () => {
     runCommand('git checkout -b "affected-tests"');
     updateFile('package.json', (f) => {
       const json = JSON.parse(f);
-      json.dependencies['@nrwl/angular'] = 'latest';
+      json.dependencies['@nrwl/angular'] = json.devDependencies['nx'];
       return JSON.stringify(json);
     });
     runPackageManagerInstall();
@@ -162,7 +162,11 @@ describe('nx-dotnet e2e', () => {
         `generate @nx-dotnet/core:app ${app} --language="C#" --template="webapi"`,
       );
       const promise = runNxCommandAsync(`lint ${app}`).then((x) => x.stderr);
-      await expect(promise).resolves.toContain('WHITESPACE');
+      await expect(promise).rejects.toThrow(
+        expect.objectContaining({
+          message: expect.stringContaining('WHITESPACE'),
+        }),
+      );
     });
   });
 
@@ -336,6 +340,7 @@ describe('nx-dotnet e2e', () => {
       expect(slnFile).toContain(app + '-test');
     });
   });
+
   describe('inferred targets', () => {
     let api: string;
     let projectFolder: string;
@@ -386,6 +391,37 @@ describe('nx-dotnet e2e', () => {
       writeFileSync(join(projectFolder, 'project.json'), projectJsonContents);
     });
   });
+
+  describe('@nx-dotnet/core:test', () => {
+    it('should test with xunit', () => {
+      const appProject = uniq('app');
+      const testProject = `${appProject}-test`;
+      runNxCommand(
+        `generate @nx-dotnet/core:app ${appProject} --language="C#" --template="webapi" --test-runner xunit`,
+      );
+
+      expect(() => runNxCommand(`test ${testProject}`)).not.toThrow();
+
+      updateFile(
+        `apps/${testProject}/UnitTest1.cs`,
+        `using Xunit;
+
+namespace Proj.${names(appProject).className}.Test;
+
+public class UnitTest1
+{
+    // This test should fail, as the e2e test is checking for test failures.
+    [Fact]
+    public void Test1()
+    {
+      Assert.Equal(1, 2)
+    }
+}`,
+      );
+
+      expect(() => runNxCommand(`test ${testProject}`)).toThrow();
+    });
+  });
 });
 
 function initializeGitRepo(cwd: string) {

nx.json

@@ -40,6 +40,10 @@
       {
         "target": "prebuild",
         "projects": "self"
+      },
+      {
+        "target": "prebuild",
+        "projects": "dependencies"
       }
     ]
   },

packages/core/project.json

@@ -21,7 +21,9 @@
     "prebuild": {
       "executor": "@nrwl/workspace:run-commands",
       "options": {
-        "commands": ["npx ts-node tools/scripts/patch-package-versions"]
+        "commands": [
+          "npx ts-node tools/scripts/patch-package-versions --project core"
+        ]
       }
     },
     "build": {

packages/core/src/executors/test/executor.ts

@@ -1,4 +1,4 @@
-import { ExecutorContext } from '@nrwl/devkit';
+import { ExecutorContext, logger } from '@nrwl/devkit';
 import { appRootPath } from '@nrwl/tao/src/utils/app-root';
 
 import { resolve } from 'path';
@@ -29,18 +29,23 @@ export default async function runExecutor(
   dotnetClient.cwd = projectDirectory;
   const { watch, ...parsedOptions } = options;
 
-  const result = dotnetClient.test(
-    resolve(appRootPath, projectFilePath),
-    watch,
-    parsedOptions,
-  );
+  try {
+    const result = dotnetClient.test(
+      resolve(appRootPath, projectFilePath),
+      watch,
+      parsedOptions,
+    );
 
-  if (watch && isChildProcess(result)) {
-    await handleChildProcessPassthrough(result);
-    await rimraf(projectDirectory + '/bin');
-    await rimraf(projectDirectory + '/obj');
+    if (watch && isChildProcess(result)) {
+      await handleChildProcessPassthrough(result);
+      await rimraf(projectDirectory + '/bin');
+      await rimraf(projectDirectory + '/obj');
+    }
+    return {
+      success: true,
+    };
+  } catch (e) {
+    logger.error(e);
+    return { success: false };
   }
-  return {
-    success: true,
-  };
 }

packages/core/src/generators/app/schema.json

@@ -44,6 +44,7 @@
       "description": "Which template should be used for creating the tests project?",
       "default": "nunit",
       "enum": ["nunit", "xunit", "mstest", "none"],
+      "aliases": ["testRunner"],
       "x-prompt": {
         "message": "Which template should be used for creating the tests project",
         "type": "list",