Skip to content

Commit 43e6a66

Browse files
committed
🐛 Ensure set is loaded in ruby 3.1
`set` doesn't need to be explicitly loaded in ruby 3.2+. Previously, we loaded it in the tests, so they passed. But running the actual code that uses Set could raise an exception.
1 parent cc13c91 commit 43e6a66

File tree

5 files changed

+5
-3
lines changed

5 files changed

+5
-3
lines changed

benchmarks/table-regexps.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
prelude: |
22
require "json"
3-
require "set"
3+
require "set" unless defined?(::Set)
44
55
all_codepoints = (0..0x10ffff).map{_1.chr("UTF-8") rescue nil}.compact
66

lib/net/imap/sequence_set.rb

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
# frozen_string_literal: true
22

3+
require "set" unless defined?(::Set)
4+
35
module Net
46
class IMAP
57

rakelib/string_prep_tables_generator.rb

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
# frozen_string_literal: true
22

3+
require "set" unless defined?(::Set)
4+
35
# Generator for stringprep regexps.
46
#
57
# Combines Unicode character classes with generated tables. Generated regexps

test/net/imap/test_sequence_set.rb

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@
22

33
require "net/imap"
44
require "test/unit"
5-
require "set"
65

76
class IMAPSequenceSetTest < Test::Unit::TestCase
87
# alias for convenience

test/net/imap/test_stringprep_tables.rb

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,6 @@
33
require "net/imap"
44
require "test/unit"
55
require "json"
6-
require "set"
76

87
require_relative "../../../rakelib/string_prep_tables_generator"
98

0 commit comments

Comments
 (0)