crabbedbushel
diff --git a/‎src/editor/commands.ts
Lines changed: 2 additions & 2 deletions b/‎src/editor/commands.ts
Lines changed: 2 additions & 2 deletions
diff --git a/‎src/services/testRunner/index.ts
Lines changed: 8 additions & 4 deletions b/‎src/services/testRunner/index.ts
Lines changed: 8 additions & 4 deletions
diff --git a/‎src/services/testRunner/parser.test.ts
Lines changed: 33 additions & 0 deletions b/‎src/services/testRunner/parser.test.ts
Lines changed: 33 additions & 0 deletions
diff --git a/‎src/services/testRunner/parser.ts
Lines changed: 5 additions & 2 deletions b/‎src/services/testRunner/parser.ts
Lines changed: 5 additions & 2 deletions
@@ -59,9 +59,9 @@ export const createCommands = ({extensionPath, workspaceState, workspaceRoot}: C
 					vscode.window.showInformationMessage('PASS')
 					webview.send({type: 'TEST_PASS', payload})
 				},
-				onFail: (payload: Payload) => {
+				onFail: (payload: Payload, message: string) => {
 					// send test fail message back to client
-					vscode.window.showWarningMessage('FAIL')
+					vscode.window.showWarningMessage(`FAIL: ${message}`)
 					webview.send({type: 'TEST_FAIL', payload})
 				},
 				onError: (payload: Payload) => {
 
@@ -9,7 +9,7 @@ export interface Payload {
 
 interface Callbacks {
 	onSuccess(payload: Payload): void
-	onFail(payload: Payload): void
+	onFail(payload: Payload, message: string): void
 	onRun(payload: Payload): void
 	onError(payload: Payload): void
 }
@@ -55,12 +55,16 @@ const createTestRunner = (config: TestRunnerConfig, callbacks: Callbacks) => {
 			callbacks.onSuccess(payload)
 			if (onSuccess) {onSuccess()}
 		} else {
-			// TODO: parse failure message
+
+			// TODO: consider logging output to channel
 			// open terminal with failed test string
 			// const channel = getOutputChannel(outputChannelName)
 			// channel.show(false)
-			// channel.appendLine(testsFailed.message)
-			callbacks.onFail(payload)
+			// channel.appendLine(tap.message)
+
+
+			const message = tap.message ? tap.message : ''
+			callbacks.onFail(payload, message)
 		}
 	}
 }
 
@@ -0,0 +1,33 @@
+import parser from './parser'
+
+
+
+describe('parser', () => {
+	test('should detect success', () => {
+		const example = `
+1..2
+ok 1 - Should pass
+ok 2 - Should also pass
+`
+		expect(parser(example)).toEqual({ok: true})
+	})
+	test('should detect failure', () => {
+		const example = `
+1..3
+ok 1 - Should pass
+not ok 2 - This one fails
+ok 3 - Also passes
+`
+		expect(parser(example).ok).toBe(false)
+	})
+	test('should return failure message', () => {
+		const example = `
+1..4
+ok 1 - Should pass
+not ok 2 - First to fail
+ok 3 - Also passes
+not ok 4 - Second to fail
+`
+		expect(parser(example).message).toBe('First to fail')
+	})
+})
@@ -1,12 +1,15 @@
 interface ParserOutput {
 	ok: boolean
+	message?: string
 }
 
 const parser = (text: string): ParserOutput => {
 	const lines = text.split('\n')
 	for (const line of lines) {
-		if (line.match(/^not ok /)) {
-			return {ok: false}
+		// parse failed test
+		const match = line.match(/^not ok \d+ - (.+)+/)
+		if (!!match) {
+			return {ok: false, message: match[1]}
 		}
 	}
 	return {ok: true}
Original file line number	Diff line number	Diff line change
`@@ -1,12 +1,15 @@`
`1`	`1`	`interface ParserOutput {`
`2`	`2`	`ok: boolean`
	`3`	`+ message?: string`
`3`	`4`	`}`
`4`	`5`
`5`	`6`	`const parser = (text: string): ParserOutput => {`
`6`	`7`	`const lines = text.split('\n')`
`7`	`8`	`for (const line of lines) {`
`8`		`- if (line.match(/^not ok /)) {`
`9`		`- return {ok: false}`
	`9`	`+ // parse failed test`
	`10`	`+ const match = line.match(/^not ok \d+ - (.+)+/)`
	`11`	`+ if (!!match) {`
	`12`	`+ return {ok: false, message: match[1]}`
`10`	`13`	`}`
`11`	`14`	`}`
`12`	`15`	`return {ok: true}`