Skip to content

Commit

Permalink
[JENKINS-15156] Found a problem with uninitialized run maps in new Ma…
Browse files Browse the repository at this point in the history
…ven modules.

Not observed in actual usage, but reproducible (for me at least, though apparently not ci.jenkins-ci.org) in a test:
java.lang.AssertionError: null
    at jenkins.model.lazy.AbstractLazyLoadRunMap.load(AbstractLazyLoadRunMap.java:628)
    at jenkins.model.lazy.AbstractLazyLoadRunMap.all(AbstractLazyLoadRunMap.java:581)
    at jenkins.model.lazy.AbstractLazyLoadRunMap.entrySet(AbstractLazyLoadRunMap.java:243)
    at java.util.AbstractMap$2$1.<init>(AbstractMap.java:378)
    at java.util.AbstractMap$2.iterator(AbstractMap.java:377)
    at hudson.util.RunList.iterator(RunList.java:103)
    at hudson.util.RunList.size(RunList.java:114)
    at hudson.maven.MavenProjectTest.testDeleteSetBuildDeletesModuleBuilds(MavenProjectTest.java:159)
Originally-Committed-As: 09c7cf6ad7cfb4d88d6d8936f29b13f3ca187875
  • Loading branch information
jglick committed Feb 21, 2013
1 parent 98399cb commit dc2215b
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions src/main/java/hudson/maven/MavenModuleSetBuild.java
Expand Up @@ -905,6 +905,7 @@ private void parsePoms(BuildListener listener, PrintStream logger, EnvVars envVa
} else {// this looks like a new module
logger.println(Messages.MavenModuleSetBuild_DiscoveredModule(pom.name,pom.displayName));
mm = new MavenModule(project,pom,getNumber());
mm.onCreatedFromScratch();
modules.put(mm.getModuleName(),mm);
needsDependencyGraphRecalculation = true;
}
Expand Down

0 comments on commit dc2215b

Please sign in to comment.