passive_record 0.3.8 → 0.3.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: f8aef98c4504dee98e437999a2fbfc3fb087363a
-  data.tar.gz: 737920031f79621eff0163e2547a93a86cc2829b
+  metadata.gz: 4f7bcf7c09ce2830df7940f8241d34b3dd79f3b0
+  data.tar.gz: 5d9844ff88d1aa22fc003765a4746ff1918d1e1b
 SHA512:
-  metadata.gz: 619873a35e03b3c0ed16ae23e9f28ff97abcbe2525a581beaa91889c508c77ad74565d043d10f619ffedf44f7c7828a6e1a65374f01ca849b8c0d7b8c61cbbd2
-  data.tar.gz: f3c817c0b0a99a41063969946f6420fce4847379f5d6bce11e431996a8ff6978846bfe2347b67db0a22d82ff0882fb53ab4bc215711dc627f94d2cb74d981c75
+  metadata.gz: ea71ea386493db3d619f9cb1fa58d738004622a822e31c309b730d2742e5585728c2e9a59f44e4d44b3bc31bfd0728c1b09808089fc9dd539ff3f27a9fdc63e8
+  data.tar.gz: 92042f4052e8e79780bdcc182fc8b784d60aad57b435c7acd0cd5324683df34b38e9472dba3b86f24f50eeaf76ba21a7498f015d54e29421186c4f92cb79b566

data/README.md

@@ -1,8 +1,7 @@
 # passive_record
 
-* [Homepage](https://rubygems.org/gems/passive_record)
-* [Documentation](http://rubydoc.info/gems/passive_record/frames)
-* [Email](mailto:jweissman1986 at gmail.com)
+* [Documentation](https://rubygems.org/gems/passive_record)
+* [Email](mailto:joe at deepc.io)
 
 [![Code Climate GPA](https://codeclimate.com/github/deepcerulean/passive_record/badges/gpa.svg)](https://codeclimate.com/github/deepcerulean/passive_record)
 [![Codeship Status for deepcerulean/passive_record](https://www.codeship.io/projects/66bb2d90-ba61-0133-af95-025ac38368ea/status)](https://codeship.com/projects/128700)
@@ -87,10 +86,12 @@ PassiveRecord may be right for you!
 
 ## PassiveRecord API
 
+
   A class including PassiveRecord will gain the following new instance and class methods.
 
 ### Instance Methods
 
+
   A class `Role` which is declared to `include PassiveRecord` will gain the following instance methods:
   - `role.update(attrs_hash)`
   - `role.to_h`
@@ -98,16 +99,19 @@ PassiveRecord may be right for you!
 
 ### Class Methods
 
+
   A class `User` which is declared to `include PassiveRecord` will gain the following class methods:
+
+  - `User.all` and `User.each`
   - `User.create(attrs_hash)`
+  - `User.descendants`
+  - `User.destroy(id)`
+  - `User.destroy_all`
+  - `User.each` enumerates over `User.all`, giving `User.count`, `User.first`, etc.
   - `User.find(id_or_ids)`
-  - `User.find_by(conditions_hash)`
   - `User.find_all_by(conditions_hash)`
-  - `User.all` and `User.each`
-  - `User.each` enumerates over `User.all`, giving `User.count`, `User.first`, etc.
+  - `User.find_by(conditions_hash)`
   - `User.where(conditions_hash)` (returns a `PassiveRecord::Query` object)
-  - `User.descendants`
-  - `User.destroy_all`
 
 ### Belongs To
 
@@ -133,11 +137,11 @@ PassiveRecord may be right for you!
   A model `Parent` which declares `has_many :children` or `has_and_belongs_to_many :children` will gain:
 
   - `parent.children` (returns a `Relation`, documented below)
-  - `parent.children_ids`
   - `parent.children=`
+  - `parent.children_ids`
   - `parent.children_ids=`
+  - `parent.children<<`
   - `parent.create_child(attrs)`
-  - `parent.children<<` (insert a related model)
   - `parent.children.all?(&predicate)`
   - `parent.children.empty?`
   - `parent.children.where(conditions)` (returns a `Core::Query`)

data/lib/passive_record/class_methods.rb

@@ -73,7 +73,7 @@ module PassiveRecord
     end
 
     def destroy(id)
-      @instances.delete(id)
+      @instances = instances_by_id.reject{|k,_| id == k }
     end
 
     def destroy_all

data/lib/passive_record/version.rb

@@ -1,4 +1,4 @@
 module PassiveRecord
   # passive_record version
-  VERSION = "0.3.8"
+  VERSION = "0.3.9"
 end

data/spec/passive_record_spec.rb

@@ -26,7 +26,8 @@ describe "passive record models" do
     describe "instance methods" do
       describe "#update" do
         it 'should update attrs' do
-          expect {model.update(foo: '123')}.to change {model.foo}.from(value).to('123')
+          expect {model.update(foo: '123')}.
+            to change {model.foo}.from(value).to('123')
         end
 
         it 'should invoke callbacks' do
@@ -38,9 +39,12 @@ describe "passive record models" do
       describe "#destroy" do
         it 'should remove the entity and freeze it' do
           doomed = SimpleModel.create
-          expect(SimpleModel.find(doomed.id)).to eq(doomed)
+          doomed_id = doomed.id
+          expect(SimpleModel.find(doomed_id)).to eq(doomed)
           doomed.destroy
-          expect(SimpleModel.find(doomed.id)).to eq(nil)
+          expect(SimpleModel.find(doomed_id)).to eq(nil)
+
+          expect{10.times{SimpleModel.create}}.to change{SimpleModel.count}.by(10)
         end
       end
 
@@ -51,13 +55,16 @@ describe "passive record models" do
 
         it 'should report relations' do
           dog = Dog.create
-          expect(dog.inspect).to eq("Family::Dog (id: #{dog.id.inspect}, breed: \"#{dog.breed}\", created_at: #{dog.created_at}, sound: \"bark\", child_id: nil)")
+          expect(dog.inspect).
+            to eq("Family::Dog (id: #{dog.id.inspect}, breed: \"#{dog.breed}\", created_at: #{dog.created_at}, sound: \"bark\", child_id: nil)")
 
           child = Child.create
           child.dogs << dog
-          expect(dog.inspect).to eq("Family::Dog (id: #{dog.id.inspect}, breed: \"#{dog.breed}\", created_at: #{dog.created_at}, sound: \"bark\", child_id: #{child.id.inspect})")
+          expect(dog.inspect).
+            to eq("Family::Dog (id: #{dog.id.inspect}, breed: \"#{dog.breed}\", created_at: #{dog.created_at}, sound: \"bark\", child_id: #{child.id.inspect})")
 
-          expect(child.inspect).to eq("Family::Child (id: #{child.id.inspect}, created_at: #{child.created_at}, name: \"Alice\", toy_id: nil, dog_ids: [#{dog.id.inspect}], parent_id: nil)")
+          expect(child.inspect).
+            to eq("Family::Child (id: #{child.id.inspect}, created_at: #{child.created_at}, name: \"Alice\", toy_id: nil, dog_ids: [#{dog.id.inspect}], parent_id: nil)")
         end
       end
 
@@ -395,6 +402,7 @@ describe "passive record models" do
     end
 
     context 'direct habtm' do
+      before(:each) { PassiveRecord.drop_all }
       let!(:user) { User.create roles: [role] }
       let(:role) { Role.create }
       let(:another_user) { User.create }
@@ -404,9 +412,12 @@ describe "passive record models" do
         expect(user.roles).to include(role)
         expect(role.user_ids).to eq([user.id])
         expect(user.role_ids).to eq([role.id])
-
         expect {role.users << another_user}.to change{role.users.count}.by(1)
       end
+
+      it 'should handle inverse relations' do
+        expect {role.users << another_user}.to change{another_user.roles.count}.by(1)
+      end
     end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: passive_record
 version: !ruby/object:Gem::Version
-  version: 0.3.8
+  version: 0.3.9
 platform: ruby
 authors:
 - Joseph Weissman
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-02-26 00:00:00.000000000 Z
+date: 2016-02-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport