Skip to content

Only run bats diagnostics on failure#24

Merged
larsoner merged 14 commits intomasterfrom
peternewman-bats-on-failure
Feb 2, 2021
Merged

Only run bats diagnostics on failure#24
larsoner merged 14 commits intomasterfrom
peternewman-bats-on-failure

Conversation

@peternewman
Copy link
Copy Markdown
Collaborator

No description provided.

@peternewman peternewman marked this pull request as ready for review February 2, 2021 01:56
@peternewman
Copy link
Copy Markdown
Collaborator Author

@peternewman peternewman requested a review from larsoner February 2, 2021 01:57
@larsoner larsoner merged commit 6de8633 into master Feb 2, 2021
@larsoner
Copy link
Copy Markdown
Member

larsoner commented Feb 2, 2021

I don't totally understand BATS but looks reasonable to me!

@larsoner larsoner deleted the peternewman-bats-on-failure branch February 2, 2021 12:43
@peternewman
Copy link
Copy Markdown
Collaborator Author

It's like pytest, but for shell commands, only without any of the useful diagnostics when the tests break to see the value of the two sides of the equality test for example, hence this workaround.

@peternewman
Copy link
Copy Markdown
Collaborator Author

I've just realised this isn't quite as simple as it should be, but probably still better than nothing!

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