rezource 0.0.5 → 0.0.6
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- data/lib/rezource.rb +3 -3
- data/rezource.gemspec +1 -1
- data/test/test_rezource.rb +25 -8
- metadata +2 -2
data/lib/rezource.rb
CHANGED
|
@@ -245,7 +245,7 @@ module Rezource
|
|
|
245
245
|
|
|
246
246
|
def resource_create(resource)
|
|
247
247
|
before_create resource
|
|
248
|
-
resource.save
|
|
248
|
+
resource.save unless resource.errors.any?
|
|
249
249
|
after_create resource
|
|
250
250
|
|
|
251
251
|
# respond_with *namespace, resource
|
|
@@ -254,11 +254,11 @@ module Rezource
|
|
|
254
254
|
|
|
255
255
|
def resource_update(resource, attributes={})
|
|
256
256
|
before_update resource
|
|
257
|
-
resource.update_attributes attributes
|
|
257
|
+
resource.update_attributes attributes unless resource.errors.any?
|
|
258
258
|
after_update resource
|
|
259
259
|
|
|
260
260
|
if _action = params[:_action]
|
|
261
|
-
|
|
261
|
+
unless resource.errors.any?
|
|
262
262
|
respond_with *namespace, resource, location: action_resource_path(_action)
|
|
263
263
|
else
|
|
264
264
|
respond_with *namespace, resource, action: _action
|
data/rezource.gemspec
CHANGED
data/test/test_rezource.rb
CHANGED
|
@@ -521,10 +521,12 @@ class RezourceTest < Test::Unit::TestCase
|
|
|
521
521
|
|
|
522
522
|
def test_resource_create__valid
|
|
523
523
|
c = SamplesController.new
|
|
524
|
+
e = mock()
|
|
524
525
|
m = mock()
|
|
525
526
|
|
|
526
527
|
c.expects(:before_create).with(m)
|
|
527
|
-
m.expects(:
|
|
528
|
+
m.expects(:errors).returns e
|
|
529
|
+
e.expects(:any?).returns false
|
|
528
530
|
m.expects(:save)
|
|
529
531
|
c.expects(:after_create).with(m)
|
|
530
532
|
c.expects(:respond_with).with(*(c.namespace + [m]))
|
|
@@ -534,10 +536,13 @@ class RezourceTest < Test::Unit::TestCase
|
|
|
534
536
|
|
|
535
537
|
def test_resource_create__invalid
|
|
536
538
|
c = SamplesController.new
|
|
539
|
+
|
|
540
|
+
e = mock()
|
|
537
541
|
m = mock()
|
|
538
542
|
|
|
539
543
|
c.expects(:before_create).with(m)
|
|
540
|
-
m.expects(:
|
|
544
|
+
m.expects(:errors).returns e
|
|
545
|
+
e.expects(:any?).returns true
|
|
541
546
|
c.expects(:after_create).with(m)
|
|
542
547
|
c.expects(:respond_with).with(*(c.namespace + [m]))
|
|
543
548
|
|
|
@@ -547,10 +552,13 @@ class RezourceTest < Test::Unit::TestCase
|
|
|
547
552
|
def test_resource_update__valid
|
|
548
553
|
c = SamplesController.new
|
|
549
554
|
attributes = {name: "joe"}
|
|
555
|
+
|
|
550
556
|
m = mock()
|
|
557
|
+
e = mock()
|
|
551
558
|
|
|
552
559
|
c.expects(:before_update).with(m)
|
|
553
|
-
m.expects(:
|
|
560
|
+
m.expects(:errors).returns e
|
|
561
|
+
e.expects(:any?).returns false
|
|
554
562
|
m.expects(:update_attributes).with(attributes)
|
|
555
563
|
c.expects(:after_update).with(m)
|
|
556
564
|
c.expects(:respond_with).with(*(c.namespace + [m]))
|
|
@@ -562,9 +570,11 @@ class RezourceTest < Test::Unit::TestCase
|
|
|
562
570
|
c = SamplesController.new
|
|
563
571
|
attributes = {name: "joe"}
|
|
564
572
|
m = mock()
|
|
573
|
+
e = mock()
|
|
565
574
|
|
|
566
575
|
c.expects(:before_update).with(m)
|
|
567
|
-
m.expects(:
|
|
576
|
+
m.expects(:errors).returns e
|
|
577
|
+
e.expects(:any?).returns true
|
|
568
578
|
c.expects(:after_update).with(m)
|
|
569
579
|
c.expects(:respond_with).with(*(c.namespace + [m]))
|
|
570
580
|
|
|
@@ -575,13 +585,16 @@ class RezourceTest < Test::Unit::TestCase
|
|
|
575
585
|
c = SamplesController.new
|
|
576
586
|
attributes = {name: "joe"}
|
|
577
587
|
m = mock()
|
|
588
|
+
e = mock()
|
|
578
589
|
|
|
579
590
|
c.expects(:before_update).with(m)
|
|
580
|
-
m.expects(:
|
|
591
|
+
m.expects(:errors).returns e
|
|
592
|
+
e.expects(:any?).returns true
|
|
581
593
|
c.expects(:after_update).with(m)
|
|
582
594
|
|
|
583
595
|
c.expects(:"params").returns({_action: "custom"})
|
|
584
|
-
m.expects(:
|
|
596
|
+
m.expects(:errors).returns e
|
|
597
|
+
e.expects(:any?).returns true
|
|
585
598
|
|
|
586
599
|
c.expects(:respond_with).with(*(c.namespace + [m] + [{action: "custom"}]))
|
|
587
600
|
|
|
@@ -592,14 +605,18 @@ class RezourceTest < Test::Unit::TestCase
|
|
|
592
605
|
c = SamplesController.new
|
|
593
606
|
attributes = {name: "joe"}
|
|
594
607
|
m = mock()
|
|
608
|
+
e = mock()
|
|
595
609
|
|
|
596
610
|
c.expects(:before_update).with(m)
|
|
597
|
-
|
|
611
|
+
|
|
612
|
+
m.expects(:errors).returns e
|
|
613
|
+
e.expects(:any?).returns false
|
|
598
614
|
m.expects(:update_attributes).with(attributes)
|
|
599
615
|
c.expects(:after_update).with(m)
|
|
600
616
|
|
|
601
617
|
c.expects(:"params").returns({_action: "custom"})
|
|
602
|
-
m.expects(:
|
|
618
|
+
m.expects(:errors).returns e
|
|
619
|
+
e.expects(:any?).returns false
|
|
603
620
|
|
|
604
621
|
c.expects(:"custom_sample_path").returns "/sample/XYZ/custom"
|
|
605
622
|
|
metadata
CHANGED
|
@@ -1,7 +1,7 @@
|
|
|
1
1
|
--- !ruby/object:Gem::Specification
|
|
2
2
|
name: rezource
|
|
3
3
|
version: !ruby/object:Gem::Version
|
|
4
|
-
version: 0.0.
|
|
4
|
+
version: 0.0.6
|
|
5
5
|
prerelease:
|
|
6
6
|
platform: ruby
|
|
7
7
|
authors:
|
|
@@ -9,7 +9,7 @@ authors:
|
|
|
9
9
|
autorequire:
|
|
10
10
|
bindir: bin
|
|
11
11
|
cert_chain: []
|
|
12
|
-
date: 2013-08-
|
|
12
|
+
date: 2013-08-21 00:00:00.000000000 Z
|
|
13
13
|
dependencies: []
|
|
14
14
|
description: Make Rails restful controller even more simple
|
|
15
15
|
email: jan.zimmek@web.de
|