Skip to content
This repository has been archived by the owner on May 24, 2023. It is now read-only.

Integration tests for authorization #30

Merged
merged 7 commits into from
Mar 6, 2019
Merged

Integration tests for authorization #30

merged 7 commits into from
Mar 6, 2019

Conversation

csomh
Copy link
Contributor

@csomh csomh commented Mar 6, 2019

Add integration tests to cover authorization, and adjust the integration test env and the other tests to match how OMPS works.

@csomh csomh requested a review from MartinBasti March 6, 2019 10:29
Hunor Csomortáni added 4 commits March 6, 2019 11:30
But don't enable it, yet.

Signed-off-by: Hunor Csomortáni <csomh@redhat.com>
It's shorter to read and type.

Signed-off-by: Hunor Csomortáni <csomh@redhat.com>
Signed-off-by: Hunor Csomortáni <csomh@redhat.com>
OMPS changed behaviour:
- organizations are not configured in the service
- if a namespace does not exist, or cannot be accessed with
  the token given, a QuayPackageError is raised.

Signed-off-by: Hunor Csomortáni <csomh@redhat.com>
Hunor Csomortáni added 3 commits March 6, 2019 13:28
Signed-off-by: Hunor Csomortáni <csomh@redhat.com>
Signed-off-by: Hunor Csomortáni <csomh@redhat.com>
OMPS has no more knowledge about namespaces used in Quay,
so organization is removed from the OMPS class of the
integration test env.

The namespace used for testing becomes a constant.

Signed-off-by: Hunor Csomortáni <csomh@redhat.com>
@MartinBasti MartinBasti merged commit 1d89b70 into release-engineering:master Mar 6, 2019
@csomh csomh deleted the int-test-authorization branch March 6, 2019 12:41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants