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

package/CHANGELOG.md

@@ -1,3 +1,20 @@
+# v1.1.7 (Tue Aug 06 2024)
+
+:tada: This release contains work from a new contributor! :tada:
+
+Thank you, Fer Riffel ([@FerRiffel-LeftHook](https://github.com/FerRiffel-LeftHook)), for all your work!
+
+#### 🐛 Bug Fix
+
+- Updated icons for all Frigg API modules [#14](https://github.com/friggframework/api-module-library/pull/14) ([@FerRiffel-LeftHook](https://github.com/FerRiffel-LeftHook))
+- Update icons for all Frigg API modules ([@FerRiffel-LeftHook](https://github.com/FerRiffel-LeftHook))
+
+#### Authors: 1
+
+- Fer Riffel ([@FerRiffel-LeftHook](https://github.com/FerRiffel-LeftHook))
+
+---
+
 # v1.1.6 (Thu Aug 01 2024)
 
 :tada: This release contains work from new contributors! :tada:

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/defaultConfig.json

@@ -3,7 +3,7 @@
   "label": "HubSpot",
   "productUrl": "https://hubspot.com",
   "apiDocs": "https://developers.hubspot.com",
-  "logoUrl": "https://friggframework.org/assets/img/hubspot.jpeg",
+  "logoUrl": "https://friggframework.org/assets/img/hubspot-icon.jpeg",
   "categories": [
     "Marketing",
     "Sales",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@friggframework/api-module-hubspot",
-  "version": "1.1.7-canary.eaa7127.0",
+  "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": "eaa7127b8c0d4843846b547209f2fa785cf4ed5d"
+  "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