Amire80 has uploaded a new change for review. https://gerrit.wikimedia.org/r/191563
Change subject: [BrowserTest] Move the screenshot uploading to a gem ...................................................................... [BrowserTest] Move the screenshot uploading to a gem * The methods from upload.rb are moved to a newly created gem: https://rubygems.org/gems/commons_upload * upload.rb is changed to simply call the method from the Gem. * Gemfile is updated accordingly. Bug: T89718 Change-Id: I229fbcd1b8c3cb03885947509aef27f0ae583a1c --- M Gemfile M Gemfile.lock M modules/ve-mw/tests/browser/upload.rb 3 files changed, 10 insertions(+), 41 deletions(-) git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/VisualEditor refs/changes/63/191563/1 diff --git a/Gemfile b/Gemfile index 42d6a77..f53b8e6 100644 --- a/Gemfile +++ b/Gemfile @@ -3,6 +3,7 @@ source 'https://rubygems.org' +gem 'commons_upload', '~> 0.0.1' gem 'mediawiki_api' gem 'mediawiki_selenium', '~> 0.4.1' gem 'rubocop', require: false diff --git a/Gemfile.lock b/Gemfile.lock index 439bee1..d76346f 100644 --- a/Gemfile.lock +++ b/Gemfile.lock @@ -1,3 +1,9 @@ +PATH + remote: /home/amire80/dev/commons_upload + specs: + commons_upload (0.0.1) + mediawiki_api (~> 0.3.0) + GEM remote: https://rubygems.org/ specs: @@ -95,6 +101,7 @@ ruby DEPENDENCIES + commons_upload (~> 0.0.1)! mediawiki_api mediawiki_selenium (~> 0.4.1) rubocop diff --git a/modules/ve-mw/tests/browser/upload.rb b/modules/ve-mw/tests/browser/upload.rb index 390bc10..3f2c332 100644 --- a/modules/ve-mw/tests/browser/upload.rb +++ b/modules/ve-mw/tests/browser/upload.rb @@ -1,42 +1,3 @@ -def license(language_code, file_name) - require 'date' - date = Date.today.to_s - "=={{int:filedesc}}== -{{Information -|description={{en|1=#{file_name}}} -|date=#{date} -|source=[[User:LanguageScreenshotBot|Automatically created by LanguageScreenshotBot]] -|author=[[User:LanguageScreenshotBot|Automatically created by LanguageScreenshotBot]] -|permission= -|other_versions= -|other_fields= -}} +require 'commons_upload' -=={{int:license-header}}== -{{Wikipedia-screenshot}} - -[[Category:VisualEditor automatic screenshots/#{language_code}]]" -end - -def upload_image(file_path, client) - language_code = ENV['LANGUAGE_SCREENSHOT_CODE'] - file_name = File.basename(file_path, '') - file_license = license(language_code, file_name) - - client.upload_image(file_name, file_path, file_license, true) - sleep 5 # Restriction in bot speed: https://commons.wikimedia.org/wiki/Commons:Bots#Bot_speed -end - -def upload_images - screenshot_directory = ENV['LANGUAGE_SCREENSHOT_PATH'] || './screenshots' - require 'mediawiki_api' - client = MediawikiApi::Client.new ENV['MEDIAWIKI_API_UPLOAD_URL'] - client.log_in ENV['MEDIAWIKI_USER'], ENV['MEDIAWIKI_PASSWORD'] - Dir["#{screenshot_directory}/*.png"].each do |file_path| - puts "Uploading #{file_path}" - upload_image file_path, client - end -end - -ENV['MEDIAWIKI_PASSWORD'] = ENV[ENV['MEDIAWIKI_PASSWORD_VARIABLE']] if ENV['MEDIAWIKI_PASSWORD_VARIABLE'] -upload_images +CommonsUpload.images -- To view, visit https://gerrit.wikimedia.org/r/191563 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I229fbcd1b8c3cb03885947509aef27f0ae583a1c Gerrit-PatchSet: 1 Gerrit-Project: mediawiki/extensions/VisualEditor Gerrit-Branch: master Gerrit-Owner: Amire80 <[email protected]> _______________________________________________ MediaWiki-commits mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
