New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
octane fixes for 5.5.4-beta #194
Conversation
…is used with old Octane
…is used with old Octane
regression in UFT discovery when new Jenkins plugin is used with old Octane
…peline directly from jenkins
@@ -546,7 +554,7 @@ public static FreeStyleProject createExecutor(DiscoveryInfo discoveryInfo) { | |||
* @param uftExecutorJobNameWithTestRunner | |||
* @return | |||
*/ | |||
public static FreeStyleProject createExecutorByJobName(String uftExecutorJobNameWithTestRunner) { | |||
public static FreeStyleProject createExecutorByJobName(String uftExecutorJobNameWithTestRunner) throws IOException { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Refactor this method to reduce its Cognitive Complexity from 25 to the 15 allowed.
} | ||
if (parameterDefinition != null && testRunnerLogicalName.equals(parameterDefinition.getDefaultParameterValue().getValue().toString())) { | ||
foundDiscoveryJob = job; | ||
|
||
if(replaceParams){ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Avoid deeply nested control flow statements.
@@ -323,7 +323,7 @@ private static FreeStyleProject getDiscoveryJobForUftExecutor(DiscoveryInfo disc | |||
} | |||
} | |||
|
|||
private static FreeStyleProject getDiscoveryJobForTestRunner(DiscoveryInfo discoveryInfo) { | |||
private static FreeStyleProject getDiscoveryJobForUftTestRunner(DiscoveryInfo discoveryInfo) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Method getDiscoveryJobForUftTestRunner
has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring.
… empty values throws exception.
Defect #733048 Adding new Octane Server configuration in jenkins with…
…peline directly from jenkins - update SDK
…peline directly from jenkins - update SDK
Code Climate has analyzed commit 40d471f and detected 3 issues on this pull request. Here's the issue category breakdown:
Note: there is 1 critical issue. View more on Code Climate. |
defect #616017: [Jenkins] can "run successfully" disabled job from Octane
fixes part of https://issues.jenkins-ci.org/browse/JENKINS-54458