page_magic 1.0.0.alpha2 → 1.0.0.alpha3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: f197806fbf5357bdcf84663abfbf87902c89f835
-  data.tar.gz: da30e148aa6cc6782c0d0c399fc45c43d00ee3d2
+  metadata.gz: 0527b24c83ab8ec283c1fac457d76771126f434a
+  data.tar.gz: 89f1a07fd4c975c82105eb551d8e2b6013cec794
 SHA512:
-  metadata.gz: 8c3b3277416e23f0767b0e0f4b63879882fd9c1872fedb290fb727be512d91b06b53056dac552c539a65cec17f20e831e4d17cd7e97ef1bdfeee941eb528e6a9
-  data.tar.gz: 8d953ec0f7af0749bedca9396caf556098b7b74b5d09f2c82a4b904bbb41fc604bc7e3b0bae87e10d367e2496546342ac31f07652be996b799ddea4b7c71ec3a
+  metadata.gz: 15099e5bb4c4f7432b0eaa82366809fc81b931581b2d3a4a14254fd0cfc35dfe5c7a5ce2357f9a2fd6d0ce7c207e1feaf54223758671baa7b3776334e011b6c2
+  data.tar.gz: 01328446a136c0254f8a1f41dff01d1675be9919e27ceb0cf70a365e8f0449131e7e2f5e7feb27363b642d9907f74f06bd8cd782e530e5180698abd1879e86d6

data/.ruby-version

@@ -1 +1 @@
-2.0.0
+2.1

data/Gemfile

@@ -1,6 +1,7 @@
 source "http://rubygems.org"
 
 gem 'capybara'
+gem 'activesupport'
 gem 'wait'
 
 group :test do

data/Gemfile.lock

@@ -1,6 +1,12 @@
 GEM
   remote: http://rubygems.org/
   specs:
+    activesupport (4.2.3)
+      i18n (~> 0.7)
+      json (~> 1.7, >= 1.7.7)
+      minitest (~> 5.1)
+      thread_safe (~> 0.3, >= 0.3.4)
+      tzinfo (~> 1.1)
     addressable (2.3.6)
     builder (3.2.2)
     capybara (2.1.0)
@@ -38,6 +44,7 @@ GEM
       guard (~> 2.0)
     hashie (2.0.5)
     highline (1.6.21)
+    i18n (0.7.0)
     jeweler (2.0.1)
       builder
       bundler (>= 1.0)
@@ -58,6 +65,7 @@ GEM
     method_source (0.8.2)
     mime-types (1.25)
     mini_portile (0.5.1)
+    minitest (5.7.0)
     multi_json (1.7.3)
     multi_xml (0.5.5)
     multipart-post (1.2.0)
@@ -113,8 +121,11 @@ GEM
       tilt (~> 1.3, >= 1.3.4)
     slop (3.5.0)
     thor (0.19.1)
+    thread_safe (0.3.5)
     tilt (1.4.1)
     timers (1.1.0)
+    tzinfo (1.2.2)
+      thread_safe (~> 0.1)
     wait (0.5.1)
     watir-webdriver (0.6.4)
       selenium-webdriver (>= 2.18.0)
@@ -127,6 +138,7 @@ PLATFORMS
   ruby
 
 DEPENDENCIES
+  activesupport
   capybara
   guard
   guard-ctags-bundler

data/README.md

@@ -1,5 +1,4 @@
-[![Build Status](https://travis-ci.org/Ladtech/page_magic.png)](https://travis-ci.org/Ladtech/page_magic)
-#PageMagic 
+#PageMagic
 PageMagic is an API for testing web applications. 
 
 It has a simple but powerful DSL which makes modelling and interacting with your pages easy.

data/VERSION

@@ -1 +1 @@
-1.0.0.alpha2
+1.0.0.alpha3

data/lib/page_magic/driver.rb

@@ -0,0 +1,19 @@
+module PageMagic
+  class Driver
+    attr_reader :supported_browsers, :handler
+    def initialize(*supported_browsers, &block)
+      @handler = block
+      @supported_browsers = supported_browsers
+    end
+
+    def support? browser
+      supported_browsers.include?(browser)
+    end
+
+    def build(app, browser:, options:{})
+      options[:browser] = browser
+      driver_clazz = handler.call
+      driver_clazz.new(app, options)
+    end
+  end
+end

data/lib/page_magic/drivers/poltergeist.rb

@@ -0,0 +1,9 @@
+module PageMagic
+  class Drivers
+    Poltergeist = Driver.new(:poltergeist) do
+      require 'capybara/poltergeist'
+      Capybara::Poltergeist::Driver
+    end
+  end
+
+end

data/lib/page_magic/drivers/rack_test.rb

@@ -0,0 +1,7 @@
+module PageMagic
+  class Drivers
+    RackTest = Driver.new(:rack_test) do
+      Capybara::RackTest::Driver
+    end
+  end
+end

data/lib/page_magic/drivers/selenium.rb

@@ -0,0 +1,8 @@
+module PageMagic
+  class Drivers
+    Selenium = Driver.new(:chrome, :firefox)  do
+      require 'watir-webdriver'
+      Capybara::Selenium::Driver
+    end
+  end
+end

data/lib/page_magic/drivers.rb

@@ -0,0 +1,31 @@
+require 'page_magic/driver'
+module PageMagic
+  class Drivers
+    def all
+      @all ||=[]
+    end
+
+    def register(driver)
+      all << driver
+    end
+
+
+    def find browser
+      all.find{|driver|driver.support?(browser)}
+    end
+
+    def load path="#{__dir__}/drivers"
+      require 'active_support/inflector'
+
+      Dir["#{path}/*.rb"].each do |driver_file|
+        require driver_file
+        driver_name = File.basename(driver_file)[/(.*)\.rb$/, 1]
+        register eval(driver_name.classify)
+      end
+    end
+
+    def == other
+      other.is_a?(Drivers) && other.all == other.all
+    end
+  end
+end

data/lib/page_magic/element_context.rb

@@ -28,19 +28,15 @@ module PageMagic
         element_locator = element_locator_factory.call(@page_element, *args)
       end
 
-      result = element_locator.browser_element
-
-      return element_locator if element_locator.section?
-
       [:set, :select_option, :unselect_option, :click].each do |action_method|
-        apply_hooks(page_element: result,
+        apply_hooks(page_element: element_locator.browser_element,
                     capybara_method: action_method,
                     before_hook: element_locator.before,
                     after_hook: element_locator.after,
         )
       end
 
-      result
+      element_locator.section? ? element_locator : element_locator.browser_element
     end
 
     def apply_hooks(options)

data/lib/page_magic/session.rb

@@ -36,8 +36,8 @@ module PageMagic
       end
     end
 
-    def visit page
-      raw_session.visit page.url
+    def visit(page, url: nil)
+      raw_session.visit url || page.url
       @current_page = page.new self
       self
     end

data/lib/page_magic.rb

@@ -1,41 +1,33 @@
 $LOAD_PATH.unshift("#{File.dirname(__FILE__)}")
 require 'capybara'
 require 'page_magic/exceptions'
-require 'page_magic/browser'
 require 'page_magic/session'
 require 'page_magic/elements'
 require 'page_magic/element_context'
 require 'page_magic/element'
 require 'page_magic/page_magic'
+require 'page_magic/drivers'
 
 module PageMagic
+  class UnspportedBrowserException < Exception;end
+
   class << self
-    def session browser=nil, options = {}
-        if browser.is_a?(Symbol)
-          application = options.delete(:application)
-
-          Capybara.register_driver browser do |app|
-            options[:browser] = browser
-            case browser
-              when :poltergeist
-                require 'capybara/poltergeist'
-                Capybara::Poltergeist::Driver.new(app)
-              when :rack_test
-                Capybara::RackTest::Driver.new(app, options)
-              else
-                require 'watir-webdriver'
-                Capybara::Selenium::Driver.new(app, options)
-            end
-
-          end
-
-          Session.new(Capybara::Session.new(browser, application))
-        else
-          Capybara.reset!
-          Capybara.app = browser[:application] if browser.is_a?(Hash) && browser[:application]
-          Session.new(Capybara.current_session)
-        end
 
+    def drivers
+      @drivers ||= Drivers.new.tap do |drivers|
+        drivers.load
+      end
+    end
+
+    def session(application: nil, browser: :rack_test, options: {})
+      driver = drivers.find(browser)
+      raise UnspportedBrowserException unless driver
+
+      Capybara.register_driver browser do |app|
+        driver.build(app, browser: browser, options: options)
+      end
+
+      Session.new(Capybara::Session.new(browser, application))
     end
 
     def included clazz
@@ -53,11 +45,10 @@ module PageMagic
           PageMagic.pages << clazz
         end
       end
-
     end
 
     def pages
       @pages||=[]
     end
   end
-end
+end

data/spec/page_magic/driver_spec.rb

@@ -0,0 +1,47 @@
+require 'page_magic/driver'
+
+module PageMagic
+  describe Driver do
+
+    let(:driver_class) do
+      Class.new do
+        attr_reader :rack_app, :options
+        def initialize rack_app, options
+          @rack_app = rack_app
+          @options = options
+        end
+        def == driver
+          driver.rack_app == rack_app && driver.options == options
+        end
+      end
+    end
+    subject do
+      described_class.new :custom_browser do
+        driver_class
+      end
+    end
+
+    describe '#supports?' do
+      context 'browser is in supported browsers' do
+        it 'returns true' do
+          expect(subject.support?(:custom_browser)).to eq(true)
+        end
+      end
+
+      context 'browser is not in supported browsers' do
+        it 'returns false' do
+          expect(subject.support?(:unsupported_browser)).to eq(false)
+        end
+      end
+    end
+    describe '#build' do
+      it 'adds the browser to the options supplied to the driver' do
+        expect(subject.build(:rack_app, browser: :custom_browser, options: {}).options).to eq(browser: :custom_browser)
+      end
+
+      it 'creates an instance of the supplied driver' do
+        expect(subject.build(:rack_app,browser: :custom_browser, options: {})).to eq(driver_class.new(:rack_app, {browser: :custom_browser}))
+      end
+    end
+  end
+end

data/spec/page_magic/drivers/poltergeist_spec.rb

@@ -0,0 +1,10 @@
+require 'page_magic/drivers/poltergeist'
+module PageMagic
+  class Drivers
+    describe Poltergeist do
+      it %q{is capybara's poltergeist driver} do
+        expect(described_class.build(:app, browser: :poltergeist, options:{})).to be_a(Capybara::Poltergeist::Driver)
+      end
+    end
+  end
+end

data/spec/page_magic/drivers/rack_test_spec.rb

@@ -0,0 +1,10 @@
+require 'page_magic/drivers/rack_test'
+module PageMagic
+  class Drivers
+    describe RackTest do
+      it %q{is capybara's rack test driver} do
+        expect(described_class.build(:app, browser: :rack_test, options:{})).to be_a(Capybara::RackTest::Driver)
+      end
+    end
+  end
+end

data/spec/page_magic/drivers/selenium_spec.rb

@@ -0,0 +1,10 @@
+require 'page_magic/drivers/selenium'
+module PageMagic
+  class Drivers
+    describe Selenium do
+      it 'is selenium' do
+        expect(described_class.build(:app, browser: :selenium, options:{})).to be_a(Capybara::Selenium::Driver)
+      end
+    end
+  end
+end

data/spec/page_magic/drivers_spec.rb

@@ -0,0 +1,41 @@
+require 'page_magic/drivers'
+
+module PageMagic
+
+  describe Drivers do
+    subject { described_class.new }
+    let(:expected_driver) { Driver.new(:browser_driver) }
+    describe '#register' do
+      it 'stores the driver' do
+        subject.register expected_driver
+        expect(subject.all).to eq([expected_driver])
+      end
+    end
+
+    describe '#find' do
+      it 'returns the registered driver' do
+        subject.register expected_driver
+        expect(subject.find(:browser_driver)).to eq(expected_driver)
+      end
+    end
+
+    describe '#load' do
+      include_context :files
+      it 'loads the drivers in the specified path' do
+
+        class_definition=<<-RUBY
+          class CustomDriver;
+            def self.support? browser
+              true
+            end
+          end
+        RUBY
+
+        File.write("#{scratch_dir}/custom_driver.rb", class_definition )
+
+        subject.load(scratch_dir)
+        expect(subject.find(:custom_browser)).to be(::CustomDriver)
+      end
+    end
+  end
+end

data/spec/page_magic/element_context_spec.rb

@@ -87,23 +87,40 @@ describe PageMagic::ElementContext do
 
   describe 'hooks' do
 
+    subject(:page) do
+      elements_page.new.tap do|p|
+        p.visit
+      end
+    end
 
-    #it 'only applies them if browser element supports event' do
-    #  page = elements_page.new
-    #end
-
-    it 'should execute a before and after action that gives access to the browser' do
-
-      page = elements_page.new
-      page.visit
-
-      selector = {text: 'a link'}
+    before do
       browser = page.browser
       browser.should_receive(:call_in_before_hook)
       browser.should_receive(:call_in_after_before_hook)
+    end
 
 
-      elements_page.link(:create, selector) do
+    context 'section' do
+      it 'applies the hooks' do
+
+        elements_page.section(:form, id: 'form') do
+          link :form_link, id: 'form_link'
+
+          before do |page_browser|
+            page_browser.call_in_before_hook
+          end
+
+          after do |page_browser|
+            page_browser.call_in_after_before_hook
+          end
+        end
+
+        described_class.new(page, page.browser, self).form.click
+      end
+    end
+
+    it 'should execute a before and after action that gives access to the browser' do
+      elements_page.link(:create, text: 'a link') do
         before do |page_browser|
           page_browser.call_in_before_hook
         end

data/spec/page_magic/session_spec.rb

@@ -70,10 +70,20 @@ describe PageMagic::Session do
     end
   end
 
-  it 'should visit the given url' do
-    browser.should_receive(:visit).with(page.url)
-    session = PageMagic::Session.new(browser).visit(page)
-    session.current_page.should be_a(page)
+  describe '#visit' do
+    context 'url supplied' do
+      it 'uses this url instead of the one defined on the page class' do
+        expect(browser).to receive(:visit).with(:custom_url)
+        session = PageMagic::Session.new(browser).visit(page, url: :custom_url)
+        session.current_page.should be_a(page)
+      end
+    end
+
+    it 'visits the url on defined on the page class' do
+      browser.should_receive(:visit).with(page.url)
+      session = PageMagic::Session.new(browser).visit(page)
+      session.current_page.should be_a(page)
+    end
   end
 
   it 'should return the current url' do

data/spec/page_magic/usage/starting_a_session_spec.rb

@@ -1,4 +1,5 @@
 describe 'PageMagic.session' do
+
   let(:app_class) do
     Class.new do
       def call env
@@ -11,35 +12,10 @@ describe 'PageMagic.session' do
     Capybara.drivers[browser].call(nil)
   end
 
-  context 'using a symbol as a parameter' do
-    it 'sets up a session using the specified browser' do
-      Capybara::Session.should_receive(:new).with(:chrome, nil).and_return(:chrome_session)
-
-      session = PageMagic.session(:chrome)
-
-      registered_driver(:chrome).should == Capybara::Selenium::Driver.new(nil, browser: :chrome)
-
-      session.raw_session.should == :chrome_session
-    end
-
-    context 'browsers' do
-      it 'supports poltergeist' do
-        session = PageMagic.session(:poltergeist, application: app_class.new)
-        session.raw_session.driver.is_a?(Capybara::Poltergeist::Driver).should be_true
-      end
-
-      it 'supports selenium' do
-        session = PageMagic.session(:firefox, application: app_class.new)
-        session.raw_session.driver.is_a?(Capybara::Selenium::Driver).should be_true
-      end
-    end
-  end
-
-  context 'defaulting the browser used from PageMagic sessions' do
-    it "uses what ever Capybara's default_driver is set to" do
-      Capybara.default_driver = :rack_test
-      session = PageMagic.session
-      session.raw_session.mode.should == :rack_test
+  context 'specificying a browser' do
+    it 'loads the driver for the specified browser' do
+      session = PageMagic.session(browser: :firefox)
+      session.raw_session.driver.is_a?(Capybara::Selenium::Driver).should be_true
     end
   end
 
@@ -52,7 +28,7 @@ describe 'PageMagic.session' do
     end
 
     it 'can run against an rack application using a particular browser' do
-      session = PageMagic.session(:rack_test, application: app_class.new)
+      session = PageMagic.session(browser: :rack_test, application: app_class.new)
       session.raw_session.mode.should == :rack_test
       session.raw_session.visit('/')
       session.raw_session.text.should == 'hello world!!'

data/spec/page_magic_spec.rb

@@ -0,0 +1,44 @@
+require 'page_magic'
+
+describe PageMagic do
+  describe '::drivers' do
+    it 'returns loaded drivers' do
+      expected_drivers = described_class::Drivers.new.tap do |drivers|
+        drivers.load
+      end
+
+      expect(described_class.drivers).to eq(expected_drivers)
+    end
+  end
+
+  describe '::session' do
+    context 'specifying the browser' do
+      it 'loads the correct driver' do
+        session = described_class.session(browser: :firefox)
+        session.raw_session.driver.is_a?(Capybara::Selenium::Driver).should be_true
+      end
+    end
+
+    context 'specifying a rack application' do
+      it 'configures capybara to run against the app' do
+        session = described_class.session(application: :rack_application)
+        expect(session.raw_session.app).to be(:rack_application)
+      end
+    end
+
+    context 'specifying options' do
+      it 'passes the options to the browser driver' do
+        options = {option: :config}
+        session = described_class.session(options: options, browser: :chrome)
+
+        expect(session.raw_session.driver.options).to include(options)
+      end
+    end
+
+    context 'driver for browser not found' do
+      it 'raises an error' do
+        expect{described_class.session(browser: :invalid)}.to raise_exception described_class::UnspportedBrowserException
+      end
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -11,7 +11,22 @@ require 'page_magic'
 require 'capybara/rspec'
 require 'helpers/capybara'
 
+shared_context :files do
+  require 'tmpdir'
+  def scratch_dir
+    @dir ||= Dir.mktmpdir
+  end
+end
 
+shared_context :rack_application do
+  let(:rack_application) do
+    Class.new do
+      def call env
+        [200, {}, ["hello world!!"]]
+      end
+    end
+  end
+end
 
 RSpec.configure do
 

metadata

@@ -1,83 +1,97 @@
 --- !ruby/object:Gem::Specification
 name: page_magic
 version: !ruby/object:Gem::Version
-  version: 1.0.0.alpha2
+  version: 1.0.0.alpha3
 platform: ruby
 authors:
 - Leon Davis
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-04-15 00:00:00.000000000 Z
+date: 2015-07-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: capybara
   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: activesupport
+  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: wait
   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: jeweler
   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: guard
   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: guard-ctags-bundler
   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'
 description: Framework for modeling and interacting with webpages which wraps capybara
@@ -87,10 +101,9 @@ extensions: []
 extra_rdoc_files:
 - README.md
 files:
-- .rspec
-- .ruby-gemset
-- .ruby-version
-- .travis.yml
+- ".rspec"
+- ".ruby-gemset"
+- ".ruby-version"
 - Gemfile
 - Gemfile.lock
 - Guardfile
@@ -99,7 +112,11 @@ files:
 - VERSION
 - lib/ext/string.rb
 - lib/page_magic.rb
-- lib/page_magic/browser.rb
+- lib/page_magic/driver.rb
+- lib/page_magic/drivers.rb
+- lib/page_magic/drivers/poltergeist.rb
+- lib/page_magic/drivers/rack_test.rb
+- lib/page_magic/drivers/selenium.rb
 - lib/page_magic/element.rb
 - lib/page_magic/element_context.rb
 - lib/page_magic/elements.rb
@@ -110,7 +127,11 @@ files:
 - spec/element_spec.rb
 - spec/helpers/capybara.rb
 - spec/member_methods_spec.rb
-- spec/page_magic/browser_spec.rb
+- spec/page_magic/driver_spec.rb
+- spec/page_magic/drivers/poltergeist_spec.rb
+- spec/page_magic/drivers/rack_test_spec.rb
+- spec/page_magic/drivers/selenium_spec.rb
+- spec/page_magic/drivers_spec.rb
 - spec/page_magic/element_context_spec.rb
 - spec/page_magic/elements_spec.rb
 - spec/page_magic/session_spec.rb
@@ -119,6 +140,7 @@ files:
 - spec/page_magic/usage/include_page_magic_spec.rb
 - spec/page_magic/usage/interacting_with_pages_spec.rb
 - spec/page_magic/usage/starting_a_session_spec.rb
+- spec/page_magic_spec.rb
 - spec/spec_helper.rb
 homepage: https://github.com/ladtech/page_magic
 licenses:
@@ -130,12 +152,12 @@ 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: 1.3.1
 requirements: []

data/.travis.yml

@@ -1,4 +0,0 @@
-rvm:
-  - '1.9.3'
-  - '2.0.0'
-

data/lib/page_magic/browser.rb

@@ -1,19 +0,0 @@
-module PageMagic
-  module Browser
-
-    class << self
-      attr_writer :default
-      def session
-        @session ||= PageMagic.session(default)
-      end
-
-      def default
-        @default || :firefox
-      end
-    end
-
-    def browser
-      Browser.session
-    end
-  end
-end

data/spec/page_magic/browser_spec.rb

@@ -1,35 +0,0 @@
-describe PageMagic::Browser do
-  let(:app) { Object.new }
-
-  before do
-    PageMagic::Browser.instance_variable_set(:@session, nil)
-    app.extend PageMagic::Browser
-  end
-
-  describe 'default' do
-    it 'should be firefox' do
-      PageMagic::Browser.default.should == :firefox
-    end
-  end
-
-  describe 'browser' do
-    it 'should return the existing session' do
-      session_instance = app.browser
-      app.browser.should == session_instance
-    end
-
-    it 'should create a session if not already set' do
-      new_session = double(:new_session)
-
-      PageMagic.should_receive(:session).with(:firefox).and_return new_session
-      app.browser.should == new_session
-    end
-
-    it 'should use custom browser' do
-      PageMagic.should_receive(:session).with(:custom_browser)
-
-      PageMagic::Browser.default = :custom_browser
-      app.browser
-    end
-  end
-end