test: refine template variable hydration and update integration test descriptions

- Updated test case to clarify that overlapping template variables should not be hydrated implicitly.
- Adjusted assertions to ensure that the expected behavior aligns with the new logic.
- Enhanced integration helper functions to include a description field for better test clarity.
- Updated snapshot tests to reflect changes in package.json descriptions and repository URLs for consistency.

Author: sdqede

packages/create-gen-app-test/src/__tests__/__snapshots__/cached-template.test.ts.snap

@@ -16,33 +16,29 @@ exports[`cached template integration tests first clone with variable replacement
   "package.json": {
     "author": "Test User test <[email protected]>",
     "bugs": {
-      "url": "https://github.com/Test User test/Test User test/issues",
+      "url": "https://github.com/tester-test/integration-test/issues",
     },
-    "description": "Test Module test",
+    "description": "Integration test module test",
     "devDependencies": {
-      "pgsql-test": "^2.13.2",
+      "makage": "0.1.8",
+      "pgsql-test": "^2.14.12",
     },
-    "homepage": "https://github.com/Test User test/Test User test",
+    "homepage": "https://github.com/tester-test/integration-test",
     "keywords": [],
     "license": "MIT",
     "name": "integration-test",
-    "pnpm": {
-      "overrides": {
-        "graphql": "14.7.0",
-      },
-    },
     "publishConfig": {
       "access": "public",
       "directory": "dist",
     },
     "repository": {
       "type": "git",
-      "url": "https://github.com/Test User test/Test User test",
+      "url": "https://github.com/tester-test/integration-test",
     },
     "scripts": {
       "lint": "eslint . --fix",
       "test": "jest",
-      "test:watch": "jest --watch",
+      "test:watch": "makage test --watch deploy --ext sql",
     },
     "version": "0.0.1",
   },
@@ -65,33 +61,29 @@ exports[`cached template integration tests second clone from cache should snapsh
   "package.json": {
     "author": "Test User cached <[email protected]>",
     "bugs": {
-      "url": "https://github.com/Test User cached/Test User cached/issues",
+      "url": "https://github.com/tester-cached/integration-cached/issues",
     },
-    "description": "Test Module cached",
+    "description": "Integration test module cached",
     "devDependencies": {
-      "pgsql-test": "^2.13.2",
+      "makage": "0.1.8",
+      "pgsql-test": "^2.14.12",
     },
-    "homepage": "https://github.com/Test User cached/Test User cached",
+    "homepage": "https://github.com/tester-cached/integration-cached",
     "keywords": [],
     "license": "MIT",
     "name": "integration-cached",
-    "pnpm": {
-      "overrides": {
-        "graphql": "14.7.0",
-      },
-    },
     "publishConfig": {
       "access": "public",
       "directory": "dist",
     },
     "repository": {
       "type": "git",
-      "url": "https://github.com/Test User cached/Test User cached",
+      "url": "https://github.com/tester-cached/integration-cached",
     },
     "scripts": {
       "lint": "eslint . --fix",
       "test": "jest",
-      "test:watch": "jest --watch",
+      "test:watch": "makage test --watch deploy --ext sql",
     },
     "version": "0.0.1",
   },

packages/create-gen-app-test/src/cli.ts

@@ -208,7 +208,9 @@ export async function runCli(
 
     const answerOverrides = extractAnswerOverrides(args);
     const noTty = Boolean(
-      args["no-tty"] ?? (args as Record<string, unknown>).noTty
+      args["no-tty"] ??
+        (args as Record<string, unknown>).noTty ??
+        (args as Record<string, unknown>).tty === false
     );
 
     // Use the createFromTemplate function which will use the same cache

packages/create-gen-app-test/src/test-utils/integration-helpers.ts

@@ -35,6 +35,7 @@ export function buildAnswers(
     email: `tester-${safeSuffix}@example.com`,
     moduleName: `integration-${safeSuffix}`,
     moduleDesc: `Integration test module ${suffix}`,
+    description: `Integration test module ${suffix}`,
     repoName: `integration-${safeSuffix}`,
     username: `tester-${safeSuffix}`,
     access: "public",
@@ -43,4 +44,3 @@ export function buildAnswers(
     ...overrides,
   };
 }
-

packages/create-gen-app/__tests__/create-gen.test.ts

@@ -378,7 +378,7 @@ module.exports = {
       expect(answers.fullName).toBe("Prompted User");
     });
 
-    it("should hydrate overlapping template variables from answers", async () => {
+    it("should not hydrate overlapping template variables implicitly", async () => {
       const { Inquirerer } = require("inquirerer");
       const mockPrompt = jest.fn().mockResolvedValue({
         description: "Prompted description",
@@ -406,7 +406,7 @@ module.exports = {
 
       const answers = await promptUser(extractedVariables, {}, false);
       expect(answers.description).toBe("Prompted description");
-      expect(answers.moduleDesc).toBe("Prompted description");
+      expect(answers.moduleDesc).toBeUndefined();
     });
   });
 

packages/create-gen-app/test-utils/integration-helpers.ts

@@ -35,6 +35,7 @@ export function buildAnswers(
     email: `tester-${safeSuffix}@example.com`,
     moduleName: `integration-${safeSuffix}`,
     moduleDesc: `Integration test module ${suffix}`,
+    description: `Integration test module ${suffix}`,
     repoName: `integration-${safeSuffix}`,
     username: `tester-${safeSuffix}`,
     access: "public",