Skip to content
This repository has been archived by the owner on Mar 16, 2021. It is now read-only.

Convert thirtyinch module tests to Kotlin #174

Closed
wants to merge 6 commits into from
Closed

Convert thirtyinch module tests to Kotlin #174

wants to merge 6 commits into from

Conversation

lukaszkalnik
Copy link
Contributor

@lukaszkalnik lukaszkalnik commented Dec 24, 2018

Adds Kotlin support to thirtyinch module tests.
As a start, converts TiTestPresenterTest and TestPresenterExampleTest to Kotlin and MockK.

Copy link
Contributor

@StefMa StefMa left a comment

Choose a reason for hiding this comment

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

☝️

thirtyinch/build.gradle Outdated Show resolved Hide resolved
@lukaszkalnik
Copy link
Contributor Author

lukaszkalnik commented Dec 24, 2018

The build before updating the gradle plugin was failing because of not finishing download of gradle plugin. Maybe it was just a temporary issue with the repo though...
Now after updating the plugin the licenses on Travis need to be accepted...
bildschirmfoto 2018-12-24 um 22 12 17

@StefMa
Copy link
Contributor

StefMa commented Dec 26, 2018

You have to update the travis.yml file too

@lukaszkalnik
Copy link
Contributor Author

Ok, I think we can do it in a separate PR.

@lukaszkalnik
Copy link
Contributor Author

As discussed in a conversation with @passsy the library is pretty stable right now and at this moment there is no need to convert the source code (especially tests) to Kotlin.
Instead of reviewing this and related PRs we can direct our efforts to more valuable goals.
So I'm closing it and all related PRs for now

@lukaszkalnik lukaszkalnik deleted the feature/kotlin_unit_tests branch January 20, 2019 13:06
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants