how to enable assertion with wisdom ? #423

Closed
hboumedane opened this Issue Jan 27, 2015 · 7 comments

Comments

Projects
None yet
2 participants
@hboumedane

No description provided.

@cescoffier

This comment has been minimized.

Show comment
Hide comment
@cescoffier

cescoffier Jan 27, 2015

Member

When launched in prod, you can configure it using:
JVM_ARGS = -ea

Member

cescoffier commented Jan 27, 2015

When launched in prod, you can configure it using:
JVM_ARGS = -ea

@hboumedane

This comment has been minimized.

Show comment
Hide comment
@hboumedane

hboumedane Jan 27, 2015

I'm in debug mode , i tried to use this option with wisdom in watch mode:

-Ddebug=5005 -Dea
            <plugin>
                <groupId>org.apache.maven.plugins</groupId>
                <artifactId>maven-surefire-plugin</artifactId>
                <version>2.16</version>
                <configuration>
                    <enableAssertions>true</enableAssertions>
                </configuration>
            </plugin>

but no way to get my assertion working ????

I'm in debug mode , i tried to use this option with wisdom in watch mode:

-Ddebug=5005 -Dea
            <plugin>
                <groupId>org.apache.maven.plugins</groupId>
                <artifactId>maven-surefire-plugin</artifactId>
                <version>2.16</version>
                <configuration>
                    <enableAssertions>true</enableAssertions>
                </configuration>
            </plugin>

but no way to get my assertion working ????

@cescoffier

This comment has been minimized.

Show comment
Hide comment
@cescoffier

cescoffier Jan 27, 2015

Member

Surefire enables the assertions by default. As your test runs in the Surefire JVM, it should work out of the box.

Member

cescoffier commented Jan 27, 2015

Surefire enables the assertions by default. As your test runs in the Surefire JVM, it should work out of the box.

@hboumedane

This comment has been minimized.

Show comment
Hide comment
@hboumedane

hboumedane Jan 27, 2015

not working !!!

not working !!!

@cescoffier

This comment has been minimized.

Show comment
Hide comment
@cescoffier

cescoffier Jan 31, 2015

Member

Oh, I miss the point where you are in watch mode, then yes, we need to pass the ea parameter to the JVM.

Member

cescoffier commented Jan 31, 2015

Oh, I miss the point where you are in watch mode, then yes, we need to pass the ea parameter to the JVM.

@cescoffier

This comment has been minimized.

Show comment
Hide comment
@cescoffier

cescoffier Jan 31, 2015

Member

So the best way would be to support a jvmArgs parameter to indicate the JVM parameters.

Member

cescoffier commented Jan 31, 2015

So the best way would be to support a jvmArgs parameter to indicate the JVM parameters.

@cescoffier

This comment has been minimized.

Show comment
Hide comment
@cescoffier

cescoffier Jan 31, 2015

Member

Fixed with #424

Just add the following argument:

mvn clean wisdom:run -Ddebug=5005 -DjvmArgs="-ea"

Member

cescoffier commented Jan 31, 2015

Fixed with #424

Just add the following argument:

mvn clean wisdom:run -Ddebug=5005 -DjvmArgs="-ea"

@cescoffier cescoffier closed this Jan 31, 2015

@cescoffier cescoffier added this to the 0.7.1 milestone Jan 31, 2015

@cescoffier cescoffier self-assigned this Jan 31, 2015

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