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
Add dolt_commit error check when autocommit is off #1074
Conversation
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.
Not quite the right way to solve this.
It's OK for the error message to mention @@autocommit, but the business logic shouldn't depend on that detail.
[[ "$output" =~ "one_pk" ]] || false | ||
|
||
# check that dolt_commit works properly when autocommit is on | ||
run dolt sql -q "SELECT DOLT_COMMIT('-a', '-m', 'Commit1')" |
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.
Also should have a test of the negative case, where there are no staged changes / no changes to stage. With and without -a.
Fails loudly when autocommit is off for dolt_commit.