-
Notifications
You must be signed in to change notification settings - Fork 1
Conversation
bdf01e0
to
5853e8a
Compare
|
||
|
||
public void testShouldIncludeRamlFilesWhenSourcePathOrClassPathSpecified() throws Exception { | ||
File pom = getTestFile("src/test/resources/generate-using-source-and-classpath/pom.xml"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can be final
public void testShouldIncludeRamlFilesWhenSourcePathOrClassPathSpecified() throws Exception { | ||
File pom = getTestFile("src/test/resources/generate-using-source-and-classpath/pom.xml"); | ||
|
||
GenerateMojo mojo = (GenerateMojo) lookupConfiguredMojo(pom, "generate"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
final
GenerateMojo mojo = (GenerateMojo) lookupConfiguredMojo(pom, "generate"); | ||
|
||
mojo.execute(); | ||
List<Pair<Raml, GeneratorConfig>> capturedGeneratorArgs = DummyGeneratorCaptor.getInstance().capturedArgs(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
final
|
||
assertThat(capturedGeneratorArgs.size(), greaterThan(5)); | ||
|
||
Raml raml = capturedGeneratorArgs.get(0).getLeft(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
final
Raml raml = capturedGeneratorArgs.get(0).getLeft(); | ||
assertThat(raml.getTitle(), equalTo("example.raml")); | ||
|
||
Path expectedSourceDirectory = Paths.get(project.getBasedir().toString(), "src", "raml"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
final and more below.....
<skip.schema.pojo.generation>true</skip.schema.pojo.generation> | ||
<skip.schema.event.pojo.generation>true</skip.schema.event.pojo.generation> | ||
<useSourcePathAndClassPath>true</useSourcePathAndClassPath> | ||
<useClassPath>false</useClassPath>--> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can this commented section be removed?
private boolean useSourcePathAndClassPath = false; | ||
|
||
@Parameter(property = "useClassPath", defaultValue = "false") | ||
private boolean useClassPath = false; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can this be an enumeration rather than two booleans?
f174900
to
22e0a80
Compare
22e0a80
to
bf9afda
Compare
No description provided.