Maven Plugin

[dodie]

Provide a Maven plugin that eases project configuration and ad-hoc usage. See the JaCoCo's maven plugin for inspiration.

[basalt79]

@dodie, I will have a look on this.

[dodie]

Awesome! 👍

[basalt79]

i have a first draft version to be able to get rid of the "maven-dependency-plugin" completly, and also the surefire plugin will be reduced

<plugin>
	<groupId>org.apache.maven.plugins</groupId>
	<artifactId>maven-surefire-plugin</artifactId>
	<version>2.19.1</version>
	<configuration>
		<properties>
			<property>
				<name>listener</name>
				<value>hu.advancedweb.scott.runtime.ScottRunListener</value>
			</property>
		</properties>
	</configuration>
</plugin>

currently i see no way to define also the listener via the scott-mojo

[dodie]

Awesome, it's already much shorter!
Is it necessary to use ScottRunListener? If the scott-agent jar is in place, the auto-instrumented ScottReportingRule should be sufficient.

[basalt79]

its working without in the junit 4 and 5 tests, but in cucumber is miss the last exception in the output

[dodie]

It sounds nice:)
Could you push the code?

[basalt79]

i prepare the pull request, you will have it soon

[dodie]

Thanks!

[dodie]

I had some time in my hands so I've checked this out. This is seriously awesome, the configuration will be much easier with this plugin, and the maven plugin itself is simple and clean. Great great job!

I've fiddled a bit with the cucumber reporting and I think the problem is caused by a small glitch in the ScottCucumberEnricher.

Currently, when processing a Result object in its result method, it has an if-else branch to handle non-failing and failing steps:

• For every non-failing step, it calls Scott to render the output.
• For the failing step, it does not render anything but relies on the Exception object that it will contain the necessary information. For this to work, it requires the ScottRunListener to enhance the exception object before this code runs.

I think this can be fixed by calling Scott to render the report for the failing step as well, just with the additional exception object. (E.g.: FailureRenderer.render(null, null, result.getError());.)

[dodie]

👍