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

Change subject: Run language screenshots from Jenkins
......................................................................


Run language screenshots from Jenkins

Bug: T139613
Change-Id: I869dd27f58e7972456fb09792262cd1be0aeeaa0
---
M build/screenshots.js
A build/screenshots.yml
2 files changed, 23 insertions(+), 3 deletions(-)

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



diff --git a/build/screenshots.js b/build/screenshots.js
index 1c57c77..a485230 100644
--- a/build/screenshots.js
+++ b/build/screenshots.js
@@ -1,17 +1,32 @@
 /* jshint node: true */
 /* global seleniumUtils, langs */
 var i, l, clientSize,
+       accessKey = process.env.SAUCE_ONDEMAND_ACCESS_KEY,
        chrome = require( 'selenium-webdriver/chrome' ),
-       test = require( 'selenium-webdriver/testing' ),
        fs = require( 'fs' ),
-       Jimp = require( 'jimp' );
+       Jimp = require( 'jimp' ),
+       test = require( 'selenium-webdriver/testing' ),
+       username = process.env.SAUCE_ONDEMAND_USERNAME,
+       webdriver = require( 'selenium-webdriver' );
 
 function runTests( lang ) {
        test.describe( 'Screenshots: ' + lang, function () {
                var driver;
 
                test.beforeEach( function () {
-                       driver = new chrome.Driver();
+                       // Use Sauce Labs when running on Jenins
+                       if ( process.env.JENKINS_URL ) {
+                               driver = new 
webdriver.Builder().withCapabilities( {
+                                               browserName: 
process.env.BROWSER,
+                                               platform: process.env.PLATFORM,
+                                               username: username,
+                                               accessKey: accessKey
+                                       } ).usingServer( 'http://' + username + 
':' + accessKey +
+                                               
'@ondemand.saucelabs.com:80/wd/hub' ).build();
+                       } else {
+                               // If not running on Jenkins, use local browser
+                               driver = new chrome.Driver();
+                       }
 
                        driver.manage().timeouts().setScriptTimeout( 20000 );
                        driver.manage().window().setSize( 1200, 1000 );
diff --git a/build/screenshots.yml b/build/screenshots.yml
new file mode 100644
index 0000000..7f5005a
--- /dev/null
+++ b/build/screenshots.yml
@@ -0,0 +1,5 @@
+BROWSER:
+  - chrome
+
+PLATFORM:
+  - Linux

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I869dd27f58e7972456fb09792262cd1be0aeeaa0
Gerrit-PatchSet: 12
Gerrit-Project: mediawiki/extensions/VisualEditor
Gerrit-Branch: master
Gerrit-Owner: Zfilipin <[email protected]>
Gerrit-Reviewer: Amire80 <[email protected]>
Gerrit-Reviewer: Esanders <[email protected]>
Gerrit-Reviewer: Jforrester <[email protected]>
Gerrit-Reviewer: Zfilipin <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to