Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
[JENKINS-15156] Found a problem with uninitialized run maps in new Ma…
…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.