Skip to content
This repository has been archived by the owner on Nov 24, 2023. It is now read-only.

test: fix test isn't suitable for an unordered output (#1372) #1374

Closed

Conversation

ti-srebot
Copy link

@ti-srebot ti-srebot commented Jan 12, 2021

cherry-pick #1372 to release-2.0
You can switch your code base to this Pull Request by using git-extras:

# In dm repo:
git pr 1374

After apply modifications, you can push your change to this PR via:

git push git@github.com:ti-srebot/dm.git pr/1374:ti-srebot:release-2.0-de9de762ef8e

What problem does this PR solve?

follow up #1366

What is changed and how it works?

change to unorder match

Check List

Tests

  • Unit test

Code changes

Side effects

Related changes

  • Need to cherry-pick to the release branch

Signed-off-by: ti-srebot <ti-srebot@pingcap.com>
@ti-srebot ti-srebot added priority/unimportant Really minor change, requires approval from secondary reviewers status/PTAL This PR is ready for review. Add this label back after committing new changes type/cherry-pick This PR is just a cherry-pick (backport) labels Jan 12, 2021
@@ -93,7 +93,13 @@ func (s *testCheckerSuite) TestDumpPrivilegeChecking(c *tc.C) {
mock := s.initMockDB(c)
mock.ExpectQuery("SHOW GRANTS").WillReturnRows(sqlmock.NewRows([]string{"Grants for User"}).
AddRow("GRANT USAGE ON *.* TO 'haha'@'%'"))
<<<<<<< HEAD
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

expected statement, found '<<' (and 2 more errors)

@lance6716
Copy link
Collaborator

will manually cherry-pick to #1371

@lance6716 lance6716 closed this Jan 12, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
priority/unimportant Really minor change, requires approval from secondary reviewers status/PTAL This PR is ready for review. Add this label back after committing new changes type/cherry-pick This PR is just a cherry-pick (backport)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants