TCK tests for sorting based on expressions#1497
Conversation
Co-authored-by-AI: IBM Bob 1.0.3
| @Assertion(id = "829", strategy = """ | ||
| Sort based on a navigable expression. | ||
| """) | ||
| public void testSortByNavigableExpression() { |
There was a problem hiding this comment.
how about use: shouldBe...
There was a problem hiding this comment.
how about use: shouldBe...
I'd rather not change just this one test. Hundreds of other tests in the TCK all start with test, including all of the other test methods within this class.
There was a problem hiding this comment.
If we decide to rename tests from test* to shouldBe*, let's do that in a separate PR that can cover all occurrences.
|
How about we move it in either as Nested classes? |
I don't understand what your question means. Are you asking me to move the 3 new test methods that cover sorting by expressions into their own class, and you are suggesting that to be a nested class? |
|
I added commit e37b116 to move the new tests to a class of their own. Is that better? |
2 participants
TCK tests for sorting based on expressions: