@friggframework/api-module-hubspot 1.1.7 → 1.1.8-canary.88be3bc.0

package/api.js

@@ -1,4 +1,5 @@
 const {OAuth2Requester, get} = require('@friggframework/core');
+const {Paginator} = require('avid-reader');
 
 class Api extends OAuth2Requester {
     constructor(params) {
@@ -71,7 +72,6 @@ class Api extends OAuth2Requester {
             listMembershipsAddRemove: (listId) => `/crm/v3/lists/${listId}/memberships/add-and-remove`,
             associations: (fromObject, toObject) => `/crm/v4/associations/${fromObject}/${toObject}`,
             associationLabels: (fromObject, toObject) => `/crm/v4/associations/${fromObject}/${toObject}/labels`,
-
         };
 
         this.authorizationUri = encodeURI(
@@ -81,6 +81,16 @@ class Api extends OAuth2Requester {
 
         this.access_token = get(params, 'access_token', null);
         this.refresh_token = get(params, 'refresh_token', null);
+        this.paginator = new Paginator(
+            this,
+            {
+                type: 'token',
+                tokenParam: 'after',
+                tokenPath: '/paging/next/after',
+                resultsKey: 'results',
+                args: { limit:100, after: 0 },
+            }
+        );
     }
 
     getAuthUri() {
@@ -217,11 +227,12 @@ class Api extends OAuth2Requester {
             url: this.baseUrl + this.URLs.contacts,
             query: {
                 limit,
-                after,
                 properties,
             }
         };
-
+        if (after) {
+            options.query.after = after;
+        }
         return this._get(options);
     }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@friggframework/api-module-hubspot",
-  "version": "1.1.7",
+  "version": "1.1.8-canary.88be3bc.0",
   "prettier": "@friggframework/prettier-config",
   "description": "HubSpot API module that lets the Frigg Framework interact with HubSpot",
   "main": "index.js",
@@ -11,8 +11,8 @@
   "author": "",
   "license": "MIT",
   "devDependencies": {
-    "@friggframework/devtools": "^1.1.2",
-    "@friggframework/test": "^1.1.2",
+    "@friggframework/devtools": "^1.1.8",
+    "@friggframework/test": "^1.1.8",
     "dotenv": "^16.0.3",
     "eslint": "^8.22.0",
     "jest": "^28.1.3",
@@ -20,7 +20,8 @@
     "prettier": "^2.7.1"
   },
   "dependencies": {
-    "@friggframework/core": "^1.1.2"
+    "@friggframework/core": "^1.1.8",
+    "avid-reader": "^0.1.1"
   },
-  "gitHead": "cdfe2d135124d7dfe8ef49ae3c6e8e95d7990b40"
+  "gitHead": "88be3bc61aad168e615780d3dc6185cbe03aace3"
 }

package/tests/api.test.js

@@ -67,24 +67,28 @@ describe(`${config.label} API tests`, () => {
     });
 
     // Skipping tests... inherited with bugs, needs refactor
-    describe.skip('HS Deals', () => {
-        it('should return a deal by id', async () => {
-            const deal_id = '2022088696';
-            const response = await api.getDealById(deal_id);
-            expect(response.id).toBe(deal_id);
-            expect(response.properties.amount).to.eq('100000');
-            expect(response.properties.dealname).to.eq('Test');
-            expect(response.properties.dealstage).to.eq('appointmentscheduled');
-        });
-
-        it('should return all deals of a company', async () => {
+    describe.only('HS Deals', () => {
+        let deal_id;
+        it('should return a page of deals', async () => {
             let response = await api.listDeals();
+            expect(response.results).toHaveProperty('length');
             expect(response.results[0]).toHaveProperty('id');
+            deal_id = response.results[0].id;
             expect(response.results[0]).toHaveProperty('properties');
             expect(response.results[0].properties).toHaveProperty('amount');
             expect(response.results[0].properties).toHaveProperty('dealname');
             expect(response.results[0].properties).toHaveProperty('dealstage');
         });
+        it('should return a deal by id', async () => {
+            const response = await api.getDealById(deal_id);
+            expect(response.id).toBe(deal_id);
+        });
+        it('should return all deals using paginator', async () => {
+            const results = await api.paginator.fetchAllPages(api.listDeals);
+            expect(results).toHaveProperty('length');
+            expect(results.length).toBeGreaterThan(100);
+        })
+
     });
 
     // Some tests skipped ... inherited with bugs, needs refactor
@@ -227,7 +231,7 @@ describe(`${config.label} API tests`, () => {
     });
 
     // Some tests skipped ... inherited with bugs, needs refactor
-    describe('HS Contacts', () => {
+    describe.only('HS Contacts', () => {
         let createResponse;
 
         it('should create a Contact', async () => {
@@ -266,6 +270,12 @@ describe(`${config.label} API tests`, () => {
             let response = await api.archiveContact(createResponse.id);
             expect(response.status).toBe(204);
         });
+
+        it('should page all contacts using paginator', async () => {
+            const results = await api.paginator.fetchAllPages(api.listContacts);
+            expect(results).toHaveProperty('length');
+            expect(results.length).toBeGreaterThan(100);
+        });
     });
 
     // Skipping tests... inherited with bugs, needs refactor