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 more specs for OpenSSL::Digest #1054

Merged
merged 9 commits into from
Oct 29, 2023
Merged

Conversation

herwinw
Copy link
Member

@herwinw herwinw commented Aug 20, 2023

The current specs are very limited

@herwinw
Copy link
Member Author

herwinw commented Aug 20, 2023

Only after pushing this change I noticed the Rubocop complaints about the style OpenSSL::Digest::SHA1.new being deprecated in favour of OpenSSL::Digest.new('SHA1'). I added an exclude in Rubocop for these lines, since we are testing this behaviour as well. I would be totally fine with removing these deprecated versions from the specs as well.

herwinw added a commit to herwinw/natalie that referenced this pull request Aug 22, 2023
Imported from ruby/spec#1054 and subject to
change.
herwinw added a commit to herwinw/natalie that referenced this pull request Aug 22, 2023
Imported from ruby/spec#1054 and subject to
change.
herwinw added a commit to herwinw/natalie that referenced this pull request Aug 24, 2023
Imported from ruby/spec#1054 and subject to
change.
They have been imported into the natalie folder to break a dependeny on
the change request of ruby-spec.
herwinw added a commit to herwinw/natalie that referenced this pull request Aug 29, 2023
Imported from ruby/spec#1054 and subject to
change.
They have been imported into the natalie folder to break a dependeny on
the change request of ruby-spec.
herwinw added a commit to herwinw/natalie that referenced this pull request Aug 30, 2023
Imported from ruby/spec#1054 and subject to
change.
They have been imported into the natalie folder to break a dependeny on
the change request of ruby-spec.
herwinw added a commit to herwinw/natalie that referenced this pull request Sep 23, 2023
Imported from ruby/spec#1054 and subject to
change.
They have been imported into the natalie folder to break a dependeny on
the change request of ruby-spec.
herwinw added a commit to herwinw/natalie that referenced this pull request Sep 24, 2023
Imported from ruby/spec#1054 and subject to
change.
They have been imported into the natalie folder to break a dependeny on
the change request of ruby-spec.
herwinw added a commit to herwinw/natalie that referenced this pull request Sep 30, 2023
Imported from ruby/spec#1054 and subject to
change.
They have been imported into the natalie folder to break a dependeny on
the change request of ruby-spec.
Copy link
Member

@andrykonchin andrykonchin left a comment

Choose a reason for hiding this comment

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

Looks good to me!

library/openssl/digest/shared/update.rb Outdated Show resolved Hide resolved
library/openssl/digest/reset_spec.rb Outdated Show resolved Hide resolved
library/openssl/digest/initialize_spec.rb Outdated Show resolved Hide resolved
library/openssl/digest/initialize_spec.rb Outdated Show resolved Hide resolved
library/openssl/digest/initialize_spec.rb Outdated Show resolved Hide resolved
library/openssl/digest/name_spec.rb Outdated Show resolved Hide resolved
library/openssl/digest/shared/update.rb Outdated Show resolved Hide resolved
library/openssl/digest/digest_length_spec.rb Outdated Show resolved Hide resolved
library/openssl/digest/digest_length_spec.rb Outdated Show resolved Hide resolved
library/openssl/digest/digest_length_spec.rb Outdated Show resolved Hide resolved
@herwinw
Copy link
Member Author

herwinw commented Oct 29, 2023

I think I've addressed all the issues in the latest version

@andrykonchin
Copy link
Member

Thank you!

@andrykonchin andrykonchin merged commit 09d73f4 into ruby:master Oct 29, 2023
10 checks passed
@herwinw herwinw deleted the openssl_digest branch October 30, 2023 05:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants