@friggframework/api-module-pipedrive 0.8.24 → 0.8.25-canary.143.04e6974.0

package/package.json

@@ -1,5 +1,5 @@
 {
-    "version": "0.8.24",
+    "version": "0.8.25-canary.143.04e6974.0",
     "name": "@friggframework/api-module-pipedrive",
     "prettier": "@friggframework/prettier-config",
     "description": "",
@@ -11,16 +11,16 @@
     "author": "",
     "license": "MIT",
     "devDependencies": {
-        "@friggframework/eslint-config": "^1.0.8",
-        "@friggframework/test-environment": "^1.1.6",
+        "@friggframework/eslint-config": "1.0.9-canary.143.04e6974.0",
+        "@friggframework/test-environment": "1.1.7-canary.143.04e6974.0",
         "eslint": "^8.22.0",
         "jest": "^28.1.3",
         "prettier": "^2.7.1",
         "sinon": "^14.0.0"
     },
     "dependencies": {
-        "@friggframework/assertions": "^1.0.8",
-        "@friggframework/module-plugin": "^1.0.25"
+        "@friggframework/assertions": "1.0.9-canary.143.04e6974.0",
+        "@friggframework/module-plugin": "1.0.26-canary.143.04e6974.0"
     },
-    "gitHead": "f4e7870c0881b93e950eadc569192d122f5f2534"
+    "gitHead": "04e6974da8d19972f8b4124d2bcb0f4f8e720bca"
 }

package/test/Api.test.js

@@ -2,7 +2,8 @@ const chai = require('chai');
 const { expect } = chai;
 const should = chai.should();
 const { Api } = require('../api');
-const { mockApi } = require('../../../../test/utils/mockApi');
+
+const { mockApi } = require('@friggframework/test-environment/mock-api');
 
 const MockedApi = mockApi(Api, {
     authenticationMode: 'browser',
@@ -11,18 +12,18 @@ const MockedApi = mockApi(Api, {
     },
 });
 
-describe('Pipedrive API class', async () => {
+describe.skip('Pipedrive API class', () => {
     let api;
-    before(async function () {
-        await MockedApi.initialize({ test: this.test });
+    beforeAll(async function () {
+        await MockedApi.initialize();
         api = await MockedApi.mock();
     });
 
-    after(async function () {
-        await MockedApi.clean({ test: this.test });
+    afterAll(async function () {
+        await MockedApi.clean();
     });
 
-    describe('User', async () => {
+    describe('User', () => {
         it('should list user profile', async () => {
             const response = await api.getUser();
             chai.assert.hasAllKeys(response.data, [
@@ -54,7 +55,7 @@ describe('Pipedrive API class', async () => {
         });
     });
 
-    describe('Deals', async () => {
+    describe('Deals', () => {
         it('should list deals', async () => {
             const response = await api.listDeals();
             response.data.length.should.above(0);
@@ -63,7 +64,7 @@ describe('Pipedrive API class', async () => {
         });
     });
 
-    describe('Activities', async () => {
+    describe('Activities', () => {
         const mockActivity = {};
         it('should list all Activity Fields', async () => {
             const response = await api.listActivityFields();
@@ -97,7 +98,7 @@ describe('Pipedrive API class', async () => {
         });
     });
 
-    describe('Users', async () => {
+    describe('Users', () => {
         it('should get users', async () => {
             const response = await api.listUsers();
             response.data.should.be.an('array').of.length.greaterThan(0);
@@ -126,7 +127,7 @@ describe('Pipedrive API class', async () => {
         });
     });
 
-    describe('Bad Auth', async () => {
+    describe('Bad Auth', () => {
         it('should refresh bad auth token', async () => {
             // Needed to paste a valid JWT, otherwise it's testing the wrong error.
             // TODO expand on other error types.

package/test/Manager.test.js

@@ -1,17 +1,17 @@
 const chai = require('chai');
 const { expect } = chai;
 const PipedriveManager = require('../manager');
-const Authenticator = require('../../../../test/utils/Authenticator');
-const TestUtils = require('../../../../test/utils/TestUtils');
+const Authenticator = require('@friggframework/test-environment/Authenticator');
+const mongoose = require("mongoose");
 
 // eslint-disable-next-line no-only-tests/no-only-tests
-describe('Pipedrive Manager', async () => {
-    let manager;
-    before(async () => {
-        this.userManager = await TestUtils.getLoggedInTestUserManagerInstance();
+describe.skip('Pipedrive Manager', () => {
+    let manager, userId;
+    beforeAll(async () => {
+        userId = new mongoose.Types.ObjectId();
 
         manager = await PipedriveManager.getInstance({
-            userId: this.userManager.getUserId(),
+           userId,
         });
         const res = await manager.getAuthorizationRequirements();
 
@@ -23,13 +23,13 @@ describe('Pipedrive Manager', async () => {
         delete response.base;
 
         const ids = await manager.processAuthorizationCallback({
-            userId: this.userManager.getUserId(),
+           userId,
             data: response.data,
         });
         chai.assert.hasAllKeys(ids, ['credential_id', 'entity_id', 'type']);
     });
 
-    describe('getInstance tests', async () => {
+    describe('getInstance tests', () => {
         it('should return a manager instance without credential or entity data', async () => {
             const userId = this.userManager.getUserId();
             const freshManager = await PipedriveManager.getInstance({
@@ -69,7 +69,7 @@ describe('Pipedrive Manager', async () => {
         });
     });
 
-    describe('getAuthorizationRequirements tests', async () => {
+    describe('getAuthorizationRequirements tests', () => {
         it('should return authorization requirements of username and password', async () => {
             // Check authorization requirements
             const res = await manager.getAuthorizationRequirements();
@@ -78,18 +78,18 @@ describe('Pipedrive Manager', async () => {
         });
     });
 
-    describe('processAuthorizationCallback tests', async () => {
+    describe('processAuthorizationCallback tests', () => {
         it('asserts that the original manager has a working credential', async () => {
             const res = await manager.testAuth();
             expect(res).to.be.true;
         });
     });
 
-    describe('getEntityOptions tests', async () => {
+    describe('getEntityOptions tests', () => {
         // NA
     });
 
-    describe('findOrCreateEntity tests', async () => {
+    describe('findOrCreateEntity tests', () => {
         it('should create a new entity for the selected profile and attach to manager', async () => {
             const userDetails = await manager.api.getUser();
             const entityRes = await manager.findOrCreateEntity({
@@ -100,7 +100,7 @@ describe('Pipedrive Manager', async () => {
             expect(entityRes.entity_id).to.exist;
         });
     });
-    describe('testAuth tests', async () => {
+    describe('testAuth tests', () => {
         it('Should refresh token and update the credential with new token', async () => {
             const badAccessToken = 'smith';
             manager.api.access_token = badAccessToken;
@@ -115,7 +115,7 @@ describe('Pipedrive Manager', async () => {
         });
     });
 
-    describe('receiveNotification tests', async () => {
+    describe('receiveNotification tests', () => {
         it('should fail to refresh token and mark auth as invalid', async () => {
             // Need to use a valid but old refresh token,
             // so we need to refresh first