Re: [PR] only initialize digestUtilsTest when necessary to speed up test MessageDigestAlgorithmsTest [commons-codec]

2024-04-14 Thread via GitHub
garydgregory closed pull request #262: only initialize digestUtilsTest when necessary to speed up test MessageDigestAlgorithmsTest URL: https://github.com/apache/commons-codec/pull/262 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Git

Re: [PR] only initialize digestUtilsTest when necessary to speed up test MessageDigestAlgorithmsTest [commons-codec]

2024-04-14 Thread via GitHub
garydgregory commented on PR #262: URL: https://github.com/apache/commons-codec/pull/262#issuecomment-2054177574 Closing; see my previous comment. -- 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

Re: [PR] only initialize digestUtilsTest when necessary to speed up test MessageDigestAlgorithmsTest [commons-codec]

2024-04-03 Thread via GitHub
garydgregory commented on PR #262: URL: https://github.com/apache/commons-codec/pull/262#issuecomment-2034359416 -1: This is not an issue worth making the test more convoluted, especially with the odd code pattern of calling tear down and set up methods manually. -- This is an automated m

[PR] only initialize digestUtilsTest when necessary to speed up test MessageDigestAlgorithmsTest [commons-codec]

2024-04-02 Thread via GitHub
TestBoost opened a new pull request, #262: URL: https://github.com/apache/commons-codec/pull/262 For the test methods in the test class MessageDigestAlgorithmsTest, not all test methods modify the contents of shared fields. We can just initialize `digestUtilsTest` once before all the test m