Skip to content

Feature/wicket 7170 followup improve tests#1312

Closed
hosea wants to merge 10 commits intoapache:masterfrom
hosea:feature/wicket-7170-followup-improve-tests
Closed

Feature/wicket 7170 followup improve tests#1312
hosea wants to merge 10 commits intoapache:masterfrom
hosea:feature/wicket-7170-followup-improve-tests

Conversation

@hosea
Copy link
Contributor

@hosea hosea commented Nov 27, 2025

This is a follow-up-PR addressing only the tests of AnnotProxyFieldValueFactory.
Tests were completly updated and rewritten during Wicket-7170 to ensure better test coverage.
The PR consists of several commits, that do a step by step migration:

  • Migrated one test after the other from AnnotProxyFieldValueFactoryTest to AnnotProxyFieldValueFactoryDefaultCandidateTest
  • After all tests are migrated: clean up empty tests (abstract AnnotProxyFieldValueFactoryTest, JakartaInjectAnnotProxyFieldValueFactoryTest, SpringBeanAnnotProxyFieldValueFactoryTest), and unused beans / interfaces
  • Rename AnnotProxyFieldValueFactoryDefaultCandidateTest to AnnotProxyFieldValueFactoryTest

{
Assertions.assertThrows(IllegalArgumentException.class, () -> new AnnotProxyFieldValueFactory(null));
}

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This test will be added later again when migrating corresponding test from AnnotProxyFieldFactoryTest...

@hosea hosea closed this Nov 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant