PR#3: New Testcases for Xmvn -

xmvn

#3 New Testcases for Xmvn

Closed Sep 1, 2016 by mizdebsk. Opened Aug 19, 2016 by rvais.

rvais/xmvn testCases into master

rvais commented Aug 19, 2016

Here are working test cases I managed to create. I merged everything to one commit. Please check commit message and other requirements.
Hope, there are no typos.

mizdebsk commented Aug 24, 2016

The build fails for me:

java.lang.AssertionError: Dependency was not removed from the list.
    at org.junit.Assert.fail(Assert.java:88)
    at org.fedoraproject.xmvn.deployer.DeploymentRequestTest.addAndGetDependenciesTest(DeploymentRequestTest.java:111)

rvais commented Aug 24, 2016

I think I know what might be the cause. This branch does not have DeploymentRequest class version with removeProperty() bug fix implemented. I have found that issue as a result of this test case and created separate branch for the fix itself, which you already merged, but I thought I should not commit change in this branch.

rebased

Aug 25, 2016

mizdebsk commented Sep 1, 2016

Moved to Github, closing.

Pull-Request has been closed by mizdebsk

Sep 1, 2016

Metadata

Assignee

None

Tags

No Tags

xmvn

#3 New Testcases for Xmvn Closed Sep 1, 2016 by mizdebsk. Opened Aug 19, 2016 by rvais. rvais/xmvn testCases into master

Metadata

#3 New Testcases for Xmvn

Closed Sep 1, 2016 by mizdebsk. Opened Aug 19, 2016 by rvais.

rvais/xmvn testCases into master