Skip to content

Support unit tests with the plugin#2734

Closed
droidpl wants to merge 1 commit intorealm:masterfrom
droidpl:patch-1
Closed

Support unit tests with the plugin#2734
droidpl wants to merge 1 commit intorealm:masterfrom
droidpl:patch-1

Conversation

@droidpl
Copy link
Copy Markdown

@droidpl droidpl commented May 4, 2016

This change will inject also the dependency for jUnit plain tests in Android

This change will inject also the dependency for jUnit plain tests in Android
@cmelchior
Copy link
Copy Markdown
Contributor

@droidpl I am not against adding this, but since you cannot run Realm on the JVM yet, you could not really use it, but perhaps you want to mock Realm in JVM tests?

👍

Please review @realm/java

@zaki50
Copy link
Copy Markdown
Contributor

zaki50 commented May 4, 2016

And we need to apply our transformer to jvm tests

@cmelchior
Copy link
Copy Markdown
Contributor

cmelchior commented May 4, 2016

Ohh, good point. Forgot about that. That isn't supported yet, right?

@droidpl
Copy link
Copy Markdown
Author

droidpl commented May 4, 2016

As far as I know with robolectric there is some way to make it work instead of mocking everything, that is why I am proposing it. What do you think?

@cmelchior
Copy link
Copy Markdown
Contributor

We have an issue here for tracking Robolectric support: #1867 , but as @zaki50 points out, right now the Transforms are not applied to JVM tests. Until that is fixed I don't think we should merge this PR. Sorry.

@droidpl
Copy link
Copy Markdown
Author

droidpl commented May 4, 2016

Sure, just keep it there and lets wait it to be fixed. Thanks :)

@bmunkholm
Copy link
Copy Markdown
Contributor

.

@cmelchior
Copy link
Copy Markdown
Contributor

Outdated. Closing

@cmelchior cmelchior closed this Jan 31, 2017
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Mar 16, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants