eclipse / kuksa.val

kuksa.val
Apache License 2.0
89 stars 52 forks source link

Add authorization integration test #708

Closed lukasmittag closed 6 months ago

lukasmittag commented 7 months ago

This adds integration tests for authorizing against databroker and checking for the right permissions.

675 needs to be merged first because it is based on the new client lib. Therefore also some changes in the existing integration test took place.

lukasmittag commented 7 months ago

@sophokles73 if you want to take a look as well

erikbosch commented 7 months ago

I looked at the Dash output and saw this. Do we need to do anything?


[main] INFO Querying Eclipse Foundation for license data for 208 items.
[main] INFO Found 46 items.
[main] INFO Querying ClearlyDefined for license data for 162 items.
[main] INFO Found 162 items.
[main] INFO License information could not be automatically verified for the following content:
[main] INFO 
[main] INFO crate/cratesio/-/kuksa/0.1.0
[main] INFO 
[main] INFO This content is either not correctly mapped by the system, or requires review.
sophokles73 commented 7 months ago

Do we need to do anything?

The Dash tool tries to find licensing information for dependencies on Clearly Defined and the Eclipse Foundation systems. The kuksa crate seems to not have been analyzed yet, neither by CD nor the EF.

What you could do is run the Dash tool with the --review switch to automatically create a corresponding review request with the EF. You would then need to provide the source location manually in the issue created in the EF GitLab instance for the review.

However, before you do so, you might want to think about