improve tests

Author: kentcdodds

exercises/01.expressions-and-output/02.problem.escaping-strings/index.ts

@@ -13,5 +13,4 @@
 // 🐨 Log tab-separated data: Name: [tab] Age: [tab] City: (like column headers)
 // 💰 Use \t to create tabs between words
 
-// 🐨 Export your variables so we can verify your work
-// 💰 export { apostrophe, quotes, newlines, tabs }
+// 🐨 No exports needed for this exercise

exercises/01.expressions-and-output/02.solution.escaping-strings/index.test.ts

@@ -1,62 +1,49 @@
 import assert from 'node:assert/strict'
+import { spawnSync } from 'node:child_process'
 import { test } from 'node:test'
-import * as solution from './index.ts'
 
-await test('apostrophe is exported', () => {
-	assert.ok(
-		'apostrophe' in solution,
-		'🚨 Make sure you export "apostrophe" - add: export { apostrophe, ... }',
-	)
-})
+function getOutput() {
+	const result = spawnSync('npm', ['start', '--silent'], { encoding: 'utf8' })
+
+	if (result.error) {
+		throw result.error
+	}
 
-await test('should have string with apostrophe', () => {
 	assert.strictEqual(
-		solution.apostrophe,
-		"It's working!",
-		'🚨 apostrophe should be "It\'s working!" - make sure you escaped the apostrophe with \\\'',
+		result.status,
+		0,
+		result.stderr || '🚨 Running the program failed',
 	)
-})
 
-await test('quotes is exported', () => {
-	assert.ok(
-		'quotes' in solution,
-		'🚨 Make sure you export "quotes" - add: export { apostrophe, quotes, ... }',
-	)
-})
+	return result.stdout.replace(/\r\n/g, '\n')
+}
 
-await test('should have string with quotes', () => {
-	assert.strictEqual(
-		solution.quotes,
-		'She said "Hi"',
-		'🚨 quotes should be: She said "Hi" - make sure you escaped the double quotes with \\"',
-	)
-})
+const output = getOutput()
 
-await test('newlines is exported', () => {
+await test('should print apostrophe string', () => {
 	assert.ok(
-		'newlines' in solution,
-		'🚨 Make sure you export "newlines" - add: export { ..., newlines, ... }',
+		output.includes("It's working!"),
+		'🚨 Output should include "It\'s working!" - make sure you escaped the apostrophe with \\\'',
 	)
 })
 
-await test('should have Hello and World on separate lines', () => {
-	assert.strictEqual(
-		solution.newlines,
-		'Hello\nWorld',
-		'🚨 newlines should be "Hello\\nWorld" - use \\n in a single string',
+await test('should print string with quotes', () => {
+	assert.ok(
+		output.includes('She said "Hi"'),
+		'🚨 Output should include: She said "Hi" - make sure you escaped the double quotes with \\"',
 	)
 })
 
-await test('tabs is exported', () => {
+await test('should print Hello and World on separate lines', () => {
 	assert.ok(
-		'tabs' in solution,
-		'🚨 Make sure you export "tabs" - add: export { ..., tabs }',
+		output.includes('Hello\nWorld'),
+		'🚨 Output should include "Hello\\nWorld" - use \\n in a single string',
 	)
 })
 
-await test('should have tab-separated values', () => {
+await test('should print tab-separated values', () => {
 	assert.ok(
-		solution.tabs.includes('\t'),
-		'🚨 tabs should include tab characters - make sure you used \\t for tabs',
+		output.includes('Name:\tAge:\tCity:'),
+		'🚨 Output should include tab characters - make sure you used \\t for tabs',
 	)
 })

exercises/01.expressions-and-output/02.solution.escaping-strings/index.ts

@@ -16,5 +16,3 @@ console.log(newlines)
 // prettier-ignore
 const tabs = 'Name:\tAge:\tCity:'
 console.log(tabs)
-
-export { apostrophe, quotes, newlines, tabs }

exercises/01.expressions-and-output/03.problem.strings/index.ts

@@ -8,5 +8,4 @@
 // 🐨 Log a sentence by concatenating multiple strings
 // 💰 Example: "I" + " " + "am" + " " + "learning" + " " + "to" + " " + "code"
 
-// 🐨 Export your variables so we can verify your work
-// 💰 export { greeting, fullName, sentence }
+// 🐨 No exports needed for this exercise

exercises/01.expressions-and-output/03.solution.strings/index.test.ts

@@ -1,41 +1,44 @@
 import assert from 'node:assert/strict'
+import { spawnSync } from 'node:child_process'
 import { readFileSync } from 'node:fs'
 import { test } from 'node:test'
-import * as solution from './index.ts'
 
-await test('greeting is exported', () => {
-	assert.ok(
-		'greeting' in solution,
-		'🚨 Make sure you export "greeting" - add: export { greeting, ... }',
-	)
-})
+function getOutput() {
+	const result = spawnSync('npm', ['start', '--silent'], { encoding: 'utf8' })
+
+	if (result.error) {
+		throw result.error
+	}
 
-await test('should have Hello TypeScript', () => {
 	assert.strictEqual(
-		solution.greeting,
-		'Hello TypeScript',
-		'🚨 greeting should be "Hello TypeScript" - concatenate "Hello" + " " + "TypeScript"',
+		result.status,
+		0,
+		result.stderr || '🚨 Running the program failed',
 	)
-})
 
-await test('fullName is exported', () => {
+	return result.stdout.replace(/\r\n/g, '\n')
+}
+
+const output = getOutput()
+
+await test('should print Hello TypeScript', () => {
 	assert.ok(
-		'fullName' in solution,
-		'🚨 Make sure you export "fullName" - add: export { greeting, fullName, ... }',
+		output.includes('Hello TypeScript'),
+		'🚨 Output should include "Hello TypeScript" - concatenate "Hello" + " " + "TypeScript"',
 	)
 })
 
-await test('fullName should have a space', () => {
+await test('should print a full name with a space', () => {
 	assert.ok(
-		solution.fullName.includes(' '),
-		'🚨 fullName should contain a space between first and last name',
+		output.includes('Kody Koala'),
+		'🚨 Output should include a full name with a space between first and last name',
 	)
 })
 
-await test('sentence is exported', () => {
+await test('should print the concatenated sentence', () => {
 	assert.ok(
-		'sentence' in solution,
-		'🚨 Make sure you export "sentence" - add: export { greeting, fullName, sentence }',
+		output.includes('I am learning to code'),
+		'🚨 Output should include "I am learning to code" - concatenate multiple strings',
 	)
 })
 

exercises/01.expressions-and-output/03.solution.strings/index.ts

@@ -8,5 +8,3 @@ const sentence = 'I' + ' ' + 'am' + ' ' + 'learning' + ' ' + 'to' + ' ' + 'code'
 console.log(greeting)
 console.log(fullName)
 console.log(sentence)
-
-export { greeting, fullName, sentence }

exercises/01.expressions-and-output/04.problem.numbers/index.ts

@@ -10,5 +10,4 @@
 // 🐨 Use console.log to show the result of (10 + 5) multiplied by 2
 // 💰 Use parentheses to control order of operations
 
-// 🐨 Export your variables so we can verify your work
-// 💰 export { addResult, multiplyResult, divideResult, groupedResult }
+// 🐨 No exports needed for this exercise

exercises/01.expressions-and-output/04.solution.numbers/index.test.ts

@@ -1,63 +1,53 @@
 import assert from 'node:assert/strict'
+import { spawnSync } from 'node:child_process'
 import { test } from 'node:test'
-import * as solution from './index.ts'
 
-await test('addResult is exported', () => {
-	assert.ok(
-		'addResult' in solution,
-		'🚨 Make sure you export "addResult" - add: export { addResult, ... }',
-	)
-})
+function getOutput() {
+	const result = spawnSync('npm', ['start', '--silent'], { encoding: 'utf8' })
+
+	if (result.error) {
+		throw result.error
+	}
 
-await test('addResult should be 42', () => {
 	assert.strictEqual(
-		solution.addResult,
-		42,
-		'🚨 addResult should be 42 (the result of 25 + 17)',
+		result.status,
+		0,
+		result.stderr || '🚨 Running the program failed',
 	)
-})
 
-await test('multiplyResult is exported', () => {
-	assert.ok(
-		'multiplyResult' in solution,
-		'🚨 Make sure you export "multiplyResult" - add: export { ..., multiplyResult, ... }',
-	)
-})
+	return result.stdout.replace(/\r\n/g, '\n')
+}
 
-await test('multiplyResult should be 48', () => {
-	assert.strictEqual(
-		solution.multiplyResult,
-		48,
-		'🚨 multiplyResult should be 48 (the result of 8 * 6)',
-	)
-})
+const output = getOutput()
+const lines = output
+	.split('\n')
+	.map((line) => line.trim())
+	.filter(Boolean)
 
-await test('divideResult is exported', () => {
+await test('should print 42', () => {
 	assert.ok(
-		'divideResult' in solution,
-		'🚨 Make sure you export "divideResult" - add: export { ..., divideResult, ... }',
+		lines.includes('42'),
+		'🚨 Output should include 42 (the result of 25 + 17)',
 	)
 })
 
-await test('divideResult should be 25', () => {
-	assert.strictEqual(
-		solution.divideResult,
-		25,
-		'🚨 divideResult should be 25 (the result of 100 / 4)',
+await test('should print 48', () => {
+	assert.ok(
+		lines.includes('48'),
+		'🚨 Output should include 48 (the result of 8 * 6)',
 	)
 })
 
-await test('groupedResult is exported', () => {
+await test('should print 25', () => {
 	assert.ok(
-		'groupedResult' in solution,
-		'🚨 Make sure you export "groupedResult" - add: export { ..., groupedResult }',
+		lines.includes('25'),
+		'🚨 Output should include 25 (the result of 100 / 4)',
 	)
 })
 
-await test('groupedResult should be 30', () => {
-	assert.strictEqual(
-		solution.groupedResult,
-		30,
-		'🚨 groupedResult should be 30 (the result of (10 + 5) * 2)',
+await test('should print 30', () => {
+	assert.ok(
+		lines.includes('30'),
+		'🚨 Output should include 30 (the result of (10 + 5) * 2)',
 	)
 })

exercises/01.expressions-and-output/04.solution.numbers/index.ts

@@ -10,5 +10,3 @@ console.log(addResult)
 console.log(multiplyResult)
 console.log(divideResult)
 console.log(groupedResult)
-
-export { addResult, multiplyResult, divideResult, groupedResult }

exercises/01.expressions-and-output/05.problem.template-literals/index.ts

@@ -9,5 +9,4 @@
 // 🐨 Log a math problem like "10 times 5 equals 50"
 // 💰 Use ${10 * 5} for the result
 
-// 🐨 Export your variables so we can verify your work
-// 💰 export { answer, greeting, math }
+// 🐨 No exports needed for this exercise