Skip to content

Commit 2f96090

Browse files
committed
🔎 Update SequenceSet#inspect output format
Similarly to how `#inspect` already outputs frozen sets as valid ruby to recreate an identical set, with `Net::IMAP::SequenceSet[str]`, now we do that for mutuble sets too, with `Net::IMAP::SequenceSet(str)`.
1 parent 19de452 commit 2f96090

File tree

2 files changed

+8
-8
lines changed

2 files changed

+8
-8
lines changed

lib/net/imap/sequence_set.rb

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1633,10 +1633,10 @@ def normalized_string
16331633
# Returns an inspection string for the SequenceSet.
16341634
#
16351635
# Net::IMAP::SequenceSet.new.inspect
1636-
# #=> "#<Net::IMAP::SequenceSet empty>"
1636+
# #=> "Net::IMAP::SequenceSet()"
16371637
#
16381638
# Net::IMAP::SequenceSet(1..5, 1024, 15, 2000).inspect
1639-
# #=> '#<Net::IMAP::SequenceSet "1:5,15,1024,2000">'
1639+
# #=> 'Net::IMAP::SequenceSet("1:5,15,1024,2000")'
16401640
#
16411641
# Frozen sets have slightly different output:
16421642
#
@@ -1648,9 +1648,9 @@ def normalized_string
16481648
#
16491649
def inspect
16501650
if empty?
1651-
(frozen? ? "%s.empty" : "#<%s empty>") % [self.class]
1651+
(frozen? ? "%s.empty" : "%s()") % [self.class]
16521652
else
1653-
(frozen? ? "%s[%p]" : "#<%s %p>") % [self.class, to_s]
1653+
(frozen? ? "%s[%p]" : "%s(%p)") % [self.class, to_s]
16541654
end
16551655
end
16561656

test/net/imap/test_sequence_set.rb

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -888,11 +888,11 @@ def obj.to_sequence_set; 192_168.001_255 end
888888

889889
data(
890890
# desc => [expected, input, freeze]
891-
"empty" => ["#<Net::IMAP::SequenceSet empty>", nil],
891+
"empty" => ["Net::IMAP::SequenceSet()", nil],
892892
"frozen empty" => ["Net::IMAP::SequenceSet.empty", nil, true],
893-
"normalized" => ['#<Net::IMAP::SequenceSet "1:2">', [2, 1]],
894-
"denormalized" => ['#<Net::IMAP::SequenceSet "2,1">', "2,1"],
895-
"star" => ['#<Net::IMAP::SequenceSet "*">', "*"],
893+
"normalized" => ['Net::IMAP::SequenceSet("1:2")', [2, 1]],
894+
"denormalized" => ['Net::IMAP::SequenceSet("2,1")', "2,1"],
895+
"star" => ['Net::IMAP::SequenceSet("*")', "*"],
896896
"frozen" => ['Net::IMAP::SequenceSet["1,3,5:*"]', [1, 3, 5..], true],
897897
)
898898
def test_inspect((expected, input, freeze))

0 commit comments

Comments
 (0)