Skip to content

Spec update 2025 11 18 - #1303

Merged
headius merged 15 commits into
ruby:masterfrom
headius:spec-update-2025-11-18
Nov 19, 2025
Merged

Spec update 2025 11 18#1303
headius merged 15 commits into
ruby:masterfrom
headius:spec-update-2025-11-18

Conversation

@headius

@headius headius commented Nov 19, 2025

Copy link
Copy Markdown
Contributor

Dry run of spec updating with latest instructions.

nobu and others added 15 commits November 18, 2025 18:19
It is already declared as:
> This behavior is slated to be removed in Ruby 4.0
that has not been obsolete.

Partially reverting ruby/ruby#15049.
Instead of an undocumented constant.
ruby/openssl#958 changed the common logic for
digest algorithm lookup:

 - If the argument is neither an OpenSSL::Digest instance nor a String,
   it is now implicitly converted to String with #to_str. This is
   consistent with algorithm name lookup logic in ruby/openssl for pkeys
   and ciphers.

 - If the name is not recognized, OpenSSL::Digest::DigestError is raised
   instead of RuntimeError.

Update the specs accordingly:

 - Remove specs that expect #to_str not to be called.

 - Relax regexps matching TypeError messages.

 - Expect OpenSSL::Digest::DigestError instead of RuntimeError for
   ruby/openssl 4.0.0 and later.
This reverts commit 5b6658a406b5f1c535aed4cb68e8e18a3cbabb81.

With a ruby spec fix.
@headius
headius merged commit c6fa667 into ruby:master Nov 19, 2025
13 checks passed
@headius

headius commented Nov 19, 2025

Copy link
Copy Markdown
Contributor Author

Ok what just happened... I did not merge that.

@headius

headius commented Nov 19, 2025

Copy link
Copy Markdown
Contributor Author

Crap, I know what it did. I ran into issues running the commands and was interrupted between mri and jruby, so I pushed what I had for MRI as a PR. I intended to continue pushing the remaining updates to that branch. But when I resumed with the jruby and truffleruby spec import, it switched my ruby/spec repo back to master. I did not realize this. When I pushed I thought I was pushing more to the PR branch but it pushed everything to master.

@eregon Not sure how you want to handle this. Specs have been synced only from impls to ruby/spec, not propgated back to the impls.

@headius
headius deleted the spec-update-2025-11-18 branch November 19, 2025 01:36
@eregon

eregon commented Nov 19, 2025

Copy link
Copy Markdown
Member

@eregon Not sure how you want to handle this. Specs have been synced only from impls to ruby/spec, not propgated back to the impls.

That's fine, it's always the first step, and needs to be merged/pushed-to-master before propagated back to the impls anyway.
LGTM.

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.

7 participants