Skip to content

Commit

Permalink
[test] Run Browserstack tests on master only (#578)
Browse files Browse the repository at this point in the history
  • Loading branch information
michaldudak authored Sep 4, 2024
1 parent dffaaec commit 3141989
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion test/karma.conf.js
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@ const webpack = require('webpack');
const CI = Boolean(process.env.CI);
// renovate PRs are based off of upstream branches.
// Their CI run will be a branch based run not PR run and therefore won't have a CIRCLE_PR_NUMBER
const isPR = Boolean(process.env.CIRCLE_PULL_REQUEST);

let build = `Base UI local ${new Date().toISOString()}`;

Expand All @@ -21,7 +22,7 @@ const browserStack = {
// Since we have limited resources on BrowserStack we often time out on PRs.
// However, BrowserStack rarely fails with a true-positive so we use it as a stop gap for release not merge.
// But always enable it locally since people usually have to explicitly have to expose their BrowserStack access key anyway.
enabled: true,
enabled: !CI || !isPR || process.env.BROWSERSTACK_FORCE === 'true',
username: process.env.BROWSERSTACK_USERNAME,
accessKey: process.env.BROWSERSTACK_ACCESS_KEY,
build,
Expand Down

0 comments on commit 3141989

Please sign in to comment.