-
-
Notifications
You must be signed in to change notification settings - Fork 481
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
/bats/arg-parsing.bats: Add skipped test for checkout panic #616
Conversation
bats/arg-parsing.bats
Outdated
@@ -45,3 +45,13 @@ teardown() { | |||
skip "Can't chain modal arguments" | |||
dolt table import -fc test `batshelper 1pk5col-ints.csv` | |||
} | |||
|
|||
@test "dolt does not panic and returns error when empty string used with checkout" { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
dolt checkout with empty string returns error
bats/arg-parsing.bats
Outdated
@test "dolt does not panic and returns error when empty string used with checkout" { | ||
run dolt checkout "" | ||
skip "Panics when attempting to checkout empty string" [[ "$output" =~ "error: cannot checkout empty string" ]] || false | ||
[ $status -eq 2 ] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we've been going with [ $status -ne 0 ]
bats/arg-parsing.bats
Outdated
|
||
@test "dolt does not panic and returns error when empty string used with checkout" { | ||
run dolt checkout "" | ||
skip "Panics when attempting to checkout empty string" [[ "$output" =~ "error: cannot checkout empty string" ]] || false |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add a newline here.
bats/arg-parsing.bats
Outdated
|
||
run dolt checkout -b "" | ||
[[ "$output" =~ "error: cannot checkout empty string" ]] || false | ||
[ $status -eq 2 ] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
-ne 0
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
SHIP IT. And thanks.
No description provided.