-
Notifications
You must be signed in to change notification settings - Fork 200
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
Snap 2698 ignoring failing tests in compatibilityTests #1194
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.
One query. Looks good otherwise.
extends StatisticsCollectionSuite | ||
with SharedSnappySessionContext { | ||
|
||
} |
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.
Do all tests in this suite fail? Won't it be better to keep the class around and mark all as ignored so that we do not miss fixing these.
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.
There are duplicates. The 2nd one got created by mistake. Hence removing it
1: SnappyStattisticsCollectionSuite and
2: SnappyStatisticsCollectionSuite
|
||
override protected def test(testName: String, testTags: Tag*)(testFun: => Unit) = { | ||
if (!excluded.contains(testName)) { | ||
super.test(testName, testTags: _*)(testFun) | ||
if (ignored.contains(testName)){ |
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.
space before opening brace
Changes proposed in this pull request
Ignoring failing tests in compatibilityTests. These tests are tracked seperately as subtasks in https://jira.snappydata.io/browse/SNAP-2263
Patch testing
Ran precheckin and check on compatibilityTests
ReleaseNotes.txt changes
No
Other PRs
No