fix: error in non tty environments (#5)

privatenumber · web-flow · commit a766c578d9f9 · 2022-01-24T11:20:12.000-05:00
diff --git a/package.json b/package.json
@@ -47,7 +47,7 @@
 		]
 	},
 	"dependencies": {
-		"terminal-columns": "^1.4.0",
+		"terminal-columns": "^1.4.1",
 		"type-flag": "^2.0.0"
 	},
 	"devDependencies": {
diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml
diff --git a/src/render-help/renderers.ts b/src/render-help/renderers.ts
@@ -19,15 +19,17 @@ export class Renderers {
 	}
 
 	bold(text: string) {
-		return `\u001B[1m${text}\u001B[22m`;
+		return stdoutHasColors
+			? `\u001B[1m${text}\u001B[22m`
+			: text.toLocaleUpperCase();
 	}
 
 	indentText({ text, spaces }: { text: string; spaces: number }) {
 		return text.replace(/^/gm, ' '.repeat(spaces));
 	}
 
 	heading(text: string) {
-		return stdoutHasColors ? this.bold(text) : text.toLocaleUpperCase();
+		return this.bold(text);
 	}
 
 	section({
diff --git a/tests/help.spec.ts b/tests/help.spec.ts
@@ -19,6 +19,10 @@ afterEach(() => {
 	mockConsoleLog.mockRestore();
 });
 
+afterAll(() => {
+	mockProcessExit.mockRestore();
+});
+
 describe('show help', () => {
 	test('empty cli', () => {
 		cli(

Original file line number	Diff line number	Diff line change
`@@ -19,15 +19,17 @@ export class Renderers {`
`19`	`19`	`}`
`20`	`20`
`21`	`21`	`bold(text: string) {`
`22`		- return `\u001B[1m${text}\u001B[22m`;
	`22`	`+ return stdoutHasColors`
	`23`	+ ? `\u001B[1m${text}\u001B[22m`
	`24`	`+ : text.toLocaleUpperCase();`
`23`	`25`	`}`
`24`	`26`
`25`	`27`	`indentText({ text, spaces }: { text: string; spaces: number }) {`
`26`	`28`	`return text.replace(/^/gm, ' '.repeat(spaces));`
`27`	`29`	`}`
`28`	`30`
`29`	`31`	`heading(text: string) {`
`30`		`- return stdoutHasColors ? this.bold(text) : text.toLocaleUpperCase();`
	`32`	`+ return this.bold(text);`
`31`	`33`	`}`
`32`	`34`
`33`	`35`	`section({`