test: stop leaking isInteractive out of project-commands

The beforeEach assigned helpers.isInteractive = () => true and never put it
back. Under mocha every test file shares one module registry, so from the
moment this file ran, everything after it saw an interactive terminal
regardless of the environment - which is why project-name-service passed in
CI despite depending on the non-interactive path, and why it only failed once
per-file isolation removed the leak.

Uses setIsInteractive(), the override the helper already exposes for this,
with an afterEach to restore - matching how project-name-service pins it.

Author: edusperoni

test/project-commands.ts

@@ -2,7 +2,7 @@ import { Yok } from "../lib/common/yok";
 import * as stubs from "./stubs";
 import { CreateProjectCommand } from "../lib/commands/create-project";
 import { StringCommandParameter } from "../lib/common/command-params";
-import * as helpers from "../lib/common/helpers";
+import { setIsInteractive } from "../lib/common/helpers";
 import * as constants from "../lib/constants";
 import { assert } from "chai";
 import { PrompterStub } from "./stubs";
@@ -132,7 +132,7 @@ class ProjectServiceMock implements IProjectService {
 	}
 
 	async createProject(
-		projectOptions: IProjectSettings
+		projectOptions: IProjectSettings,
 	): Promise<ICreateProjectData> {
 		createProjectCalledWithForce = projectOptions.force;
 		selectedTemplateName = projectOptions.template;
@@ -220,8 +220,7 @@ describe("Project commands tests", () => {
 
 	beforeEach(() => {
 		testInjector = createTestInjector();
-		// @ts-expect-error
-		helpers.isInteractive = () => true;
+		setIsInteractive(() => true);
 		isProjectCreated = false;
 		validateProjectCallsCount = 0;
 		createProjectCalledWithForce = false;
@@ -230,6 +229,10 @@ describe("Project commands tests", () => {
 		createProjectCommand = testInjector.resolve("$createCommand");
 	});
 
+	afterEach(() => {
+		setIsInteractive();
+	});
+
 	describe("#CreateProjectCommand", () => {
 		it("should not fail when using only --ng.", async () => {
 			options.ng = true;
@@ -366,7 +369,7 @@ describe("Project commands tests", () => {
 
 			assert.deepStrictEqual(
 				selectedTemplateName,
-				"@nativescript/template-hello-world-ng"
+				"@nativescript/template-hello-world-ng",
 			);
 			assert.equal(validateProjectCallsCount, 1);
 			assert.isTrue(createProjectCalledWithForce);
@@ -382,7 +385,7 @@ describe("Project commands tests", () => {
 
 			assert.deepStrictEqual(
 				selectedTemplateName,
-				"@nativescript/template-drawer-navigation-ts"
+				"@nativescript/template-drawer-navigation-ts",
 			);
 			assert.equal(validateProjectCallsCount, 1);
 			assert.isTrue(createProjectCalledWithForce);
@@ -398,7 +401,7 @@ describe("Project commands tests", () => {
 
 			assert.deepStrictEqual(
 				selectedTemplateName,
-				"@nativescript/template-tab-navigation"
+				"@nativescript/template-tab-navigation",
 			);
 			assert.equal(validateProjectCallsCount, 1);
 			assert.isTrue(createProjectCalledWithForce);
@@ -414,7 +417,7 @@ describe("Project commands tests", () => {
 
 			assert.deepStrictEqual(
 				selectedTemplateName,
-				"@nativescript/template-drawer-navigation-vue"
+				"@nativescript/template-drawer-navigation-vue",
 			);
 			assert.equal(validateProjectCallsCount, 1);
 			assert.isTrue(createProjectCalledWithForce);
@@ -430,7 +433,7 @@ describe("Project commands tests", () => {
 
 			assert.deepStrictEqual(
 				selectedTemplateName,
-				"@nativescript/template-blank-react"
+				"@nativescript/template-blank-react",
 			);
 			assert.equal(validateProjectCallsCount, 1);
 			assert.isTrue(createProjectCalledWithForce);
@@ -446,7 +449,7 @@ describe("Project commands tests", () => {
 
 			assert.deepStrictEqual(
 				selectedTemplateName,
-				"@nativescript/template-blank-svelte"
+				"@nativescript/template-blank-svelte",
 			);
 			assert.equal(validateProjectCallsCount, 1);
 			assert.isTrue(createProjectCalledWithForce);