yykaue opened a new pull request, #6917: URL: https://github.com/apache/shenyu/pull/6917
Fixes #6816 ## What this PR does Explicitly initialize the sign verification delay in `SignServiceVersionOneTest` and `SignServiceVersionTwoTest`. Previously, both test classes constructed `ComposableSignService` directly with `new`. As a result, Spring did not process its `@Value` field, leaving `delay` at the JVM default value of `0`. This could make success-expecting tests fail intermittently when timestamp creation and verification crossed a minute boundary. This PR: - Uses an explicit `DELAY = 5` test constant. - Injects the delay into manually constructed `ComposableSignService` instances with `ReflectionTestUtils`. - Removes the ineffective `@Value` fields from the test classes. - Adds regression tests verifying that valid signatures from one minute earlier succeed. - Updates overdue tests to use the same configured delay. ## Tests ```text ./mvnw -pl shenyu-plugin/shenyu-plugin-security/shenyu-plugin-sign test Tests run: 41, Failures: 0, Errors: 0, Skipped: 0 -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
