chemistrykit 3.9.0.rc2 → 3.9.0.rc3

data/spec/integration/lib/chemistrykit/split_testing/provider_factory_spec.rb

@@ -0,0 +1,15 @@
+# Encoding: utf-8
+
+require 'spec_helper'
+require 'chemistrykit/split_testing/provider_factory'
+require 'chemistrykit/config/split_testing'
+
+describe ChemistryKit::SplitTesting::ProviderFactory do
+
+  it 'should return the correctly configured provider based on configuration' do
+    config = ChemistryKit::Config::SplitTesting.new(provider: 'optimizely', opt_out: true)
+    provider = ChemistryKit::SplitTesting::ProviderFactory.build(config)
+    provider.should be_an_instance_of ChemistryKit::SplitTesting::OptimizelyProvider
+  end
+
+end

data/spec/spec_helper.rb

@@ -13,12 +13,6 @@ require 'rubygems'
 require 'bundler/setup'
 require 'fileutils'
 
-# not sure it this is the right way to include all the files under test.
-require_relative '../lib/chemistrykit/cli/helpers/formula_loader'
-require_relative '../lib/chemistrykit/formula/base'
-require_relative '../lib/chemistrykit/catalyst'
-require_relative '../lib/chemistrykit/configuration'
-
 require_relative '../spec/support/formulas/sub_module/basic_formula'
 require_relative '../spec/support/formulas/sub_module/chemist_formula'
 require_relative '../spec/support/formulas/sub_module/alt_chemist_formula'

data/spec/support/config.yaml

@@ -10,3 +10,11 @@ log:
 selenium_connect:
     log: 'evidence'
     host: 'localhost'
+basic_auth:
+    username: 'user'
+    password: 'pass'
+    http_path: '/basic'
+    https_path: '/secure'
+split_testing:
+    provider: 'optimizely'
+    opt_out: true

data/spec/unit/lib/chemistrykit/catalyst_spec.rb

@@ -1,6 +1,7 @@
 # Encoding: utf-8
 
 require 'spec_helper'
+require 'chemistrykit/catalyst'
 
 describe ChemistryKit::Catalyst do
 

data/spec/unit/lib/chemistrykit/cli/helpers/formula_loader_spec.rb

@@ -1,6 +1,7 @@
 # Encoding: utf-8
 
 require 'spec_helper'
+require 'chemistrykit/cli/helpers/formula_loader'
 # might be able to use something like this: https://github.com/alexeypetrushin/class_loader
 
 describe ChemistryKit::CLI::Helpers::FormulaLoader do

data/spec/unit/lib/chemistrykit/config/basic_auth_spec.rb

@@ -0,0 +1,76 @@
+# Encoding: utf-8
+
+require 'spec_helper'
+require 'chemistrykit/config/basic_auth'
+
+describe ChemistryKit::Config::BasicAuth do
+
+  VALID_USERNAME = 'user'
+  VALID_PASSWORD = 'pass'
+  VALID_URL = 'http://testsite.com'
+  VALID_HTTP_PATH = '/'
+  VALID_HTTPS_PATH = '/secure'
+
+  before(:each) do
+    @opts = { username: VALID_USERNAME,
+      password: VALID_PASSWORD,
+      base_url: VALID_URL,
+      http_path: VALID_HTTP_PATH,
+      https_path: VALID_HTTPS_PATH
+      }
+      @basic_auth = ChemistryKit::Config::BasicAuth.new(@opts)
+  end
+
+  it 'should be initialized with an hash of options' do
+    @basic_auth.should be_an_instance_of ChemistryKit::Config::BasicAuth
+  end
+
+  it 'should get the basic configuration values' do
+    @basic_auth.username.should eq VALID_USERNAME
+    @basic_auth.password.should eq VALID_PASSWORD
+    @basic_auth.base_url.should eq VALID_URL
+    @basic_auth.http_path.should eq VALID_HTTP_PATH
+    @basic_auth.https_path.should eq VALID_HTTPS_PATH
+  end
+
+  it 'should raise an error for invalid config parameters' do
+    expect do
+      ChemistryKit::Config::BasicAuth.new(@opts.merge(bad: 'value'))
+    end.to raise_error(ArgumentError, 'The config key: "bad" is unknown!')
+  end
+
+  it 'should report on the availability of http path' do
+    @basic_auth.http?.should be_true
+    basic_auth = ChemistryKit::Config::BasicAuth.new(@opts.merge(http_path: nil))
+    basic_auth.http?.should be_false
+    basic_auth = ChemistryKit::Config::BasicAuth.new({})
+    basic_auth.http?.should be_false
+  end
+
+  it 'should report on the availability of https path' do
+    @basic_auth.https?.should be_true
+    basic_auth = ChemistryKit::Config::BasicAuth.new(@opts.merge(https_path: nil))
+    basic_auth.https?.should be_false
+    basic_auth = ChemistryKit::Config::BasicAuth.new({})
+    basic_auth.https?.should be_false
+  end
+
+  it 'should return the http url' do
+    @basic_auth.http_url.should eq "http://#{VALID_USERNAME}:#{VALID_PASSWORD}@testsite.com/"
+  end
+
+  it 'should return the http url by default' do
+    basic_auth = ChemistryKit::Config::BasicAuth.new(@opts.merge(http_path: nil))
+    basic_auth.http_url.should eq "http://#{VALID_USERNAME}:#{VALID_PASSWORD}@testsite.com"
+  end
+
+  it 'should return the https url' do
+    @basic_auth.https_url.should eq "https://#{VALID_USERNAME}:#{VALID_PASSWORD}@testsite.com/secure"
+  end
+
+  it 'should return the https url by default' do
+    basic_auth = ChemistryKit::Config::BasicAuth.new(@opts.merge(https_path: nil))
+    basic_auth.https_url.should eq "https://#{VALID_USERNAME}:#{VALID_PASSWORD}@testsite.com"
+  end
+
+end

data/spec/unit/lib/chemistrykit/config/split_testing_spec.rb

@@ -0,0 +1,31 @@
+# Encoding: utf-8
+
+require 'spec_helper'
+require 'chemistrykit/config/split_testing'
+
+describe ChemistryKit::Config::SplitTesting do
+
+  VALID_PROVIDER = 'optimizely'
+
+  before(:each) do
+    @opts = { provider: VALID_PROVIDER, opt_out: true, base_url: 'http://google.com'
+      }
+      @split_testing = ChemistryKit::Config::SplitTesting.new(@opts)
+  end
+
+  it 'should be initialized with an hash of options' do
+    @split_testing.should be_an_instance_of ChemistryKit::Config::SplitTesting
+  end
+
+  it 'should get the basic configuration values' do
+    @split_testing.provider.should eq VALID_PROVIDER
+    @split_testing.base_url.should eq 'http://google.com'
+    @split_testing.opt_out?.should be_true
+  end
+
+  it 'should raise an error for invalid config parameters' do
+    expect do
+      ChemistryKit::Config::SplitTesting.new(@opts.merge(bad: 'value'))
+    end.to raise_error(ArgumentError, 'The config key: "bad" is unknown!')
+  end
+end

data/spec/unit/lib/chemistrykit/configuration_spec.rb

@@ -1,6 +1,7 @@
 # Encoding: utf-8
 
 require 'spec_helper'
+require 'chemistrykit/configuration'
 
 describe ChemistryKit::Configuration do
 
@@ -30,10 +31,18 @@ describe ChemistryKit::Configuration do
     config.concurrency.should eq VALID_CONCURRENCY
     config.retries_on_failure.should eq VALID_RETRIES_ON_FAILURE
     config.base_url.should eq VALID_BASE_URL
+
+    # log configurations
     config.log.path.should eq VALID_LOG_PATH
     config.log.results_file.should eq VALID_JUNIT
     config.log.format.should eq VALID_JUNIT_FORMAT_OUT
+
+    # selenium connect configurations
     config.selenium_connect.should eq @valid_selenium_connect_hash
+
+    # basic auth configurations
+
+    # a/b testing configurations
   end
 
   it 'should initialize with sane defaults' do
@@ -44,6 +53,8 @@ describe ChemistryKit::Configuration do
     config.log.results_file.should eq VALID_JUNIT
     config.log.format.should eq VALID_JUNIT_FORMAT_OUT
     config.selenium_connect.should eq({ log: VALID_LOG_PATH })
+    config.basic_auth.should be_nil
+    config.split_testing.should be_nil
   end
 
   it 'should initialize with a hash of configurations' do
@@ -80,4 +91,23 @@ describe ChemistryKit::Configuration do
     config =  ChemistryKit::Configuration.new selenium_connect: { log: 'sc-log', sauce_opts: { job_name: 'test' } }
     config.selenium_connect.should eq log: 'sc-log', sauce_opts: { job_name: 'test' }
   end
+
+  it 'should return the basic auth object if it is set' do
+    config =  ChemistryKit::Configuration.new basic_auth: { username: 'user' }
+    config.basic_auth.should be_an_instance_of ChemistryKit::Config::BasicAuth
+    config.basic_auth.username.should eq 'user'
+  end
+
+  it 'should pass the base url to the auth object' do
+    yaml_file = File.join(Dir.pwd, 'spec', 'support', VALID_CONFIG_FILE)
+    config = ChemistryKit::Configuration.initialize_with_yaml yaml_file
+    config.basic_auth.http_url.should eq 'http://user:pass@google.com/basic'
+  end
+
+  it 'should return the split testing object if it is set' do
+    yaml_file = File.join(Dir.pwd, 'spec', 'support', VALID_CONFIG_FILE)
+    config = ChemistryKit::Configuration.initialize_with_yaml yaml_file
+    config.split_testing.should be_an_instance_of ChemistryKit::Config::SplitTesting
+    config.split_testing.base_url.should eq VALID_BASE_URL
+  end
 end

data/spec/unit/lib/chemistrykit/formula/base_spec.rb

@@ -1,6 +1,7 @@
 # Encoding: utf-8
 
 require 'spec_helper'
+require 'chemistrykit/formula/base'
 
 describe ChemistryKit::Formula::Base do
 

data/spec/unit/lib/chemistrykit/split_testing/optimizely_provider_spec.rb

@@ -0,0 +1,42 @@
+# Encoding: utf-8
+
+require 'spec_helper'
+require 'chemistrykit/split_testing/optimizely_provider'
+
+describe ChemistryKit::SplitTesting::OptimizelyProvider do
+
+  before(:each) do
+    @config = double 'ChemistryKit::Config::SplitTesting'
+    @provider = ChemistryKit::SplitTesting::OptimizelyProvider.new(@config)
+  end
+
+  it 'should be initialized with the split testing config object' do
+    @provider.should be_an_instance_of ChemistryKit::SplitTesting::OptimizelyProvider
+  end
+
+  it 'should respond to config' do
+    @config.should_receive :provider
+    @provider.should respond_to :config
+    @provider.config.provider
+  end
+
+  it 'should set the cookie correctly on split' do
+
+    @config.should_receive :base_url
+    @config.should_receive(:opt_out?).and_return(true)
+
+    driver = double 'Selenium::WebDriver::Driver'
+    driver.should_receive(:get)
+
+    manage = double 'manage'
+    manage.should_receive(:add_cookie)
+    driver.should_receive(:manage).and_return(manage)
+
+    navigate = double 'navigate'
+    navigate.should_receive(:refresh)
+    driver.should_receive(:navigate).and_return(navigate)
+
+    @provider.split(driver)
+
+  end
+end

data/spec/unit/lib/chemistrykit/split_testing/provider_factory_spec.rb

@@ -0,0 +1,22 @@
+# Encoding: utf-8
+
+require 'spec_helper'
+require 'chemistrykit/split_testing/provider_factory'
+
+describe ChemistryKit::SplitTesting::ProviderFactory do
+
+  let(:config) { double 'ChemistryKit::Config::SplitTesting' }
+
+  it 'should determine what kind of provider to build' do
+    config.should_receive(:provider).and_return('optimizely')
+    ChemistryKit::SplitTesting::ProviderFactory.build(config)
+  end
+
+  it 'should raise an error for an un known provider' do
+    config.should_receive(:provider).twice.and_return('bad')
+    expect do
+      ChemistryKit::SplitTesting::ProviderFactory.build(config)
+    end.to raise_error(ArgumentError, 'The provider: "bad" is unknown!')
+  end
+
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: chemistrykit
 version: !ruby/object:Gem::Version
-  version: 3.9.0.rc2
+  version: 3.9.0.rc3
   prerelease: 6
 platform: ruby
 authors:
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-08-14 00:00:00.000000000 Z
+date: 2013-08-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: thor
@@ -99,7 +99,7 @@ dependencies:
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: 3.5.0
+        version: 3.6.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -107,7 +107,7 @@ dependencies:
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: 3.5.0
+        version: 3.6.0
 - !ruby/object:Gem::Dependency
   name: parallel_tests
   requirement: !ruby/object:Gem::Requirement
@@ -243,6 +243,7 @@ files:
 - features/multi-config.feature
 - features/new.feature
 - features/reporting.feature
+- features/split_testing.feature
 - features/step_definitions/steps.rb
 - features/support/env.rb
 - features/tags.feature
@@ -255,6 +256,8 @@ files:
 - lib/chemistrykit/cli/formula.rb
 - lib/chemistrykit/cli/helpers/formula_loader.rb
 - lib/chemistrykit/cli/new.rb
+- lib/chemistrykit/config/basic_auth.rb
+- lib/chemistrykit/config/split_testing.rb
 - lib/chemistrykit/configuration.rb
 - lib/chemistrykit/formula/base.rb
 - lib/chemistrykit/formula/chemist_aware.rb
@@ -263,6 +266,8 @@ files:
 - lib/chemistrykit/reporting/html_report_assembler.rb
 - lib/chemistrykit/rspec/html_formatter.rb
 - lib/chemistrykit/rspec/j_unit_formatter.rb
+- lib/chemistrykit/split_testing/optimizely_provider.rb
+- lib/chemistrykit/split_testing/provider_factory.rb
 - lib/templates/beaker.tt
 - lib/templates/beaker_with_formula.tt
 - lib/templates/chemistrykit/beakers/.gitkeep
@@ -300,6 +305,7 @@ files:
 - spec/integration/lib/chemistrykit/.gitkeep
 - spec/integration/lib/chemistrykit/formula/formula_lab_spec.rb
 - spec/integration/lib/chemistrykit/reporting/html_reporting_assembler_spec.rb
+- spec/integration/lib/chemistrykit/split_testing/provider_factory_spec.rb
 - spec/spec_helper.rb
 - spec/support/bad_chemists.csv
 - spec/support/chemists.csv
@@ -316,11 +322,15 @@ files:
 - spec/unit/lib/chemistrykit/chemist/repository/csv_chemist_repository_spec.rb
 - spec/unit/lib/chemistrykit/chemist_spec.rb
 - spec/unit/lib/chemistrykit/cli/helpers/formula_loader_spec.rb
+- spec/unit/lib/chemistrykit/config/basic_auth_spec.rb
+- spec/unit/lib/chemistrykit/config/split_testing_spec.rb
 - spec/unit/lib/chemistrykit/configuration_spec.rb
 - spec/unit/lib/chemistrykit/formula/base_spec.rb
 - spec/unit/lib/chemistrykit/formula/chemist_aware_spec.rb
 - spec/unit/lib/chemistrykit/formula/formula_lab_spec.rb
 - spec/unit/lib/chemistrykit/reporting/html_reporting_assembler_spec.rb
+- spec/unit/lib/chemistrykit/split_testing/optimizely_provider_spec.rb
+- spec/unit/lib/chemistrykit/split_testing/provider_factory_spec.rb
 homepage: https://github.com/arrgyle/chemistrykit
 licenses:
 - MIT
@@ -342,7 +352,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: 1.3.1
 requirements: []
 rubyforge_project: 
-rubygems_version: 1.8.24
+rubygems_version: 1.8.25
 signing_key: 
 specification_version: 3
 summary: A simple and opinionated web testing framework for Selenium that follows
@@ -360,12 +370,14 @@ test_files:
 - features/multi-config.feature
 - features/new.feature
 - features/reporting.feature
+- features/split_testing.feature
 - features/step_definitions/steps.rb
 - features/support/env.rb
 - features/tags.feature
 - spec/integration/lib/chemistrykit/.gitkeep
 - spec/integration/lib/chemistrykit/formula/formula_lab_spec.rb
 - spec/integration/lib/chemistrykit/reporting/html_reporting_assembler_spec.rb
+- spec/integration/lib/chemistrykit/split_testing/provider_factory_spec.rb
 - spec/spec_helper.rb
 - spec/support/bad_chemists.csv
 - spec/support/chemists.csv
@@ -382,8 +394,12 @@ test_files:
 - spec/unit/lib/chemistrykit/chemist/repository/csv_chemist_repository_spec.rb
 - spec/unit/lib/chemistrykit/chemist_spec.rb
 - spec/unit/lib/chemistrykit/cli/helpers/formula_loader_spec.rb
+- spec/unit/lib/chemistrykit/config/basic_auth_spec.rb
+- spec/unit/lib/chemistrykit/config/split_testing_spec.rb
 - spec/unit/lib/chemistrykit/configuration_spec.rb
 - spec/unit/lib/chemistrykit/formula/base_spec.rb
 - spec/unit/lib/chemistrykit/formula/chemist_aware_spec.rb
 - spec/unit/lib/chemistrykit/formula/formula_lab_spec.rb
 - spec/unit/lib/chemistrykit/reporting/html_reporting_assembler_spec.rb
+- spec/unit/lib/chemistrykit/split_testing/optimizely_provider_spec.rb
+- spec/unit/lib/chemistrykit/split_testing/provider_factory_spec.rb