Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: test/browsers/chromium.js

Issue 29891666: Noissue - Enable tests to run on other browser versions (Closed)
Patch Set: Created Sept. 25, 2018, 7:37 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« test/all.js ('K') | « test/all.js ('k') | test/browsers/firefox.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/browsers/chromium.js
===================================================================
--- a/test/browsers/chromium.js
+++ b/test/browsers/chromium.js
@@ -17,24 +17,20 @@
"use strict";
-// The Chromium version is a build number, quite obscure.
-// Chromium 63.0.3239.x is 508578
-// Chromium 65.0.3325.0 is 530368
-// We currently want Chromiun 63, as we still support it and that's the
-// loweset version that supports WebDriver.
-const CHROMIUM_REVISION = 508578;
-
const webdriver = require("selenium-webdriver");
const chrome = require("selenium-webdriver/chrome");
const {ensureChromium} = require("../../adblockpluscore/test/runners/" +
"chromium_download");
exports.platform = "chrome";
+exports.ensureBrowser = ensureChromium;
-exports.ensureBrowser = function()
-{
- return ensureChromium(CHROMIUM_REVISION);
-};
+// The Chromium version is a build number, quite obscure.
+// Chromium 63.0.3239.x is 508578
+// Chromium 65.0.3325.0 is 530368
+// We currently want Chromiun 63, as we still support it and that's the
+// loweset version that supports WebDriver.
+exports.oldestCompatibleVersion = 508578;
exports.getDriver = function(browserBinary, devenvPath)
{
« test/all.js ('K') | « test/all.js ('k') | test/browsers/firefox.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld