Skip to content

Conversation

@dogukanteber
Copy link
Contributor

What this PR does:
This PR adds unit test to resolveConflicts function

Which issue(s) this PR fixes:
Fixes #5079

Checklist

  • Tests updated
  • Documentation added
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]

Signed-off-by: Doğukan Teber <[email protected]>
@dogukanteber dogukanteber requested a review from friedrichg March 30, 2023 19:48
@alvinlin123 alvinlin123 merged commit fbc4a76 into cortexproject:master Mar 31, 2023
yeya24 pushed a commit to yeya24/cortex that referenced this pull request Apr 28, 2023
* Add unit test for resolveConflicts function

Signed-off-by: Doğukan Teber <[email protected]>

* Clean up the test logic

Signed-off-by: Doğukan Teber <[email protected]>

---------

Signed-off-by: Doğukan Teber <[email protected]>
Signed-off-by: Ben Ye <[email protected]>
alexqyle pushed a commit to alexqyle/cortex that referenced this pull request May 2, 2023
* Add unit test for resolveConflicts function

Signed-off-by: Doğukan Teber <[email protected]>

* Clean up the test logic

Signed-off-by: Doğukan Teber <[email protected]>

---------

Signed-off-by: Doğukan Teber <[email protected]>
Signed-off-by: Alex Le <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

resolveConflicts is not covered by unit tests

3 participants