[GitHub] [commons-lang] garydgregory commented on pull request #849: [LANG-1662] Create methods on ReflectionToStringBuilder to reflect only select fields

[GitBox <gi...@apache.org>]

garydgregory commented on PR #849:
URL: https://github.com/apache/commons-lang/pull/849#issuecomment-1175237551

   When I apply the patch locally and run the Maven build from the command line, I get failures because it seems like the test order matters:
   
   ```
   [ERROR] org.apache.commons.lang3.builder.ToStringBuilderTest.testDoubleArray  Time elapsed: 0.001 s  <<< FAILURE!
   org.opentest4j.AssertionFailedError: Expected null, actual: {org.apache.commons.lang3.builder.ReflectionToStringBuilderIncludeTest$TestFeature@58e9cc9a=null, org.apache.commons.lang3.builder.ReflectionToStringBuil
   derIncludeTest$TestFeature@7dca6126=null} ==> expected: <null> but was: <{org.apache.commons.lang3.builder.ReflectionToStringBuilderIncludeTest$TestFeature@58e9cc9a=null, org.apache.commons.lang3.builder.Reflectio
   nToStringBuilderIncludeTest$TestFeature@7dca6126=null}>
           at org.junit.jupiter.api.AssertionUtils.fail(AssertionUtils.java:55)
           at org.junit.jupiter.api.AssertNull.failNotNull(AssertNull.java:54)
           at org.junit.jupiter.api.AssertNull.assertNull(AssertNull.java:38)
   ```


-- 
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@commons.apache.org

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