jenkins-bot has submitted this change and it was merged.

Change subject: [BrowserTest] Client logins only once to upload all screenshots
......................................................................


[BrowserTest] Client logins only once to upload all screenshots

Change-Id: Ic242d8b2f35ced23809d4454a44f28f260bb3eb7
---
M modules/ve-mw/tests/browser/upload.rb
1 file changed, 5 insertions(+), 5 deletions(-)

Approvals:
  Zfilipin: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/modules/ve-mw/tests/browser/upload.rb 
b/modules/ve-mw/tests/browser/upload.rb
index a4930cc..34dbe4f 100644
--- a/modules/ve-mw/tests/browser/upload.rb
+++ b/modules/ve-mw/tests/browser/upload.rb
@@ -18,23 +18,23 @@
 [[Category:VisualEditor-#{language_code}]]"
 end
 
-def upload_image file_path
+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)
 
-  require 'mediawiki_api'
-  client = MediawikiApi::Client.new ENV["MEDIAWIKI_API_UPLOAD_URL"]
-  client.log_in ENV["MEDIAWIKI_USER"], ENV["MEDIAWIKI_PASSWORD"]
   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
+    upload_image file_path, client
   end
 end
 

-- 
To view, visit https://gerrit.wikimedia.org/r/148373
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ic242d8b2f35ced23809d4454a44f28f260bb3eb7
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/extensions/VisualEditor
Gerrit-Branch: master
Gerrit-Owner: Vikassy <vikasyaligar...@gmail.com>
Gerrit-Reviewer: Amire80 <amir.ahar...@mail.huji.ac.il>
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

Reply via email to