datamappify 0.60.0 → 0.70.0.beta1

data/spec/repository/finders/different_providers_spec.rb

@@ -0,0 +1,81 @@
+require 'spec_helper'
+
+describe "attributes from different data providers" do
+  before do
+    pending
+
+    HeroUserRepository.save!(HeroUser.new(:first_name => 'Fred', :last_name => 'Wu'))
+    HeroUserRepository.save!(HeroUser.new(:first_name => 'Sheldon', :last_name => 'Cooper'))
+    HeroUserRepository.save!(HeroUser.new(:first_name => 'Fred', :last_name => 'Cooper'))
+  end
+
+  context "example 1" do
+    let(:records) { HeroUserRepository.find(:first_name => 'Fred') }
+    subject       { records }
+
+    it { should have(2).users }
+
+    describe "record" do
+      subject { records.first }
+
+      its(:first_name) { should == 'Fred' }
+      its(:last_name)  { should == 'Wu' }
+    end
+  end
+
+  context "example 2" do
+    let(:records) { HeroUserRepository.find(:first_name => 'Sheldon') }
+    subject       { records }
+
+    it { should have(1).user }
+
+    describe "record" do
+      subject { records.first }
+
+      its(:first_name) { should == 'Sheldon' }
+      its(:last_name)  { should == 'Cooper' }
+    end
+  end
+
+  context "example 3" do
+    let(:records) { HeroUserRepository.find(:last_name => 'Cooper') }
+    subject       { records }
+
+    it { should have(2).users }
+
+    describe "record" do
+      subject { records.first }
+
+      its(:first_name) { should == 'Sheldon' }
+      its(:last_name)  { should == 'Cooper' }
+    end
+  end
+
+  context "example 4" do
+    let(:records) { HeroUserRepository.find(:first_name => 'Sheldon', :last_name => 'Cooper') }
+    subject       { records }
+
+    it { should have(1).user }
+
+    describe "record" do
+      subject { records.first }
+
+      its(:first_name) { should == 'Sheldon' }
+      its(:last_name)  { should == 'Cooper' }
+    end
+  end
+
+  context "example 5" do
+    let(:records) { HeroUserRepository.find(:first_name => 'Leonard', :last_name => 'Cooper') }
+    subject       { records }
+
+    it { should be_empty }
+  end
+
+  context "example 6" do
+    let(:records) { HeroUserRepository.find(:first_name => 'Fred', :last_name => 'Woo') }
+    subject       { records }
+
+    it { should be_empty }
+  end
+end

data/spec/repository/finders/find_spec.rb

@@ -0,0 +1,21 @@
+require 'spec_helper'
+
+shared_examples_for "finder find" do |data_provider|
+  include_context "user repository for finders", data_provider
+
+  context "#{data_provider}" do
+    describe "#find" do
+      describe "by id" do
+        subject { user_repository.find(existing_user.id) }
+
+        its(:id) { should == existing_user.id }
+      end
+    end
+  end
+end
+
+describe Datamappify::Repository do
+  DATA_PROVIDERS.each do |data_provider|
+    it_behaves_like "finder find", data_provider
+  end
+end

data/spec/repository/finders/match_spec.rb

@@ -0,0 +1,28 @@
+require 'spec_helper'
+
+shared_examples_for "finder match" do |data_provider|
+  include_context "user repository for finders", data_provider
+
+  context "#{data_provider}" do
+    describe "#match" do
+      context "case sensitive" do
+        include_context "where or match context"
+        it_behaves_like "where or match finder", :match, data_provider
+      end
+
+      context "case insensitive" do
+        let(:bob)        { 'bob' }
+        let(:license)    { 'yeslicense' }
+        let(:no_license) { 'No_License' }
+
+        it_behaves_like "where or match finder", :match, data_provider
+      end
+    end
+  end
+end
+
+describe Datamappify::Repository do
+  DATA_PROVIDERS.each do |data_provider|
+    it_behaves_like "finder match", data_provider
+  end
+end

data/spec/repository/finders/where_spec.rb

@@ -0,0 +1,18 @@
+require 'spec_helper'
+
+shared_examples_for "finder where" do |data_provider|
+  include_context "user repository for finders", data_provider
+
+  context "#{data_provider}" do
+    describe "#where" do
+      include_context "where or match context"
+      it_behaves_like "where or match finder", :where, data_provider
+    end
+  end
+end
+
+describe Datamappify::Repository do
+  DATA_PROVIDERS.each do |data_provider|
+    it_behaves_like "finder where", data_provider
+  end
+end

data/spec/repository/reverse_mapped_spec.rb

@@ -6,6 +6,8 @@ shared_examples_for "reverse mapped" do |data_provider|
   let(:new_author)         { Reversed::Author.new(:name => 'George R. R. Martin', :bio => 'GoT') }
   let(:new_post)           { Reversed::Post.new(:title => 'Hello world', :author_name => 'Fred Wu', :author_bio => 'x')}
   let(:new_post_2)         { Reversed::Post.new(:title => 'Hello earth', :author_name => 'Sheldon Cooper', :author_bio => 'y')}
+  let(:data_authors)       { "Datamappify::Data::Record::#{data_provider}::Reversed::Author".constantize }
+  let(:data_posts)         { "Datamappify::Data::Record::#{data_provider}::Reversed::Post".constantize }
 
   context "#{data_provider}" do
     before do
@@ -46,9 +48,24 @@ shared_examples_for "reverse mapped" do |data_provider|
         end
 
         describe "update record" do
+          let(:record) { post_repository.find(saved_post.id) }
+
+          describe "data records" do
+            it "does not create extra primary data records" do
+              expect { post_repository.save!(record) }.to change { data_posts.count }.by(0)
+            end
+
+            it "does not create extra secondary data records" do
+              expect { post_repository.save!(record) }.to change { data_authors.count }.by(0)
+            end
+
+            it "does not change data record's foreign key value" do
+              expect { post_repository.save!(record) }.not_to change { data_posts.last.author_id }
+            end
+          end
+
           context "example 1" do
-            let(:record) { post_repository.find(saved_post.id) }
-            subject      { post_repository.save!(record) }
+            subject { post_repository.save!(record) }
 
             its(:title)       { should == 'Hello world' }
             its(:author_name) { should == 'Fred Wu' }
@@ -56,8 +73,7 @@ shared_examples_for "reverse mapped" do |data_provider|
           end
 
           context "example 2" do
-            let(:record) { post_repository.find(saved_post.id) }
-            subject      { post_repository.find(record.id) }
+            subject { post_repository.find(record.id) }
 
             before do
               record.title = 'Hello mars'

data/spec/support/entities/group.rb

@@ -1,5 +1,9 @@
+require_relative 'super_user'
+
 class Group
   include Datamappify::Entity
 
   attribute :name, String
+
+  has_many :users, :via => SuperUser
 end

data/spec/support/entities/super_user.rb

@@ -0,0 +1,9 @@
+require_relative 'user'
+
+class SuperUser
+  include Datamappify::Entity
+
+  attributes_from User
+
+  belongs_to :group
+end

data/spec/support/entities/user.rb

@@ -1,3 +1,5 @@
+require_relative 'user_info'
+
 class User
   include Datamappify::Entity
 
@@ -15,6 +17,9 @@ class User
   validates :driver_license, :presence => true,
                              :length   => { :minimum => 8 }
 
+  attributes_from UserInfo, :prefix_with => :personal
+  attributes_from UserInfo, :prefix_with => :business
+
   def full_name
     "#{first_name} #{last_name}"
   end

data/spec/support/entities/user_info.rb

@@ -0,0 +1,5 @@
+class UserInfo
+  include Datamappify::Entity
+
+  attribute :content, String
+end

data/spec/support/repositories/active_record/group_repository.rb

@@ -1,6 +1,10 @@
+require_relative 'super_user_repository'
+
 class GroupRepositoryActiveRecord
   include Datamappify::Repository
 
   for_entity Group
   default_provider :ActiveRecord
+
+  references :users, :via => SuperUserRepositoryActiveRecord
 end

data/spec/support/repositories/active_record/super_user_repository.rb

@@ -0,0 +1,23 @@
+require_relative 'user_repository'
+
+class SuperUserRepositoryActiveRecord
+  include Datamappify::Repository
+
+  for_entity SuperUser
+  default_provider :ActiveRecord
+
+  group :via => :base_user_id do
+    map_attribute :first_name, :to => 'User#first_name'
+    map_attribute :last_name,  :to => 'User#surname'
+    map_attribute :age,        :to => 'User#age'
+  end
+
+  group :reference_key => :user_id do
+    map_attribute :driver_license, :to => 'UserDriverLicense#number'
+    map_attribute :passport,       :to => 'UserPassport#number'
+    map_attribute :health_care,    :to => 'UserHealthCare#number'
+  end
+
+  map_attribute :personal_content, :to => 'UserInfo#content', :via => :personal_info_id
+  map_attribute :business_content, :to => 'UserInfo#content', :via => :business_info_id
+end

data/spec/support/repositories/active_record/user_repository.rb

@@ -4,8 +4,10 @@ class UserRepositoryActiveRecord
   for_entity User
   default_provider :ActiveRecord
 
-  map_attribute :last_name,      :to => 'User#surname'
-  map_attribute :driver_license, :to => 'UserDriverLicense#number'
-  map_attribute :passport,       :to => 'UserPassport#number'
-  map_attribute :health_care,    :to => 'UserHealthCare#number'
+  map_attribute :last_name,        :to => 'User#surname'
+  map_attribute :driver_license,   :to => 'UserDriverLicense#number'
+  map_attribute :passport,         :to => 'UserPassport#number'
+  map_attribute :health_care,      :to => 'UserHealthCare#number'
+  map_attribute :personal_content, :to => 'UserInfo#content', :via => :personal_info_id
+  map_attribute :business_content, :to => 'UserInfo#content', :via => :business_info_id
 end

data/spec/support/repositories/sequel/group_repository.rb

@@ -1,6 +1,10 @@
+require_relative 'super_user_repository'
+
 class GroupRepositorySequel
   include Datamappify::Repository
 
   for_entity Group
   default_provider :Sequel
+
+  references :users, :via => SuperUserRepositorySequel
 end

data/spec/support/repositories/sequel/super_user_repository.rb

@@ -0,0 +1,23 @@
+require_relative 'user_repository'
+
+class SuperUserRepositorySequel
+  include Datamappify::Repository
+
+  for_entity SuperUser
+  default_provider :Sequel
+
+  group :via => :base_user_id do
+    map_attribute :first_name, :to => 'User#first_name'
+    map_attribute :last_name,  :to => 'User#surname'
+    map_attribute :age,        :to => 'User#age'
+  end
+
+  group :reference_key => :user_id do
+    map_attribute :driver_license, :to => 'UserDriverLicense#number'
+    map_attribute :passport,       :to => 'UserPassport#number'
+    map_attribute :health_care,    :to => 'UserHealthCare#number'
+  end
+
+  map_attribute :personal_content, :to => 'UserInfo#content', :via => :personal_info_id
+  map_attribute :business_content, :to => 'UserInfo#content', :via => :business_info_id
+end

data/spec/support/repositories/sequel/user_repository.rb

@@ -4,8 +4,10 @@ class UserRepositorySequel
   for_entity User
   default_provider :Sequel
 
-  map_attribute :last_name,      :to => 'User#surname'
-  map_attribute :driver_license, :to => 'UserDriverLicense#number'
-  map_attribute :passport,       :to => 'UserPassport#number'
-  map_attribute :health_care,    :to => 'UserHealthCare#number'
+  map_attribute :last_name,        :to => 'User#surname'
+  map_attribute :driver_license,   :to => 'UserDriverLicense#number'
+  map_attribute :passport,         :to => 'UserPassport#number'
+  map_attribute :health_care,      :to => 'UserHealthCare#number'
+  map_attribute :personal_content, :to => 'UserInfo#content', :via => :personal_info_id
+  map_attribute :business_content, :to => 'UserInfo#content', :via => :business_info_id
 end

data/spec/support/shared/contexts.rb

@@ -1,5 +1,5 @@
 shared_context "user repository" do |data_provider|
-  let!(:user_repository)     { "UserRepository#{data_provider}".constantize }
+  let(:user_repository)      { "UserRepository#{data_provider}".constantize }
   let!(:existing_user)       { user_repository.save(User.new(:first_name => 'Fred', :driver_license => 'FREDWU42')) }
   let(:new_valid_user)       { User.new(:first_name => 'Batman', :driver_license => 'ARKHAMCITY') }
   let(:new_valid_user2)      { User.new(:first_name => 'Ironman', :driver_license => 'NEWYORKCITY') }
@@ -8,3 +8,10 @@ shared_context "user repository" do |data_provider|
   let(:data_passports)       { "Datamappify::Data::Record::#{data_provider}::UserPassport".constantize }
   let(:data_driver_licenses) { "Datamappify::Data::Record::#{data_provider}::UserDriverLicense".constantize }
 end
+
+shared_context "user repository for finders" do |data_provider|
+  include_context "user repository", data_provider
+
+  let!(:existing_user_1) { user_repository.save(new_valid_user.dup) }
+  let!(:existing_user_2) { user_repository.save(new_valid_user.dup) }
+end

data/spec/support/shared/examples/repository/finders/where_or_match.rb

@@ -0,0 +1,130 @@
+shared_context "where or match context" do
+  let(:bob)        { 'Bob' }
+  let(:license)    { 'YESLICENSE' }
+  let(:no_license) { 'NO_LICENSE' }
+end
+
+shared_examples_for "where or match finder" do |finder, data_provider|
+  subject { records }
+
+  before do
+    user_repository.save!(User.new(:first_name => 'Mary', :driver_license => 'YESLICENSE'))
+    user_repository.save!(User.new(:first_name => 'Bob',  :driver_license => 'YESLICENSE', :personal_content => 'Ok'))
+    user_repository.save!(User.new(:first_name => 'Jane', :driver_license => 'NO_LICENSE', :business_content => 'Ok'))
+    user_repository.save!(User.new(:first_name => 'Bob',  :driver_license => 'NO_LICENSE'))
+    user_repository.save!(User.new(:first_name => 'Bob',  :driver_license => 'YESLICENSE', :business_content => 'No'))
+    user_repository.save!(User.new(:first_name => 'Bobb', :driver_license => 'YESLICENSE'))
+    user_repository.save!(User.new(:first_name => 'John', :driver_license => 'YESLICENSE'))
+  end
+
+  describe "by attribute that does not exist" do
+    let(:records) { user_repository.send(finder, :blah => bob) }
+
+    it { expect { records }.to raise_exception(Datamappify::Data::EntityAttributeInvalid) }
+  end
+
+  describe "by primary attribute" do
+    let(:records) { user_repository.send(finder, :first_name => bob) }
+
+    it { should have(3).users }
+
+    describe "record" do
+      subject { records.first }
+
+      its(:first_name)     { should == 'Bob' }
+      its(:driver_license) { should == 'YESLICENSE' }
+    end
+  end
+
+  describe "by secondary attribute" do
+    let(:records) { user_repository.send(finder, :driver_license => no_license) }
+
+    it { should have(2).users }
+
+    describe "record" do
+      subject { records.first }
+
+      its(:first_name)     { should == 'Jane' }
+      its(:driver_license) { should == 'NO_LICENSE' }
+    end
+  end
+
+  describe "by primary and secondary attributes" do
+    context "example 1" do
+      let(:records) { user_repository.send(finder, :first_name => bob, :driver_license => no_license) }
+      subject       { records }
+
+      it { should have(1).user }
+
+      describe "record" do
+        subject { records.first }
+
+        its(:first_name)     { should == 'Bob' }
+        its(:driver_license) { should == 'NO_LICENSE' }
+      end
+    end
+
+    context "example 2" do
+      let(:records) { user_repository.send(finder, :first_name => bob, :driver_license => license) }
+      subject       { records }
+
+      it { should have(2).users }
+
+      describe "record" do
+        subject { records.first }
+
+        its(:first_name)     { should == 'Bob' }
+        its(:driver_license) { should == 'YESLICENSE' }
+      end
+    end
+
+    context "example 3" do
+      subject { user_repository.send(finder, :first_name => 'Nope', :driver_license => license) }
+
+      it { should be_empty }
+    end
+
+    context "example 4" do
+      subject { user_repository.send(finder, :first_name => bob, :driver_license => 'NOPE') }
+
+      it { should be_empty }
+    end
+
+    context "example 5 (string keys)" do
+      let(:records) { user_repository.send(finder, 'first_name' => bob, 'driver_license' => license) }
+
+      it { should have(2).users }
+
+      describe "record" do
+        subject { records.first }
+
+        its(:first_name)     { should == 'Bob' }
+        its(:driver_license) { should == 'YESLICENSE' }
+      end
+    end
+  end
+
+  describe "by reverse mapped attribute" do
+    let(:records) { user_repository.send(finder, :business_content => 'Ok') }
+
+    it { should have(1).user }
+
+    context "record" do
+      subject { records.first }
+
+      its(:personal_content) { should be_nil }
+    end
+
+    context "data record" do
+      subject { "Datamappify::Data::Record::#{data_provider}::User".constantize.last }
+
+      it "has proper personal_info association" do
+        -> { subject.personal_info }.should_not raise_error
+      end
+
+      it "has proper business_info association" do
+        -> { subject.business_info }.should_not raise_error
+      end
+    end
+  end
+end

data/spec/support/tables/active_record.rb

@@ -13,18 +13,33 @@ ActiveRecord::Migration.suppress_messages do
       t.timestamps
     end
 
+    create_table :super_users do |t|
+      t.references :personal_info
+      t.references :business_info
+      t.references :base_user
+      t.references :group
+      t.timestamps
+    end
+
     create_table :users do |t|
       t.string :first_name, :null => false
       t.string :surname
       t.integer :age
       t.integer :level
-      t.references :role
+      t.references :personal_info
+      t.references :business_info
+      t.timestamps
+    end
+
+    create_table :user_infos do |t|
+      t.string :content
+      t.references :user
       t.timestamps
     end
 
     create_table :comments do |t|
       t.string :content
-      t.belongs_to :user
+      t.references :user
       t.timestamps
     end
 
@@ -38,29 +53,24 @@ ActiveRecord::Migration.suppress_messages do
       t.timestamps
     end
 
-    create_table :groups_users do |t|
-      t.belongs_to :user
-      t.belongs_to :group
-    end
-
     create_table :user_driver_licenses do |t|
       t.string :number
       t.string :expiry
-      t.belongs_to :user
+      t.references :user
       t.timestamps
     end
 
     create_table :user_passports do |t|
       t.string :number
       t.string :expiry, :null => false
-      t.belongs_to :user
+      t.references :user
       t.timestamps
     end
 
     create_table :user_health_cares do |t|
       t.string :number
       t.string :expiry
-      t.belongs_to :user
+      t.references :user
       t.timestamps
     end
 

data/spec/support/tables/sequel.rb

@@ -11,13 +11,32 @@ DB.create_table :hero_user_last_names do
   DateTime :updated_at
 end
 
+DB.create_table :super_users do
+  primary_key :id
+  foreign_key :personal_info_id
+  foreign_key :business_info_id
+  foreign_key :base_user_id
+  foreign_key :group_id
+  DateTime :created_at
+  DateTime :updated_at
+end
+
 DB.create_table :users do
   primary_key :id
   String :first_name, :null => false
   String :surname
   Integer :age
   Integer :level
-  foreign_key :role_id
+  foreign_key :personal_info_id
+  foreign_key :business_info_id
+  DateTime :created_at
+  DateTime :updated_at
+end
+
+DB.create_table :user_infos do
+  primary_key :id
+  String :content
+  foreign_key :user_id
   DateTime :created_at
   DateTime :updated_at
 end
@@ -44,12 +63,6 @@ DB.create_table :groups do
   DateTime :updated_at
 end
 
-DB.create_table :groups_users do
-  primary_key :id
-  foreign_key :user_id
-  foreign_key :group_id
-end
-
 DB.create_table :user_driver_licenses do
   primary_key :id
   String :number

data/spec/unit/entity/{relation_spec.rb → association/reference_spec.rb}

@@ -1,21 +1,21 @@
 require 'spec_helper'
 
 module Datamappify::Entity
-  describe Datamappify::Entity::Relation do
-    class DummyEntity
+  describe Datamappify::Entity::Association::Reference do
+    class DummyEntityA
       include Datamappify::Entity
 
       references :another_entity
     end
 
-    class AnotherEntity
+    class DummyEntityB
       include Datamappify::Entity
     end
 
-    subject { DummyEntity.new }
+    subject { DummyEntityA.new }
 
     describe "#references" do
-      let(:another_entity) { AnotherEntity.new.tap { |e| e.id = 42 } }
+      let(:another_entity) { DummyEntityB.new.tap { |e| e.id = 42 } }
 
       it { should respond_to(:another_entity_id) }
       it { should respond_to(:another_entity_id=) }

data/spec/unit/repository/query_method_spec.rb

@@ -37,8 +37,15 @@ module Datamappify::Repository::QueryMethod
       it { should_not be_a_writer }
     end
 
-    describe FindMultiple do
-      subject { FindMultiple }
+    describe Where do
+      subject { Where }
+
+      it { should     be_a_reader }
+      it { should_not be_a_writer }
+    end
+
+    describe Criteria do
+      subject { Criteria }
 
       it { should     be_a_reader }
       it { should_not be_a_writer }