eclipse-edc / MinimumViableDataspace

Guidance on documentation, scripts and integration steps on using the EDC project results
Apache License 2.0
58 stars 167 forks source link

feat: extract response ids in postman collection #352

Closed nikschul closed 3 weeks ago

nikschul commented 1 month ago

What this PR changes/adds

The functionality to carry over variables from previous responses to following request if applicable. This should make it easier for the user to execute the chain of requests necessary to download data from the other dataspace.

Why it does that

Postman pre- and post-request scripts were added to accomplish this behavior. Also a pre-request check for the respective requests is added to make sure the variables carried over are defined before executing the request

Further notes

Linked Issue(s)

Closes #351

paullatzelsperger commented 1 month ago

also, you should rebase onto main

nikschul commented 3 weeks ago

@paullatzelsperger I've added some tests for the collection which are executed in a new GitHub action workflow "newman_test". Also I had to add an additional terraform setup step in all workflow scrips, as the workflows were failing this morning with "terraform: command not found". I hope this is ok and does not break anything else.

nikschul commented 3 weeks ago

Here is one example of a failing test run, even though I did not change anything regarding this test: https://github.com/eclipse-edc/MinimumViableDataspace/actions/runs/11252232440/job/31284932435

paullatzelsperger commented 3 weeks ago

@paullatzelsperger I've added some tests for the collection which are executed in a new GitHub action workflow "newman_test". Also I had to add an additional terraform setup step in all workflow scrips, as the workflows were failing this morning with "terraform: command not found". I hope this is ok and does not break anything else.

huh? that is .... very interesting. stumbled across the same thing too.