Fix/notification response (#181)

* fix(notifications): reply original tweet data

* fix(notifications): reply original tweet data

* fix(notifications): reply original tweet data

* fix(notifications): reply original tweet data

* fix(notifications): extra data

* fix(notifications): extra data

* fix(notifications): push notification format

* fix(notifications): push notification format

* fix(notifications): push notification format

* fix(notifications): push notification format

* fix(notifications): push notification format

* fix(notifications): push notification format

* fix(notifications): push notification format

* fix(notifications): push notification format

* fix(notifications): push notification format

* fix(notifications): push notification format

* fix(notifications): push notification format

Author: MoBahgat010

dump.rdb

@@ -379,7 +379,7 @@ describe('FCMService', () => {
             );
         });
 
-        it('should return false when payload is invalid', async () => {
+        it('should send notification with fallback content when payload is missing fields', async () => {
             const payload = {
                 // Missing required fields
                 tweet_id: 'tweet-123',
@@ -391,7 +391,15 @@ describe('FCMService', () => {
                 payload
             );
 
-            expect(result).toBe(false);
+            expect(result).toBe(true);
+            expect(mock_expo_instance.sendPushNotificationsAsync).toHaveBeenCalledWith(
+                expect.arrayContaining([
+                    expect.objectContaining({
+                        title: 'Liked by Someone',
+                        body: 'your post',
+                    }),
+                ])
+            );
         });
 
         it('should return false and warn if user has no FCM token', async () => {

src/expo/expo.service.spec.ts

@@ -128,49 +128,65 @@ export class FCMService {
         type: NotificationType,
         payload: any
     ): { title: string; body: string; data?: any } {
+        console.log(payload);
         switch (type) {
             case NotificationType.FOLLOW:
                 return {
                     title: 'yapper',
-                    body: `@${payload.follower_username} followed you!`,
+                    body: `@${payload.follower_username || 'Someone'} followed you!`,
                     data: { username: payload.follower_id },
                 };
             case NotificationType.MENTION:
                 return {
-                    title: `Mentioned by ${payload.mentioned_by?.name}:`,
-                    body: payload.tweet?.content,
-                    data: { tweet_id: payload.tweet?.id },
+                    title: `Mentioned by ${payload.mentioned_by?.name || 'Someone'}:`,
+                    body: payload.tweet?.content || 'You were mentioned in a post',
+                    data: { tweet_id: payload.tweet?.id || payload.tweet?.tweet_id },
                 };
             case NotificationType.REPLY:
                 return {
-                    title: `${payload.replier?.name} replied:`,
-                    body: payload.reply_tweet?.content,
-                    data: { tweet_id: payload.reply_tweet?.id },
+                    title: `${payload.replier?.name || 'Someone'} replied:`,
+                    body: payload.reply_tweet?.content || 'replied to your post',
+                    data: { tweet_id: payload.reply_tweet?.id || payload.reply_tweet?.tweet_id },
                 };
             case NotificationType.QUOTE:
                 return {
                     title: 'yapper',
-                    body: `@${payload.quoted_by?.username} quoted your post and said: ${
-                        payload.quote?.content || ''
+                    body: `@${payload.quoted_by?.username || 'Someone'} quoted your post${
+                        payload.quote?.content ? ` and said: ${payload.quote.content}` : ''
                     }`,
-                    data: { tweet_id: payload.quote_tweet?.id },
+                    data: { tweet_id: payload.quote?.id || payload.quote?.tweet_id },
                 };
-            case NotificationType.LIKE:
+            case NotificationType.LIKE: {
+                // Handle both array format (likers/tweets) and singular format (liker/tweet)
+                const liker_name = payload.liker?.name || payload.likers?.[0]?.name || 'Someone';
+                const liked_tweet_content =
+                    payload.tweet?.content || payload.tweets?.[0]?.content || 'your post';
+                const liked_tweet_id =
+                    payload.tweet?.tweet_id || payload.tweet?.id || payload.tweets?.[0]?.id;
                 return {
-                    title: `Liked by ${payload.likers[0].name}`,
-                    body: payload.tweets[0].content,
-                    data: { tweet_id: payload.tweets[0].id },
+                    title: `Liked by ${liker_name}`,
+                    body: liked_tweet_content,
+                    data: { tweet_id: liked_tweet_id },
                 };
-            case NotificationType.REPOST:
+            }
+            case NotificationType.REPOST: {
+                // Handle both array format (reposters/tweets) and singular format (reposter/tweet)
+                const reposter_name =
+                    payload.reposter?.name || payload.reposters?.[0]?.name || 'Someone';
+                const reposted_tweet_content =
+                    payload.tweet?.content || payload.tweets?.[0]?.content || 'your post';
+                const reposted_tweet_id =
+                    payload.tweet?.tweet_id || payload.tweet?.id || payload.tweets?.[0]?.id;
                 return {
-                    title: `Reposted by ${payload.reposters[0].name}:`,
-                    body: payload.tweets[0].content,
-                    data: { tweet_id: payload.tweets[0].id },
+                    title: `Reposted by ${reposter_name}:`,
+                    body: reposted_tweet_content,
+                    data: { tweet_id: reposted_tweet_id },
                 };
+            }
             case NotificationType.MESSAGE:
                 return {
-                    title: payload.sender?.name,
-                    body: payload.message?.content,
+                    title: payload.sender?.name || 'New Message',
+                    body: payload.message?.content || 'You have a new message',
                 };
             default:
                 return {