Skip to content

Conversation

reaperhulk
Copy link
Member

Also restructures the tests. Refs #184

This was referenced Oct 30, 2013
@alex
Copy link
Member

alex commented Oct 31, 2013

I'm not sure stuff is really fully consistent under this patch, there seems to be ciphers/Camellia/, ciphers/AES/KAT/, and hashes/SHABYTE/. I think we should try to tend towards only having one form, not sure what the components should be, but we should use algorithm/origin consistently.

@reaperhulk
Copy link
Member Author

SHABYTE should definitely be changed (and split up for SHA-1/SHA-2 family). I was unsure what to do about the KAT/MMT vectors for AES so I punted on that one at the time. We could split them up by mode, which would be a bit nicer.

@alex
Copy link
Member

alex commented Oct 31, 2013

Going by mode for ciphers sounds like a great idea.

On Thu, Oct 31, 2013 at 9:37 AM, Paul Kehrer notifications@github.comwrote:

SHABYTE should definitely be changed (and split up for SHA-1/SHA-2
family). I was unsure what to do about the KAT/MMT vectors for AES so I
punted on that one at the time. We could split them up by mode, which would
be a bit nicer.


Reply to this email directly or view it on GitHubhttps://github.com//pull/198#issuecomment-27502403
.

"I disapprove of what you say, but I will defend to the death your right to
say it." -- Evelyn Beatrice Hall (summarizing Voltaire)
"The people's good is the highest law." -- Cicero
GPG Key fingerprint: 125F 5C67 DFE9 4084

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

test_OpenSSL doesn't seem like a very good name anymore? Probably this should just be test_{mode}? That's at least what TestAESCTR.test_OpenSSL appears to have ended up as. Maybe they should even be moved to a single TestCamellia class?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think that makes sense. We could also collapse the test classes for the other ciphers to be consistent.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Want to rename these to be test_{mode} and collapse in a separate pull request?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Up to you since you're reviewing. I'm fine with a separate PR.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, lets rename the methods here, and then I'll merge it, and we can do the collapse in a separate pull request.

dreid added a commit that referenced this pull request Oct 31, 2013
Restructure vectors into ciphers/hashes/hmac
@dreid dreid merged commit 6adaf27 into pyca:master Oct 31, 2013
joerichter-stash pushed a commit to kiwigrid/cryptography that referenced this pull request Nov 15, 2017
Remove the out-of-date rationale document.
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 11, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants