Merge pull request #9 from airwallex/feature/APAM-27-support-staging-environment

[APAM-27] Support staging environment

Author: aw-hector

RiskSdk/src/main/java/com/airwallex/risk/AirwallexRisk.kt

@@ -9,7 +9,7 @@ class AirwallexRisk private constructor() {
     companion object {
         private const val sharedPreferencesFilename: String = Constants.sharedPreferencesFilename
 
-        internal var shared: AirwallexRiskInternal? = null
+        private var shared: AirwallexRiskInternal? = null
 
         /**
          * Airwallex risk request header.

RiskSdk/src/main/java/com/airwallex/risk/Environment.kt

@@ -1,6 +1,6 @@
 package com.airwallex.risk
 
-internal enum class Environment(internal val host: String) {
+enum class Environment(internal val host: String) {
     PRODUCTION("https://bws.airwallex.com/bws/v2/m/"),
     DEMO("https://bws-demo.airwallex.com/bws/v2/m/"),
     STAGING("https://bws-staging.airwallex.com/bws/v2/m/")

RiskSdk/src/main/java/com/airwallex/risk/RiskConfiguration.kt

@@ -2,18 +2,26 @@ package com.airwallex.risk
 
 /**
  * Configuration context for the Airwallex Risk SDK.
- * @param isProduction set to `false` for test environments.
+ * @param environment Airwallex risk environment, set to production for release builds.
  * @Param tenant only modify if requested by Airwallex.
  * @param bufferTimeMillis requests buffer time in milliseconds.
  */
 class RiskConfiguration(
-    isProduction: Boolean = true,
+    val environment: Environment = Environment.PRODUCTION,
     val tenant: Tenant = Tenant.SCALE,
     val bufferTimeMillis: Long = 20_000L
 ) {
-    internal val environment: Environment
 
-    init {
-        this.environment = if (isProduction) Environment.PRODUCTION else Environment.DEMO
-    }
+    /**
+     * Configuration context for the Airwallex Risk SDK.
+     * @param isProduction set to `false` for test environments.
+     * @Param tenant only modify if requested by Airwallex.
+     */
+    constructor(
+        isProduction: Boolean = true,
+        tenant: Tenant = Tenant.SCALE
+    ) : this(
+        environment = if (isProduction) Environment.PRODUCTION else Environment.DEMO,
+        tenant = tenant
+    )
 }

RiskSdk/src/test/java/com/airwallex/risk/RiskConfigurationTest.kt

@@ -7,14 +7,14 @@ class RiskConfigurationTest {
 
     @Test
     fun `test configuration`() {
-        val config1 = RiskConfiguration(isProduction = true, tenant = Tenant.PA)
+        val config1 = RiskConfiguration(environment = Environment.PRODUCTION, tenant = Tenant.PA, bufferTimeMillis = 5_000L)
         assertEquals(config1.environment, Environment.PRODUCTION)
         assertEquals(config1.tenant, Tenant.PA)
-        assertEquals(config1.bufferTimeMillis, 20_000L)
+        assertEquals(config1.bufferTimeMillis, 5_000L)
 
-        val config2 = RiskConfiguration(isProduction = false, tenant = Tenant.SCALE, bufferTimeMillis = 5_000L)
+        val config2 = RiskConfiguration(isProduction = false, tenant = Tenant.SCALE)
         assertEquals(config2.environment, Environment.DEMO)
         assertEquals(config2.tenant, Tenant.SCALE)
-        assertEquals(config2.bufferTimeMillis, 5_000L)
+        assertEquals(config2.bufferTimeMillis, 20_000L)
     }
 }

app/src/main/java/com/airwallex/risk/sample/SampleApplication.kt

@@ -15,8 +15,7 @@ class SampleApplication : Application() {
             applicationContext = applicationContext,
             accountId = accountId,
             configuration = RiskConfiguration(
-                isProduction = !BuildConfig.DEBUG,
-                bufferTimeMillis = 5_000L
+                isProduction = !BuildConfig.DEBUG
             )
         )