@itentialopensource/adapter-slack 2.6.1 → 3.0.0

package/CHANGELOG.md

@@ -1,4 +1,20 @@
 
+## 3.0.0 [02-15-2023]
+
+* Major/adapt 2297
+
+See merge request itentialopensource/adapters/notification-messaging/adapter-slack!14
+
+---
+
+## 2.6.2 [08-03-2022]
+
+* Add parameters for postFilesupload
+
+See merge request itentialopensource/adapters/notification-messaging/adapter-slack!13
+
+---
+
 ## 2.6.1 [07-25-2022]
 
 * Fix schema for threadTs

package/adapter.js

@@ -15,13 +15,13 @@ const path = require('path');
 const AdapterBaseCl = require(path.join(__dirname, 'adapterBase.js'));
 
 /**
- * This is the adapter/interface into Slacktest
+ * This is the adapter/interface into Slack
  */
 
 /* GENERAL ADAPTER FUNCTIONS */
-class Slacktest extends AdapterBaseCl {
+class Slack extends AdapterBaseCl {
   /**
-   * Slacktest Adapter
+   * Slack Adapter
    * @constructor
    */
   /* Working on changing the way we do Emit methods due to size and time constrainsts
@@ -5283,10 +5283,11 @@ class Slacktest extends AdapterBaseCl {
    * @param {string} content - File contents via a POST variable. If omitting this parameter, you must provide a `file`.
    * @param {string} token - Authentication token. Requires scope: `files:write:user`
    * @param {string} file - File contents via `multipart\/form-data`. If omitting this parameter, you must submit `content`.
+   * @param {number} threadTs - Unique identifier of a thread's parent message
    * @param {getCallback} callback - a callback function to return the result
    */
   /* YOU CAN CHANGE THE PARAMETERS YOU TAKE IN HERE AND IN THE pronghorn.json FILE */
-  postFilesupload(channels, title, initialComment, filetype, filename, content, token, file, callback) {
+  postFilesupload(channels, title, initialComment, filetype, filename, content, token, file, threadTs, callback) {
     const meth = 'adapter-postFilesupload';
     const origin = `${this.id}-${meth}`;
     log.trace(origin);
@@ -5303,7 +5304,7 @@ class Slacktest extends AdapterBaseCl {
     const queryParamsAvailable = {};
     const queryParams = {};
     const pathVars = [];
-    const bodyVars = {};
+    const bodyVars = { channels, title, initialComment, filetype, filename, content, file, threadTs };
 
     // loop in template. long callback arg name to avoid identifier conflicts
     Object.keys(queryParamsAvailable).forEach((thisKeyInQueryParamsAvailable) => {
@@ -10722,4 +10723,4 @@ class Slacktest extends AdapterBaseCl {
   }
 }
 
-module.exports = Slacktest;
+module.exports = Slack;

package/entities/Channels/schema.json

@@ -29,7 +29,7 @@
       "external_name": "ph_request_type"
     },
     "threadTs": {
-      "type": "string",
+      "type": ["string", "number"],
       "description": "Unique identifier of a thread's parent message",
       "parse": false,
       "encode": false,

package/entities/Chat/schema.json

@@ -21,7 +21,7 @@
       "external_name": "ph_request_type"
     },
     "threadTs": {
-      "type": "string",
+      "type": ["string", "number"],
       "description": "Unique identifier of a thread's parent message",
       "parse": false,
       "encode": false,

package/entities/Files/schema.json

@@ -2,7 +2,7 @@
   "$id": "schema.json",
   "type": "object",
   "schema": "http://json-schema.org/draft-07/schema#",
-  "translate": false,
+  "translate": true,
   "dynamicfields": true,
   "properties": {
     "ph_request_type": {
@@ -18,6 +18,17 @@
         "postFilesupload"
       ],
       "external_name": "ph_request_type"
+    },
+    "threadTs": {
+      "type": ["string", "number"],
+      "description": "Unique identifier of a thread's parent message",
+      "parse": false,
+      "encode": false,
+      "encrypt": {
+        "type": "AES",
+        "key": ""
+      },
+      "external_name": "thread_ts"
     }
   },
   "definitions": {}

package/entities/Groups/schema.json

@@ -30,7 +30,7 @@
       "external_name": "ph_request_type"
     },
     "threadTs": {
-      "type": "string",
+      "type": ["string", "number"],
       "description": "Unique identifier of a thread's parent message",
       "parse": false,
       "encode": false,

package/entities/Im/schema.json

@@ -20,7 +20,7 @@
       "external_name": "ph_request_type"
     },
     "threadTs": {
-      "type": "string",
+      "type": ["string", "number"],
       "description": "Unique identifier of a thread's parent message",
       "parse": false,
       "encode": false,

package/entities/Mpim/schema.json

@@ -20,7 +20,7 @@
       "external_name": "ph_request_type"
     },
     "threadTs": {
-      "type": "string",
+      "type": ["string", "number"],
       "description": "Unique identifier of a thread's parent message",
       "parse": false,
       "encode": false,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@itentialopensource/adapter-slack",
-  "version": "2.6.1",
+  "version": "3.0.0",
   "description": "This adapter integrates with system Slack",
   "main": "adapter.js",
   "systemName": "Slack",
@@ -55,7 +55,7 @@
   "author": "Itential",
   "homepage": "https://gitlab.com/itentialopensource/adapters/notification-messaging/adapter-slack#readme",
   "dependencies": {
-    "@itentialopensource/adapter-utils": "^4.45.6",
+    "@itentialopensource/adapter-utils": "^4.48.7",
     "ajv": "^6.12.0",
     "axios": "^0.21.0",
     "commander": "^2.20.0",

package/pronghorn.json

@@ -1,7 +1,7 @@
 {
   "id": "@itentialopensource/adapter-slack",
   "type": "Adapter",
-  "export": "Slacktest",
+  "export": "Slack",
   "title": "Slack",
   "src": "adapter.js",
   "roles": [
@@ -4332,6 +4332,16 @@
             "title": "file",
             "type": "string"
           }
+        },
+        {
+          "name": "threadTs",
+          "type": "number",
+          "info": "Unique identifier of a thread's parent message",
+          "required": false,
+          "schema": {
+            "title": "threadTs",
+            "type": "number"
+          }
         }
       ],
       "output": {

package/report/adapterInfo.json

@@ -1,10 +1,10 @@
 {
-  "version": "2.5.5",
-  "configLines": 10347,
+  "version": "2.6.1",
+  "configLines": 10357,
   "scriptLines": 1707,
-  "codeLines": 12509,
+  "codeLines": 12510,
   "testLines": 9108,
   "testCases": 394,
-  "totalCodeLines": 23324,
+  "totalCodeLines": 23325,
   "wfTasks": 155
 }

package/sampleProperties.json

@@ -1,6 +1,6 @@
 {
   "id": "slack",
-  "type": "Slacktest",
+  "type": "Slack",
   "properties": {
     "host": "slack.com",
     "port": 443,

package/test/integration/adapterTestIntegration.js

@@ -58,7 +58,7 @@ global.pronghornProps = {
   adapterProps: {
     adapters: [{
       id: 'Test-slack',
-      type: 'Slacktest',
+      type: 'Slack',
       properties: samProps
     }]
   }
@@ -234,12 +234,12 @@ function saveMockData(entityName, actionName, descriptor, responseData) {
 }
 
 // require the adapter that we are going to be using
-const Slacktest = require('../../adapter');
+const Slack = require('../../adapter');
 
 // begin the testing - these should be pretty well defined between the describe and the it!
 describe('[integration] Slack Adapter Test', () => {
-  describe('Slacktest Class Tests', () => {
-    const a = new Slacktest(
+  describe('Slack Class Tests', () => {
+    const a = new Slack(
       pronghornProps.adapterProps.adapters[0].id,
       pronghornProps.adapterProps.adapters[0].properties
     );
@@ -2848,7 +2848,7 @@ describe('[integration] Slack Adapter Test', () => {
     describe('#postFilesupload - errors', () => {
       it('should work if integrated or standalone with mockdata', (done) => {
         try {
-          a.postFilesupload('fakedata', 'fakedata', 'fakedata', 'fakedata', 'fakedata', 'fakedata', 'fakedata', 'fakedata', (data, error) => {
+          a.postFilesupload('fakedata', 'fakedata', 'fakedata', 'fakedata', 'fakedata', 'fakedata', 'fakedata', 'fakedata', 0, (data, error) => {
             try {
               if (stub) {
                 assert.equal(undefined, error);

package/test/unit/adapterTestUnit.js

@@ -54,7 +54,7 @@ global.pronghornProps = {
   adapterProps: {
     adapters: [{
       id: 'Test-slack',
-      type: 'Slacktest',
+      type: 'Slack',
       properties: samProps
     }]
   }
@@ -116,7 +116,7 @@ function runErrorAsserts(data, error, code, origin, displayStr) {
 }
 
 // require the adapter that we are going to be using
-const Slacktest = require('../../adapter');
+const Slack = require('../../adapter');
 
 // delete the .DS_Store directory in entities -- otherwise this will cause errors
 const dirPath = path.join(__dirname, '../../entities/.DS_Store');
@@ -131,8 +131,8 @@ if (fs.existsSync(dirPath)) {
 
 // begin the testing - these should be pretty well defined between the describe and the it!
 describe('[unit] Slack Adapter Test', () => {
-  describe('Slacktest Class Tests', () => {
-    const a = new Slacktest(
+  describe('Slack Class Tests', () => {
+    const a = new Slack(
       pronghornProps.adapterProps.adapters[0].id,
       pronghornProps.adapterProps.adapters[0].properties
     );
@@ -363,7 +363,7 @@ describe('[unit] Slack Adapter Test', () => {
           const pronghornDotJson = require('../../pronghorn.json');
           assert.notEqual(-1, pronghornDotJson.id.indexOf('slack'));
           assert.equal('Adapter', pronghornDotJson.type);
-          assert.equal('Slacktest', pronghornDotJson.export);
+          assert.equal('Slack', pronghornDotJson.export);
           assert.equal('Slack', pronghornDotJson.title);
           assert.equal('adapter.js', pronghornDotJson.src);
           done();
@@ -729,7 +729,7 @@ describe('[unit] Slack Adapter Test', () => {
         try {
           const sampleDotJson = require('../../sampleProperties.json');
           assert.notEqual(-1, sampleDotJson.id.indexOf('slack'));
-          assert.equal('Slacktest', sampleDotJson.type);
+          assert.equal('Slack', sampleDotJson.type);
           assert.notEqual(undefined, sampleDotJson.properties);
           assert.notEqual(null, sampleDotJson.properties);
           assert.notEqual('', sampleDotJson.properties);