-
Notifications
You must be signed in to change notification settings - Fork 10
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
Adds management URL #41
Conversation
android/build.gradle
Outdated
@@ -98,4 +98,7 @@ dependencies { | |||
repositories { | |||
google() | |||
jcenter() | |||
maven { | |||
url "http://oss.sonatype.org/content/repositories/snapshots/" | |||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added these to be able to test, we should remove it before merging
val appContext = InstrumentationRegistry.getInstrumentation().targetContext | ||
assertEquals("com.revenuecat.purchases.common.test", appContext.packageName) | ||
} | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
removed it since it's an example and we are not doing instrumented tests
context("with a non null managementURL") { | ||
it("should map to a null managementURL") { | ||
val expected = "https://www.url.com/" | ||
val mockkUri = mockk<Uri>(relaxed = true) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Uri
is an Android class, so it's easier to create a relaxed mock
@aboedo don't review it yet, sorry I requested review. I will add the iOS changes here too |
665fb41
to
3044772
Compare
@vegaro heads up that I added the iOS side on this one. I can't request a review because you opened the PR, but take a look! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can't approve it either because it's mine. LGTM!
Changes
TODO
Update Android version when released