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

Dependency update and migration to AndroidX #9

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

Conversation

xmellado
Copy link

@xmellado xmellado commented Nov 30, 2018

Hello,

I will use your template for a new project at work and we have decided to update its dependencies. I have updated all library versions and migrate everything to AndroidX, including tests. There is a changelog file with all the changes and I have added a plugin for checking dependency updates.

I hope you find it useful,
Xavi

Xavier Mellado Esteban added 7 commits November 28, 2018 13:42
@LukasAnda
Copy link

Very good migration but I am unable to execute instrumental tests

1) Mockito Android 2.23.x is not compatible with current stable Android Gradle Plugin. Android Gradle Plugin 3.3.0-beta02 or higher is needed. See: mockito/mockito#1511
2) Koin 1.0.2 makes browser activity tests fail. It is probably an issue with the updated declareMock function. See: InsertKoinIO/koin#243
@xmellado
Copy link
Author

xmellado commented Jan 5, 2019

Hi,
@LukasAnda I have "fixed" instrumentation tests. There were two problems:

  1. The compilation issue was introduced by Mockito Android 2.23.x, which is not compatible with current Android Gradle Plugin. When a new version of the Android Gradle Plugin becomes available, this dependency can be updated.
  2. Koin 1.0.2 makes the browser activity test fail, since the mocked BufferooRepository does not work properly. So I downgraded the version to 1.0.1 and it works fine. If you or anyone reading this thread can help me with this issue, I will appreciate it.

Thanks for pointing this problem,
Xavi

@LukasAnda
Copy link

Hi @xmellado thanks for response :) I've been trying to solve it by myself, and I submitted an issue to Koin team. Here you can view the issue https://github.com/InsertKoinIO/koin/issues/326

@xmellado
Copy link
Author

Hi,
@LukasAnda I have updated Koin to version 2.0 and I have fixed the instrumental tests ;)
Regards,
Xavi

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