Skip to content

Conversation

@cbeust
Copy link
Contributor

@cbeust cbeust commented Jan 20, 2017

I don't expect you to merge this, just showing you what your project looks like when built by Kobalt.

Run with:

./kobaltw test

Note that I renamed your test class from unsigned to UnsignedTest, otherwise Kobalt won't find
it by default (you can configure the test runner to look at all the .class files but that might take
a while, so by default, Kobalt only looks for **/*Test.class).

Configuring your build to upload your project to bintray as a Maven artifact is just a few more lines, I can show you how if you're curious.

I don't expect you to merge this, just showing you what your project looks like when built by Kobalt.

Run with:

./kobaltw test

Note that I renamed your test class from `unsigned` to `UnsignedTest`, otherwise Kobalt won't find
it by default (you can configure the test runner to look at all the .class files but that might take
a while, so by default, Kobalt only looks for **/*Test.class).
@elect86
Copy link
Collaborator

elect86 commented Jan 20, 2017

Cool, thanks!

but that might take a while

What do you mean? To implement or to write in the script?

@cbeust
Copy link
Contributor Author

cbeust commented Jan 20, 2017

If the build system just parses all the **/*.class files, it might take a while to load them and introspect them in order to find out if they are test classes (Do they have test annotations? Do they have the right constructor? ...).

By narrowing to **/*Test.class, you make sure the build system only introspects real test classes and doesn't waste its time on helper classes.

@elect86 elect86 merged commit f373509 into kotlin-graphics:master Jan 22, 2017
@cbeust cbeust deleted the KobaltBuild branch January 22, 2017 20:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants