Merge pull request #1 from rudderlabs/test-action

test action
rudderlabs · Apr 27, 2021 · bdc972c · bdc972c
2 parents aa477bb + fe57fe4
commit bdc972c
Showing 6 changed files with 80 additions and 877 deletions.
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -1,6 +1,12 @@
 name: Test
 
-on: [push, pull_request]
+on: 
+  push:
+    branches:
+      - main
+  pull_request:
+    branches:
+      - main
 
 jobs:
   request:
@@ -13,7 +19,7 @@ jobs:
         uses: ./
         with:
           metaPath: './code/meta.json'
-          email: 'sayan+10@rudderlabs.com'
-          accessKey: ${{ secrets.ACCESS_KEY }}
-          serverEndpoint: 'https://1a3bd53e8df2.ngrok.io'
+          email: 'test1619427853838@rudderlabs.com'
+          accessKey: '1rhrWotHszP3eVma57BzAP67QWA'
+          serverEndpoint: 'https://aa88e07d9346.ngrok.io'
           uploadTestArtifact: true
diff --git a/code/expected.json b/code/expected.json
@@ -3,14 +3,14 @@
     "revenue": 0,
     "price": 0,
     "profit": 0,
-    "city": "kolkata",
-    "country": "India",
+    "city": "no data found",
+    "country": "no data found",
     "street": "no data found"
   },
   {
     "revenue": 0,
     "price": 20,
-    "profit": 0,
+    "profit": 20,
     "city": "no data found",
     "country": "no data found",
     "street": "no data found"

diff --git a/code/meta.json b/code/meta.json
@@ -2,23 +2,35 @@
   "transformations": [
     {
       "file": "./code/code.js",
-      "name": "action-T13",
-      "description": "action-T13",
+      "name": "action-T14",
+      "description": "action-T14",
       "type": "transformation",
       "test-input-file": "./code/events.json",
       "expected-output": "./code/expected.json"
     },
     {
       "file": "./code/code_2.js",
-      "name": "action-T13_2",
+      "name": "action-T14_2",
       "description": "action-T13_2",
       "type": "transformation"
     },
     {
       "file": "./code/code_3.js",
-      "name": "action-T13_3",
+      "name": "action-T14_3",
       "description": "action-T13_3",
       "type": "transformation"
     }
+  ],
+  "libraries": [
+    {
+      "file": "./code/lib1.js",
+      "name": "getFinanceData13",
+      "description": "action-T13-lib"
+    },
+    {
+      "file": "./code/lib2.js",
+      "name": "getUserAddress13",
+      "description": "action-T13_2-lib"
+    }
   ]
 }
diff --git a/dist/index.js b/dist/index.js
diff --git a/src/apiCalls.js b/src/apiCalls.js
@@ -1,6 +1,6 @@
 const axios = require("axios");
 const core = require("@actions/core");
-const serverEndpoint = core.getInput("serverEndpoint");
+const serverEndpoint = core.getInput("serverEndpoint") || 'https://api.rudderstack.com';
 const createTransformerEndpoint = `${serverEndpoint}/transformations`;
 const createLibraryEndpoint = `${serverEndpoint}/libraries`;
 const testEndpoint = `${serverEndpoint}/transformations/libraries/test`;

diff --git a/src/index.js b/src/index.js
@@ -96,7 +96,6 @@ async function testAndPublish() {
       }
       transformationDict[res.data.versionId] = { ...tr, id: res.data.id };
     }
-
     core.info("transformations create/update done!");
 
     for (let i = 0; i < libraries.length; i++) {
@@ -115,7 +114,6 @@ async function testAndPublish() {
       }
       libraryDict[res.data.versionId] = { ...lib, id: res.data.id };
     }
-
     core.info("libraries create/update done!");
 
     let transformationTest = [];
@@ -171,8 +169,13 @@ async function testAndPublish() {
       fs.mkdirSync(testOutputDir);
     }
     for (let i = 0; i < successResults.length; i++) {
+      let transformerVersionID = successResults[i].transformerVersionID;
+      if (!transformationDict.hasOwnProperty(transformerVersionID) || !transformationDict[transformerVersionID].hasOwnProperty("expected-output")) {
+        continue;
+      }
+
       let expectedOutputfile =
-        transformationDict[successResults[i].transformerVersionID][
+        transformationDict[transformerVersionID][
           "expected-output"
         ];
       let expectedOutput = expectedOutputfile
@@ -182,11 +185,11 @@ async function testAndPublish() {
       let apiOutput = successResults[i].result.output;
 
       let transformationName =
-        transformationDict[successResults[i].transformerVersionID].name;
+        transformationDict[transformerVersionID].name;
 
       fs.writeFileSync(
         `${testOutputDir}/${transformationName}_output.json`,
-        JSON.stringify(apiOutput)
+        JSON.stringify(apiOutput, null, 2)
       );
       testOutputFiles.push(
         `${testOutputDir}/${transformationName}_output.json`
@@ -199,13 +202,13 @@ async function testAndPublish() {
       if (!isEqual(expectedOutput, apiOutput)) {
         errorResults.push(
           `Transformer name: ${
-            transformationDict[successResults[i].transformerVersionID].name
+            transformationDict[transformerVersionID].name
           } test outputs don't match`
         );
 
         fs.writeFileSync(
           `${testOutputDir}/${transformationName}_diff.json`,
-          JSON.stringify(detailedDiff(expectedOutput, apiOutput))
+          JSON.stringify(detailedDiff(expectedOutput, apiOutput), null, 2)
         );
 
         testOutputFiles.push(