Skip to content

Commit

Permalink
FIX : get jobs inside folders
Browse files Browse the repository at this point in the history
  • Loading branch information
sokratis.sidereas committed Dec 6, 2023
1 parent f24f7f9 commit 5228546
Show file tree
Hide file tree
Showing 2 changed files with 221 additions and 206 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -42,9 +42,6 @@
import hudson.tasks.Publisher;
import hudson.util.FormValidation;
import hudson.util.Secret;

import hudson.util.VariableResolver;

import jenkins.model.Jenkins;
import jenkins.tasks.SimpleBuildStep;
import net.sf.json.JSONObject;
Expand Down Expand Up @@ -185,7 +182,7 @@ public void perform(Run<?, ?> run, FilePath workspace, EnvVars env, Launcher lau
aggregatedSavedData = getPreviousData(run.getPreviousSuccessfulBuild(), validatedData);
}
// Collect Data
Collector collector = new Collector(jenkinsUrl, desc.getUsername(), desc.getPassword(), listener.getLogger());
Collector collector = new Collector(jenkinsUrl, desc.getUsername(), desc.getPassword(), listener.getLogger(), validatedData);
collector.collectResults(validatedData, compareWithPrevious(), ignoreRunningJobs());
collector.closeJenkinsConnection();
// Analyze Results
Expand Down Expand Up @@ -225,7 +222,7 @@ public boolean perform(final AbstractBuild build, final Launcher launcher, final
aggregatedSavedData = getPreviousData(build, validatedData);
}
// Collect Data
Collector collector = new Collector(jenkinsUrl, desc.getUsername(), desc.getPassword(), listener.getLogger());
Collector collector = new Collector(jenkinsUrl, desc.getUsername(), desc.getPassword(), listener.getLogger(), validatedData);

Check warning on line 225 in src/main/java/com/jenkins/testresultsaggregator/TestResultsAggregator.java

View check run for this annotation

ci.jenkins.io / Code Coverage

Not covered lines

Lines 185-225 are not covered by tests
collector.collectResults(validatedData, compareWithPrevious(), ignoreRunningJobs());
collector.closeJenkinsConnection();
// Analyze Results
Expand Down Expand Up @@ -347,7 +344,7 @@ public FormValidation doCheckOutOfDateResults(@QueryParameter final String outOf

@RequirePOST
public FormValidation doTestApiConnection(@QueryParameter final String jenkinsUrl, @QueryParameter final String username, @QueryParameter final Secret password) {
// https://www.jenkins.io/doc/developer/security/form-validation/
// https://www.jenkins.io/doc/developer/security/form-validation/
Jenkins.get().checkPermission(Jenkins.ADMINISTER);
try {
JenkinsServer jenkins = new JenkinsServer(new URI(jenkinsUrl), username, password.getPlainText());
Expand Down
Loading

0 comments on commit 5228546

Please sign in to comment.