Merge pull request #1 from sinstein/track_travis_sessions

Suffix username to mark BrowserStack sessions run via Travis
JuliaCI · May 2, 2017 · b661f9a · b661f9a
2 parents b7ac3e7 + fb0431c
commit b661f9a
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/lib/travis/build/addons/browserstack.rb b/lib/travis/build/addons/browserstack.rb
@@ -44,7 +44,7 @@ def before_before_script
             sh.echo 'Starting BrowserStack Local', ansi: :yellow
             sh.cmd "#{build_start_command(browserstack_key)}"
             browserstack_user = username.to_s
-            sh.export ENV_USER, browserstack_user, echo: true unless browserstack_user.empty?
+            sh.export ENV_USER, browserstack_user + "-travis", echo: true unless browserstack_user.empty?
             sh.export ENV_KEY, browserstack_key, echo: false
             sh.export ENV_LOCAL, 'true', echo: true
           end

diff --git a/spec/build/addons/browserstack_spec.rb b/spec/build/addons/browserstack_spec.rb
@@ -40,7 +40,7 @@
     let(:config) { { os: 'linux', username: 'user1', access_key: 'accesskey' } }
 
     it_behaves_like 'installs browserstack local'
-    it { should include_sexp [:export, ["#{described_class::ENV_USER}", config[:username]], {:echo => true}] }
+    it { should include_sexp [:export, ["#{described_class::ENV_USER}", config[:username] + "-travis"], {:echo => true}] }
     it { should include_sexp [:cmd, "#{described_class::BROWSERSTACK_HOME}/BrowserStackLocal -d start #{config[:access_key]} -localIdentifier $BROWSERSTACK_LOCAL_IDENTIFIER"] }
   end