Skip to content

Update architecture docs for unified test provider - #3313

Merged
olamy merged 6 commits into
apache:masterfrom
sebtiem:update-architecture-docs-for-unified-provider
Mar 17, 2026
Merged

olamy merged 6 commits into
apache:masterfrom
sebtiem:update-architecture-docs-for-unified-provider

Conversation

@sebtiem

@sebtiem sebtiem commented Mar 9, 2026

Copy link
Copy Markdown
Contributor

@olamy Following this checklist to help us incorporate your
contribution quickly and easily:

  • Each commit in the pull request should have a meaningful subject line and body.
  • Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
  • Run mvn clean install to make sure basic checks pass. A more thorough check will
    be performed on your pull request automatically.
  • You have run the integration tests successfully (mvn -Prun-its clean install).

If your pull request is about ~20 lines of code you don't need to sign an
Individual Contributor License Agreement if you are unsure
please ask on the developers list.

To make clear that you license your contribution under
the Apache License Version 2.0, January 2004
you have to acknowledge this by using the following check-box.

sebtiem added 5 commits March 9, 2026 20:24
Signed-off-by: Sebastian Tiemann <setie@mailbox.org>
…ormation

Signed-off-by: Sebastian Tiemann <setie@mailbox.org>
Signed-off-by: Sebastian Tiemann <setie@mailbox.org>
…tures to architecture.md

Signed-off-by: Sebastian Tiemann <setie@mailbox.org>
Signed-off-by: Sebastian Tiemann <setie@mailbox.org>
@olamy olamy self-assigned this Mar 10, 2026
@olamy olamy added the documentation Improvements or additions to documentation label Mar 10, 2026
@olamy olamy added this to the 3.6.0.M1 milestone Mar 10, 2026
@olamy
olamy requested a review from sparsick March 11, 2026 07:49

@olamy olamy left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think with the change in 3.6.x we will not have dedicated provider module for each test framework.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

correct. not anymore true.


| Provider | Module | Test framework | Key classes |
|----------|--------|---------------|-------------|
| **JUnit 3 + POJO** | `surefire-junit3` | JUnit 3.x, plain POJOs | `JUnit3Provider`, `PojoTestSetExecutor` |

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I would not delete the table, but move it to subchapter 3.5

Signed-off-by: Sebastian Tiemann <setie@mailbox.org>
@olamy
olamy merged commit 498bb7f into apache:master Mar 17, 2026
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants