Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add managed-access integration test for verbatim manifests #6098

Closed
nadove-ucsc opened this issue Mar 26, 2024 · 3 comments
Closed

Add managed-access integration test for verbatim manifests #6098

nadove-ucsc opened this issue Mar 26, 2024 · 3 comments
Assignees
Labels
+ [priority] High debt [type] A defect incurring continued engineering cost enh [type] New feature or request manifests [subject] Generation and contents of manifests no demo [process] Not to be demonstrated at the end of the sprint orange [process] Done by the Azul team test [subject] Unit and integration test code

Comments

@nadove-ucsc
Copy link
Contributor

nadove-ucsc commented Mar 26, 2024

Add a managed access subtest to the IT that generates three small verbatim manifests. The first should use filters that include files from both public and private sources and be requested using service account credentials. The second should use the same filters, but the default credentials. The third should use filters that include only the files from the private sources used by the first two manifests. Assert that the third manifest is the set difference of the first manifest and the second, i.e., M3 = M1 \ M2.

[edit: @hannes-ucsc] The test should cover all verbatim formats currently implemented.

@nadove-ucsc nadove-ucsc added the orange [process] Done by the Azul team label Mar 26, 2024
@hannes-ucsc hannes-ucsc changed the title Add managed access integration test for verbatim manifest Add managed-access integration test for verbatim manifest Mar 27, 2024
@achave11-ucsc
Copy link
Member

Assignee to provide description.

@nadove-ucsc nadove-ucsc removed their assignment Mar 28, 2024
@achave11-ucsc
Copy link
Member

Assignee to include this in the PR #6029 for #6028.

@achave11-ucsc achave11-ucsc added enh [type] New feature or request debt [type] A defect incurring continued engineering cost test [subject] Unit and integration test code manifests [subject] Generation and contents of manifests + [priority] High labels Mar 28, 2024
@hannes-ucsc
Copy link
Member

Assignee to include this in the PR #6029 for #6028.

Decided not to include it.

@hannes-ucsc hannes-ucsc changed the title Add managed-access integration test for verbatim manifest Add managed-access integration test for verbatim manifests Apr 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
+ [priority] High debt [type] A defect incurring continued engineering cost enh [type] New feature or request manifests [subject] Generation and contents of manifests no demo [process] Not to be demonstrated at the end of the sprint orange [process] Done by the Azul team test [subject] Unit and integration test code
Projects
None yet
Development

No branches or pull requests

3 participants