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

INTEGRATION [PR#1031 > development/7.7] bugfix: S3C-2899 implement v1 format for DelimiterVersions listing #1047

Merged

Conversation

bert-e
Copy link
Contributor

@bert-e bert-e commented May 19, 2020

This pull request has been created automatically.
It is linked to its parent pull request #1031.

Do not edit this pull request directly.
If you need to amend/cancel the changeset on branch
w/7.7/bugfix/S3C-2899-vformatV1delimiterVersions, please follow this
procedure:

 $ git fetch
 $ git checkout w/7.7/bugfix/S3C-2899-vformatV1delimiterVersions
 $ # <amend or cancel the changeset by _adding_ new commits>
 $ git push origin w/7.7/bugfix/S3C-2899-vformatV1delimiterVersions

Please always comment pull request #1031 instead of this one.

jonathan-gramain and others added 3 commits May 19, 2020 16:44
New helper function to convert listing params from v0 to v1, when a
listing of master keys is requested. This logic is shared between
DelimiterMaster and MPU listing, hence a shared helper is useful.

Also, update the test function performListing to prepare for v1
testing of listing algos, by adding the vFormat parameter. Also check
that getMDParams() returns a valid object to enhance coverage.
Implement the v1 versioning key format for DelimiterVersions listing
method, in addition to v0.

Enhance existing unit tests to check the result of getMDParams()
@bert-e bert-e merged commit 0d49eff into development/7.7 May 21, 2020
@bert-e bert-e deleted the w/7.7/bugfix/S3C-2899-vformatV1delimiterVersions branch May 21, 2020 22:42
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