You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@maven.apache.org by GitBox <gi...@apache.org> on 2022/04/26 21:41:55 UTC

[GitHub] [maven-surefire] zoltanmeze commented on pull request #529: [SUREFIRE-2076] BufferOverflowException when encoding message with null runMode

zoltanmeze commented on PR #529:
URL: https://github.com/apache/maven-surefire/pull/529#issuecomment-1110276947

   > After the #518 is merged, you have to rebase this PR. Then we will run the CI again.
   
   Rebased onto latest master.
   
   > #518 was merged, let's concentrate on this one.
   
   The issue should be easily reproducible with  [AbstractStreamEncoderTest#testStdErrStreamEmptyMessageNullRunMode](https://github.com/apache/maven-surefire/pull/529/files#diff-d64bde0920c0b26eb5bec21735b15e8f57e2ca8b6fbd1b387f2f47ccf7e2d5d3R1255). Test implemented similar way as `testStdErrStreamEmptyMessageNullTestId` from [SUREFIRE-2056](https://issues.apache.org/jira/browse/SUREFIRE-2056).
   
   Expected length also changed in [AbstractStreamEncoderTest#shouldComputeStreamPreemptiveLength
   ](https://github.com/apache/maven-surefire/pull/529/files#diff-1b8c3d63f48dfb2501c224494ef498d3613047545954ddfd7606f4163a7d8ad0R140) in cases where runMode is set to null, in those cases length is now greater by 3. Also included how the output should look like (without testId) in the comments.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@maven.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org