test: add missing unit and integration test (#472)

* unit tests

* fix uuid encoding issue in presence apis

* handling empty channel case for presence apis

* lint fix for get_state.ts

* fix test url for presence endpoints and fix typescript compilation error

* additional integration tests

* added integration tests for presence error cases

* removed minor tweaks in presence endpoints for paramters structuring,
rename message actions test directory

* removed invalid case of where now, fix undefined uuid case in get state

* updated package.json-lock and libs

---------

Co-authored-by: Mohit Tejani <mohit.tejani@Mohits-MacBook-Pro.local>
Co-authored-by: Serhii Mamontov <parfeon@me.com>

Author: 3 people

dist/web/pubnub.js

@@ -11210,7 +11210,7 @@
 	    }
 	    get path() {
 	        const { keySet: { subscribeKey }, uuid, channels, } = this.parameters;
-	        return `/v2/presence/sub-key/${subscribeKey}/channel/${encodeNames(channels !== null && channels !== void 0 ? channels : [], ',')}/uuid/${uuid}`;
+	        return `/v2/presence/sub-key/${subscribeKey}/channel/${encodeNames(channels !== null && channels !== void 0 ? channels : [], ',')}/uuid/${encodeString(uuid !== null && uuid !== void 0 ? uuid : '')}`;
 	    }
 	    get queryParameters() {
 	        const { channelGroups } = this.parameters;
@@ -11243,7 +11243,7 @@
 	        const { keySet: { subscribeKey }, state, channels = [], channelGroups = [], } = this.parameters;
 	        if (!subscribeKey)
 	            return 'Missing Subscribe Key';
-	        if (!state)
+	        if (state === undefined)
 	            return 'Missing State';
 	        if ((channels === null || channels === void 0 ? void 0 : channels.length) === 0 && (channelGroups === null || channelGroups === void 0 ? void 0 : channelGroups.length) === 0)
 	            return 'Please provide a list of channels and/or channel-groups';
@@ -11309,7 +11309,7 @@
 	        const query = { heartbeat: `${heartbeat}` };
 	        if (channelGroups && channelGroups.length !== 0)
 	            query['channel-group'] = channelGroups.join(',');
-	        if (state)
+	        if (state !== undefined)
 	            query.state = JSON.stringify(state);
 	        return query;
 	    }

dist/web/pubnub.min.js

@@ -59,7 +59,7 @@ class GetPresenceStateRequest extends request_1.AbstractRequest {
     }
     get path() {
         const { keySet: { subscribeKey }, uuid, channels, } = this.parameters;
-        return `/v2/presence/sub-key/${subscribeKey}/channel/${(0, utils_1.encodeNames)(channels !== null && channels !== void 0 ? channels : [], ',')}/uuid/${uuid}`;
+        return `/v2/presence/sub-key/${subscribeKey}/channel/${(0, utils_1.encodeNames)(channels !== null && channels !== void 0 ? channels : [], ',')}/uuid/${(0, utils_1.encodeString)(uuid !== null && uuid !== void 0 ? uuid : '')}`;
     }
     get queryParameters() {
         const { channelGroups } = this.parameters;

lib/core/endpoints/presence/get_state.js

@@ -59,7 +59,7 @@ class HeartbeatRequest extends request_1.AbstractRequest {
         const query = { heartbeat: `${heartbeat}` };
         if (channelGroups && channelGroups.length !== 0)
             query['channel-group'] = channelGroups.join(',');
-        if (state)
+        if (state !== undefined)
             query.state = JSON.stringify(state);
         return query;
     }

lib/core/endpoints/presence/heartbeat.js

@@ -39,7 +39,7 @@ class SetPresenceStateRequest extends request_1.AbstractRequest {
         const { keySet: { subscribeKey }, state, channels = [], channelGroups = [], } = this.parameters;
         if (!subscribeKey)
             return 'Missing Subscribe Key';
-        if (!state)
+        if (state === undefined)
             return 'Missing State';
         if ((channels === null || channels === void 0 ? void 0 : channels.length) === 0 && (channelGroups === null || channelGroups === void 0 ? void 0 : channelGroups.length) === 0)
             return 'Please provide a list of channels and/or channel-groups';

lib/core/endpoints/presence/set_state.js

@@ -9,7 +9,7 @@ import { AbstractRequest } from '../../components/request';
 import RequestOperation from '../../constants/operations';
 import { KeySet, Payload, Query } from '../../types/api';
 import * as Presence from '../../types/api/presence';
-import { encodeNames } from '../../utils';
+import { encodeNames, encodeString } from '../../utils';
 
 // --------------------------------------------------------
 // ------------------------ Types -------------------------
@@ -103,7 +103,10 @@ export class GetPresenceStateRequest extends AbstractRequest<Presence.GetPresenc
       channels,
     } = this.parameters;
 
-    return `/v2/presence/sub-key/${subscribeKey}/channel/${encodeNames(channels ?? [], ',')}/uuid/${uuid}`;
+    return `/v2/presence/sub-key/${subscribeKey}/channel/${encodeNames(
+      channels ?? [],
+      ',',
+    )}/uuid/${encodeString(uuid ?? '')}`;
   }
 
   protected get queryParameters(): Query {

package-lock.json

@@ -91,7 +91,7 @@ export class HeartbeatRequest extends AbstractRequest<Presence.PresenceHeartbeat
     const query: Record<string, string> = { heartbeat: `${heartbeat}` };
 
     if (channelGroups && channelGroups.length !== 0) query['channel-group'] = channelGroups.join(',');
-    if (state) query.state = JSON.stringify(state);
+    if (state !== undefined) query.state = JSON.stringify(state);
 
     return query;
   }

src/core/endpoints/presence/get_state.ts

@@ -80,7 +80,7 @@ export class SetPresenceStateRequest extends AbstractRequest<Presence.SetPresenc
     } = this.parameters;
 
     if (!subscribeKey) return 'Missing Subscribe Key';
-    if (!state) return 'Missing State';
+    if (state === undefined) return 'Missing State';
     if (channels?.length === 0 && channelGroups?.length === 0)
       return 'Please provide a list of channels and/or channel-groups';
   }

src/core/endpoints/presence/heartbeat.ts

@@ -2549,7 +2549,7 @@ export class PubNubCore<
 
       const request = new GetPresenceStateRequest({
         ...parameters,
-        uuid: parameters.uuid ?? this._configuration.userId,
+        uuid: parameters.uuid ?? this._configuration.userId!,
         keySet: this._configuration.keySet,
       });
       const logResponse = (response: Presence.GetPresenceStateResponse | null) => {