Skip to content

Commit fffd7f4

Browse files
committed
Drop orphaned C8::S1 comments in rdoc_store_test.rb
tompng pointed out on #1690 that the "C8::S1 does not exist. It should not be in the list." comments in test_all_classes_and_modules and test_all_classes were anchored to the Ripper-only workaround ("expected = (expected + ['C8::S1']).sort unless using_prism_ruby_parser?") that the previous commit removed. With no workaround for them to explain, the comments are orphaned -- drop them. The matching comment in test_load_single_class stays; it explains the remaining `assert_nil @c8_s1` assertion.
1 parent d4da155 commit fffd7f4

1 file changed

Lines changed: 0 additions & 6 deletions

File tree

test/rdoc/rdoc_store_test.rb

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -166,9 +166,6 @@ def test_all_classes_and_modules
166166
Parent
167167
]
168168

169-
# C8::S1 does not exist. It should not be in the list.
170-
# class C8; class << something; class S1; end; end; end
171-
172169
assert_equal expected,
173170
@store.all_classes_and_modules.map { |m| m.full_name }.sort
174171
end
@@ -220,9 +217,6 @@ def test_classes
220217
Parent
221218
]
222219

223-
# C8::S1 does not exist. It should not be in the list.
224-
# class C8; class << something; class S1; end; end; end
225-
226220
assert_equal expected, @store.all_classes.map { |m| m.full_name }.sort
227221
end
228222

0 commit comments

Comments
 (0)