[GEODE-9885] StringsDUnitTest.givenBucketsMoveDuringAppend_thenDataIsNotLost fails with duplicated append

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.15.0
• Fix Version/s: 1.15.0
• Component/s: redis
• Labels: pull-request-available testing
• Assignee: Donal Evans
• Reporter: Ray Ingles
• Created: 09/Dec/21 20:31
• Updated: 22/Jun/22 20:46
• Resolved: 21/Jan/22 23:28

Description

The test appends a lot of strings to a key. It wound up adding (at least one) extra string to the stored string:

{{java.util.concurrent.ExecutionException: java.lang.AssertionError: unexpected -

{append0}

-key-3-27680 at index 27681 iterationCount=61995 in string}}

The string "{append0}-key-3-27680" appeared twice in sequence.

Additional to this failure, the test should be modified to produce a more useful failure message. The current assertion prints the entire String upon failure, which can contain upwards of 50,000 repeats of the "{append0}-key-3*" String, making the output large and unreadable. Consider using an assertion with .withFailureMessage() to produce a more useful error message rather than the currently used Assert.fail().

Issue Links

• links to: GitHub Pull Request #7290