Skip to content

Commit 2c2bbe0

Browse files
committed
Remove comment and spec slop
1 parent 4894509 commit 2c2bbe0

3 files changed

Lines changed: 0 additions & 4 deletions

File tree

lib/grape-swagger/doc_methods.rb

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -147,7 +147,6 @@ def setup_formatter(formatter)
147147
def pop_desc(doc)
148148
result = doc.key?(:desc) ? doc.delete(:desc) : doc.delete(:description)
149149
# Also remove the alias so it does not leak into **doc kwargs.
150-
# This is a no-op when :desc was absent.
151150
doc.delete(:description)
152151
result
153152
end

lib/grape-swagger/request_param_parsers/route.rb

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,6 @@ def fetch_inherited_params(stackable_values)
4949
end
5050

5151
# Grape 3.2+ serializes `type: [A, B]` via VariantCollectionCoercer#to_s, losing the type list.
52-
# Recover it from the matching CoerceValidator entry in stackable_values[:validations].
5352
# Grape 3.2+ stores validator metadata as Hash entries; older supported versions use
5453
# CoerceValidator object instances and require private-ivar reads below.
5554
# If the internal structure changes in a future Grape version this silently returns {}.

spec/swagger_v2/api_documentation_spec.rb

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,6 @@
6868
end
6969

7070
it 'respects the explicit nil and does not fall through to :description' do
71-
expect { api }.not_to raise_error
7271
expect(subject.pluck(:description)).not_to include('fallback')
7372
expect(subject.first[:description]).to be_nil
7473
end
@@ -95,7 +94,6 @@
9594
end
9695

9796
it 'passes params through to the main desc call' do
98-
expect { api }.not_to raise_error
9997
expect(subject.first[:description]).to eq('With params')
10098
end
10199
end

0 commit comments

Comments
 (0)