iknow_view_models 3.6.7 → 3.7.1
Sign up to get free protection for your applications and to get access to all the features.
- checksums.yaml +4 -4
- data/iknow_view_models.gemspec +1 -1
- data/lib/iknow_view_models/version.rb +1 -1
- data/lib/view_model/callbacks.rb +2 -2
- data/lib/view_model/migrator.rb +23 -3
- data/lib/view_model/reference.rb +8 -0
- data/lib/view_model/references.rb +4 -3
- data/lib/view_model.rb +2 -2
- data/test/helpers/arvm_test_utilities.rb +2 -2
- data/test/helpers/viewmodel_spec_helpers.rb +8 -0
- data/test/unit/view_model/active_record/controller_nested_test.rb +14 -14
- data/test/unit/view_model/active_record/controller_test.rb +5 -5
- data/test/unit/view_model/active_record/customization_test.rb +3 -3
- data/test/unit/view_model/active_record/has_many_test.rb +6 -6
- data/test/unit/view_model/active_record/migration_test.rb +98 -4
- data/test/unit/view_model/active_record_test.rb +3 -3
- data/test/unit/view_model/callbacks_test.rb +29 -0
- data/test/unit/view_model/record_test.rb +2 -2
- metadata +3 -3
checksums.yaml
CHANGED
@@ -1,7 +1,7 @@
|
|
1
1
|
---
|
2
2
|
SHA256:
|
3
|
-
metadata.gz:
|
4
|
-
data.tar.gz:
|
3
|
+
metadata.gz: 1fb4421931099dc325d159573afcd9ea3529f6e29b4ac80f60d3628c9812df4c
|
4
|
+
data.tar.gz: 97d730c4d0a28ec9ce257906f69a1dd79e1b99c5fe025067ad57ba4ca9774931
|
5
5
|
SHA512:
|
6
|
-
metadata.gz:
|
7
|
-
data.tar.gz:
|
6
|
+
metadata.gz: 53aff699c92efd1c334333b2f417444498c504b2f91ea57a00741e3e6e1e259c988723b23e68a7f2c16fba1a6906786471748f5adf5dc2797ca2a4d0924f10df
|
7
|
+
data.tar.gz: b739f19369e3282f6e897a000f04bb56b432796acbe78ca70777e349eef8f39a4058488032f4b42eeda0c4c7c4f60e9b4dba1fba1f2cf253a3c2bd960ff753e3
|
data/iknow_view_models.gemspec
CHANGED
@@ -19,7 +19,7 @@ Gem::Specification.new do |spec|
|
|
19
19
|
spec.test_files = spec.files.grep(%r{^(test|spec|features)/})
|
20
20
|
spec.require_paths = ['lib']
|
21
21
|
|
22
|
-
spec.required_ruby_version = '>= 2.7'
|
22
|
+
spec.required_ruby_version = '>= 2.7.3'
|
23
23
|
|
24
24
|
spec.add_dependency 'actionpack', '>= 5.0'
|
25
25
|
spec.add_dependency 'activerecord', '>= 5.0'
|
data/lib/view_model/callbacks.rb
CHANGED
@@ -11,9 +11,9 @@ module ViewModel::Callbacks
|
|
11
11
|
# callbacks instance with additional instance method access to the view,
|
12
12
|
# context and extra context-dependent parameters.
|
13
13
|
module CallbackEnvContext
|
14
|
-
def method_missing(method,
|
14
|
+
def method_missing(method, ...)
|
15
15
|
if _callbacks.respond_to?(method, true)
|
16
|
-
_callbacks.send(method,
|
16
|
+
_callbacks.send(method, ...)
|
17
17
|
else
|
18
18
|
super
|
19
19
|
end
|
data/lib/view_model/migrator.rb
CHANGED
@@ -39,7 +39,26 @@ class ViewModel
|
|
39
39
|
def migrate!(serialization)
|
40
40
|
references = (serialization['references'] ||= {})
|
41
41
|
|
42
|
-
|
42
|
+
# First visit everything except references; there's no issue with adding
|
43
|
+
# new references during this.
|
44
|
+
migrate_tree!(serialization.except('references'), references: references)
|
45
|
+
|
46
|
+
# While visiting references itself, we need to take care that we can
|
47
|
+
# concurrently modify them (e.g. by adding new referenced views).
|
48
|
+
# Moreover, such added references must themselves be visited, as they'll
|
49
|
+
# be synthesized at the current version and so may need to be migrated
|
50
|
+
# down to the client's requested version.
|
51
|
+
visited_refs = []
|
52
|
+
loop do
|
53
|
+
unvisited_refs = references.keys - visited_refs
|
54
|
+
break if unvisited_refs.empty?
|
55
|
+
|
56
|
+
unvisited_refs.each do |ref|
|
57
|
+
migrate_tree!(references[ref], references: references)
|
58
|
+
end
|
59
|
+
|
60
|
+
visited_refs.concat(unvisited_refs)
|
61
|
+
end
|
43
62
|
|
44
63
|
GarbageCollection.garbage_collect_references!(serialization)
|
45
64
|
|
@@ -112,10 +131,11 @@ class ViewModel
|
|
112
131
|
path = @paths[view_name]
|
113
132
|
return false unless path
|
114
133
|
|
115
|
-
# We assume that an unspecified source version is the same as the required
|
116
|
-
# version.
|
117
134
|
required_version, current_version = @versions[view_name]
|
135
|
+
return false if source_version == current_version
|
118
136
|
|
137
|
+
# We assume that an unspecified source version is the same as the required
|
138
|
+
# version (i.e. the version demanded by the client request).
|
119
139
|
unless source_version.nil? || source_version == required_version
|
120
140
|
raise ViewModel::Migration::UnspecifiedVersionError.new(view_name, source_version)
|
121
141
|
end
|
data/lib/view_model/reference.rb
CHANGED
@@ -29,5 +29,13 @@ class ViewModel
|
|
29
29
|
def hash
|
30
30
|
[viewmodel_class, model_id].hash
|
31
31
|
end
|
32
|
+
|
33
|
+
# Generate a stable reference key for this viewmodel using type name and id
|
34
|
+
def stable_reference
|
35
|
+
raise RuntimeError.new('Model id required to generate a stable reference') unless model_id
|
36
|
+
|
37
|
+
hash = Digest::SHA256.base64digest("#{viewmodel_class.name}.#{model_id}")
|
38
|
+
"ref:h:#{hash}"
|
39
|
+
end
|
32
40
|
end
|
33
41
|
end
|
@@ -37,12 +37,13 @@ class ViewModel
|
|
37
37
|
|
38
38
|
private
|
39
39
|
|
40
|
-
# Ensure stable reference
|
40
|
+
# Ensure stable reference keys for the same (persisted) viewmodels. For
|
41
|
+
# unpersisted viewmodels, use a counter to generate a reference key unique
|
42
|
+
# to this serialization.
|
41
43
|
def new_ref!(viewmodel)
|
42
44
|
vm_ref = viewmodel.to_reference
|
43
45
|
if vm_ref.model_id
|
44
|
-
|
45
|
-
"ref:h:#{hash}"
|
46
|
+
vm_ref.stable_reference
|
46
47
|
else
|
47
48
|
format('ref:i:%06<count>d', count: (@last_ref += 1))
|
48
49
|
end
|
data/lib/view_model.rb
CHANGED
@@ -303,12 +303,12 @@ class ViewModel
|
|
303
303
|
end
|
304
304
|
end
|
305
305
|
|
306
|
-
def
|
306
|
+
def serialize_to_hash(serialize_context: self.class.new_serialize_context)
|
307
307
|
Jbuilder.new { |json| serialize(json, serialize_context: serialize_context) }.attributes!
|
308
308
|
end
|
309
309
|
|
310
310
|
def to_json(serialize_context: self.class.new_serialize_context)
|
311
|
-
ViewModel.encode_json(self.
|
311
|
+
ViewModel.encode_json(self.serialize_to_hash(serialize_context: serialize_context))
|
312
312
|
end
|
313
313
|
|
314
314
|
# Render this viewmodel to a jBuilder. Usually overridden in subclasses.
|
@@ -98,7 +98,7 @@ module ARVMTestUtilities
|
|
98
98
|
end
|
99
99
|
|
100
100
|
def assert_serializes(vm, model, serialize_context: vm.new_serialize_context)
|
101
|
-
h = vm.new(model).
|
101
|
+
h = vm.new(model).serialize_to_hash(serialize_context: serialize_context)
|
102
102
|
assert_kind_of(Hash, h)
|
103
103
|
refs = serialize_context.serialize_references_to_hash
|
104
104
|
assert_kind_of(Hash, refs)
|
@@ -106,7 +106,7 @@ module ARVMTestUtilities
|
|
106
106
|
|
107
107
|
def refute_serializes(vm, model, message = nil, serialize_context: vm.new_serialize_context)
|
108
108
|
ex = assert_raises(ViewModel::AccessControlError) do
|
109
|
-
vm.new(model).
|
109
|
+
vm.new(model).serialize_to_hash(serialize_context: serialize_context)
|
110
110
|
serialize_context.serialize_references_to_hash
|
111
111
|
end
|
112
112
|
assert_match(message, ex.message) if message
|
@@ -295,6 +295,14 @@ module ViewModelSpecHelpers
|
|
295
295
|
end
|
296
296
|
end
|
297
297
|
|
298
|
+
module ReferencedList
|
299
|
+
extend ActiveSupport::Concern
|
300
|
+
include ViewModelSpecHelpers::List
|
301
|
+
def model_attributes
|
302
|
+
super.merge(viewmodel: ->(_v) { root! })
|
303
|
+
end
|
304
|
+
end
|
305
|
+
|
298
306
|
module ParentAndHasOneChild
|
299
307
|
extend ActiveSupport::Concern
|
300
308
|
include ViewModelSpecHelpers::Base
|
@@ -44,7 +44,7 @@ class ViewModel::ActiveRecord::ControllerNestedTest < ActiveSupport::TestCase
|
|
44
44
|
assert_equal(200, childcontroller.status)
|
45
45
|
|
46
46
|
expected_children = @parent.children
|
47
|
-
assert_equal({ 'data' => expected_children.map { |c| ChildView.new(c).
|
47
|
+
assert_equal({ 'data' => expected_children.map { |c| ChildView.new(c).serialize_to_hash } },
|
48
48
|
childcontroller.hash_response)
|
49
49
|
|
50
50
|
assert_all_hooks_nested_inside_parent_hook(childcontroller.hook_trace)
|
@@ -59,7 +59,7 @@ class ViewModel::ActiveRecord::ControllerNestedTest < ActiveSupport::TestCase
|
|
59
59
|
assert_equal(200, childcontroller.status)
|
60
60
|
|
61
61
|
expected_children = @parent.children + distractor.children
|
62
|
-
assert_equal({ 'data' => expected_children.map { |c| ChildView.new(c).
|
62
|
+
assert_equal({ 'data' => expected_children.map { |c| ChildView.new(c).serialize_to_hash } },
|
63
63
|
childcontroller.hash_response)
|
64
64
|
end
|
65
65
|
|
@@ -79,7 +79,7 @@ class ViewModel::ActiveRecord::ControllerNestedTest < ActiveSupport::TestCase
|
|
79
79
|
@parent.reload
|
80
80
|
|
81
81
|
assert_equal(%w[c1 c2 c3], @parent.children.order(:position).pluck(:name))
|
82
|
-
assert_equal({ 'data' => ChildView.new(@parent.children.last).
|
82
|
+
assert_equal({ 'data' => ChildView.new(@parent.children.last).serialize_to_hash },
|
83
83
|
childcontroller.hash_response)
|
84
84
|
|
85
85
|
assert_all_hooks_nested_inside_parent_hook(childcontroller.hook_trace)
|
@@ -102,7 +102,7 @@ class ViewModel::ActiveRecord::ControllerNestedTest < ActiveSupport::TestCase
|
|
102
102
|
@parent.reload
|
103
103
|
|
104
104
|
assert_equal(%w[c1 c2 c3 c4], @parent.children.order(:position).pluck(:name))
|
105
|
-
new_children_hashes = @parent.children.last(2).map { |c| ChildView.new(c).
|
105
|
+
new_children_hashes = @parent.children.last(2).map { |c| ChildView.new(c).serialize_to_hash }
|
106
106
|
assert_equal({ 'data' => new_children_hashes },
|
107
107
|
childcontroller.hash_response)
|
108
108
|
|
@@ -265,7 +265,7 @@ class ViewModel::ActiveRecord::ControllerNestedTest < ActiveSupport::TestCase
|
|
265
265
|
old_child.reload
|
266
266
|
|
267
267
|
assert_equal('new_name', old_child.name)
|
268
|
-
assert_equal({ 'data' => ChildView.new(old_child).
|
268
|
+
assert_equal({ 'data' => ChildView.new(old_child).serialize_to_hash },
|
269
269
|
childcontroller.hash_response)
|
270
270
|
end
|
271
271
|
|
@@ -275,7 +275,7 @@ class ViewModel::ActiveRecord::ControllerNestedTest < ActiveSupport::TestCase
|
|
275
275
|
childcontroller = ChildController.new(params: { id: old_child.id })
|
276
276
|
childcontroller.invoke(:show)
|
277
277
|
|
278
|
-
assert_equal({ 'data' => ChildView.new(old_child).
|
278
|
+
assert_equal({ 'data' => ChildView.new(old_child).serialize_to_hash },
|
279
279
|
childcontroller.hash_response)
|
280
280
|
|
281
281
|
assert_equal(200, childcontroller.status)
|
@@ -323,7 +323,7 @@ class ViewModel::ActiveRecord::ControllerNestedTest < ActiveSupport::TestCase
|
|
323
323
|
|
324
324
|
assert_equal(200, labelcontroller.status, labelcontroller.hash_response)
|
325
325
|
|
326
|
-
assert_equal({ 'data' => LabelView.new(old_label).
|
326
|
+
assert_equal({ 'data' => LabelView.new(old_label).serialize_to_hash },
|
327
327
|
labelcontroller.hash_response)
|
328
328
|
|
329
329
|
assert_all_hooks_nested_inside_parent_hook(labelcontroller.hook_trace)
|
@@ -367,7 +367,7 @@ class ViewModel::ActiveRecord::ControllerNestedTest < ActiveSupport::TestCase
|
|
367
367
|
old_label.reload
|
368
368
|
|
369
369
|
assert_equal('new label', old_label.text)
|
370
|
-
assert_equal({ 'data' => LabelView.new(old_label).
|
370
|
+
assert_equal({ 'data' => LabelView.new(old_label).serialize_to_hash },
|
371
371
|
labelcontroller.hash_response)
|
372
372
|
|
373
373
|
assert_all_hooks_nested_inside_parent_hook(labelcontroller.hook_trace)
|
@@ -381,7 +381,7 @@ class ViewModel::ActiveRecord::ControllerNestedTest < ActiveSupport::TestCase
|
|
381
381
|
|
382
382
|
assert_equal(200, labelcontroller.status, labelcontroller.hash_response)
|
383
383
|
|
384
|
-
assert_equal({ 'data' => LabelView.new(old_label).
|
384
|
+
assert_equal({ 'data' => LabelView.new(old_label).serialize_to_hash },
|
385
385
|
labelcontroller.hash_response)
|
386
386
|
end
|
387
387
|
|
@@ -414,7 +414,7 @@ class ViewModel::ActiveRecord::ControllerNestedTest < ActiveSupport::TestCase
|
|
414
414
|
old_label.reload
|
415
415
|
|
416
416
|
assert_equal('new label', old_label.text)
|
417
|
-
assert_equal({ 'data' => LabelView.new(old_label).
|
417
|
+
assert_equal({ 'data' => LabelView.new(old_label).serialize_to_hash },
|
418
418
|
labelcontroller.hash_response)
|
419
419
|
end
|
420
420
|
|
@@ -472,11 +472,11 @@ class ViewModel::ActiveRecord::ControllerNestedTest < ActiveSupport::TestCase
|
|
472
472
|
'updates' => [
|
473
473
|
{
|
474
474
|
'id' => @parent.id,
|
475
|
-
'update' => TargetView.new(target).
|
475
|
+
'update' => TargetView.new(target).serialize_to_hash,
|
476
476
|
},
|
477
477
|
{
|
478
478
|
'id' => other_parent.id,
|
479
|
-
'update' => TargetView.new(other_target).
|
479
|
+
'update' => TargetView.new(other_target).serialize_to_hash,
|
480
480
|
},
|
481
481
|
].sort_by { |x| x.fetch('id') }
|
482
482
|
}
|
@@ -535,7 +535,7 @@ class ViewModel::ActiveRecord::ControllerNestedTest < ActiveSupport::TestCase
|
|
535
535
|
|
536
536
|
assert_equal(
|
537
537
|
{
|
538
|
-
ref_key => CategoryView.new(@parent.category).
|
538
|
+
ref_key => CategoryView.new(@parent.category).serialize_to_hash,
|
539
539
|
},
|
540
540
|
references,
|
541
541
|
)
|
@@ -591,7 +591,7 @@ class ViewModel::ActiveRecord::ControllerNestedTest < ActiveSupport::TestCase
|
|
591
591
|
|
592
592
|
assert_equal(
|
593
593
|
{
|
594
|
-
ref_key => TagView.new(@parent.parent_tags.first.tag).
|
594
|
+
ref_key => TagView.new(@parent.parent_tags.first.tag).serialize_to_hash,
|
595
595
|
},
|
596
596
|
references,
|
597
597
|
)
|
@@ -37,7 +37,7 @@ class ViewModel::ActiveRecord::ControllerTest < ActiveSupport::TestCase
|
|
37
37
|
parentcontroller = ParentController.new(params: { id: @parent.id })
|
38
38
|
parentcontroller.invoke(:show)
|
39
39
|
|
40
|
-
assert_equal({ 'data' => @parent_view.
|
40
|
+
assert_equal({ 'data' => @parent_view.serialize_to_hash },
|
41
41
|
parentcontroller.hash_response)
|
42
42
|
|
43
43
|
assert_equal(200, parentcontroller.status)
|
@@ -52,7 +52,7 @@ class ViewModel::ActiveRecord::ControllerTest < ActiveSupport::TestCase
|
|
52
52
|
|
53
53
|
parentcontroller.invoke(:show)
|
54
54
|
|
55
|
-
expected_view = @parent_view.
|
55
|
+
expected_view = @parent_view.serialize_to_hash
|
56
56
|
.except('name')
|
57
57
|
.merge('old_name' => @parent.name,
|
58
58
|
ViewModel::VERSION_ATTRIBUTE => 1,
|
@@ -88,7 +88,7 @@ class ViewModel::ActiveRecord::ControllerTest < ActiveSupport::TestCase
|
|
88
88
|
assert_equal(200, parentcontroller.status)
|
89
89
|
|
90
90
|
assert_equal(parentcontroller.hash_response,
|
91
|
-
{ 'data' => [@parent_view.
|
91
|
+
{ 'data' => [@parent_view.serialize_to_hash, p2_view.serialize_to_hash] })
|
92
92
|
|
93
93
|
assert_all_hooks_nested_inside_parent_hook(parentcontroller.hook_trace)
|
94
94
|
end
|
@@ -112,7 +112,7 @@ class ViewModel::ActiveRecord::ControllerTest < ActiveSupport::TestCase
|
|
112
112
|
p2_view = ParentView.new(p2)
|
113
113
|
assert(p2.present?, 'p2 created')
|
114
114
|
|
115
|
-
assert_equal({ 'data' => p2_view.
|
115
|
+
assert_equal({ 'data' => p2_view.serialize_to_hash }, parentcontroller.hash_response)
|
116
116
|
|
117
117
|
assert_all_hooks_nested_inside_parent_hook(parentcontroller.hook_trace)
|
118
118
|
end
|
@@ -160,7 +160,7 @@ class ViewModel::ActiveRecord::ControllerTest < ActiveSupport::TestCase
|
|
160
160
|
@parent.reload
|
161
161
|
|
162
162
|
assert_equal('new', @parent.name)
|
163
|
-
assert_equal({ 'data' => @parent_view.
|
163
|
+
assert_equal({ 'data' => @parent_view.serialize_to_hash },
|
164
164
|
parentcontroller.hash_response)
|
165
165
|
|
166
166
|
assert_all_hooks_nested_inside_parent_hook(parentcontroller.hook_trace)
|
@@ -283,13 +283,13 @@ class ViewModel::ActiveRecord::FlattenAssociationTest < ActiveSupport::TestCase
|
|
283
283
|
|
284
284
|
def test_serialize
|
285
285
|
v = SectionView.new(@simplesection)
|
286
|
-
assert_equal(@simplesection_view, v.
|
286
|
+
assert_equal(@simplesection_view, v.serialize_to_hash)
|
287
287
|
|
288
288
|
v = SectionView.new(@quizsection)
|
289
|
-
assert_equal(@quizsection_view, v.
|
289
|
+
assert_equal(@quizsection_view, v.serialize_to_hash)
|
290
290
|
|
291
291
|
v = SectionView.new(@vocabsection)
|
292
|
-
assert_equal(@vocabsection_view, v.
|
292
|
+
assert_equal(@vocabsection_view, v.serialize_to_hash)
|
293
293
|
end
|
294
294
|
|
295
295
|
def new_view_like(view)
|
@@ -469,8 +469,8 @@ class ViewModel::ActiveRecord::HasManyTest < ActiveSupport::TestCase
|
|
469
469
|
old_children = @model1.children.order(:position)
|
470
470
|
moved_child = old_children[1]
|
471
471
|
|
472
|
-
view = viewmodel_class.new(@model2).
|
473
|
-
view['children'] << child_viewmodel_class.new(moved_child).
|
472
|
+
view = viewmodel_class.new(@model2).serialize_to_hash
|
473
|
+
view['children'] << child_viewmodel_class.new(moved_child).serialize_to_hash
|
474
474
|
|
475
475
|
retained_children = old_children - [moved_child]
|
476
476
|
release_view = { '_type' => 'Model', 'id' => @model1.id,
|
@@ -503,7 +503,7 @@ class ViewModel::ActiveRecord::HasManyTest < ActiveSupport::TestCase
|
|
503
503
|
def test_has_many_append_and_update_existing_association
|
504
504
|
child = @model1.children[1]
|
505
505
|
|
506
|
-
cv = child_viewmodel_class.new(child).
|
506
|
+
cv = child_viewmodel_class.new(child).serialize_to_hash
|
507
507
|
cv['name'] = 'newname'
|
508
508
|
|
509
509
|
viewmodel_class.new(@model1).append_associated(:children, cv)
|
@@ -558,7 +558,7 @@ class ViewModel::ActiveRecord::HasManyTest < ActiveSupport::TestCase
|
|
558
558
|
def test_move_and_edit_child_to_new
|
559
559
|
child = @model1.children[1]
|
560
560
|
|
561
|
-
child_view = child_viewmodel_class.new(child).
|
561
|
+
child_view = child_viewmodel_class.new(child).serialize_to_hash
|
562
562
|
child_view['name'] = 'changed'
|
563
563
|
|
564
564
|
view = { '_type' => 'Model',
|
@@ -592,9 +592,9 @@ class ViewModel::ActiveRecord::HasManyTest < ActiveSupport::TestCase
|
|
592
592
|
def test_move_and_edit_child_to_existing
|
593
593
|
old_child = @model1.children[1]
|
594
594
|
|
595
|
-
old_child_view = child_viewmodel_class.new(old_child).
|
595
|
+
old_child_view = child_viewmodel_class.new(old_child).serialize_to_hash
|
596
596
|
old_child_view['name'] = 'changed'
|
597
|
-
view = viewmodel_class.new(@model2).
|
597
|
+
view = viewmodel_class.new(@model2).serialize_to_hash
|
598
598
|
view['children'] << old_child_view
|
599
599
|
|
600
600
|
release_view = { '_type' => 'Model', 'id' => @model1.id,
|
@@ -436,16 +436,17 @@ class ViewModel::ActiveRecord::Migration < ActiveSupport::TestCase
|
|
436
436
|
let(:migrator) { down_migrator }
|
437
437
|
let(:subject) do
|
438
438
|
ser = current_serialization.deep_dup
|
439
|
-
raise ArgumentError.new(
|
439
|
+
raise ArgumentError.new('Expected no references') if ser.has_key?('references')
|
440
|
+
|
440
441
|
ser
|
441
442
|
end
|
442
443
|
|
443
444
|
let(:expected_result) do
|
444
445
|
{
|
445
446
|
'data' => v1_serialization_data.deep_dup.deep_merge(
|
446
|
-
{ ViewModel::MIGRATED_ATTRIBUTE => true }
|
447
|
+
{ ViewModel::MIGRATED_ATTRIBUTE => true },
|
447
448
|
),
|
448
|
-
'references' => v1_serialization_references
|
449
|
+
'references' => v1_serialization_references,
|
449
450
|
}
|
450
451
|
end
|
451
452
|
|
@@ -460,7 +461,8 @@ class ViewModel::ActiveRecord::Migration < ActiveSupport::TestCase
|
|
460
461
|
let(:migrator) { up_migrator }
|
461
462
|
let(:subject) do
|
462
463
|
ser = v1_serialization.deep_dup
|
463
|
-
raise ArgumentError.new(
|
464
|
+
raise ArgumentError.new('Expected references') unless ser.has_key?('references')
|
465
|
+
|
464
466
|
ser
|
465
467
|
end
|
466
468
|
|
@@ -480,4 +482,96 @@ class ViewModel::ActiveRecord::Migration < ActiveSupport::TestCase
|
|
480
482
|
end
|
481
483
|
end
|
482
484
|
end
|
485
|
+
|
486
|
+
describe 'concurrently inserting a reference' do
|
487
|
+
include ViewModelSpecHelpers::ReferencedList
|
488
|
+
let(:migration_versions) { { viewmodel_class => 1 } }
|
489
|
+
|
490
|
+
# Use a list with two members
|
491
|
+
def new_model
|
492
|
+
model_class.new(name: 'root',
|
493
|
+
next: model_class.new(name: 'old-tail'))
|
494
|
+
end
|
495
|
+
|
496
|
+
# Define a down migration that matches the old tail to insert a new tail
|
497
|
+
# after it, and the new tail to change its name.
|
498
|
+
def model_attributes
|
499
|
+
super.merge(
|
500
|
+
viewmodel: ->(v) {
|
501
|
+
self.schema_version = 2
|
502
|
+
|
503
|
+
migrates from: 1, to: 2 do
|
504
|
+
down do |view, refs|
|
505
|
+
case view['name']
|
506
|
+
when 'old-tail'
|
507
|
+
view['next'] = { ViewModel::REFERENCE_ATTRIBUTE => 'ref:s:new_tail' }
|
508
|
+
refs['ref:s:new_tail'] = {
|
509
|
+
ViewModel::TYPE_ATTRIBUTE => v.view_name,
|
510
|
+
ViewModel::VERSION_ATTRIBUTE => v.schema_version,
|
511
|
+
'id' => 100, # entirely fake
|
512
|
+
'name' => 'new-tail',
|
513
|
+
'next' => nil,
|
514
|
+
}
|
515
|
+
|
516
|
+
when 'new-tail'
|
517
|
+
view['name'] = 'newer-tail'
|
518
|
+
end
|
519
|
+
end
|
520
|
+
end
|
521
|
+
},
|
522
|
+
)
|
523
|
+
end
|
524
|
+
|
525
|
+
let(:v1_serialization_data) do
|
526
|
+
{
|
527
|
+
ViewModel::TYPE_ATTRIBUTE => viewmodel_class.view_name,
|
528
|
+
ViewModel::VERSION_ATTRIBUTE => 1,
|
529
|
+
ViewModel::ID_ATTRIBUTE => viewmodel.id,
|
530
|
+
'name' => viewmodel.name,
|
531
|
+
'next' => { ViewModel::REFERENCE_ATTRIBUTE => viewmodel.next.to_reference.stable_reference },
|
532
|
+
ViewModel::MIGRATED_ATTRIBUTE => true,
|
533
|
+
}
|
534
|
+
end
|
535
|
+
|
536
|
+
let(:v1_serialization_references) do
|
537
|
+
old_tail = viewmodel.next
|
538
|
+
old_tail_ref = old_tail.to_reference.stable_reference
|
539
|
+
{
|
540
|
+
old_tail_ref => {
|
541
|
+
ViewModel::TYPE_ATTRIBUTE => viewmodel_class.view_name,
|
542
|
+
ViewModel::VERSION_ATTRIBUTE => 1,
|
543
|
+
ViewModel::ID_ATTRIBUTE => old_tail.id,
|
544
|
+
'name' => 'old-tail',
|
545
|
+
'next' => { ViewModel::REFERENCE_ATTRIBUTE => 'ref:s:new_tail' },
|
546
|
+
ViewModel::MIGRATED_ATTRIBUTE => true,
|
547
|
+
},
|
548
|
+
'ref:s:new_tail' => {
|
549
|
+
ViewModel::TYPE_ATTRIBUTE => viewmodel_class.view_name,
|
550
|
+
ViewModel::VERSION_ATTRIBUTE => 1,
|
551
|
+
ViewModel::ID_ATTRIBUTE => 100,
|
552
|
+
'name' => 'newer-tail',
|
553
|
+
'next' => nil,
|
554
|
+
ViewModel::MIGRATED_ATTRIBUTE => true,
|
555
|
+
},
|
556
|
+
}
|
557
|
+
end
|
558
|
+
|
559
|
+
let(:v1_serialization) do
|
560
|
+
{
|
561
|
+
'data' => v1_serialization_data,
|
562
|
+
'references' => v1_serialization_references,
|
563
|
+
}
|
564
|
+
end
|
565
|
+
|
566
|
+
describe 'downwards' do
|
567
|
+
let(:migrator) { down_migrator }
|
568
|
+
let(:subject) { current_serialization.deep_dup }
|
569
|
+
|
570
|
+
it 'migrates' do
|
571
|
+
migrate!
|
572
|
+
|
573
|
+
assert_equal(v1_serialization, subject)
|
574
|
+
end
|
575
|
+
end
|
576
|
+
end
|
483
577
|
end
|
@@ -146,7 +146,7 @@ class ViewModel::ActiveRecordTest < ActiveSupport::TestCase
|
|
146
146
|
|
147
147
|
assert_raises(ViewModel::AccessControlError) do
|
148
148
|
no_view_context = ViewModelBase.new_serialize_context(can_view: false)
|
149
|
-
parentview.
|
149
|
+
parentview.serialize_to_hash(serialize_context: no_view_context)
|
150
150
|
end
|
151
151
|
|
152
152
|
assert_raises(ViewModel::AccessControlError) do
|
@@ -191,7 +191,7 @@ class ViewModel::ActiveRecordTest < ActiveSupport::TestCase
|
|
191
191
|
|
192
192
|
ex = assert_raises(ViewModel::AccessControlError) do
|
193
193
|
# edit
|
194
|
-
v = ParentView.new(@parent1).
|
194
|
+
v = ParentView.new(@parent1).serialize_to_hash.merge('name' => 'p2')
|
195
195
|
ParentView.deserialize_from_view(v, deserialize_context: no_edit_context)
|
196
196
|
end
|
197
197
|
assert_match(/Illegal edit/, ex.message)
|
@@ -214,7 +214,7 @@ class ViewModel::ActiveRecordTest < ActiveSupport::TestCase
|
|
214
214
|
|
215
215
|
ex = assert_raises(ViewModel::AccessControlError) do
|
216
216
|
# edit
|
217
|
-
v = ParentView.new(@parent1).
|
217
|
+
v = ParentView.new(@parent1).serialize_to_hash.merge('name' => 'p2')
|
218
218
|
ParentView.deserialize_from_view(v, deserialize_context: no_edit_context)
|
219
219
|
end
|
220
220
|
assert_match(/Illegal edit/, ex.message)
|
@@ -554,6 +554,35 @@ class ViewModel::CallbacksTest < ActiveSupport::TestCase
|
|
554
554
|
end
|
555
555
|
end
|
556
556
|
|
557
|
+
describe 'delegates to methods on the callback object' do
|
558
|
+
class TestCallback
|
559
|
+
include ViewModel::Callbacks
|
560
|
+
|
561
|
+
attr_reader :a_args, :b_args
|
562
|
+
|
563
|
+
def a(*args)
|
564
|
+
@a_args = args
|
565
|
+
end
|
566
|
+
|
567
|
+
def b(*args, **kwargs)
|
568
|
+
@b_args = [args, kwargs]
|
569
|
+
end
|
570
|
+
|
571
|
+
before_visit do
|
572
|
+
a(1, 2, x: 1, y: 2)
|
573
|
+
b(1, 2, x: 1, y: 2)
|
574
|
+
end
|
575
|
+
end
|
576
|
+
|
577
|
+
let(:callback) { TestCallback.new }
|
578
|
+
|
579
|
+
it 'delegates to callback methods including kwargs' do
|
580
|
+
serialize(vm)
|
581
|
+
value(callback.a_args).must_equal([1, 2, { x: 1, y: 2 }])
|
582
|
+
value(callback.b_args).must_equal([[1, 2], { x: 1, y: 2 }])
|
583
|
+
end
|
584
|
+
end
|
585
|
+
|
557
586
|
describe 'provides details to the execution environment' do
|
558
587
|
class EnvCallback
|
559
588
|
include ViewModel::Callbacks
|
@@ -204,7 +204,7 @@ class ViewModel::RecordTest < ActiveSupport::TestCase
|
|
204
204
|
def self.included(base)
|
205
205
|
base.instance_eval do
|
206
206
|
it 'can serialize to the expected view' do
|
207
|
-
h = viewmodel_class.new(subject_model).
|
207
|
+
h = viewmodel_class.new(subject_model).serialize_to_hash
|
208
208
|
assert_equal(expected_view, h)
|
209
209
|
end
|
210
210
|
end
|
@@ -318,7 +318,7 @@ class ViewModel::RecordTest < ActiveSupport::TestCase
|
|
318
318
|
it 'raises correctly on an undeserializable value' do
|
319
319
|
bad_model = subject_model.tap { |m| m.moment = 2.7 }
|
320
320
|
ex = assert_raises(ViewModel::SerializationError) do
|
321
|
-
viewmodel_class.new(bad_model).
|
321
|
+
viewmodel_class.new(bad_model).serialize_to_hash
|
322
322
|
end
|
323
323
|
assert_match(/Could not serialize invalid value.*'moment'.*Incorrect type/, ex.detail)
|
324
324
|
end
|
metadata
CHANGED
@@ -1,14 +1,14 @@
|
|
1
1
|
--- !ruby/object:Gem::Specification
|
2
2
|
name: iknow_view_models
|
3
3
|
version: !ruby/object:Gem::Version
|
4
|
-
version: 3.
|
4
|
+
version: 3.7.1
|
5
5
|
platform: ruby
|
6
6
|
authors:
|
7
7
|
- iKnow Team
|
8
8
|
autorequire:
|
9
9
|
bindir: bin
|
10
10
|
cert_chain: []
|
11
|
-
date: 2022-
|
11
|
+
date: 2022-11-17 00:00:00.000000000 Z
|
12
12
|
dependencies:
|
13
13
|
- !ruby/object:Gem::Dependency
|
14
14
|
name: actionpack
|
@@ -502,7 +502,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
|
|
502
502
|
requirements:
|
503
503
|
- - ">="
|
504
504
|
- !ruby/object:Gem::Version
|
505
|
-
version:
|
505
|
+
version: 2.7.3
|
506
506
|
required_rubygems_version: !ruby/object:Gem::Requirement
|
507
507
|
requirements:
|
508
508
|
- - ">="
|