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#1018 > development/8.1] bugfix: S3C-2899 helper for v1 genMDParams() of master keys listing #1042

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 #1018.

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

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

Please always comment pull request #1018 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.
…mp/octopus/w/7.7/bugfix/S3C-2899-helperForListingAlgoGenMDParams
…into tmp/octopus/w/8.1/bugfix/S3C-2899-helperForListingAlgoGenMDParams
@bert-e bert-e merged commit b8c4ae4 into development/8.1 May 20, 2020
@bert-e bert-e deleted the w/8.1/bugfix/S3C-2899-helperForListingAlgoGenMDParams branch May 20, 2020 04:21
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