diff --git a/app/renderer/actions/Session.js b/app/renderer/actions/Session.js index 6d9ea9ab1e..61cca9f3e6 100644 --- a/app/renderer/actions/Session.js +++ b/app/renderer/actions/Session.js @@ -422,13 +422,12 @@ export function newSession(caps, attachSessId = null) { if (!desiredCapabilities['tb:options']) { desiredCapabilities['tb:options'] = {}; } - desiredCapabilities['tb:options'].key = session.server.testingbot.key || process.env.TB_KEY; - desiredCapabilities['tb:options'].secret = - session.server.testingbot.secret || process.env.TB_SECRET; - if ( - !(session.server.testingbot.key || process.env.TB_KEY) || - !(session.server.testingbot.secret || process.env.TB_SECRET) - ) { + username = session.server.testingbot.username || process.env.TB_KEY; + accessKey = session.server.testingbot.accessKey || process.env.TB_SECRET; + desiredCapabilities['tb:options'].key = username; + desiredCapabilities['tb:options'].secret = accessKey; + desiredCapabilities['tb:options'].source = 'appiumdesktop'; + if (!username || !accessKey) { showError(new Error(i18n.t('testingbotCredentialsRequired'))); return false; } diff --git a/app/renderer/components/Session/ServerTabTestingbot.js b/app/renderer/components/Session/ServerTabTestingbot.js index 935894f994..469d484a93 100644 --- a/app/renderer/components/Session/ServerTabTestingbot.js +++ b/app/renderer/components/Session/ServerTabTestingbot.js @@ -3,14 +3,14 @@ import React from 'react'; import {INPUT} from '../AntdTypes'; -const testingbotKeyPlaceholder = (t) => { +const testingbotUsernamePlaceholder = (t) => { if (process.env.TB_KEY) { return t('usingDataFoundIn', {environmentVariable: 'TB_KEY'}); } return t('yourUsername'); }; -const testingbotSecretPlaceholder = (t) => { +const testingbotAccessKeyPlaceholder = (t) => { if (process.env.TB_SECRET) { return t('usingDataFoundIn', {environmentVariable: 'TB_SECRET'}); } @@ -24,10 +24,10 @@ const ServerTabTestingbot = ({server, setServerParam, t}) => ( setServerParam('key', e.target.value)} + value={server.testingbot.username} + onChange={(e) => setServerParam('username', e.target.value)} /> @@ -36,10 +36,10 @@ const ServerTabTestingbot = ({server, setServerParam, t}) => ( setServerParam('secret', e.target.value)} + value={server.testingbot.accessKey} + onChange={(e) => setServerParam('accessKey', e.target.value)} />