[MIT-1839] Support deeplink in AuthorizingPaymentActivity (#273)

* feat: open external app if authorize_uri is custom scheme * test: add failure test * chore: remove unnecessary handling exception * feat: create bank app for testing * feat: create PaymentResultActivity for receiving result from bank app * chore: return all results to requestor app * test: authorize payment with deeplink tests * test: revamp and add deep-link tests * fix: fix build failed * test: fix failed test * chore(CI): run instrumentation test on api-level 30 * Revert "chore(CI): run instrumentation test on api-level 30" This reverts commit edac178. * test: fix failed test * test: fix failed test
omise · Sep 27, 2023 · 8e89a4d · 8e89a4d
1 parent 60b0c5c
commit 8e89a4d
Show file tree

Hide file tree

Showing 38 changed files with 771 additions and 133 deletions.
diff --git a/app/src/main/AndroidManifest.xml b/app/src/main/AndroidManifest.xml
@@ -9,7 +9,19 @@
         android:supportsRtl="true"
         android:usesCleartextTraffic="false"
         android:theme="@style/OmiseTheme">
-        <activity android:name=".PaymentResultActivity"></activity>
+        <activity android:name=".PaymentResultActivity">
+            <intent-filter android:label="Payment Result">
+                <action android:name="android.intent.action.VIEW" />
+
+                <category android:name="android.intent.category.DEFAULT" />
+                <category android:name="android.intent.category.BROWSABLE" />
+
+                <data
+                    android:host="omise.co"
+                    android:path="/authorize_return"
+                    android:scheme="sampleapp" />
+            </intent-filter>
+        </activity>
         <activity android:name=".CheckoutActivity">
             <intent-filter>
                 <action android:name="android.intent.action.MAIN" />

diff --git a/app/src/main/java/co/omise/android/example/PaymentResultActivity.kt b/app/src/main/java/co/omise/android/example/PaymentResultActivity.kt
@@ -0,0 +1,23 @@
+package co.omise.android.example
+
+import androidx.appcompat.app.AppCompatActivity
+import android.os.Bundle
+import android.widget.TextView
+
+/**
+ * The example activity to receive the result of the payment.
+ */
+class PaymentResultActivity : AppCompatActivity() {
+    override fun onCreate(savedInstanceState: Bundle?) {
+        super.onCreate(savedInstanceState)
+        setContentView(R.layout.activity_payment_result)
+
+        val resultText = findViewById<TextView>(R.id.payment_result_text)
+
+        intent.data?.getQueryParameter("result")?.let { result ->
+            resultText.text = "result: $result"
+        } ?: run {
+            resultText.text = "No result found"
+        }
+    }
+}
diff --git a/app/src/main/res/layout/activity_payment_result.xml b/app/src/main/res/layout/activity_payment_result.xml
@@ -0,0 +1,16 @@
+<?xml version="1.0" encoding="utf-8"?>
+<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
+    xmlns:tools="http://schemas.android.com/tools"
+    android:layout_width="match_parent"
+    android:layout_height="match_parent"
+    android:gravity="center"
+    android:orientation="vertical"
+    tools:context=".PaymentResultActivity">
+
+    <TextView
+        android:id="@+id/payment_result_text"
+        android:layout_width="wrap_content"
+        android:layout_height="wrap_content"
+        tools:text="result: n/a" />
+
+</LinearLayout>
diff --git a/bankapp/.gitignore b/bankapp/.gitignore
@@ -0,0 +1 @@
+/build
diff --git a/bankapp/build.gradle b/bankapp/build.gradle
@@ -0,0 +1,45 @@
+plugins {
+    id 'com.android.application'
+    id 'org.jetbrains.kotlin.android'
+}
+
+android {
+    namespace 'co.omise.android.bankapp'
+    compileSdk 33
+
+    defaultConfig {
+        applicationId "co.omise.android.bankapp"
+        minSdk 21
+        targetSdk 33
+        versionCode 1
+        versionName "1.0"
+
+        testInstrumentationRunner "androidx.test.runner.AndroidJUnitRunner"
+    }
+
+    buildTypes {
+        release {
+            minifyEnabled false
+            proguardFiles getDefaultProguardFile('proguard-android-optimize.txt'), 'proguard-rules.pro'
+        }
+    }
+    compileOptions {
+        sourceCompatibility JavaVersion.VERSION_1_8
+        targetCompatibility JavaVersion.VERSION_1_8
+    }
+    kotlinOptions {
+        jvmTarget = '1.8'
+    }
+}
+
+dependencies {
+
+    implementation 'androidx.core:core-ktx:1.9.0'
+    implementation platform('org.jetbrains.kotlin:kotlin-bom:1.8.0')
+    implementation 'androidx.appcompat:appcompat:1.6.1'
+    implementation 'com.google.android.material:material:1.9.0'
+    implementation 'androidx.constraintlayout:constraintlayout:2.1.4'
+    testImplementation 'junit:junit:4.13.2'
+    androidTestImplementation 'androidx.test.ext:junit:1.1.5'
+    androidTestImplementation 'androidx.test.espresso:espresso-core:3.5.1'
+}
diff --git a/bankapp/proguard-rules.pro b/bankapp/proguard-rules.pro
@@ -0,0 +1,21 @@
+# Add project specific ProGuard rules here.
+# You can control the set of applied configuration files using the
+# proguardFiles setting in build.gradle.
+#
+# For more details, see
+#   http://developer.android.com/guide/developing/tools/proguard.html
+
+# If your project uses WebView with JS, uncomment the following
+# and specify the fully qualified class name to the JavaScript interface
+# class:
+#-keepclassmembers class fqcn.of.javascript.interface.for.webview {
+#   public *;
+#}
+
+# Uncomment this to preserve the line number information for
+# debugging stack traces.
+#-keepattributes SourceFile,LineNumberTable
+
+# If you keep the line number information, uncomment this to
+# hide the original source file name.
+#-renamesourcefileattribute SourceFile
diff --git a/bankapp/src/androidTest/java/co/omise/android/bankapp/ExampleInstrumentedTest.kt b/bankapp/src/androidTest/java/co/omise/android/bankapp/ExampleInstrumentedTest.kt
@@ -0,0 +1,24 @@
+package co.omise.android.bankapp
+
+import androidx.test.platform.app.InstrumentationRegistry
+import androidx.test.ext.junit.runners.AndroidJUnit4
+
+import org.junit.Test
+import org.junit.runner.RunWith
+
+import org.junit.Assert.*
+
+/**
+ * Instrumented test, which will execute on an Android device.
+ *
+ * See [testing documentation](http://d.android.com/tools/testing).
+ */
+@RunWith(AndroidJUnit4::class)
+class ExampleInstrumentedTest {
+    @Test
+    fun useAppContext() {
+        // Context of the app under test.
+        val appContext = InstrumentationRegistry.getInstrumentation().targetContext
+        assertEquals("co.omise.android.bankapp", appContext.packageName)
+    }
+}
diff --git a/bankapp/src/main/AndroidManifest.xml b/bankapp/src/main/AndroidManifest.xml
@@ -0,0 +1,33 @@
+<?xml version="1.0" encoding="utf-8"?>
+<manifest xmlns:android="http://schemas.android.com/apk/res/android">
+
+    <application
+        android:allowBackup="true"
+        android:icon="@mipmap/ic_launcher"
+        android:label="@string/app_name"
+        android:roundIcon="@mipmap/ic_launcher_round"
+        android:supportsRtl="true"
+        android:theme="@style/Theme.Omiseandroid">
+        <activity
+            android:name=".MainActivity"
+            android:exported="true">
+            <intent-filter>
+                <action android:name="android.intent.action.MAIN" />
+
+                <category android:name="android.intent.category.LAUNCHER" />
+            </intent-filter>
+
+            <intent-filter android:label="Bank app">
+                <action android:name="android.intent.action.VIEW" />
+
+                <category android:name="android.intent.category.DEFAULT" />
+                <category android:name="android.intent.category.BROWSABLE" />
+
+                <data
+                    android:host="omise.co"
+                    android:scheme="bankapp" />
+            </intent-filter>
+        </activity>
+    </application>
+
+</manifest>
diff --git a/bankapp/src/main/java/co/omise/android/bankapp/MainActivity.kt b/bankapp/src/main/java/co/omise/android/bankapp/MainActivity.kt
@@ -0,0 +1,26 @@
+package co.omise.android.bankapp
+
+import android.content.Intent
+import android.net.Uri
+import android.os.Bundle
+import android.widget.Button
+import androidx.appcompat.app.AppCompatActivity
+import com.google.android.material.snackbar.Snackbar
+
+class MainActivity : AppCompatActivity() {
+    override fun onCreate(savedInstanceState: Bundle?) {
+        super.onCreate(savedInstanceState)
+        setContentView(R.layout.activity_main)
+
+        val goBackButton = findViewById<Button>(R.id.go_back_button)
+
+        // test uri: bankapp://omise.co/authorize?return_uri=sampleapp://omise.co/authorize_return?result=success
+        goBackButton.setOnClickListener {
+            intent.data?.getQueryParameter("return_uri")?.let { returnUri ->
+                startActivity(Intent(Intent.ACTION_VIEW, Uri.parse(returnUri)))
+            } ?: run {
+                Snackbar.make(it, "No return_uri found", Snackbar.LENGTH_SHORT).show()
+            }
+        }
+    }
+}
diff --git a/bankapp/src/main/res/drawable-v24/ic_launcher_foreground.xml b/bankapp/src/main/res/drawable-v24/ic_launcher_foreground.xml
@@ -0,0 +1,30 @@
+<vector xmlns:android="http://schemas.android.com/apk/res/android"
+    xmlns:aapt="http://schemas.android.com/aapt"
+    android:width="108dp"
+    android:height="108dp"
+    android:viewportWidth="108"
+    android:viewportHeight="108">
+    <path android:pathData="M31,63.928c0,0 6.4,-11 12.1,-13.1c7.2,-2.6 26,-1.4 26,-1.4l38.1,38.1L107,108.928l-32,-1L31,63.928z">
+        <aapt:attr name="android:fillColor">
+            <gradient
+                android:endX="85.84757"
+                android:endY="92.4963"
+                android:startX="42.9492"
+                android:startY="49.59793"
+                android:type="linear">
+                <item
+                    android:color="#44000000"
+                    android:offset="0.0" />
+                <item
+                    android:color="#00000000"
+                    android:offset="1.0" />
+            </gradient>
+        </aapt:attr>
+    </path>
+    <path
+        android:fillColor="#FFFFFF"
+        android:fillType="nonZero"
+        android:pathData="M65.3,45.828l3.8,-6.6c0.2,-0.4 0.1,-0.9 -0.3,-1.1c-0.4,-0.2 -0.9,-0.1 -1.1,0.3l-3.9,6.7c-6.3,-2.8 -13.4,-2.8 -19.7,0l-3.9,-6.7c-0.2,-0.4 -0.7,-0.5 -1.1,-0.3C38.8,38.328 38.7,38.828 38.9,39.228l3.8,6.6C36.2,49.428 31.7,56.028 31,63.928h46C76.3,56.028 71.8,49.428 65.3,45.828zM43.4,57.328c-0.8,0 -1.5,-0.5 -1.8,-1.2c-0.3,-0.7 -0.1,-1.5 0.4,-2.1c0.5,-0.5 1.4,-0.7 2.1,-0.4c0.7,0.3 1.2,1 1.2,1.8C45.3,56.528 44.5,57.328 43.4,57.328L43.4,57.328zM64.6,57.328c-0.8,0 -1.5,-0.5 -1.8,-1.2s-0.1,-1.5 0.4,-2.1c0.5,-0.5 1.4,-0.7 2.1,-0.4c0.7,0.3 1.2,1 1.2,1.8C66.5,56.528 65.6,57.328 64.6,57.328L64.6,57.328z"
+        android:strokeWidth="1"
+        android:strokeColor="#00000000" />
+</vector>