remote_factory_girl 1.0.2 → 1.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 568879d0cb0a73b0a0b4cfbc8d24a7e55db06052
-  data.tar.gz: 050ebe624b42a226271972e4174d635750a1b998
+  metadata.gz: 29a342e578e2499f6d8cd58bfc6d52b669b62d4f
+  data.tar.gz: 3eef162182966244ea79a39b0d7daaca444198ca
 SHA512:
-  metadata.gz: 7b9105b90699050925579b0e0dc4fd9b7af917f6c6073b1f90798ab9c9ac030627b12f5d7169adf05677b984e8cdc0541a8ffb1af3c1923fac5a0f1f789d2311
-  data.tar.gz: 6d1c20f5438cb6eebdb16f019415843c628ff4c43267c9c85a422e1f55eb27521fb2bc9a407ea5f3bb62482070b477598127a91fa854074c04ffab9709af5f92
+  metadata.gz: 72b597be3b6f6798478bd2e9f15b2408ff8726bc8c5d670a322871a806e0bd6cbbc4bcf3f739391c2ae7e790d1e84665fdd36bd2105f214e23d0dbf6f5c55200
+  data.tar.gz: 7b8c81a70d02f0c6569ce6e64ca7cb0a4e94debb928d77c61a0ac29a791835cffc8edb000a5ba921dd362086dd635ef20c1b07d2f05cc3b0201e78ba2d3a4f29

data/CHANGELOG.md

@@ -1,3 +1,6 @@
+### 1.1.0
+1. Configure and create test data in multiple "home"/"remote" applications
+2. Remove RemoteFactoryGirl.reset. Instead use, RemoteFactoryGirl.remotes_config.reset
 ### 1.0.2 - Aug. 19, 2014
 1. Update specs to use Rspec '3.0.0' syntax
 

data/lib/remote_factory_girl.rb

@@ -1,5 +1,6 @@
 require "remote_factory_girl/version"
 require 'remote_factory_girl/config'
+require 'remote_factory_girl/remotes_config'
 require 'remote_factory_girl/http'
 require 'remote_factory_girl/config_applier'
 require 'remote_factory_girl/hash_to_dot'
@@ -23,32 +24,35 @@ module RemoteFactoryGirl
     end
   end
 
-  def self.configure(config = Config, &block)
+  def self.configure(remote_name = remotes_config.default_remote_name, config = Config, &block)
     configuration = Config.new
     yield(configuration)
-    self.config = configuration
+    remotes_config.remotes[remote_name] = configuration
   end
 
   def self.factories(params = {}, http = Http)
-    factory = RemoteFactoryGirl.new(config)
+    config_for_remote = config(remotes_config.current_remote)
+    factory = RemoteFactoryGirl.new(config_for_remote)
     factory.factories(params, http).to_hash
   end
 
   def self.create(factory_name, attributes = {}, config_applier = ConfigApplier, http = Http)
-    factory = RemoteFactoryGirl.new(config)
+    config_for_remote = config(remotes_config.current_remote)
+    factory = RemoteFactoryGirl.new(config_for_remote)
     factory.create(factory_name, attributes, http)
-    config_applier.apply_config(factory.response.to_hash, config.to_hash)
+    config_applier.apply_config(factory.response.to_hash, config_for_remote.to_hash)
   end
 
-  def self.config
-    @config
+  def self.with_remote(remote_name = remotes_config.default_remote_name)
+    remotes_config.current_remote = remote_name
+    self
   end
 
-  def self.config=(config)
-    @config = config
+  def self.config(remote_name = remotes_config.default_remote_name)
+    remotes_config.remotes[remote_name]
   end
 
-  def self.reset(config = Config.new)
-    self.config = config
+  def self.remotes_config
+    @remotes_config ||= RemotesConfig.new
   end
 end

data/lib/remote_factory_girl/remotes_config.rb

@@ -0,0 +1,30 @@
+require 'remote_factory_girl/config'
+
+module RemoteFactoryGirl
+  class RemotesConfig
+
+    attr_writer :current_remote
+    attr_accessor :remotes
+
+    DEFAULT_REMOTE_NAME = :default
+
+    alias_method :to_hash, :remotes
+
+    def initialize
+      @remotes = {}
+    end
+
+    def current_remote
+      @current_remote || default_remote_name
+    end
+
+    def default_remote_name
+      DEFAULT_REMOTE_NAME
+    end
+
+    def reset(config = Config.new)
+      self.current_remote = default_remote_name
+      self.remotes        = { default_remote_name => config}
+    end
+  end
+end

data/lib/remote_factory_girl/version.rb

@@ -1,3 +1,3 @@
 module RemoteFactoryGirl
-  VERSION = "1.0.2"
+  VERSION = "1.1.0"
 end

data/spec/integration/remote_factory_girl_spec.rb

@@ -1,8 +1,9 @@
 require 'remote_factory_girl'
+require 'spec_helper'
 
 describe RemoteFactoryGirl do
 
-  before { RemoteFactoryGirl.reset }
+  before { RemoteFactoryGirl.remotes_config.reset }
 
   describe 'configuration' do
     it 'should be configured with correct defaults' do
@@ -49,6 +50,34 @@ describe RemoteFactoryGirl do
       RemoteFactoryGirl.config.https = true
       expect(RemoteFactoryGirl.config.https).to eq true
     end
+
+    context 'when configuring multiple remotes' do
+
+      before do
+        configure_remote_factory_girl(remote_name: :travis,
+                                      host: 'localhost',
+                                      port: 3000,
+                                      end_point: '/remote_factory_girl/travis/home')
+        configure_remote_factory_girl(remote_name: :casey,
+                                      host: 'over_the_rainbow',
+                                      port: 6000,
+                                      end_point: '/remote_factory_girl/casey/home')
+      end
+
+      it 'should return configuration for remote "travis"' do
+        expect(RemoteFactoryGirl.config(:travis).home).to eq({:host      => 'localhost',
+                                                              :port      => 3000,
+                                                              :end_point => '/remote_factory_girl/travis/home'})
+        expect(RemoteFactoryGirl.config(:travis).return_with_root).to eq(true)
+        expect(RemoteFactoryGirl.config(:travis).return_response_as).to eq(:as_hash)
+      end
+
+      it 'should return configuration for remote "casey"' do
+        expect(RemoteFactoryGirl.config(:casey).home).to eq({:host      => 'over_the_rainbow',
+                                                             :port      => 6000,
+                                                             :end_point => '/remote_factory_girl/casey/home'})
+      end
+    end
   end
 
   describe 'errors' do
@@ -71,15 +100,114 @@ describe RemoteFactoryGirl do
     end
 
     describe '.factories' do
+      context 'when multiple remotes are configured' do
 
-      before { RemoteFactoryGirl.config.home[:host] = 'localhost' }
+        before do
+          configure_remote_factory_girl(remote_name: :travis,
+                                        host: 'localhost',
+                                        port: 3000,
+                                        end_point: '/remote_factory_girl/travis/home')
+          configure_remote_factory_girl(remote_name: :casey,
+                                        host: 'over_the_rainbow',
+                                        port: 6000,
+                                        end_point: '/remote_factory_girl/casey/home')
+        end
 
-      it 'should return all factories available' do
-        expect(RemoteFactoryGirl.factories).to match_array(['user', 'user_admin'])
+        context 'for remote "travis"' do
+          it 'should return all factories available' do
+            expect(RemoteFactoryGirl.with_remote(:travis).factories).to match_array(['user', 'user_admin'])
+          end
+
+          it 'should be configured to send HTTP request to remote "travis"' do
+            remote_factory_girl = RemoteFactoryGirl.with_remote(:travis)
+
+            expect(remote_factory_girl).to receive(:factories) do
+              expect(
+                remote_factory_girl.remotes_config.current_remote
+              ).to eq(:travis)
+              expect(
+                remote_factory_girl.config(:travis).home_url
+              ).to eq('http://localhost:3000/remote_factory_girl/travis/home')
+            end
+            remote_factory_girl.factories
+          end
+        end
+
+        context 'for remote "casey"' do
+          it 'should return all factories available' do
+            expect(RemoteFactoryGirl.with_remote(:casey).factories).to match_array(['user', 'user_admin'])
+          end
+
+          it 'should be configured to send HTTP request to remote "casey"' do
+            remote_factory_girl = RemoteFactoryGirl.with_remote(:casey)
+
+            expect(remote_factory_girl).to receive(:factories) do
+              expect(
+                remote_factory_girl.remotes_config.current_remote
+              ).to eq(:casey)
+              expect(
+                remote_factory_girl.config(:casey).home_url
+              ).to eq('http://over_the_rainbow:6000/remote_factory_girl/casey/home')
+            end
+            remote_factory_girl.factories
+          end
+        end
+      end
+
+      context 'when configured with remote "default"' do
+
+        before do
+          configure_remote_factory_girl(host: 'not_configured_with_name',
+                                        port: 9000)
+        end
+
+        it 'should return all factories available' do
+          expect(RemoteFactoryGirl.factories).to match_array(['user', 'user_admin'])
+        end
+
+        it 'should be configured to send HTTP request to remote "default"' do
+          remote_factory_girl = RemoteFactoryGirl
+
+          expect(remote_factory_girl).to receive(:factories) do
+            expect(
+              remote_factory_girl.remotes_config.current_remote
+            ).to eq(:default)
+            expect(
+              remote_factory_girl.config.home_url
+            ).to eq('http://not_configured_with_name:9000/remote_factory_girl/home')
+          end
+          remote_factory_girl.factories
+        end
       end
     end
 
     describe '.create' do
+      context 'when configured with multiple remotes' do
+        before do
+          configure_remote_factory_girl(remote_name: :travis,
+                                        host: 'localhost',
+                                        port: 3000,
+                                        end_point: '/remote_factory_girl/travis/home',
+                                        return_with_root: false)
+          configure_remote_factory_girl(remote_name: :casey,
+                                        host: 'over_the_rainbow',
+                                        port: 6000,
+                                        end_point: '/remote_factory_girl/casey/home',
+                                        return_response_as: :dot_notation,
+                                        return_with_root: false)
+
+        end
+
+        it 'should be able to create a factory with "travis" remote' do
+          user = RemoteFactoryGirl.with_remote(:travis).create(:user)
+          expect(user['first_name']).to eq('Sam')
+        end
+
+        it 'should be able to create a factory with "casey" remote' do
+          user = RemoteFactoryGirl.with_remote(:casey).create(:user)
+          expect(user.first_name).to eq('Sam')
+        end
+      end
 
       describe 'default .home' do
 
@@ -109,11 +237,10 @@ describe RemoteFactoryGirl do
       end
 
       it 'should not return root hash key and should return an object that responds to dot notation' do
-        RemoteFactoryGirl.configure do |config|
-          config.home               = { :host => 'localhost' }
-          config.return_response_as = :dot_notation
-          config.return_with_root   = false
-        end
+        configure_remote_factory_girl(host: 'localhost',
+                                      port: 3000,
+                                      return_response_as: :dot_notation,
+                                      return_with_root: false)
         user = RemoteFactoryGirl.create(:user)
         expect(user.first_name).to eq('Sam')
       end

data/spec/models/remote_factory_girl_spec.rb

@@ -1,36 +1,146 @@
 require 'remote_factory_girl'
+require 'spec_helper'
 
 describe RemoteFactoryGirl do
+  describe '#create' do
 
-  let(:config) { double('config', :to_hash => {}) }
+    let(:http) { double('RemoteFactoryGirl::Http') }
+    let(:factory_attributes) { { :first_name => 'Sam' } }
 
-  describe '#create' do
-    it 'should send a post request' do
-      http       = double('RemoteFactoryGirl::Http')
-      attributes = { :first_name => 'Sam' }
-      expect(http).to receive(:post).with(config, {:factory => 'user', :attributes => { :first_name => 'Sam'}})
-      RemoteFactoryGirl::RemoteFactoryGirl.new(config).create('user', attributes, http)
+    context 'when remote configuration does not specify a remote name' do
+
+      before do
+        configure_remote_factory_girl(host: 'localhost', 
+                                      port: 3000)
+      end
+
+      it "should send a post request to remote" do
+        config     = RemoteFactoryGirl.config
+        expect(http).to receive(:post).with(config, {:factory => :user, :attributes => factory_attributes})
+        RemoteFactoryGirl::RemoteFactoryGirl.new(config).create(:user, factory_attributes, http)
+      end
+    end
+
+    context 'when multiple remotes are configured' do
+      before do
+        configure_remote_factory_girl(remote_name: :travis,
+                                      host: 'localhost',
+                                      port: 3000,
+                                      end_point: '/remote_factory_girl/travis/home')
+        configure_remote_factory_girl(remote_name: :casey,
+                                      host: 'over_the_rainbow',
+                                      port: 6000,
+                                      end_point: '/remote_factory_girl/casey/home')
+      end
+
+      it "should be configured to send HTTP requests to 'travis' remote" do
+        remote_config_travis = RemoteFactoryGirl.config(:travis)
+        remote_factory_girl  = RemoteFactoryGirl::RemoteFactoryGirl.new(remote_config_travis)
+
+        expect(remote_factory_girl).to receive(:create).with(:user, factory_attributes, http) do |_, _, _|
+          expect(
+            remote_factory_girl.config.home_url
+          ).to eq('http://localhost:3000/remote_factory_girl/travis/home')
+        end
+
+        remote_factory_girl.create(:user, factory_attributes, http)
+      end
+
+      it "should be configured to send HTTP requests to 'casey' remote" do
+        remote_config_casey = RemoteFactoryGirl.config(:casey)
+        remote_factory_girl = RemoteFactoryGirl::RemoteFactoryGirl.new(remote_config_casey)
+
+        expect(remote_factory_girl).to receive(:create).with(:user, factory_attributes, http) do |_, _, _|
+          expect(
+            remote_factory_girl.config.home_url
+          ).to eq('http://over_the_rainbow:6000/remote_factory_girl/casey/home')
+        end
+
+        remote_factory_girl.create(:user, factory_attributes, http)
+      end
     end
   end
 
   xit 'should be able to configure with a block' do
+    # TODO: Remove
     pending
   end
 
-  describe '.config' do
-    it 'should be able to set and get config' do
-      config = double('config')
-      RemoteFactoryGirl.config = config
-      expect(RemoteFactoryGirl.config).to equal(config)
+  describe '.reset' do
+    context 'when multiple remotes are configured' do
+
+      before do
+        configure_remote_factory_girl(remote_name: :travis,
+                                      host: 'localhost',
+                                      port: 3000,
+                                      end_point: '/remote_factory_girl/travis/home')
+        configure_remote_factory_girl(remote_name: :casey,
+                                      host: 'over_the_rainbow',
+                                      port: 6000,
+                                      end_point: '/remote_factory_girl/casey/home')
+
+      end
+
+      it 'should be able to reset the configuration' do
+        RemoteFactoryGirl.remotes_config.reset
+        expect(RemoteFactoryGirl.remotes_config.to_hash.keys).to eq([:default])
+      end
+    end
+
+    context 'when only one remote configuration and does not specify a remote name' do
+      before do
+        configure_remote_factory_girl(host: 'not_configured_with_name',
+                                      port: 9000)
+      end
+
+      it 'should be able to reset the configuration' do
+        RemoteFactoryGirl.remotes_config.reset
+        expect(RemoteFactoryGirl.remotes_config.to_hash.keys).to eq([:default])
+      end
     end
   end
 
-  describe '.reset' do
-    it 'should be able to reset the configuration' do
-      config = double('config')
-      RemoteFactoryGirl.config = config
-      RemoteFactoryGirl.reset(double('config'))
-      expect(RemoteFactoryGirl.config).to_not equal(config)
+  describe '.factories' do
+    context 'when multiple remotes are configured' do
+
+      let(:http) { double('RemoteFactoryGirl::Http') }
+
+      before do
+        configure_remote_factory_girl(remote_name: :travis,
+                                      host: 'localhost',
+                                      port: 3000,
+                                      end_point: '/remote_factory_girl/travis/home')
+        configure_remote_factory_girl(remote_name: :casey,
+                                      host: 'over_the_rainbow',
+                                      port: 6000,
+                                      end_point: '/remote_factory_girl/casey/home')
+      end
+
+      it "should be configured to send HTTP requests to 'travis' remote" do
+        remote_config_travis = RemoteFactoryGirl.config(:travis)
+        remote_factory_girl  = RemoteFactoryGirl::RemoteFactoryGirl.new(remote_config_travis)
+
+        expect(remote_factory_girl).to receive(:factories).with({}, http) do |_, _|
+          expect(
+            remote_factory_girl.config.home_url
+          ).to eq('http://localhost:3000/remote_factory_girl/travis/home')
+        end
+
+        remote_factory_girl.factories({}, http)
+      end
+
+      it "should be configured to send HTTP requests to 'casey' remote" do
+        remote_config_casey = RemoteFactoryGirl.config(:casey)
+        remote_factory_girl = RemoteFactoryGirl::RemoteFactoryGirl.new(remote_config_casey)
+
+        expect(remote_factory_girl).to receive(:factories).with({}, http) do |_, _|
+          expect(
+            remote_factory_girl.config.home_url
+          ).to eq('http://over_the_rainbow:6000/remote_factory_girl/casey/home')
+        end
+
+        remote_factory_girl.factories({}, http)
+      end
     end
   end
 

data/spec/models/remotes_config_spec.rb

@@ -0,0 +1,42 @@
+require 'remote_factory_girl/remotes_config.rb'
+
+describe RemoteFactoryGirl::Config do
+
+  let(:remotes_config) { RemoteFactoryGirl::RemotesConfig.new }
+  let(:home_1_config) { double('config') }
+  let(:home_2_config) { double('config') }
+  let(:default_remote_name) { RemoteFactoryGirl::RemotesConfig::DEFAULT_REMOTE_NAME }
+
+  describe '#initialize' do
+    context 'default configuration' do
+      it 'should be configured with correct defaults' do
+        expect(remotes_config.remotes).to eq({})
+      end
+    end
+  end
+
+  context 'adding a remote' do
+    before do
+      remotes_config.remotes[:home_1] = home_1_config
+    end
+
+    it 'should return configuration for home_1' do
+      expect(remotes_config.remotes[:home_1]).to eq(home_1_config)
+    end
+  end
+
+  describe '#reset' do
+
+    let(:reset_config) { double('config') }
+
+    before do
+      remotes_config.remotes[:home_1] = home_1_config
+      remotes_config.remotes[:home_2] = home_2_config
+      remotes_config.reset(reset_config)
+    end
+
+    it 'should return an empty hash' do
+      expect(remotes_config.to_hash).to eq({:default => reset_config })
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -6,3 +6,35 @@ require 'remote_factory_girl'
 RSpec.configure do |config|
 
 end
+
+def configure_remote_factory_girl(remote_name: nil,
+                                  host: nil, 
+                                  port: nil, 
+                                  end_point: '/remote_factory_girl/home',
+                                  return_response_as: :as_hash, 
+                                  return_with_root: true,
+                                  return_as_active_resource: false,
+                                  https: false)
+  if remote_name.nil?
+    RemoteFactoryGirl.configure do |config|
+        config.home                     = {:host      => host,
+                                           :port      => port,
+                                           :end_point => end_point }
+        config.return_response_as        = return_response_as
+        config.return_with_root          = return_with_root
+        config.return_as_active_resource = return_as_active_resource
+        config.https                     = https
+    end
+
+  else
+    RemoteFactoryGirl.configure(remote_name) do |config|
+        config.home                     = {:host      => host,
+                                           :port      => port,
+                                           :end_point => end_point }
+        config.return_response_as        = return_response_as
+        config.return_with_root          = return_with_root
+        config.return_as_active_resource = return_as_active_resource
+        config.https                     = https
+    end
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: remote_factory_girl
 version: !ruby/object:Gem::Version
-  version: 1.0.2
+  version: 1.1.0
 platform: ruby
 authors:
 - tdouce
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-08-19 00:00:00.000000000 Z
+date: 2014-10-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -88,6 +88,7 @@ files:
 - lib/remote_factory_girl/hash_to_dot.rb
 - lib/remote_factory_girl/http.rb
 - lib/remote_factory_girl/json_to_active_resource.rb
+- lib/remote_factory_girl/remotes_config.rb
 - lib/remote_factory_girl/version.rb
 - remote_factory_girl.gemspec
 - spec/integration/remote_factory_girl_spec.rb
@@ -97,6 +98,7 @@ files:
 - spec/models/remote_factory_girl/http_spec.rb
 - spec/models/remote_factory_girl/json_to_active_resource_spec.rb
 - spec/models/remote_factory_girl_spec.rb
+- spec/models/remotes_config_spec.rb
 - spec/spec_helper.rb
 - wiki.md
 homepage: https://github.com/tdouce/remote_factory_girl
@@ -131,4 +133,5 @@ test_files:
 - spec/models/remote_factory_girl/http_spec.rb
 - spec/models/remote_factory_girl/json_to_active_resource_spec.rb
 - spec/models/remote_factory_girl_spec.rb
+- spec/models/remotes_config_spec.rb
 - spec/spec_helper.rb