Commit95f9621

committed

configure test failure message

1 parent59f27f6 commit95f9621Copy full SHA for 95f9621

File tree

+48

-8

lines changed

+48

-8

lines changed

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -59,9 +59,9 @@ export const createCommands = ({extensionPath, workspaceState, workspaceRoot}: C`
`59`	`59`	`vscode.window.showInformationMessage('PASS')`
`60`	`60`	`webview.send({type:'TEST_PASS', payload})`
`61`	`61`	`},`
`62`		`-onFail:(payload:Payload)=>{`
	`62`	`+onFail:(payload:Payload,message:string)=>{`
`63`	`63`	`// send test fail message back to client`
`64`		`-vscode.window.showWarningMessage('FAIL')`
	`64`	+vscode.window.showWarningMessage(`FAIL:${message}`)
`65`	`65`	`webview.send({type:'TEST_FAIL', payload})`
`66`	`66`	`},`
`67`	`67`	`onError:(payload:Payload)=>{`

Lines changed: 8 additions & 4 deletions

Original file line number	Diff line number	Diff line change
`@@ -9,7 +9,7 @@ export interface Payload {`
`9`	`9`
`10`	`10`	`interfaceCallbacks{`
`11`	`11`	`onSuccess(payload:Payload):void`
`12`		`-onFail(payload:Payload):void`
	`12`	`+onFail(payload:Payload,message:string):void`
`13`	`13`	`onRun(payload:Payload):void`
`14`	`14`	`onError(payload:Payload):void`
`15`	`15`	`}`
`@@ -55,12 +55,16 @@ const createTestRunner = (config: TestRunnerConfig, callbacks: Callbacks) => {`
`55`	`55`	`callbacks.onSuccess(payload)`
`56`	`56`	`if(onSuccess){onSuccess()}`
`57`	`57`	`}else{`
`58`		`-// TODO: parse failure message`
	`58`	`+`
	`59`	`+// TODO: consider logging output to channel`
`59`	`60`	`// open terminal with failed test string`
`60`	`61`	`// const channel = getOutputChannel(outputChannelName)`
`61`	`62`	`// channel.show(false)`
`62`		`-// channel.appendLine(testsFailed.message)`
`63`		`-callbacks.onFail(payload)`
	`63`	`+// channel.appendLine(tap.message)`
	`64`	`+`
	`65`	`+`
	`66`	`+constmessage=tap.message ?tap.message :''`
	`67`	`+callbacks.onFail(payload,message)`
`64`	`68`	`}`
`65`	`69`	`}`
`66`	`70`	`}`

Lines changed: 33 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,33 @@`
	`1`	`+importparserfrom'./parser'`
	`2`	`+`
	`3`	`+`
	`4`	`+`
	`5`	`+describe('parser',()=>{`
	`6`	`+test('should detect success',()=>{`
	`7`	+constexample=`
	`8`	`+1..2`
	`9`	`+ok 1 - Should pass`
	`10`	`+ok 2 - Should also pass`
	`11`	+`
	`12`	`+expect(parser(example)).toEqual({ok:true})`
	`13`	`+})`
	`14`	`+test('should detect failure',()=>{`
	`15`	+constexample=`
	`16`	`+1..3`
	`17`	`+ok 1 - Should pass`
	`18`	`+not ok 2 - This one fails`
	`19`	`+ok 3 - Also passes`
	`20`	+`
	`21`	`+expect(parser(example).ok).toBe(false)`
	`22`	`+})`
	`23`	`+test('should return failure message',()=>{`
	`24`	+constexample=`
	`25`	`+1..4`
	`26`	`+ok 1 - Should pass`
	`27`	`+not ok 2 - First to fail`
	`28`	`+ok 3 - Also passes`
	`29`	`+not ok 4 - Second to fail`
	`30`	+`
	`31`	`+expect(parser(example).message).toBe('First to fail')`
	`32`	`+})`
	`33`	`+})`

Lines changed: 5 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -1,12 +1,15 @@`
`1`	`1`	`interfaceParserOutput{`
`2`	`2`	`ok:boolean`
	`3`	`+message?:string`
`3`	`4`	`}`
`4`	`5`
`5`	`6`	`constparser=(text:string):ParserOutput=>{`
`6`	`7`	`constlines=text.split('\n')`
`7`	`8`	`for(constlineoflines){`
`8`		`-if(line.match(/^notok/)){`
`9`		`-return{ok:false}`
	`9`	`+// parse failed test`
	`10`	`+constmatch=line.match(/^notok\d+-(.+)+/)`
	`11`	`+if(!!match){`
	`12`	`+return{ok:false,message:match[1]}`
`10`	`13`	`}`
`11`	`14`	`}`
`12`	`15`	`return{ok:true}`

Comments

(0)