jenkins-bot has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/403633 )
Change subject: Remove deprecated ruby browser tests ...................................................................... Remove deprecated ruby browser tests As they got replaced in by nodejs selenium test in I17b04e365b3df2aba9ee14f26b75896288158a69 Bug: T180246 Depends-On: I9b9f9a1c7f32a86b61e7db35027393354462e28c Change-Id: I7cc93c30bbc499f9d389bc8c751a443b75a88655 --- D .rubocop.yml D .rubocop_todo.yml D Gemfile D Gemfile.lock D tests/browser/README.mediawiki D tests/browser/environments.yml D tests/browser/features/download_screen.feature D tests/browser/features/support/env.rb D tests/browser/features/support/hooks.rb D tests/browser/features/support/pages/download_screen_page.rb D tests/browser/features/support/pages/mainpage_page.rb D tests/browser/features/support/step_definitions/download_screen.rb D tests/browser/features/support/step_definitions/mainpage.rb 13 files changed, 0 insertions(+), 246 deletions(-) Approvals: Zfilipin: Looks good to me, approved jenkins-bot: Verified diff --git a/.rubocop.yml b/.rubocop.yml deleted file mode 100644 index 3c8a650..0000000 --- a/.rubocop.yml +++ /dev/null @@ -1,22 +0,0 @@ -inherit_from: .rubocop_todo.yml - -AllCops: - StyleGuideCopsOnly: true - -Metrics/LineLength: - Max: 100 - -Metrics/MethodLength: - Enabled: false - -Style/Alias: - Enabled: false - -Style/SignalException: - Enabled: false - -Style/StringLiterals: - EnforcedStyle: single_quotes - -Style/TrivialAccessors: - ExactNameMatch: true diff --git a/.rubocop_todo.yml b/.rubocop_todo.yml deleted file mode 100644 index 285a55f..0000000 --- a/.rubocop_todo.yml +++ /dev/null @@ -1,22 +0,0 @@ -# This configuration was generated by -# `rubocop --auto-gen-config` -# on 2017-12-06 13:21:11 +0000 using RuboCop version 0.48.1. -# The point is for the user to remove these configuration records -# one by one as the offenses are removed from the code base. -# Note that changes in the inspected code, or installation of new -# versions of RuboCop, may require this file to be generated again. - -# Offense count: 2 -# Configuration parameters: AllowHeredoc, AllowURI, URISchemes, IgnoreCopDirectives, IgnoredPatterns. -# URISchemes: http, https -Metrics/LineLength: - Max: 119 - -# Offense count: 2 -# Cop supports --auto-correct. -# Configuration parameters: EnforcedStyle, SupportedStyles. -# SupportedStyles: final_newline, final_blank_line -Style/TrailingBlankLines: - Exclude: - - 'tests/browser/features/support/env.rb' - - 'tests/browser/features/support/hooks.rb' diff --git a/Gemfile b/Gemfile deleted file mode 100644 index e011699..0000000 --- a/Gemfile +++ /dev/null @@ -1,6 +0,0 @@ -source 'https://rubygems.org' - -gem 'chunky_png', '~> 1.3.4' -gem 'mediawiki_selenium', '~> 1.7.0' -gem 'rake', '~> 10.4', '>= 10.4.2' -gem 'rubocop', '~> 0.51.0', require: false diff --git a/Gemfile.lock b/Gemfile.lock deleted file mode 100644 index 384059c..0000000 --- a/Gemfile.lock +++ /dev/null @@ -1,122 +0,0 @@ -GEM - remote: https://rubygems.org/ - specs: - ast (2.3.0) - builder (3.2.2) - childprocess (0.5.9) - ffi (~> 1.0, >= 1.0.11) - chunky_png (1.3.7) - cucumber (1.3.20) - builder (>= 2.1.2) - diff-lcs (>= 1.1.3) - gherkin (~> 2.12) - multi_json (>= 1.7.5, < 2.0) - multi_test (>= 0.1.2) - data_magic (0.22) - faker (>= 1.1.2) - yml_reader (>= 0.6) - diff-lcs (1.2.5) - domain_name (0.5.20161021) - unf (>= 0.0.5, < 1.0.0) - faker (1.6.6) - i18n (~> 0.5) - faraday (0.9.2) - multipart-post (>= 1.2, < 3) - faraday-cookie_jar (0.0.6) - faraday (>= 0.7.4) - http-cookie (~> 1.0.0) - faraday_middleware (0.10.0) - faraday (>= 0.7.4, < 0.10) - ffi (1.9.14) - ffi (1.9.14-x86-mingw32) - gherkin (2.12.2) - multi_json (~> 1.3) - gherkin (2.12.2-x86-mingw32) - multi_json (~> 1.3) - headless (2.3.1) - http-cookie (1.0.3) - domain_name (~> 0.5) - i18n (0.7.0) - json (1.8.3) - mediawiki_api (0.7.0) - faraday (~> 0.9, >= 0.9.0) - faraday-cookie_jar (~> 0.0, >= 0.0.6) - faraday_middleware (~> 0.10, >= 0.10.0) - mediawiki_selenium (1.7.2) - cucumber (~> 1.3, >= 1.3.20) - headless (~> 2.0, >= 2.1.0) - json (~> 1.8, >= 1.8.1) - mediawiki_api (~> 0.7, >= 0.7.0) - page-object (~> 1.0) - rest-client (~> 1.6, >= 1.6.7) - rspec-core (~> 2.14, >= 2.14.4) - rspec-expectations (~> 2.14, >= 2.14.4) - syntax (~> 1.2, >= 1.2.0) - thor (~> 0.19, >= 0.19.1) - mime-types (2.99.3) - multi_json (1.12.1) - multi_test (0.1.2) - multipart-post (2.0.0) - netrc (0.11.0) - page-object (1.2.0) - page_navigation (>= 0.9) - selenium-webdriver (>= 2.44.0) - watir-webdriver (>= 0.6.11) - page_navigation (0.10) - data_magic (>= 0.22) - parallel (1.12.0) - parser (2.4.0.2) - ast (~> 2.3) - powerpack (0.1.1) - rainbow (2.2.2) - rake - rake (10.5.0) - rest-client (1.8.0) - http-cookie (>= 1.0.2, < 2.0) - mime-types (>= 1.16, < 3.0) - netrc (~> 0.7) - rest-client (1.8.0-x86-mingw32) - ffi (~> 1.9) - http-cookie (>= 1.0.2, < 2.0) - mime-types (>= 1.16, < 3.0) - netrc (~> 0.7) - rspec-core (2.99.2) - rspec-expectations (2.99.2) - diff-lcs (>= 1.1.3, < 2.0) - rubocop (0.51.0) - parallel (~> 1.10) - parser (>= 2.3.3.1, < 3.0) - powerpack (~> 0.1) - rainbow (>= 2.2.2, < 3.0) - ruby-progressbar (~> 1.7) - unicode-display_width (~> 1.0, >= 1.0.1) - ruby-progressbar (1.9.0) - rubyzip (1.2.1) - selenium-webdriver (3.0.1) - childprocess (~> 0.5) - rubyzip (~> 1.0) - websocket (~> 1.0) - syntax (1.2.1) - thor (0.19.1) - unf (0.1.4) - unf_ext - unf_ext (0.0.7.2) - unf_ext (0.0.7.2-x86-mingw32) - unicode-display_width (1.3.0) - watir-webdriver (0.9.9) - selenium-webdriver (>= 2.46.2) - websocket (1.2.3) - yml_reader (0.7) - -PLATFORMS - ruby - x86-mingw32 - -DEPENDENCIES - chunky_png (~> 1.3.4) - mediawiki_selenium (~> 1.7.0) - rake (~> 10.4, >= 10.4.2) - rubocop (~> 0.51.0) - -BUNDLED WITH - 1.16.0 diff --git a/tests/browser/README.mediawiki b/tests/browser/README.mediawiki deleted file mode 100644 index 22092c4..0000000 --- a/tests/browser/README.mediawiki +++ /dev/null @@ -1,2 +0,0 @@ -For updated information about running browser tests for MediaWiki extension please see -https://www.mediawiki.org/wiki/Browser_testing/Running_tests diff --git a/tests/browser/environments.yml b/tests/browser/environments.yml deleted file mode 100644 index 4e84d0a..0000000 --- a/tests/browser/environments.yml +++ /dev/null @@ -1,30 +0,0 @@ -# Customize this configuration as necessary to provide defaults for various -# test environments. -# -# The set of defaults to use is determined by the MEDIAWIKI_ENVIRONMENT -# environment variable. -# -# export MEDIAWIKI_ENVIRONMENT=mw-vagrant-host -# bundle exec cucumber -# -# Additional variables set by the environment will override the corresponding -# defaults defined here. -# -# export MEDIAWIKI_ENVIRONMENT=mw-vagrant-host -# export MEDIAWIKI_USER=Selenium_user2 -# bundle exec cucumber -# -mw-vagrant-host: &default - mediawiki_url: http://127.0.0.1:8080/wiki/ - user_factory: true - -mw-vagrant-guest: - mediawiki_url: http://127.0.0.1/wiki/ - user_factory: true - -integration: - browser: chrome - user_factory: true - # mediawiki_url: THIS WILL BE SET BY JENKINS - -default: *default \ No newline at end of file diff --git a/tests/browser/features/download_screen.feature b/tests/browser/features/download_screen.feature deleted file mode 100644 index eef3013..0000000 --- a/tests/browser/features/download_screen.feature +++ /dev/null @@ -1,8 +0,0 @@ -@chrome @en.wikipedia.beta.wmflabs.org @firefox @integration -Feature: ElectronPdfService Download Screen - Background: - Given I am on the Main Page - - Scenario: Download screen is shown with correct data - When I click Download as PDF - Then I should see a print form with a download button diff --git a/tests/browser/features/support/env.rb b/tests/browser/features/support/env.rb deleted file mode 100644 index 3c896c2..0000000 --- a/tests/browser/features/support/env.rb +++ /dev/null @@ -1,3 +0,0 @@ -require 'mediawiki_selenium/cucumber' -require 'mediawiki_selenium/pages' -require 'mediawiki_selenium/step_definitions' \ No newline at end of file diff --git a/tests/browser/features/support/hooks.rb b/tests/browser/features/support/hooks.rb deleted file mode 100644 index 9e8e4cd..0000000 --- a/tests/browser/features/support/hooks.rb +++ /dev/null @@ -1,6 +0,0 @@ -# Needed for cucumber --dry-run -f stepdefs -require_relative 'env' - -Before('@skip') do |scenario| - scenario.skip_invoke! -end \ No newline at end of file diff --git a/tests/browser/features/support/pages/download_screen_page.rb b/tests/browser/features/support/pages/download_screen_page.rb deleted file mode 100644 index 06a0d9b..0000000 --- a/tests/browser/features/support/pages/download_screen_page.rb +++ /dev/null @@ -1,6 +0,0 @@ -class DownloadScreenPage - include PageObject - - hidden_field(:redirect_to_electron_action, css: '.mw-electronPdfService-selection-form [value=redirect-to-electron]') - button(:pdf_download_button, css: '.mw-electronPdfService-selection-form .oo-ui-buttonElement-button') -end diff --git a/tests/browser/features/support/pages/mainpage_page.rb b/tests/browser/features/support/pages/mainpage_page.rb deleted file mode 100644 index ee9f3f4..0000000 --- a/tests/browser/features/support/pages/mainpage_page.rb +++ /dev/null @@ -1,7 +0,0 @@ -class MainPage - include PageObject - - li(:download_as_pdf, css: '#coll-download-as-rl') - - page_url 'Main_Page' -end diff --git a/tests/browser/features/support/step_definitions/download_screen.rb b/tests/browser/features/support/step_definitions/download_screen.rb deleted file mode 100644 index 2641da1..0000000 --- a/tests/browser/features/support/step_definitions/download_screen.rb +++ /dev/null @@ -1,5 +0,0 @@ -Then(/^I should see a print form with a download button$/) do - on(DownloadScreenPage) do |page| - expect(page.pdf_download_button_element).to be_visible - end -end diff --git a/tests/browser/features/support/step_definitions/mainpage.rb b/tests/browser/features/support/step_definitions/mainpage.rb deleted file mode 100644 index 7eb84a2..0000000 --- a/tests/browser/features/support/step_definitions/mainpage.rb +++ /dev/null @@ -1,7 +0,0 @@ -Given(/^I am on the Main Page$/) do - visit(MainPage) -end - -When(/^I click Download as PDF$/) do - on(MainPage).download_as_pdf_element.when_visible.click -end -- To view, visit https://gerrit.wikimedia.org/r/403633 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7cc93c30bbc499f9d389bc8c751a443b75a88655 Gerrit-PatchSet: 2 Gerrit-Project: mediawiki/extensions/ElectronPdfService Gerrit-Branch: master Gerrit-Owner: WMDE-Fisch <christoph.jau...@wikimedia.de> Gerrit-Reviewer: Addshore <addshorew...@gmail.com> Gerrit-Reviewer: Andrew-WMDE <andrew.kos...@wikimedia.de> Gerrit-Reviewer: Pmiazga <pmia...@wikimedia.org> Gerrit-Reviewer: Tobias Gritschacher <tobias.gritschac...@wikimedia.de> Gerrit-Reviewer: Zfilipin <zfili...@wikimedia.org> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits