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

Write specs for new Ruby 3.3 features and changes #1216

Open
30 of 51 tasks
andrykonchin opened this issue Nov 15, 2024 · 3 comments
Open
30 of 51 tasks

Write specs for new Ruby 3.3 features and changes #1216

andrykonchin opened this issue Nov 15, 2024 · 3 comments

Comments

@andrykonchin
Copy link
Member

andrykonchin commented Nov 15, 2024

ruby/spec already contains some specs for 3.3, but we should aim to cover all new features and important changes.
This will improve the test coverage of these features (and maybe discover a few bugs along the way), allow other Ruby implementations to implement the changes faster with more confidence and document clearly the new behavior.

The new specs should be within a version guard block:

ruby_version_is "3.3" do
  # New specs
end

NOTE: https://rubyreferences.github.io/rubychanges/3.3.html gives more details for many features and changes.

From https://github.com/ruby/ruby/blob/master/doc/NEWS/NEWS-3.3.0.md:

NEWS for Ruby 3.3.0

Core classes updates

Note: We're only listing outstanding class updates.

Array

  • Array#pack now raises ArgumentError for unknown directives. [Bug #19150]

Dir

  • Dir.for_fd added for returning a Dir object for the directory specified
    by the provided directory file descriptor. [Feature #19347]
  • Dir.fchdir added for changing the directory to the directory specified
    by the provided directory file descriptor. [Feature #19347]
  • Dir#chdir added for changing the directory to the directory specified by
    the provided Dir object. [Feature #19347]

Encoding

  • Encoding#replicate has been removed, it was already deprecated. [Feature #18949]

Fiber

fiber = Fiber.new do
  while true
    puts "Yielding..."
    Fiber.yield
  end
ensure
  puts "Exiting..."
end

fiber.resume
# Yielding...
fiber.kill
# Exiting...

MatchData

  • MatchData#named_captures now accepts optional symbolize_names
    keyword. [Feature #19591]

Module

  • Module#set_temporary_name added for setting a temporary name for a
    module. [Feature #19521]

ObjectSpace::WeakKeyMap

  • New core class to build collections with weak references.
    The class use equality semantic to lookup keys like a regular hash,
    but it doesn't hold strong references on the keys. [Feature #18498]

ObjectSpace::WeakMap

  • ObjectSpace::WeakMap#delete was added to eagerly clear weak map
    entries. [Feature #19561]

Proc

  • Now Proc#dup and Proc#clone call #initialize_dup and #initialize_clone
    hooks respectively. [Feature #19362]

Process

  • New Process.warmup method that notify the Ruby virtual machine that the boot sequence is finished,
    and that now is a good time to optimize the application. This is useful
    for long-running applications. The actual optimizations performed are entirely
    implementation-specific and may change in the future without notice. [Feature #18885]

Process::Status

  • Process::Status#& and Process::Status#>> are deprecated. [Bug #19868]

Range

  • Range#reverse_each can now process beginless ranges with an Integer endpoint. [Feature #18515]
  • Range#reverse_each now raises TypeError for endless ranges. [Feature #18551]
  • Range#overlap? added for checking if two ranges overlap. [Feature #19839]

Refinement

  • Add Refinement#target as an alternative of Refinement#refined_class.
    Refinement#refined_class is deprecated and will be removed in Ruby
    3.4. [Feature #19714]

Regexp

  • The cache-based optimization now supports lookarounds and atomic groupings. That is, match
    for Regexp containing these extensions can now also be performed in linear time to the length
    of the input string. However, these cannot contain captures and cannot be nested. [Feature #19725]

String

  • String#unpack now raises ArgumentError for unknown directives. [Bug #19150]
  • String#bytesplice now accepts new arguments index/length or range of the
    source string to be copied. [Feature #19314]

Thread::Queue

  • Thread::Queue#freeze now raises TypeError. [Bug #17146]

Thread::SizedQueue

  • Thread::SizedQueue#freeze now raises TypeError. [Bug #17146]

Time

  • Time.new with a string argument became stricter. [Bug #19293]
Time.new('2023-12-20')
#  no time information (ArgumentError)

TracePoint

  • TracePoint supports rescue event. When the raised exception was rescued,
    the TracePoint will fire the hook. rescue event only supports Ruby-level
    rescue. [Feature #19572]

Socket

  • Socket#recv and Socket#recv_nonblock returns nil instead of an empty string on closed
    connections. Socket#recvmsg and Socket#recvmsg_nonblock returns nil instead of an empty packet on closed
    connections. [Bug #19012]

  • Name resolution such as Socket.getaddrinfo, Socket.getnameinfo, Addrinfo.getaddrinfo, etc.
    can now be interrupted. [Feature #19965]

Random

  • Random::Formatter#alphanumeric is extended to accept optional chars
    keyword argument. [Feature #18183]

Compatibility issues

  • Subprocess creation/forking via the following file open methods is deprecated. [Feature #19630]

    • Kernel#open
    • URI.open
    • IO.binread
    • IO.foreach
    • IO.readlines
    • IO.read
    • IO.write
  • When given a non-lambda, non-literal block, Kernel#lambda with now raises
    ArgumentError instead of returning it unmodified. These usages have been
    issuing warnings under the Warning[:deprecated] category since Ruby 3.0.0.
    [Feature #19777]

  • it calls without arguments in a block with no ordinary parameters are
    deprecated. it will be a reference to the first block parameter in Ruby 3.4.
    [Feature #18980]

  • Error message for NoMethodError have changed to not use the target object's #inspect
    for efficiency, and says "instance of ClassName" instead. [Feature #18285]

    ([1] * 100).nonexisting
    # undefined method `nonexisting' for an instance of Array (NoMethodError)
  • Now anonymous parameters forwarding is disallowed inside a block
    that uses anonymous parameters. [Feature #19370]

C API updates

rb_postponed_job updates

  • New APIs and deprecated APIs (see comments for details)
    • added: rb_postponed_job_preregister()
    • added: rb_postponed_job_trigger()
    • deprecated: rb_postponed_job_register() (and semantic change. see below)
    • deprecated: rb_postponed_job_register_one()
  • The postponed job APIs have been changed to address some rare crashes.
    To solve the issue, we introduced new two APIs and deprecated current APIs.
    The semantics of these functions have also changed slightly; rb_postponed_job_register
    now behaves like the once variant in that multiple calls with the same
    func might be coalesced into a single execution of the func
    [Feature #20057]

Some updates for internal thread event hook APIs

  • rb_profile_thread_frames() is introduced to get a frames from
    a specific thread.
    [Feature #10602]

  • rb_data_define() is introduced to define Data. [Feature #19757]

  • rb_ext_resolve_symbol() is introduced to search a function from
    extension libraries. [Feature #20005]

IO related updates:

  • The details of rb_io_t will be hidden and deprecated attributes
    are added for each members. [Feature #19057]
  • rb_io_path(VALUE io) is introduced to get a path of io.
  • rb_io_closed_p(VALUE io) to get opening or closing of io.
  • rb_io_mode(VALUE io) to get the mode of io.
  • rb_io_open_descriptor() is introduced to make an IO object from a file
    descriptor.
@herwinw
Copy link
Member

herwinw commented Nov 16, 2024

A couple of these items have already been implemented and can be marked as done:

  • Array#pack now raises ArgumentError for unknown directives (9d69b95)
  • Dir.fchdir added for changing the directory to the directory specified
    by the provided directory file descriptor (3d7233e)
  • Encoding#replicate has been removed, it was already deprecated (4f4dd51, 81669f6)
  • Introduce Fiber#kill (29c6f1e)
  • MatchData#named_captures now accepts optional symbolize_names
    keyword (43d5579)
  • Module#set_temporary_name added for setting a temporary name for a
    module (07a66bf, 3db32e8)
  • ObjectSpace::WeakKeyMap: New core class to build collections with weak references.
    The class use equality semantic to lookup keys like a regular hash,
    but it doesn't hold strong references on the keys (923a768)
  • ObjectSpace::WeakMap#delete was added to eagerly clear weak map
    entries (b19eb32)
  • New Process.warmup method that notify the Ruby virtual machine that the boot sequence is finished ... (6ac8ed2, 6f959e3)
  • Range#reverse_eachcan now process beginless ranges with an Integer endpoint: Add specs for Range#reverse_each #1169
  • Range#reverse_each now raises TypeError for endless ranges: Add specs for Range#reverse_each #1169
  • String#unpack now raises ArgumentError for unknown directives (9d69b95)
  • String#bytesplice now accepts new arguments index/length or range of the source string to be copied (15c4b26)
  • Thread::Queue#freeze now raises TypeError (ae05448)
  • Thread::SizedQueue#freeze now raises TypeError (ae05448)
  • Time.new with a string argument became stricter (7ae08ba)
  • Subprocess creation/forking via the following file open methods is deprecated (a38bf16)
  • When given a non-lambda, non-literal block, Kernel#lambda with now raises ArgumentError instead of returning it unmodified (316ea5c)
  • Error message for NoMethodError have changed to not use the target object's #inspect for efficiency, and says "instance of ClassName" instead: There is an implicit test at
    -> { map[BasicObject.new] = 1 }.should raise_error(NoMethodError, /undefined method [`']hash' for an instance of BasicObject/)
    , but this should probably get a dedicated test.
  • rb_io_path(VALUE io) is introduced to get a path of io (f6102af)
  • rb_io_mode(VALUE io) to get the mode of io (f6102af)

@andrykonchin
Copy link
Member Author

@herwinw Thank you! I have updated the list.

@andrykonchin
Copy link
Member Author

In addition

Error message for NoMethodError have changed to not use the target object's #inspect for efficiency,

It seems it was also addressed, in 72cd673

Array#pack now raises ArgumentError for unknown directives
String#unpack now raises ArgumentError for unknown directives

The mentioned tests check for NULL byte. I would say we need more explicit tests that check not supported directives

Time.new with a string argument became stricter

There is no any 3.3 specific test for String argument validation. So I assume that the change wasn't covered with specs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

2 participants