Refactored revokeRefreshToken to use a new helper method buildRevokeRefreshTokenRequest

Added test for buildRevokeRefreshTokenRequest

Author: wmathurin

libs/SalesforceSDK/src/com/salesforce/androidsdk/auth/OAuth2.java

@@ -471,19 +471,23 @@ public static TokenEndpointResponse refreshAuthToken(HttpAccess httpAccessor, UR
      * @param reason The reason the refresh token is being revoked.
      */
     public static void revokeRefreshToken(HttpAccess httpAccessor, URI loginServer, String refreshToken, LogoutReason reason) {
-        final String requestUrl = loginServer.toString() + OAUTH_REVOKE_PATH;
-        final FormBody body = new FormBody.Builder()
-                .add(TOKEN, refreshToken)
-                .add(REVOKE_REASON, reason.toString())
-                .build();
-        final Request request = new Request.Builder().url(requestUrl).post(body).build();
+        final Request request = buildRevokeRefreshTokenRequest(loginServer, refreshToken, reason);
         try {
             httpAccessor.getOkHttpClient().newCall(request).execute();
         } catch (IOException e) {
             SalesforceSDKLogger.w(TAG, "Exception thrown while revoking refresh token", e);
         }
     }
 
+    protected static Request buildRevokeRefreshTokenRequest(URI loginServer, String refreshToken, LogoutReason reason) {
+        final String requestUrl = loginServer.toString() + OAUTH_REVOKE_PATH;
+        final FormBody body = new FormBody.Builder()
+                .add(TOKEN, refreshToken)
+                .add(REVOKE_REASON, reason.toString())
+                .build();
+        return new Request.Builder().url(requestUrl).post(body).build();
+    }
+
     /**
      * Swaps a JWT for regular OAuth tokens. This is typically the first step after
      * receiving a JWT from a link. In addition, this will also call the identity

libs/test/SalesforceSDKTest/src/com/salesforce/androidsdk/auth/OAuth2Test.java

@@ -61,6 +61,7 @@
 import okhttp3.HttpUrl;
 import okhttp3.Request;
 import okhttp3.Response;
+import okio.Buffer;
 
 /**
  * Tests for OAuth2.
@@ -482,4 +483,36 @@ public void testGetOpenIDToken() {
                 TestCredentials.CLIENT_ID, TestCredentials.REFRESH_TOKEN);
         Assert.assertNotNull("OpenID token should not be null", openIdToken);
     }
+
+    /**
+     * Testing buildRevokeRefreshTokenRequest.
+     */
+    @Test
+    public void testBuildRevokeRefreshTokenRequest() throws Exception {
+        String refreshToken = "test_refresh_token_123";
+        OAuth2.LogoutReason reason = OAuth2.LogoutReason.USER_LOGOUT;
+        URI loginServer = new URI(TestCredentials.LOGIN_URL);
+
+        Request request = OAuth2.buildRevokeRefreshTokenRequest(loginServer, refreshToken, reason);
+
+        // Verify URL
+        String expectedUrl = TestCredentials.LOGIN_URL + "/services/oauth2/revoke";
+        Assert.assertEquals(expectedUrl, request.url().toString());
+
+        // Verify method
+        Assert.assertEquals("POST", request.method());
+
+        // Verify body contains expected parameters
+        String body = getRequestBodyAsString(request);
+        Assert.assertTrue(body.contains("token=" + refreshToken));
+        Assert.assertTrue(body.contains("revoke_reason=" + reason.toString()));
+    }
+
+    private String getRequestBodyAsString(Request request) throws IOException {
+        okio.Buffer buffer = new okio.Buffer();
+        if (request.body() != null) {
+            request.body().writeTo(buffer);
+        }
+        return buffer.readUtf8();
+    }
 }