passive_record 0.3.9 → 0.3.10
Sign up to get free protection for your applications and to get access to all the features.
- checksums.yaml +4 -4
- data/lib/passive_record.rb +0 -14
- data/lib/passive_record/class_methods.rb +2 -2
- data/lib/passive_record/core/identifier.rb +3 -9
- data/lib/passive_record/version.rb +1 -1
- data/spec/passive_record_spec.rb +4 -17
- metadata +1 -1
checksums.yaml
CHANGED
@@ -1,7 +1,7 @@
|
|
1
1
|
---
|
2
2
|
SHA1:
|
3
|
-
metadata.gz:
|
4
|
-
data.tar.gz:
|
3
|
+
metadata.gz: 7a864962f49adde921cd337981fea7046b072a91
|
4
|
+
data.tar.gz: f6dfa54709c92ab2628fbe07a2831f71f714b930
|
5
5
|
SHA512:
|
6
|
-
metadata.gz:
|
7
|
-
data.tar.gz:
|
6
|
+
metadata.gz: 9aba9642fa965b2b5460e5eec9521d426fe3041dfb53c5e234000eebb36d41caba198f88f011a399900dfdea4c6b68f721aa3421873d3a7edc26a3f54ee3ac04
|
7
|
+
data.tar.gz: 19fb03f821db6c570d24a546960d3b729c0b4e78f29189bee1c9ae820d87d227b74fb765b1eade54c6db602d903653d7683387f741c066253a6c6fb19b6b657e
|
data/lib/passive_record.rb
CHANGED
@@ -40,18 +40,4 @@ module PassiveRecord
|
|
40
40
|
def self.drop_all
|
41
41
|
(model_classes + model_classes.flat_map(&:descendants)).each(&:destroy_all)
|
42
42
|
end
|
43
|
-
|
44
|
-
def self.configure
|
45
|
-
yield configuration
|
46
|
-
end
|
47
|
-
|
48
|
-
def self.configuration
|
49
|
-
@config ||= default_configuration
|
50
|
-
end
|
51
|
-
|
52
|
-
def self.default_configuration
|
53
|
-
OpenStruct.new(
|
54
|
-
:identify_using => Identifier
|
55
|
-
)
|
56
|
-
end
|
57
43
|
end
|
@@ -53,7 +53,7 @@ module PassiveRecord
|
|
53
53
|
instance = new
|
54
54
|
|
55
55
|
instance.singleton_class.class_eval { attr_accessor :id }
|
56
|
-
instance.send(:"id=",
|
56
|
+
instance.send(:"id=", SecureRandomIdentifier.generate(self))
|
57
57
|
|
58
58
|
register(instance)
|
59
59
|
|
@@ -73,7 +73,7 @@ module PassiveRecord
|
|
73
73
|
end
|
74
74
|
|
75
75
|
def destroy(id)
|
76
|
-
@instances
|
76
|
+
@instances.reject! {|k,_| id == k }
|
77
77
|
end
|
78
78
|
|
79
79
|
def destroy_all
|
@@ -1,11 +1,11 @@
|
|
1
1
|
module PassiveRecord
|
2
|
-
class
|
2
|
+
class SecureRandomIdentifier < Struct.new(:value)
|
3
3
|
def self.generate(klass)
|
4
4
|
new(generate_id_value_for(klass))
|
5
5
|
end
|
6
6
|
|
7
|
-
def self.generate_id_value_for(
|
8
|
-
|
7
|
+
def self.generate_id_value_for(*)
|
8
|
+
SecureRandom.uuid
|
9
9
|
end
|
10
10
|
|
11
11
|
def ==(other_id)
|
@@ -16,10 +16,4 @@ module PassiveRecord
|
|
16
16
|
value
|
17
17
|
end
|
18
18
|
end
|
19
|
-
|
20
|
-
class SecureRandomIdentifier < Identifier
|
21
|
-
def self.generate_id_value_for(*)
|
22
|
-
SecureRandom.uuid
|
23
|
-
end
|
24
|
-
end
|
25
19
|
end
|
data/spec/passive_record_spec.rb
CHANGED
@@ -44,6 +44,7 @@ describe "passive record models" do
|
|
44
44
|
doomed.destroy
|
45
45
|
expect(SimpleModel.find(doomed_id)).to eq(nil)
|
46
46
|
|
47
|
+
SimpleModel.destroy_all
|
47
48
|
expect{10.times{SimpleModel.create}}.to change{SimpleModel.count}.by(10)
|
48
49
|
end
|
49
50
|
end
|
@@ -423,23 +424,9 @@ describe "passive record models" do
|
|
423
424
|
end
|
424
425
|
|
425
426
|
describe "configuration" do
|
426
|
-
|
427
|
-
|
428
|
-
|
429
|
-
expect(Model.create.id).to be_a(PassiveRecord::Identifier)
|
430
|
-
end
|
431
|
-
end
|
432
|
-
|
433
|
-
context 'with a new id factory configured' do
|
434
|
-
before do
|
435
|
-
PassiveRecord.configure do |passive_records|
|
436
|
-
passive_records.identify_using = PassiveRecord::SecureRandomIdentifier
|
437
|
-
end
|
438
|
-
end
|
439
|
-
|
440
|
-
it 'should generate uuids' do
|
441
|
-
expect(Model.create.id).to be_a(PassiveRecord::SecureRandomIdentifier)
|
442
|
-
end
|
427
|
+
context 'with default config' do
|
428
|
+
it 'should generate simple identifiers' do
|
429
|
+
expect(Model.create.id).to be_a(PassiveRecord::SecureRandomIdentifier)
|
443
430
|
end
|
444
431
|
end
|
445
432
|
end
|