diff --git a/pom.xml b/pom.xml index 0df4dbe..0e3bd51 100644 --- a/pom.xml +++ b/pom.xml @@ -10,7 +10,7 @@ browserstack-integration - 1.2.1-SNAPSHOT + 1.2.2-SNAPSHOT hpi BrowserStack diff --git a/src/main/java/com/browserstack/automate/ci/jenkins/BrowserStackBuildWrapper.java b/src/main/java/com/browserstack/automate/ci/jenkins/BrowserStackBuildWrapper.java index cd7d93d..133ad70 100644 --- a/src/main/java/com/browserstack/automate/ci/jenkins/BrowserStackBuildWrapper.java +++ b/src/main/java/com/browserstack/automate/ci/jenkins/BrowserStackBuildWrapper.java @@ -19,6 +19,7 @@ import hudson.util.DescribableList; import org.apache.commons.lang.StringUtils; import org.kohsuke.stapler.DataBoundConstructor; +import org.kohsuke.stapler.DataBoundSetter; import java.io.IOException; import java.io.PrintStream; @@ -31,15 +32,19 @@ public class BrowserStackBuildWrapper extends BuildWrapper { private static final char CHAR_MASK = '*'; - private final LocalConfig localConfig; + private LocalConfig localConfig; private String credentialsId; private String username; private String accesskey; @DataBoundConstructor - public BrowserStackBuildWrapper(String credentialsId, LocalConfig localConfig) { + public BrowserStackBuildWrapper(String credentialsId) { this.credentialsId = credentialsId; + } + + @DataBoundSetter + public void setLocalConfig(LocalConfig localConfig) { this.localConfig = localConfig; } diff --git a/src/test/java/com/browserstack/automate/ci/jenkins/AutomateTestActionTest.java b/src/test/java/com/browserstack/automate/ci/jenkins/AutomateTestActionTest.java index cccd9bb..e5adaa4 100644 --- a/src/test/java/com/browserstack/automate/ci/jenkins/AutomateTestActionTest.java +++ b/src/test/java/com/browserstack/automate/ci/jenkins/AutomateTestActionTest.java @@ -124,7 +124,8 @@ public void addBuildStep() throws IOException { localConfig.setLocalOptions("-force"); BrowserStackBuildWrapper buildWrapper = - new BrowserStackBuildWrapper(credentialsId, localConfig); + new BrowserStackBuildWrapper(credentialsId); + buildWrapper.setLocalConfig(localConfig); project.getBuildWrappersList().add(buildWrapper); } diff --git a/src/test/java/com/browserstack/automate/ci/jenkins/AutomateTestDataPublisherTest.java b/src/test/java/com/browserstack/automate/ci/jenkins/AutomateTestDataPublisherTest.java index e0ea504..f0bd7cf 100644 --- a/src/test/java/com/browserstack/automate/ci/jenkins/AutomateTestDataPublisherTest.java +++ b/src/test/java/com/browserstack/automate/ci/jenkins/AutomateTestDataPublisherTest.java @@ -115,7 +115,8 @@ public void addBuildStep() throws IOException { LocalConfig localConfig = new LocalConfig(); localConfig.setLocalOptions("-force"); - BrowserStackBuildWrapper buildWrapper = new BrowserStackBuildWrapper(credentialsId, localConfig); + BrowserStackBuildWrapper buildWrapper = new BrowserStackBuildWrapper(credentialsId); + buildWrapper.setLocalConfig(localConfig); project.getBuildWrappersList().add(buildWrapper); }