fix: binary file handling

Author: exuanbo

.github/workflows/ci.yml

@@ -25,8 +25,8 @@ jobs:
       - name: Setup Node.js
         uses: actions/setup-node@v3
         with:
-          node-version: "${{ steps.nvm.outputs.NVMRC }}"
-          cache: "npm"
+          node-version: ${{ steps.nvm.outputs.NVMRC }}
+          cache: npm
 
       - name: Update npm
         run: npm install -g npm@latest
@@ -44,7 +44,7 @@ jobs:
           INPUT_TOKEN: ${{ secrets.TOKEN }}
         run: npm run all
 
-  action-with-all-inputs:
+  with-all-inputs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v3
@@ -57,8 +57,9 @@ jobs:
           gist_description: "foo bar"
           gist_file_name: foo.bar
           file_path: ./__tests__/foo.bar
+          file_type: text
 
-  action-with-minimum-inputs:
+  with-minimum-inputs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v3
@@ -69,3 +70,16 @@ jobs:
           token: ${{ secrets.TOKEN }}
           gist_id: e885afa349a0e5d1cfb408e46d6a37bc
           file_path: ./__tests__/foo.bar
+
+  with-binary-file:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v3
+
+      - name: Test action
+        uses: ./
+        with:
+          token: ${{ secrets.TOKEN }}
+          gist_id: e885afa349a0e5d1cfb408e46d6a37bc
+          file_path: ./__tests__/dummy.pdf
+          file_type: binary

.gitignore

@@ -96,4 +96,7 @@ Thumbs.db
 
 # Ignore built ts files
 __tests__/runner/*
-lib/**/*
+lib/**/*
+
+# Tests
+tmp

__tests__/dummy.pdf

@@ -1,5 +1,5 @@
 import { execSync } from 'child_process'
-import path from 'path'
+import { join } from 'path'
 import dotenv from 'dotenv'
 import { getInput } from '../src/input'
 
@@ -23,13 +23,33 @@ test('input', () => {
 })
 
 test('run', () => {
-  process.env.GITHUB_WORKSPACE = process.cwd()
+  const cwd = process.cwd()
+
+  process.env.RUNNER_TEMP = join(cwd, 'tmp')
+  process.env.GITHUB_WORKSPACE = cwd
+
   process.env.INPUT_GIST_ID = 'e885afa349a0e5d1cfb408e46d6a37bc'
   process.env.INPUT_GIST_DESCRIPTION = 'foo bar'
+
+  const run = (): void => {
+    try {
+      const stdout = execSync(`node ${join(__dirname, '../lib/index.js')}`, {
+        env: process.env,
+        encoding: 'utf8'
+      })
+      console.log(stdout)
+    } catch (err) {
+      console.error(err)
+      expect(err).toBeFalsy()
+    }
+  }
+
   process.env.INPUT_GIST_FILE_NAME = 'foo.bar'
   process.env.INPUT_FILE_PATH = '__tests__/foo.bar'
+  run()
 
-  const filePath = path.join(__dirname, '../lib/index.js')
-  const stdoutBuffer = execSync(`node ${filePath}`, { env: process.env })
-  console.log(stdoutBuffer.toString())
+  process.env.INPUT_GIST_FILE_NAME = 'dummy.pdf'
+  process.env.INPUT_FILE_PATH = '__tests__/dummy.pdf'
+  process.env.INPUT_FILE_TYPE = 'binary'
+  run()
 })

__tests__/index.test.ts

@@ -20,6 +20,10 @@ inputs:
   file_path:
     required: true
     description: "Set path of the file to be deployed."
+  file_type:
+    required: false
+    description: "Set file type of the file to be deployed."
+    default: "text"
 runs:
   using: "node16"
   main: "dist/index.js"