Skip to content

Comments

chore: restore methods for downstream projects#926

Merged
wolf4ood merged 1 commit intoeclipse-edc:mainfrom
wolf4ood:chore/restore_methods
Feb 18, 2026
Merged

chore: restore methods for downstream projects#926
wolf4ood merged 1 commit intoeclipse-edc:mainfrom
wolf4ood:chore/restore_methods

Conversation

@wolf4ood
Copy link
Contributor

What this PR changes/adds

restore methods used in downstream projects

Why it does that

Briefly state why the change was necessary.

Further notes

List other areas of code that have changed but are not necessarily linked to the main feature. This could be method
signature changes, package declarations, bugs that were encountered and were fixed inline, etc.

Who will sponsor this feature?

Please @-mention the committer that will sponsor your feature.

Linked Issue(s)

Closes # <-- insert Issue number if one exists

Please be sure to take a look at the contributing guidelines and our etiquette for pull requests.

@wolf4ood wolf4ood requested a review from a team as a code owner February 18, 2026 09:38
@wolf4ood wolf4ood added the build label Feb 18, 2026
@wolf4ood wolf4ood merged commit 49c06ed into eclipse-edc:main Feb 18, 2026
19 of 20 checks passed
@wolf4ood wolf4ood deleted the chore/restore_methods branch February 18, 2026 10:18
.body().jsonPath();
}

public void createHolder(String participantContextId, String holderId, String holderDid, String holderName) {
Copy link
Member

Choose a reason for hiding this comment

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

generally speaking, to make the e2e more effective, we should have this methods to hit the http endpoints, and eventually using them for the IH e2e tests so they won't be deleted by accident :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Agree we already started doing the shift to expose http client testing api to use in e2e tests

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants