feat: support identify user by id (#1115)

Jira ID: MOB-13746

Author: abdelhamid-f-nasser

CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## [Unreleased](https://github.com/Instabug/Instabug-React-Native/compare/v12.6.0...dev)
+
+### Added
+
+- Support user identification using ID ([#1115](https://github.com/Instabug/Instabug-React-Native/pull/1115))
+
 ## [12.6.0](https://github.com/Instabug/Instabug-React-Native/compare/v12.5.0...v12.6.0) (January 14, 2024)
 
 ### Changed

android/src/main/java/com/instabug/reactlibrary/RNInstabugReactnativeModule.java

@@ -294,16 +294,22 @@ public void run() {
      * Set the user identity.
      * Instabug will pre-fill the user email in reports.
      *
-     * @param userName  Username.
      * @param userEmail User's default email
+     * @param userName  Username.
+     * @param userId User's ID
      */
     @ReactMethod
-    public void identifyUser(final String userEmail, final String userName) {
+    public void identifyUser(
+            final String userEmail,
+            final String userName,
+            @Nullable final String userId
+    ) {
         MainThreadHandler.runOnMainThread(new Runnable() {
             @Override
             public void run() {
                 try {
-                    Instabug.identifyUser(userName, userEmail);
+                    // The arguments get re-ordered here to match the API signature.
+                    Instabug.identifyUser(userName, userEmail, userId);
                 } catch (Exception e) {
                     e.printStackTrace();
                 }

android/src/test/java/com/instabug/reactlibrary/RNInstabugReactnativeModuleTest.java

@@ -200,16 +200,29 @@ public void tearDown() {
     }
 
     @Test
-    public void givenArgs$identifyUser_whenQuery_thenShouldCallNativeApiWithArgs() {
+    public void testIdentifyUserWithNoId() {
         // given
 
         String email = "sali@instabug.com";
         String userName = "salmaali";
+        String id = null;
         // when
-        rnModule.identifyUser(email, userName);
+        rnModule.identifyUser(email, userName, id);
         // then
-        verify(Instabug.class,times(1));
-        Instabug.identifyUser(userName, email);
+        mockInstabug.verify(() -> Instabug.identifyUser(userName, email, id));
+    }
+
+    @Test
+    public void testIdentifyUserWithId() {
+        // given
+
+        String email = "sali@instabug.com";
+        String userName = "salmaali";
+        String id = "salmaali";
+        // when
+        rnModule.identifyUser(email, userName, id);
+        // then
+        mockInstabug.verify(() -> Instabug.identifyUser(userName, email, id));
     }
 
     @Test

examples/default/ios/InstabugTests/InstabugSampleTests.m

@@ -188,8 +188,19 @@ - (void)testIdentifyUser {
   NSString *name = @"this is my name";
 
   OCMStub([mock identifyUserWithEmail:email name:name]);
-  [self.instabugBridge identifyUser:email name:name];
-  OCMVerify([mock identifyUserWithEmail:email name:name]);
+  [self.instabugBridge identifyUser:email name:name userId:nil];
+  OCMVerify([mock identifyUserWithID:nil email:email name:name]);
+}
+
+- (void)testIdentifyUserWithID {
+  id mock = OCMClassMock([Instabug class]);
+  NSString *email = @"em@il.com";
+  NSString *name = @"this is my name";
+  NSString *userId = @"this is my id";
+
+  OCMStub([mock identifyUserWithID:userId email:email name:name]);
+  [self.instabugBridge identifyUser:email name:name userId:userId];
+  OCMVerify([mock identifyUserWithID:userId email:email name:name]);
 }
 
 - (void)testLogOut {

ios/RNInstabug/InstabugReactBridge.h

@@ -49,7 +49,7 @@
 
 - (void)setString:(NSString *)value toKey:(NSString *)key;
 
-- (void)identifyUser:(NSString *)email name:(NSString *)name;
+- (void)identifyUser:(NSString *)email name:(NSString *)name userId:(nullable NSString *)userId;
 
 - (void)logOut;
 

ios/RNInstabug/InstabugReactBridge.m

@@ -191,8 +191,8 @@ - (dispatch_queue_t)methodQueue {
     [Instabug clearFileAttachments];
 }
 
-RCT_EXPORT_METHOD(identifyUser:(NSString *)email name:(NSString *)name) {
-    [Instabug identifyUserWithEmail:email name:name];
+RCT_EXPORT_METHOD(identifyUser:(NSString *)email name:(NSString *)name userId:(nullable NSString *)userId) {
+    [Instabug identifyUserWithID:userId email:email name:name];
 }
 
 RCT_EXPORT_METHOD(logOut) {

src/modules/Instabug.ts

@@ -196,15 +196,16 @@ export const setString = (key: StringKey, string: string) => {
 };
 
 /**
- * Sets the default value of the user's email and hides the email field from the reporting UI
+ * Sets the default value of the user's email and ID and hides the email field from the reporting UI
  * and set the user's name to be included with all reports.
  * It also reset the chats on device to that email and removes user attributes,
  * user data and completed surveys.
  * @param email Email address to be set as the user's email.
  * @param name Name of the user to be set.
+ * @param [id] ID of the user to be set.
  */
-export const identifyUser = (email: string, name: string) => {
-  NativeInstabug.identifyUser(email, name);
+export const identifyUser = (email: string, name: string, id?: string) => {
+  NativeInstabug.identifyUser(email, name, id);
 };
 
 /**

src/native/NativeInstabug.ts

@@ -62,7 +62,7 @@ export interface InstabugNativeModule extends NativeModule {
   clearLogs(): void;
 
   // User APIs //
-  identifyUser(email: string, name: string): void;
+  identifyUser(email: string, name: string, id?: string): void;
   logOut(): void;
   logUserEvent(name: string): void;
   setUserData(data: string): void;

test/modules/Instabug.spec.ts

@@ -409,7 +409,17 @@ describe('Instabug Module', () => {
     Instabug.identifyUser(email, name);
 
     expect(NativeInstabug.identifyUser).toBeCalledTimes(1);
-    expect(NativeInstabug.identifyUser).toBeCalledWith(email, name);
+    expect(NativeInstabug.identifyUser).toBeCalledWith(email, name, undefined);
+  });
+
+  it('identifyUser when id is defined should call the native method identifyUser', () => {
+    const email = 'foo@instabug.com';
+    const name = 'Instabug';
+    const id = 'instabug-id';
+    Instabug.identifyUser(email, name, id);
+
+    expect(NativeInstabug.identifyUser).toBeCalledTimes(1);
+    expect(NativeInstabug.identifyUser).toBeCalledWith(email, name, id);
   });
 
   it('should call the native method logOut', () => {