Fixed the endTime not modified bug to pass sonar cloud tests.

[Quanzzzz]

Fixed the bug of endTime not modified by plusNanos() by reassigning the modified result to itself;
Added corresponding test to test if the change works;
Modified the stub of mockClock.instant() to make it pass all the tests.

Testing

The reason why stubbing mockClock.instant() is moved ahead of creating timer is that while creating the timer there are two calls of mockClock.instant(). In original version, these two calls were ignored so the startTime and endTime were null and the new added test will fail to this. After changing the position of the stubbing and adding two more values to the stub, all the tests are able to pass.

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)

Checklist

• I have read the CONTRIBUTING document
• Local run of mvn install succeeds
• My code follows the code style of this project
• My change requires a change to the Javadoc documentation
• I have updated the Javadoc documentation accordingly
• I have read the README document
• I have added tests to cover my changes
• All new and existing tests passed
• A short description of the change has been added to the CHANGELOG
• My change is to implement 1.11 parity feature and I have updated LaunchChangelog

License

• I confirm that this pull request can be released under the Apache 2 license

[sonarqubecloud]

SonarCloud Quality Gate failed.

0 Bugs
0 Vulnerabilities (and 0 Security Hotspots to review)
0 Code Smells

0.0% Coverage
0.0% Duplication

[millems]

Not sure I like this class or how it's implemented, but that's outside of the scope of this review.