Skip to content

Commit d904381

Browse files
committed
Merge pull request ruby-grape#63 from Nerian/cannot_reopen_exposure
Test that you can reopen nested exposures
2 parents 4a6be72 + 3112419 commit d904381

File tree

2 files changed

+12
-6
lines changed

2 files changed

+12
-6
lines changed

lib/grape_entity/entity.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -195,7 +195,7 @@ def nested_exposures
195195
value = nested_exposures_hash
196196

197197
if superclass.respond_to? :nested_exposures
198-
value = superclass.nested_exposures.merge(value)
198+
value = superclass.nested_exposures.deep_merge(value)
199199
end
200200

201201
value

spec/grape_entity/entity_spec.rb

Lines changed: 11 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -137,7 +137,7 @@ class ClassRoom < Grape::Entity
137137

138138
class Person < Grape::Entity
139139
expose :user do
140-
expose(:id) { |_| 'value' }
140+
expose(:in_first) { |_| 'value' }
141141
end
142142
end
143143

@@ -155,12 +155,18 @@ class Parent < Person
155155
ClassRoom.represent({}).serializable_hash.should == {
156156
parents: [
157157
{
158-
user: { id: "value" },
159-
children: [{ user: { user_id: "value", display_id: "value" } }, { user: { user_id: "value", display_id: "value" } }]
158+
user: { in_first: 'value' },
159+
children: [
160+
{ user: { in_first: 'value', user_id: "value", display_id: "value" } },
161+
{ user: { in_first: 'value', user_id: "value", display_id: "value" } }
162+
]
160163
},
161164
{
162-
user: { id: "value" },
163-
children: [{ user: { user_id: "value", display_id: "value" } }, { user: { user_id: "value", display_id: "value" } }]
165+
user: { in_first: 'value' },
166+
children: [
167+
{ user: { in_first: 'value', user_id: "value", display_id: "value" } },
168+
{ user: { in_first: 'value', user_id: "value", display_id: "value" } }
169+
]
164170
}
165171
]
166172
}

0 commit comments

Comments
 (0)