Skip to content

Commit

Permalink
fixed tests considering new TestLinkBuilder constructor param
Browse files Browse the repository at this point in the history
(debatable move, might have created additional constructor, but I think it is unnecessary )
  • Loading branch information
Mike Saravayskiy committed Apr 14, 2016
1 parent 0989242 commit fb3ce3c
Showing 1 changed file with 5 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,7 @@ public class TestTestLinkBuilder {
@Before
public void setUp() throws Exception {
builder = new TestLinkBuilder("No testlink", "No project", "No plan", "No platform", "No build",
"class, time, sample-job-$BUILD_ID", Boolean.FALSE, Boolean.FALSE, Boolean.FALSE, Boolean.FALSE, null,
"class, time, sample-job-$BUILD_ID", "host, user",Boolean.FALSE, Boolean.FALSE, Boolean.FALSE, Boolean.FALSE, null,
null, null, null, Boolean.FALSE, Boolean.FALSE, Boolean.FALSE, Boolean.FALSE, null);
}

Expand Down Expand Up @@ -129,7 +129,7 @@ public void testGetters() {
List<BuildStep> singleBuildSteps = new ArrayList<BuildStep>();
singleBuildSteps.add(shell);

builder = new TestLinkBuilder("No testlink", "No project", "No plan", "No platform", "No build", "class, time",
builder = new TestLinkBuilder("No testlink", "No project", "No plan", "No platform", "No build", "class, time", "host, user",
Boolean.FALSE, Boolean.FALSE, Boolean.FALSE, Boolean.FALSE, singleBuildSteps, null, null, null,
Boolean.FALSE, Boolean.FALSE, Boolean.FALSE, Boolean.FALSE, null);

Expand Down Expand Up @@ -160,6 +160,9 @@ public void testGetters() {
assertNotNull(builder.getCustomFields());
assertEquals(builder.getCustomFields(), "class, time");

assertNotNull(builder.getTestPlanCustomFields());
assertEquals(builder.getTestPlanCustomFields(), "host, user");

assertFalse(builder.getTransactional());
assertFalse(builder.getFailIfNoResults());
assertFalse(builder.getFailOnNotRun());
Expand Down

0 comments on commit fb3ce3c

Please sign in to comment.