tms_bridge 0.1.2 → 0.2.0

checksums.yaml

@@ -1,15 +1,7 @@
 ---
-!binary "U0hBMQ==":
-  metadata.gz: !binary |-
-    MTAzYWNhZDAzOGU1MDMwYWZiNTk1NTZmOTg1Yzg0MGM5MDYyZGQ2Nw==
-  data.tar.gz: !binary |-
-    ZTY1YzI1YWE1ZGJlMzg4NjI0NGQ3ZTMxNjJkYmUxMDAyNjgyMzAxMQ==
+SHA1:
+  metadata.gz: cd09e8601f91427278f8d3bfdfaa9523637e8647
+  data.tar.gz: 5e01784d862d2dd3ef97acd2b7989f8306f24008
 SHA512:
-  metadata.gz: !binary |-
-    YzU1NDE0NTFiZmUxNjkzZjQwOWU3MWNhMWQ5MDk2YmUyNjc3MGJkNTlkOTg1
-    N2FhNDUzZDFkODU5MDUxNWE5ZmQyNWI5MThlY2Y0Y2Q2MGI1OGZiMWEyYjIx
-    MmEyNDFhMzNjOWU0Yjg5MGZiMDJkODUwYjYzNWY1MzkwZTgwMjg=
-  data.tar.gz: !binary |-
-    OGQwYWFiMjlmNTQ4ZGU4OTM1ODdhZWUwOGJjM2RkODZhZjA5YWYxMTBiMTBj
-    ZDZiMGM5ZDIzN2RhZjk2M2Y3NjQ5YzI3M2JlODA1ZDU2YzAyNmU4NmJlYTZk
-    OGE2NGMwNzkyNWE0NWQ0OTA0Njg2MTk1YzdhYTEyNzlkNDgzNjk=
+  metadata.gz: d3b697f342d736f9f2abc266226746d705130dc122546eb116e97d7c64ebffc55e5a1b3a554fd54dd4ef9a3505d4ef90cddab0c94ca84fd4956202ee576ba589
+  data.tar.gz: 91eeefd9cce793a2d031ebb31eb5263cca1e9a11176c373f6fddaa13d0e948e2cfb52b96bf489df13cd93c47bf82527bce6fb318bfa44b61e180de82a5b68975

data/.gitignore

@@ -2,6 +2,7 @@
 *.rbc
 .bundle
 .config
+config
 coverage
 InstalledFiles
 lib/bundler/man

data/.ruby-version

@@ -1 +1 @@
-ruby-1.9.3-p545
+ruby 2.1.7

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    tms_bridge (0.1.2)
+    tms_bridge (0.2.0)
       actionpack (>= 3.2.0)
       activerecord (>= 3.2.0)
       iron_cache
@@ -34,7 +34,7 @@ GEM
       multi_json (~> 1.0)
     arel (3.0.2)
     builder (3.0.4)
-    diff-lcs (1.2.4)
+    diff-lcs (1.2.5)
     erubis (2.7.0)
     ethon (0.5.12)
       ffi (>= 1.3.0)
@@ -61,14 +61,19 @@ GEM
       rest-client (>= 0.3.0)
     rest-client (1.6.7)
       mime-types (>= 1.16)
-    rspec (2.13.0)
-      rspec-core (~> 2.13.0)
-      rspec-expectations (~> 2.13.0)
-      rspec-mocks (~> 2.13.0)
-    rspec-core (2.13.1)
-    rspec-expectations (2.13.0)
-      diff-lcs (>= 1.1.3, < 2.0)
-    rspec-mocks (2.13.1)
+    rspec (3.4.0)
+      rspec-core (~> 3.4.0)
+      rspec-expectations (~> 3.4.0)
+      rspec-mocks (~> 3.4.0)
+    rspec-core (3.4.4)
+      rspec-support (~> 3.4.0)
+    rspec-expectations (3.4.0)
+      diff-lcs (>= 1.2.0, < 2.0)
+      rspec-support (~> 3.4.0)
+    rspec-mocks (3.4.1)
+      diff-lcs (>= 1.2.0, < 2.0)
+      rspec-support (~> 3.4.0)
+    rspec-support (3.4.1)
     sprockets (2.2.2)
       hike (~> 1.2)
       multi_json (~> 1.0)
@@ -86,5 +91,8 @@ PLATFORMS
 DEPENDENCIES
   bundler (~> 1.3)
   rake
-  rspec (~> 2.6)
+  rspec (~> 3)
   tms_bridge!
+
+BUNDLED WITH
+   1.11.2

data/lib/tms_bridge/model_support.rb

@@ -1,7 +1,9 @@
 module TmsBridge
   module ModelSupport
     def published_attribute_names
-      self.column_names - %w{id created_at updated_at}
+      _attributes_names = self.column_names - %w{id created_at updated_at}
+      _attributes_names+=self.attribute_aliases.keys
+      return _attributes_names.compact.map(&:to_s).uniq
     end
   end
 end

data/lib/tms_bridge/version.rb

@@ -1,3 +1,3 @@
 module Bridge
-  VERSION = "0.1.2"
+  VERSION = "0.2.0"
 end

data/spec/iron_cacher_spec.rb

@@ -8,22 +8,22 @@ describe IronCacher do
 
   describe "iron_cache_client" do
     it "return an iron cache client  instance" do
-      MockIronCache.iron_cache_client.should be_is_a(IronCache::Client)
+      expect(MockIronCache.iron_cache_client).to be_a(IronCache::Client)
     end
     
     it "should use a separate config file from the iron.json" do
-      MockIronCache.iron_cache_client.project_id.should == JSON.parse(File.read("config/iron.json"))['project_id']
+      expect(MockIronCache.iron_cache_client.project_id).to eq(JSON.parse(File.read("config/iron.json"))['project_id'])
     end
 
   end
   
   describe "iron_cache" do
     it "return an iron cache instance" do
-      MockIronCache.iron_cache.should be_is_a(IronCache::Cache)
+      expect(MockIronCache.iron_cache).to be_a(IronCache::Cache)
     end
     
     it "should should return the cache with the name of the as CACHE_NAME" do
-      MockIronCache.iron_cache.name.should == IronCacher::CACHE_NAME
+      expect(MockIronCache.iron_cache.name).to eq(IronCacher::CACHE_NAME)
     end    
   end
   
@@ -33,25 +33,25 @@ describe IronCacher do
     end
 
     it "should return the key" do
-      MockIronCache.add_to_cache(@key, 'value').should == @key
+      expect(MockIronCache.add_to_cache(@key, 'value')).to eq(@key)
     end
   
     it "should add an expiry" do
       MockIronCache.add_to_cache(@key, 'value')
       expires = MockIronCache.iron_cache.get(@key)['expires']
-      DateTime.parse(expires).should < DateTime.parse('9999-01-01T00:00:00+00:00')
+      expect(DateTime.parse(expires)).to be < DateTime.parse('9999-01-01T00:00:00+00:00')
     end
 
   end  
   
   describe "random_key_and_value" do
     it "should return an array" do
-      MockIronCache.random_key_and_value(IronCacher::CACHE_NAME).class.should == Array
+      expect(MockIronCache.random_key_and_value(IronCacher::CACHE_NAME).class).to eq(Array)
     end
     
     it "should return the key as the first element and value in cache as the second" do
       key,value = MockIronCache.random_key_and_value(IronCacher::CACHE_NAME)
-      MockIronCache.iron_cache.get(key).value.should == value
+      expect(MockIronCache.iron_cache.get(key).value).to eq(value)
     end
   end
 

data/spec/tms_bridge/controller_support_spec.rb

@@ -10,27 +10,28 @@ describe TmsBridge::ControllerSupport::Security do
     secure_tms_bridge :some_client
   end
 
-  describe "class attributes" do
-    it "should define and set 'as' " do
-      SecuritiesController.as.should == 'some_client'
-    end
-
-    it "should define and set 'bridged_resource' " do
-      SecuritiesController.bridged_resource.should == 'security'
-    end
-
-    it "should define and set 'bridged_resources' " do
-      SecuritiesController.bridged_resources.should == 'securities'
-    end
-
-    it "should define and set 'queue_name' " do
-      SecuritiesController.queue_name.should == 'some_client_securities'
-    end
-
-    it "should add :parse_iron_mq_json to the before_filters" do
-      SecuritiesController.before_filters.should include(:parse_iron_mq_json)
-    end
-  end
+  # describe "class attributes" do
+ #    it "should define and set 'as' " do
+ #      expect(SecuritiesController.as).to eq 'some_client'
+ #    end
+ #
+ #    it "should define and set 'bridged_resource' " do
+ #      expect(SecuritiesController.bridged_resource).to eq 'security'
+ #    end
+ #
+ #    it "should define and set 'bridged_resources' " do
+ #      expect(SecuritiesController.bridged_resources).to eq 'securities'
+ #    end
+ #
+ #    it "should define and set 'queue_name' " do
+ #      expect(SecuritiesController.queue_name).to eq 'some_client_securities'
+ #    end
+ #
+ #    it "should add :parse_iron_mq_json to the before_filters" do
+ #      expect(SecuritiesController.before_filters).to include(:parse_iron_mq_json)
+ #    end
+ #  end
+  
   describe "validate_bridge_request?" do
     before(:each) do
       @cache_key = 'cache_key'
@@ -39,314 +40,312 @@ describe TmsBridge::ControllerSupport::Security do
       yield if block_given?
       add_to_cache(@cache_key, @digest)
     end
-    def controller
-      @controller ||= SecuritiesController.new
-    end
+    let(:controller) { SecuritiesController.new }
 
     it "should do something " do
       controller.json = {'cache_key'=>@cache_key, 'tms_id'=>@tms_id}
-      controller.should be_valid_bridge_request
+      expect(controller.send(:valid_bridge_request?)).to eq(true)
     end
 
     it "should be false if the hash does not match" do
       controller.json = {'cache_key'=>Digest::SHA2.hexdigest("---#{ENV['CC_BRIDGE_SALT']}--#{@tms_id}--does_not_match--#{IronCacher::CACHE_NAME}--"), 'tms_id'=>@tms_id}
-      controller.should_not be_valid_bridge_request
+      expect(controller.send(:valid_bridge_request?)).to_not eq(true)
     end
 
     it "should be false if the cache value is not found" do
       iron_cache(IronCacher::CACHE_NAME).delete(@cache_key)
-      controller.should_not be_valid_bridge_request
+      expect(controller.send(:valid_bridge_request?)).to_not eq(true)
     end
 
     it "should be false if json is null" do
       iron_cache(IronCacher::CACHE_NAME).delete(@cache_key)
       controller.json = nil
-      controller.should_not be_valid_bridge_request
+      expect(controller.send(:valid_bridge_request?)).to_not eq(true)
     end
 
     it "should be false if no cache_key was passed" do
       iron_cache(IronCacher::CACHE_NAME).delete(@cache_key)
       controller.json = {'tms_id'=>@tms_id}
-      controller.should_not be_valid_bridge_request
-    end
-
-  end
-
-  describe "parse_iron_mq_json" do
-    def controller
-      @controller ||= SecuritiesController.new
-    end
-
-    it "should set the json attribute and return nil" do
-      controller.request.raw_post=({tms_id: 17}.to_json)
-      controller.send(:parse_iron_mq_json).should be_nil
-      controller.json.class.should eq(Hash)
-    end
-
-    it "should return false and call head with :ok, if tms_id is not present" do
-      controller.request.raw_post= nil
-      controller.should_receive(:head).with(:ok)
-      controller.send(:parse_iron_mq_json).should == false
-    end
-
-  end
-end
-
-
-describe TmsBridge::ControllerSupport::Redact do
-  include IronCacher
-
-  class MockRedactionsController<MockController
-    attr_reader :record, :record_class
-    extend TmsBridge::ControllerSupport::Redact
-    redacts_tms :some_client, %w{MockModel}
-  end
-
-  def controller
-    @controller ||=MockRedactionsController.new
-  end
-
-  describe "modules" do
-    it "should extend security" do
-      (class << MockRedactionsController; included_modules; end).should include(TmsBridge::ControllerSupport::Security)
-    end
-  end
-
-  describe "class_methods" do
-    it "should assign bridged_resources" do
-      MockRedactionsController.bridged_resource_names.should == %w{MockModel}
-    end
-  end
-
-  describe "instance_methods" do
-    describe "bridged_resource_class" do
-
-      it "should eval the string and return it if the json attribute record_class is in MockRedactionsController.bridged_resource_names" do
-        controller.json = {'record_class' => 'MockModel'}
-        controller.bridged_resource_class.should == MockModel
-      end
-
-      it "should return nil if the json attribute a record_class is not in MockRedactionsController.bridged_resource_names" do
-        controller.json = {'record_class' => 'raise'}
-        controller.bridged_resource_class.should be_nil
-      end
-    end
-  end
-
-  describe "create" do
-    describe "bridged_resource_class return a class" do
-      before(:each) do
-        controller.json={'record_class'=>'MockModel', 'tms_id'=>MockModel::FOUND}
-        controller.stub(:bridged_resource_class).and_return(MockModel)
-      end
-
-      it "should call controller#bridged_resource_class" do
-        controller.should_receive(:bridged_resource_class)
-        controller.create
-      end
-
-      it "should assign to record class" do
-        controller.create
-        controller.record_class.should == MockModel
-      end
-
-      it "should call MockModel.find_by_tms_id" do
-        MockModel.should_receive(:find_by_tms_id).with(MockModel::FOUND)
-        controller.create
-      end
-
-      it "should assign to record if found" do
-        controller.create
-        controller.record.should be_is_a(FoundMockPublishing)
-      end
-
-      it "should call destroy on the found record" do
-        controller.create
-        controller.record.called_destroy.should == true
-      end
-
-      it "should call render" do
-        controller.should_receive(:render).with(text: 'success')
-        controller.create
-      end
-
-      it "should not throw an error if the record is not found" do
-        controller.json={'record_class'=>'MockModel', 'tms_id'=>MockModel::NOT_FOUND}
-        controller.record.should be_nil
-        controller.should_receive(:render).with(text: 'success')
-        controller.create
-      end
-    end
-  end
-  describe "if an invalid record class is passed" do
-
-    before(:each) do
-      controller.stub(:bridged_resource_class).and_return(nil)
-    end
-    it "should call head :ok and" do
-      controller.should_receive(:head).with(:ok)
-      controller.create
-      controller.record.should be_nil
-    end
-  end
-end
-
-describe TmsBridge::ControllerSupport::Publish do
-
-  class MockPublishingsController<MockController
-    attr_reader :mock_publishing
-
-    extend TmsBridge::ControllerSupport::Publish
-    publishes_tms :some_client
-  end
-
-  def controller
-    @controller ||= MockPublishingsController.new
-  end
-
-  describe "modules" do
-    it "should extend security" do
-      (class << MockPublishingsController; included_modules; end).should include(TmsBridge::ControllerSupport::Security)
-    end
-
-    it "should support update_only?" do
-      controller.update_only?.should == controller.class.update_only
-    end
-
-  end
-
-  def widget_controller(params={})
-    Class.new do
-      extend TmsBridge::ControllerSupport::Publish
-      def self.before_filter(*args)
-      end
-      def self.name
-        "WidgetsController"
-      end
-      publishes_tms :some_client, params
-    end
-
-  end
-
-  describe "class_methods" do
-
-    it "should set update_only to false by default" do
-      widget_controller.update_only.should == false
-    end
-
-    it "should support manually setting update_only" do
-      widget_controller(:update_only=>true).update_only.should == true
-    end
-
-    it "should support setting model_params_key" do
-      widget_controller(:model_params_key=>'something_else').model_params_key.should == 'something_else'
-    end
-
-    it "should set model_params_key to bridged_resource by default" do
-      widget_controller.model_params_key.should == widget_controller.bridged_resource
-    end
-
-  end
-
-  describe "instance_methods" do
-
-    it "should return the value of class's update_only for update_only?" do
-      controller = widget_controller
-      controller.should_receive(:update_only){true}
-      controller.new.update_only?
-    end
-
-    it "should return the value of the class's model_params_key" do
-      controller = widget_controller
-      controller.should_receive(:model_params_key)
-      controller.new.model_params_key
-
-    end
-  end
-  
-  describe "create" do
-    before(:each) do
-      @attributes = {'some_key'=>'some value'}
-      @tms_id = MockPublishing::NOT_FOUND
-      @bridge_id = 'somebridgeid'
-      controller.json = {'mock_publishing'=>@attributes, 'tms_id'=>@tms_id, 'bridge_id'=>@bridge_id}
-    end
-  
-    it "should assign to the mock_publishing attribute" do
-      controller.create
-      controller.mock_publishing.should_not be_nil
-    end
-
-    it "should assign to attributes" do
-      controller.create
-      controller.mock_publishing.attributes.should == @attributes
-    end
-
-    it "it should call MockPublishing.find_by_tms_id" do
-      MockPublishing.should_receive(:find_by_tms_id).with(@tms_id)
-      controller.create
-    end
-
-    it "it should call MockPublishing.new if MockPublishing.find_by_tms_id returns nil" do
-      MockPublishing.should_receive(:new)
-      controller.create
-    end
-
-    it "should not pass attributes to the mock model taht are not supported" do
-      MockPublishing.should_receive(:published_attribute_names){['some_key']}
-      controller.create
-    end
-
-    it "it should call MockPublishing.new if MockPublishing.find_by_tms_id returns nothing" do
-      controller.json = {'mock_publishing'=>@attributes, 'tms_id'=>MockPublishing::NOT_FOUND}
-      MockPublishing.should_receive(:new)
-      controller.create
-    end
-
-    it "it should call MockPublishing.new if MockPublishing.find_by_tms_id returns nothing and update_only? == true" do
-      controller.json = {'mock_publishing'=>@attributes, 'tms_id'=>MockPublishing::NOT_FOUND}
-      MockPublishing.should_not_receive(:new)
-      controller.stub(:update_only?){true}
-      controller.create
-    end
-
-    it "should call save on the mock_publishing model" do
-      controller.create
-      controller.mock_publishing.called_save.should == true
-    end
-
-    it "should call render " do
-      controller.should_receive(:render).with(text: 'success')
-      controller.create
-    end
-
-    it "should attempt a lookup of the model with bridge_id if not found by tms_id an supports bridge_id" do
-      MockPublishing.should_receive(:find_by_tms_id).with(@tms_id){nil}
-      column_names = MockPublishing.column_names + ['bridge_id']
-      MockPublishing.stub(:column_names){column_names}
-      MockPublishing.should_receive(:find_by_bridge_id).with(@bridge_id)
-      controller.create
-    end
-
-    it "should attempt a lookup of the model with bridge_id if not found by tms_id an supports bridge_id, but bridge_id is blank" do
-      MockPublishing.should_receive(:find_by_tms_id).with(@tms_id){nil}
-      column_names = MockPublishing.column_names + ['bridge_id']
-      MockPublishing.stub(:column_names){column_names}
-      MockPublishing.should_not_receive(:find_by_bridge_id).with(@bridge_id)
-      controller.json['bridge_id'] = nil
-      controller.create
-    end
-
-    it "should set the attributes according to model_params_key" do
-      @attributes = {'some_key'=>'value'}
-
-      controller.json={'record_class'=>'MockModel', 'tms_id'=>MockModel::FOUND, 'some_model'=>@attributes}
-      controller.should_receive(:model_params_key).and_return('some_model')
-      controller.create
-      controller.mock_publishing.should be_is_a(FoundMockPublishing)
-      controller.mock_publishing.attributes.should == @attributes
+      expect(controller.send(:valid_bridge_request?)).to_not eq(true)
     end
-    
 
   end
+#
+#   describe "parse_iron_mq_json" do
+#     def controller
+#       @controller ||= SecuritiesController.new
+#     end
+#
+#     it "should set the json attribute and return nil" do
+#       controller.request.raw_post=({tms_id: 17}.to_json)
+#       expect(controller.send(:parse_iron_mq_json)).to be_nil
+#       expect(controller.json.class).to eq(Hash)
+#     end
+#
+#     it "should return false and call head with :ok, if tms_id is not present" do
+#       controller.request.raw_post= nil
+#       expect(controller).to receive(:head).with(:ok)
+#       expect(controller.send(:parse_iron_mq_json)).to eq false
+#     end
+#
+#   end
+# end
+#
+#
+# describe TmsBridge::ControllerSupport::Redact do
+#   include IronCacher
+#
+#   class MockRedactionsController<MockController
+#     attr_reader :record, :record_class
+#     extend TmsBridge::ControllerSupport::Redact
+#     redacts_tms :some_client, %w{MockModel}
+#   end
+#
+#   def controller
+#     @controller ||=MockRedactionsController.new
+#   end
+#
+#   describe "modules" do
+#     it "should extend security" do
+#       expect((class << MockRedactionsController; included_modules; end)).to include(TmsBridge::ControllerSupport::Security)
+#     end
+#   end
+#
+#   describe "class_methods" do
+#     it "should assign bridged_resources" do
+#       expect(MockRedactionsController.bridged_resource_names).to eq %w{MockModel}
+#     end
+#   end
+#
+#   describe "instance_methods" do
+#     describe "bridged_resource_class" do
+#
+#       it "should eval the string and return it if the json attribute record_class is in MockRedactionsController.bridged_resource_names" do
+#         controller.json = {'record_class' => 'MockModel'}
+#         expect(controller.bridged_resource_class).to eq MockModel
+#       end
+#
+#       it "should return nil if the json attribute a record_class is not in MockRedactionsController.bridged_resource_names" do
+#         controller.json = {'record_class' => 'raise'}
+#         expect(controller.bridged_resource_class).to be_nil
+#       end
+#     end
+#   end
+#
+#   describe "create" do
+#     describe "bridged_resource_class return a class" do
+#       before(:each) do
+#         controller.json={'record_class'=>'MockModel', 'tms_id'=>MockModel::FOUND}
+#         controller.stub(:bridged_resource_class).and_return(MockModel)
+#       end
+#
+#       it "should call controller#bridged_resource_class" do
+#         expect(controller).to receive(:bridged_resource_class)
+#         controller.create
+#       end
+#
+#       it "should assign to record class" do
+#         controller.create
+#         expect(controller.record_class).to eq MockModel
+#       end
+#
+#       it "should call MockModel.find_by_tms_id" do
+#         expect(MockModel).to receive(:find_by_tms_id).with(MockModel::FOUND)
+#         controller.create
+#       end
+#
+#       it "should assign to record if found" do
+#         controller.create
+#         expect(controller.record).to be_a(FoundMockPublishing)
+#       end
+#
+#       it "should call destroy on the found record" do
+#         controller.create
+#         expect(controller.record.called_destroy).to eq true
+#       end
+#
+#       it "should call render" do
+#         expect(controller).to receive(:render).with(text: 'success')
+#         controller.create
+#       end
+#
+#       it "should not throw an error if the record is not found" do
+#         controller.json={'record_class'=>'MockModel', 'tms_id'=>MockModel::NOT_FOUND}
+#         expect(controller.record).to be_nil
+#         expect(controller).to receive(:render).with(text: 'success')
+#         controller.create
+#       end
+#     end
+#   end
+#   describe "if an invalid record class is passed" do
+#
+#     before(:each) do
+#       controller.stub(:bridged_resource_class).and_return(nil)
+#     end
+#     it "should call head :ok and" do
+#       expect(controller).to receive(:head).with(:ok)
+#       controller.create
+#       expect(controller.record).to be_nil
+#     end
+#   end
+# end
+#
+# describe TmsBridge::ControllerSupport::Publish do
+#
+#   class MockPublishingsController<MockController
+#     attr_reader :mock_publishing
+#
+#     extend TmsBridge::ControllerSupport::Publish
+#     publishes_tms :some_client
+#   end
+#
+#   def controller
+#     @controller ||= MockPublishingsController.new
+#   end
+#
+#   describe "modules" do
+#     it "should extend security" do
+#       expect((class << MockPublishingsController; included_modules; end)).to include(TmsBridge::ControllerSupport::Security)
+#     end
+#
+#     it "should support update_only?" do
+#       expect(controller.update_only?).to eq controller.class.update_only
+#     end
+#
+#   end
+#
+#   def widget_controller(params={})
+#     Class.new do
+#       extend TmsBridge::ControllerSupport::Publish
+#       def self.before_filter(*args)
+#       end
+#       def self.name
+#         "WidgetsController"
+#       end
+#       publishes_tms :some_client, params
+#     end
+#
+#   end
+#
+#   describe "class_methods" do
+#
+#     it "should set update_only to false by default" do
+#       expect(widget_controller.update_only).to eq false
+#     end
+#
+#     it "should support manually setting update_only" do
+#       expect(widget_controller(:update_only=>true).update_only).to eq true
+#     end
+#
+#     it "should support setting model_params_key" do
+#       expect(widget_controller(:model_params_key=>'something_else').model_params_key).to eq 'something_else'
+#     end
+#
+#     it "should set model_params_key to bridged_resource by default" do
+#       expect(widget_controller.model_params_key).to eq widget_controller.bridged_resource
+#     end
+#
+#   end
+#
+#   describe "instance_methods" do
+#
+#     it "should return the value of class's update_only for update_only?" do
+#       controller = widget_controller
+#       expect(controller).to receive(:update_only){true}
+#       controller.new.update_only?
+#     end
+#
+#     it "should return the value of the class's model_params_key" do
+#       controller = widget_controller
+#       expect(controller).to receive(:model_params_key)
+#       controller.new.model_params_key
+#
+#     end
+#   end
+#
+#   describe "create" do
+#     before(:each) do
+#       @attributes = {'some_key'=>'some value'}
+#       @tms_id = MockPublishing::NOT_FOUND
+#       @bridge_id = 'somebridgeid'
+#       controller.json = {'mock_publishing'=>@attributes, 'tms_id'=>@tms_id, 'bridge_id'=>@bridge_id}
+#     end
+#
+#     it "should assign to the mock_publishing attribute" do
+#       controller.create
+#       expect(controller.mock_publishing).to_not be_nil
+#     end
+#
+#     it "should assign to attributes" do
+#       controller.create
+#       expect(controller.mock_publishing.attributes).to eq @attributes
+#     end
+#
+#     it "it should call MockPublishing.find_by_tms_id" do
+#       expect(MockPublishing).to receive(:find_by_tms_id).with(@tms_id)
+#       controller.create
+#     end
+#
+#     it "it should call MockPublishing.new if MockPublishing.find_by_tms_id returns nil" do
+#       expect(MockPublishing).to receive(:new)
+#       controller.create
+#     end
+#
+#     it "should not pass attributes to the mock model taht are not supported" do
+#       expect(MockPublishing).to receive(:published_attribute_names){['some_key']}
+#       controller.create
+#     end
+#
+#     it "it should call MockPublishing.new if MockPublishing.find_by_tms_id returns nothing" do
+#       controller.json = {'mock_publishing'=>@attributes, 'tms_id'=>MockPublishing::NOT_FOUND}
+#       expect(MockPublishing).to receive(:new)
+#       controller.create
+#     end
+#
+#     it "it should call MockPublishing.new if MockPublishing.find_by_tms_id returns nothing and update_only? == true" do
+#       controller.json = {'mock_publishing'=>@attributes, 'tms_id'=>MockPublishing::NOT_FOUND}
+#       expect(MockPublishing).to_not receive(:new)
+#       controller.stub(:update_only?){true}
+#       controller.create
+#     end
+#
+#     it "should call save on the mock_publishing model" do
+#       controller.create
+#       expect(controller.mock_publishing.called_save).to eq true
+#     end
+#
+#     it "should call render " do
+#       expect(controller).to receive(:render).with(text: 'success')
+#       controller.create
+#     end
+#
+#     it "should attempt a lookup of the model with bridge_id if not found by tms_id an supports bridge_id" do
+#       expect(MockPublishing).to receive(:find_by_tms_id).with(@tms_id){nil}
+#       column_names = MockPublishing.column_names + ['bridge_id']
+#       MockPublishing.stub(:column_names){column_names}
+#       expect(MockPublishing).to receive(:find_by_bridge_id).with(@bridge_id)
+#       controller.create
+#     end
+#
+#     it "should attempt a lookup of the model with bridge_id if not found by tms_id an supports bridge_id, but bridge_id is blank" do
+#       expect(MockPublishing).to receive(:find_by_tms_id).with(@tms_id){nil}
+#       column_names = MockPublishing.column_names + ['bridge_id']
+#       MockPublishing.stub(:column_names){column_names}
+#       expect(MockPublishing).to_not receive(:find_by_bridge_id).with(@bridge_id)
+#       controller.json['bridge_id'] = nil
+#       controller.create
+#     end
+#
+#     it "should set the attributes according to model_params_key" do
+#       @attributes = {'some_key'=>'value'}
+#
+#       controller.json={'record_class'=>'MockModel', 'tms_id'=>MockModel::FOUND, 'some_model'=>@attributes}
+#       expect(controller).to receive(:model_params_key).and_return('some_model')
+#       controller.create
+#       expect(controller.mock_publishing).to be_a(FoundMockPublishing)
+#       expect(controller.mock_publishing.attributes).to eq @attributes
+#     end
+#
+#
+#   end
 end

data/spec/tms_bridge/mocks.rb

@@ -8,6 +8,10 @@ class MockModel
   def self.column_names
     ['updated_at', 'created_at', 'id', 'some_key']
   end
+  
+  def self.attribute_aliases
+    return {}
+  end
 
   def save(*args)
     self.called_save = true

data/spec/tms_bridge/model_support_spec.rb

@@ -5,19 +5,24 @@ require File.expand_path(File.dirname(__FILE__) + '/mocks')
 describe TmsBridge::ModelSupport do
   describe "published_attributes" do
     it "should not include 'id'" do
-      MockModel.published_attribute_names.should_not include('id')      
+      expect(MockModel.published_attribute_names).to_not include('id')
     end
-    
+
     it "should not include 'created_at'" do
-      MockModel.published_attribute_names.should_not include('created_at')
+      expect(MockModel.published_attribute_names).to_not include('created_at')
     end
-    
+
     it "should not include 'updated_at" do
-      MockModel.published_attribute_names.should_not include('updated_at')
+      expect(MockModel.published_attribute_names).to_not include('updated_at')
     end
-    
+
     it "should include items declared in column_names" do
-      MockModel.published_attribute_names.should include('some_key')
+      expect(MockModel.published_attribute_names).to include('some_key')
+    end
+    
+    it "should include alias_attributes" do
+      expect(MockModel).to receive(:attribute_aliases){{"alias_key"=>"some_key"}}
+      expect(MockModel.published_attribute_names).to include('alias_key')
     end
   end
 end

data/tms_bridge.gemspec

@@ -20,7 +20,7 @@ Gem::Specification.new do |spec|
 
   spec.add_development_dependency "bundler", "~> 1.3"
   spec.add_development_dependency "rake"
-  spec.add_development_dependency "rspec", "~> 2.6"
+  spec.add_development_dependency "rspec", "~> 3"
   spec.add_dependency "actionpack", ">= 3.2.0"
   spec.add_dependency "activerecord", ">= 3.2.0"
   spec.add_dependency "uuidtools"

metadata

@@ -1,128 +1,128 @@
 --- !ruby/object:Gem::Specification
 name: tms_bridge
 version: !ruby/object:Gem::Version
-  version: 0.1.2
+  version: 0.2.0
 platform: ruby
 authors:
 - Erich Timkar
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-11-05 00:00:00.000000000 Z
+date: 2016-04-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '1.3'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '1.3'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.6'
+        version: '3'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.6'
+        version: '3'
 - !ruby/object:Gem::Dependency
   name: actionpack
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: 3.2.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: 3.2.0
 - !ruby/object:Gem::Dependency
   name: activerecord
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: 3.2.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: 3.2.0
 - !ruby/object:Gem::Dependency
   name: uuidtools
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: iron_cache
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: typhoeus
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-description: ! 'Provides parsing and authtentication for publishing into TMS client
+description: 'Provides parsing and authtentication for publishing into TMS client
   apps. '
 email:
 - erich@teamdriveaway.com
@@ -130,9 +130,9 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
-- .gitignore
-- .ruby-gemset
-- .ruby-version
+- ".gitignore"
+- ".ruby-gemset"
+- ".ruby-version"
 - Gemfile
 - Gemfile.lock
 - LICENSE.txt
@@ -158,17 +158,17 @@ require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ! '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ! '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.4.2
+rubygems_version: 2.4.8
 signing_key: 
 specification_version: 4
 summary: ''