Skip to content

Conversation

@saudkhan116
Copy link

WHAT

  • Added Bruno collection to enable EDC API testing
  • Added a setup README

WHY

To enable API testing for the data exchange

FURTHER NOTES

Closes # <-- CXAP4-58

@saudkhan116 saudkhan116 requested review from dominikgoertz and removed request for dominikgoertz February 9, 2026 14:02
@saudkhan116 saudkhan116 changed the base branch from develop to main February 9, 2026 14:03
@eschrewe
Copy link

Hi Saud,

just saw this. As you know, I informed you personally via Email on last friday, that this feature branch is nearing completion, which also contains a comprehensive docker compose setup and a bruno collection. In that email, I also asked you to provide feedback (which I haven't received so far...).

So the point is: you were aware, that a Bruno collection was already in preparation when you created this PR yesterday, which is something that I don't quite understand...

From my point of view, this PR is not needed and should be closed.

BR

Ernst

@saudkhan116
Copy link
Author

Hi Saud,

just saw this. As you know, I informed you personally via Email on last friday, that this feature branch is nearing completion, which also contains a comprehensive docker compose setup and a bruno collection. In that email, I also asked you to provide feedback (which I haven't received so far...).

So the point is: you were aware, that a Bruno collection was already in preparation when you created this PR yesterday, which is something that I don't quite understand...

From my point of view, this PR is not needed and should be closed.

BR

Ernst

Hi @eschrewe,

Yes you are right. If the bruno collection from your changes is still testable, then we can close this PR. However, the collection included in this PR is based on upstream tractusx-EDC version and also currently being used and tested by the Construct-X federated services.

@saudkhan116
Copy link
Author

On the other hand, I think, this PR could be considered as its easier to enable local API testing with the current shared services setup, and without enabling the other dependencies such as identityhub and issuance services (until their docker images are publicly available).

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.

2 participants