Skip to content

cmake/TestSalmonQuasi.cmake: more verbose test failure messages.#810

Merged
rob-p merged 3 commits intoCOMBINE-lab:developfrom
emollier:more-verbose-test-failure
Nov 18, 2022
Merged

cmake/TestSalmonQuasi.cmake: more verbose test failure messages.#810
rob-p merged 3 commits intoCOMBINE-lab:developfrom
emollier:more-verbose-test-failure

Conversation

@emollier
Copy link
Copy Markdown
Contributor

@emollier emollier commented Nov 5, 2022

Greetings,

When trying to debug test failure, I ended up with an incomplete error messages which turned out to stem from a typo in the variable name, so I took the liberty to make the whole message a bit more verbose to help with the present and future debugging.

I thought you might be interested.
Have a nice day, :)
Étienne.

rob-p and others added 3 commits June 22, 2022 23:04
When trying to debug test failure, I ended up with an incomplete error
messages which turned out to stem from a typo in the variable name, so
I took the liberty to make the whole message a bit more verbose to
help with the present and future debugging.
@rob-p rob-p changed the base branch from master to develop November 18, 2022 02:16
@rob-p
Copy link
Copy Markdown
Collaborator

rob-p commented Nov 18, 2022

Thanks @emollier !

@rob-p rob-p merged commit 22db092 into COMBINE-lab:develop Nov 18, 2022
@emollier emollier deleted the more-verbose-test-failure branch December 18, 2022 17:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants