Skip to content
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

Ability to add test listener #46

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Artyomcool
Copy link

Hey! I'm working on internal plugin for tracking tests state. For that reason I have to add listener to the spoon runner. Looks like you originally planned this ability, so I've decided to make a PR instead of maintaining own fork.

google()
maven { url "https://oss.sonatype.org/content/repositories/snapshots" } // For Spoon snapshot, until 2.0.0 is released
jcenter()
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We need maven { url "https://oss.sonatype.org/content/repositories/snapshots" } for Spoon 2.0.

@@ -1,5 +1,5 @@
GROUP=com.jaredsburrows
VERSION_NAME=1.4.1-SNAPSHOT
VERSION_NAME=1.4.1-SNAPSHOT-fork-1.0.0
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is this fork?

@jaredsburrows
Copy link
Owner

Have you tried doing this via instrumentation args?

@Artyomcool
Copy link
Author

Sorry, I'm just forgotten that I've done this PR. So, I'm using my master-branch for own development. If you still interested in PR, I can make it in a separate branch (actually, only 44350b commit is relevant).

@jaredsburrows
Copy link
Owner

Have you tested this locally? Maybe we can add some tests?

@Artyomcool
Copy link
Author

Yep, I've tested it. And I can add more checks to Task/Plugin Specs. If you are feel ok about the idea, I'll prepare separate PR and close this one.

@jaredsburrows
Copy link
Owner

You can just clean up this PR :)

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