From 613edd4a3c8e9555b6bd0e32db98d27e19c6bb83 Mon Sep 17 00:00:00 2001
From: Zeljko Filipin <zeljko.filipin@gmail.com>
Date: Wed, 2 Oct 2013 18:16:11 +0200
Subject: [PATCH] Moving shared code to mediawiki-selenium Ruby gem

Bug: 53579
Change-Id: I3164a9640ec878723b622f42c7684ffdeedf9d8c
---
 tests/browser/Gemfile                         |  1 +
 tests/browser/Gemfile.lock                    |  2 +
 .../features/step_definitions/common_steps.rb |  2 +-
 tests/browser/features/support/env.rb         | 91 +------------------
 tests/browser/features/support/hooks.rb       | 25 -----
 tests/browser/features/support/sauce.rb       | 10 --
 6 files changed, 6 insertions(+), 125 deletions(-)
 delete mode 100644 tests/browser/features/support/sauce.rb

diff --git a/tests/browser/Gemfile b/tests/browser/Gemfile
index d38e5939..c723f578 100755
--- a/tests/browser/Gemfile
+++ b/tests/browser/Gemfile
@@ -5,6 +5,7 @@ source 'https://rubygems.org'
 
 gem 'cucumber'
 gem 'json'
+gem 'mediawiki-selenium'
 gem 'net-http-persistent'
 gem 'page-object'
 gem 'rspec-expectations'
diff --git a/tests/browser/Gemfile.lock b/tests/browser/Gemfile.lock
index 70d47da0..1dd2ab7c 100644
--- a/tests/browser/Gemfile.lock
+++ b/tests/browser/Gemfile.lock
@@ -21,6 +21,7 @@ GEM
       multi_json (~> 1.3)
     i18n (0.6.5)
     json (1.8.0)
+    mediawiki-selenium (0.1.5)
     multi_json (1.8.0)
     multi_test (0.0.2)
     net-http-persistent (2.9)
@@ -50,6 +51,7 @@ PLATFORMS
 DEPENDENCIES
   cucumber
   json
+  mediawiki-selenium
   net-http-persistent
   page-object
   rspec-expectations
diff --git a/tests/browser/features/step_definitions/common_steps.rb b/tests/browser/features/step_definitions/common_steps.rb
index 2660db3d..1d62f63e 100644
--- a/tests/browser/features/step_definitions/common_steps.rb
+++ b/tests/browser/features/step_definitions/common_steps.rb
@@ -6,7 +6,7 @@ Given(/^I am logged out$/) do
 end
 
 Given(/^I am logged in$/) do
-	visit(LoginPage).login_with(@mediawiki_username, ENV['MEDIAWIKI_PASSWORD'])
+	visit(LoginPage).login_with(ENV['MEDIAWIKI_USER'], ENV['MEDIAWIKI_PASSWORD'])
 	# Assert that login worked
 	loggedin = !@browser.execute_script( "return mw.user.isAnon();" )
 	loggedin.should be_true
diff --git a/tests/browser/features/support/env.rb b/tests/browser/features/support/env.rb
index e6c01012..b97a64e8 100644
--- a/tests/browser/features/support/env.rb
+++ b/tests/browser/features/support/env.rb
@@ -1,89 +1,2 @@
-# before all
-require 'bundler/setup'
-require 'page-object'
-require 'page-object/page_factory'
-require 'watir-webdriver'
-require 'yaml'
-
-World(PageObject::PageFactory)
-
-def browser(environment, test_name, language)
-  if environment == :saucelabs
-    sauce_browser(test_name, language)
-  else
-    local_browser(language)
-  end
-end
-def environment
-  if ENV['BROWSER_LABEL'] and ENV['SAUCE_ONDEMAND_USERNAME'] and ENV['SAUCE_ONDEMAND_ACCESS_KEY']
-    :saucelabs
-  else
-    :local
-  end
-end
-def local_browser(language)
-  if ENV['BROWSER_LABEL']
-    browser_label = ENV['BROWSER_LABEL'].to_sym
-  else
-    browser_label = :firefox
-  end
-
-  if language == 'default'
-    Watir::Browser.new browser_label
-  else
-    if browser_label == :firefox
-      profile = Selenium::WebDriver::Firefox::Profile.new
-    elsif browser_label == :chrome
-      profile = Selenium::WebDriver::Chrome::Profile.new
-    else
-      raise "Changing default language is currently supported only for Firefox and Chrome!"
-    end
-    profile['intl.accept_languages'] = language
-    Watir::Browser.new browser_label, :profile => profile
-  end
-end
-def sauce_api(json)
-  %x{curl -H 'Content-Type:text/json' -s -X PUT -d '#{json}' http://#{ENV['SAUCE_ONDEMAND_USERNAME']}:#{ENV['SAUCE_ONDEMAND_ACCESS_KEY']}@saucelabs.com/rest/v1/#{ENV['SAUCE_ONDEMAND_USERNAME']}/jobs/#{$session_id}}
-end
-def sauce_browser(test_name, language)
-  config = YAML.load_file('config/config.yml')
-  browser_label = config[ENV['BROWSER_LABEL']]
-
-  if language == 'default'
-    caps = Selenium::WebDriver::Remote::Capabilities.send(browser_label['name'])
-  elsif browser_label['name'] == 'firefox'
-    profile = Selenium::WebDriver::Firefox::Profile.new
-    profile['intl.accept_languages'] = language
-    caps = Selenium::WebDriver::Remote::Capabilities.firefox(:firefox_profile => profile)
-  elsif browser_label['name'] == 'chrome'
-    profile = Selenium::WebDriver::Chrome::Profile.new
-    profile['intl.accept_languages'] = language
-    caps = Selenium::WebDriver::Remote::Capabilities.chrome('chrome.profile' => profile.as_json['zip'])
-  end
-
-  caps.platform = browser_label['platform']
-  caps.version = browser_label['version']
-  caps[:name] = "#{test_name} #{ENV['JOB_NAME']}##{ENV['BUILD_NUMBER']}"
-
-  require 'selenium/webdriver/remote/http/persistent' # http_client
-  browser = Watir::Browser.new(
-    :remote,
-    http_client: Selenium::WebDriver::Remote::Http::Persistent.new,
-    url: "http://#{ENV['SAUCE_ONDEMAND_USERNAME']}:#{ENV['SAUCE_ONDEMAND_ACCESS_KEY']}@ondemand.saucelabs.com:80/wd/hub",
-    desired_capabilities: caps)
-
-  browser.wd.file_detector = lambda do |args|
-    # args => ['/path/to/file']
-    str = args.first.to_s
-    str if File.exist?(str)
-  end
-
-  browser
-end
-def test_name(scenario)
-  if scenario.respond_to? :feature
-    "#{scenario.feature.name}: #{scenario.name}"
-  elsif scenario.respond_to? :scenario_outline
-    "#{scenario.scenario_outline.feature.name}: #{scenario.scenario_outline.name}: #{scenario.name}"
-  end
-end
+require_relative 'hooks'
+require 'mediawiki/selenium'
diff --git a/tests/browser/features/support/hooks.rb b/tests/browser/features/support/hooks.rb
index ef42a9cd..d5358c1c 100644
--- a/tests/browser/features/support/hooks.rb
+++ b/tests/browser/features/support/hooks.rb
@@ -1,25 +1,8 @@
-config = YAML.load_file('config/config.yml')
-mediawiki_username = config['mediawiki_username']
-
-Before do |scenario|
-  @config = config
-  @random_string = Random.new.rand.to_s
-  @mediawiki_username = mediawiki_username
-  unless @language
-    @browser = browser(environment, test_name(scenario), 'default')
-    $session_id = @browser.driver.instance_variable_get(:@bridge).session_id
-  end
-end
-
 Before('@language') do |scenario|
   @language = true
   @scenario = scenario
 end
 
-Before('@login') do
-  puts "MEDIAWIKI_PASSWORD environment variable is not defined! Please export a value for that variable before proceeding." unless ENV['MEDIAWIKI_PASSWORD']
-end
-
 Before('@uls-in-personal-only') do |scenario|
   if uls_position() != 'personal'
     scenario.skip_invoke!
@@ -32,14 +15,6 @@ Before('@uls-in-sidebar-only') do |scenario|
   end
 end
 
-After do |scenario|
-  if environment == :saucelabs
-    sauce_api(%Q{{"passed": #{scenario.passed?}}})
-    sauce_api(%Q{{"public": true}})
-  end
-  @browser.close unless ENV['KEEP_BROWSER_OPEN'] == 'true'
-end
-
 After('@reset-preferences-after') do |scenario|
   visit(ResetPreferencesPage)
   on(ResetPreferencesPage).submit_element.click
diff --git a/tests/browser/features/support/sauce.rb b/tests/browser/features/support/sauce.rb
deleted file mode 100644
index 63eaeaaf..00000000
--- a/tests/browser/features/support/sauce.rb
+++ /dev/null
@@ -1,10 +0,0 @@
-require 'cucumber/formatter/junit'
-
-module Cucumber::Formatter
-  class Sauce < Junit
-    def format_exception(exception)
-      sauce_job_page = "Sauce Labs job URL: http://saucelabs.com/jobs/#{$session_id}\n"
-      ([sauce_job_page] + ["#{exception.message} (#{exception.class})"] + exception.backtrace).join("\n")
-    end
-  end
-end