New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

update test PolynomFunction #38

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
3 participants
@marcheto

marcheto commented Jun 3, 2016

Add test for method "negate()" from file "PolynomialFunction.java"

@@ -37,5 +37,6 @@ public void testCrossProduct() {
Vector2D p5 = new Vector2D(2, 1);
Assert.assertEquals(-1.0, p5.crossProduct(p1, p2), epsilon);

This comment has been minimized.

@atodorov

atodorov Jun 8, 2016

Remove this extra line and amend this commit

<artifactId>jacoco-maven-plugin</artifactId>
<version>${commons.jacoco.version}</version>
<scope>test</scope>
</dependency>
</dependencies>

This comment has been minimized.

@atodorov

atodorov Jun 8, 2016

Guys,
I'm not sure if we really need this, someone please comment if this should stay or be removed. Thanks,

@chtompki

This comment has been minimized.

Member

chtompki commented Jun 11, 2016

We also currently work in the develop branch as opposed to master. I know that may seem a tad backwards, but that's where the work is currently going on.

@atodorov

This comment has been minimized.

atodorov commented Jun 16, 2016

Rebased in #40. Please close this one.

@marcheto

This comment has been minimized.

marcheto commented Jun 16, 2016

Close this pull request according to previous comment. This pull request is rebased in #40.

@marcheto marcheto closed this Jun 16, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment