Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
[JENKINS-22811] Remove unnecessary check in test unit. Fix misleading…
… test unit comment.

(cherry picked from commit 3661ed5)
  • Loading branch information
christ66 authored and olivergondza committed Feb 1, 2015
1 parent 937ed1b commit 13774a1
Showing 1 changed file with 2 additions and 8 deletions.
10 changes: 2 additions & 8 deletions test/src/test/java/hudson/model/ItemGroupMixInTest.java
Expand Up @@ -104,8 +104,8 @@ public void xmlFileFailsToLoad() throws Exception {
}

/**
* This test unit makes sure that any exception that is thrown is caught, however the folder
* will continue to load not impacting any other jobs in the folder.
* This test unit makes sure that jobs that contain bad get*Action methods will continue to
* load the project.
*/
@LocalData
@Issue("JENKINS-22811")
Expand All @@ -115,12 +115,6 @@ public void xmlFileReadExceptionOnLoad() throws Exception {
assertNotNull(d);
Collection<TopLevelItem> items = d.getItems();
assertEquals(5, items.size());
Iterator<TopLevelItem> iterator = items.iterator();
assertEquals("badBuildStep", iterator.next().getName());
assertEquals("badBuildTrigger", iterator.next().getName());
assertEquals("badBuildWrapper", iterator.next().getName());
assertEquals("badPublisher", iterator.next().getName());
assertEquals("valid", iterator.next().getName());
}

@TestExtension
Expand Down

0 comments on commit 13774a1

Please sign in to comment.