Expand Unit Test Coverage

README.md

@@ -17,10 +17,11 @@ actions can be run directly on your workstation.
 The following table tracks the versions of the GitHub Actions Toolkit that are
 currently implemented by this tool.
 
-| Package             | Version  |
-| ------------------- | -------- |
-| `@actions/artifact` | `2.2.0`  |
-| `@actions/core`     | `1.11.1` |
+| Package                                                                | Version  |
+| ---------------------------------------------------------------------- | -------- |
+| [`@actions/artifact`](https://www.npmjs.com/package/@actions/artifact) | `2.3.2`  |
+| [`@actions/core`](https://www.npmjs.com/package/@actions/core)         | `1.11.1` |
+| [`@actions/github`](https://www.npmjs.com/package/@actions/github)     | `6.0.0`  |
 
 ## Changelog
 

__fixtures__/@actions/github.ts

@@ -0,0 +1,3 @@
+import * as octokit from '../@octokit/rest.js'
+
+export const getOctokit = () => octokit

__fixtures__/@actions/http-client.ts

@@ -0,0 +1,9 @@
+import { jest } from '@jest/globals'
+
+export const getAgent = jest.fn()
+export const getAgentDispatcher = jest.fn()
+
+export class HttpClient {
+  getAgent = getAgent
+  getAgentDispatcher = getAgentDispatcher
+}

__fixtures__/@octokit/rest.ts

@@ -0,0 +1,18 @@
+import { jest } from '@jest/globals'
+import { Endpoints } from '@octokit/types'
+
+export const graphql = jest.fn()
+export const paginate = jest.fn()
+export const request = jest.fn()
+export const rest = {
+  actions: {
+    deleteArtifact:
+      jest.fn<
+        () => Endpoints['DELETE /repos/{owner}/{repo}/actions/artifacts/{artifact_id}']['response']
+      >(),
+    listWorkflowRunArtifacts:
+      jest.fn<
+        () => Endpoints['GET /repos/{owner}/{repo}/actions/runs/{run_id}/artifacts']['response']
+      >()
+  }
+}

__fixtures__/fs.ts

@@ -7,6 +7,7 @@ export const existsSync = jest.fn()
 export const mkdirSync = jest.fn()
 export const readFileSync = jest.fn()
 export const rmSync = jest.fn()
+export const statSync = jest.fn()
 
 export default {
   accessSync,
@@ -15,5 +16,6 @@ export default {
   existsSync,
   mkdirSync,
   readFileSync,
-  rmSync
+  rmSync,
+  statSync
 }

__fixtures__/typescript/success-yaml/.env.fixture

@@ -0,0 +1,2 @@
+ACTIONS_STEP_DEBUG=false
+INPUT_MILLISECONDS=2400

__fixtures__/typescript/success-yaml/action.yaml

@@ -0,0 +1,16 @@
+name: TypeScript (Success)
+description: This action returns without error
+
+inputs:
+  myInput:
+    description: An input
+    required: true
+    default: value
+
+outputs:
+  myOutput:
+    description: An output
+
+runs:
+  using: node20
+  main: dist/index.js

__fixtures__/typescript/success-yaml/package.json

@@ -0,0 +1,15 @@
+{
+  "name": "typescript-action",
+  "description": "GitHub Actions TypeScript template",
+  "version": "0.0.0",
+  "engines": {
+    "node": ">=20"
+  },
+  "dependencies": {
+    "@actions/core": "^1.10.1"
+  },
+  "devDependencies": {
+    "@types/node": "^22.2.0",
+    "typescript": "^5.5.4"
+  }
+}

__fixtures__/typescript/success-yaml/src/index.ts

@@ -0,0 +1,3 @@
+import { run } from './main'
+
+run()

__fixtures__/typescript/success-yaml/src/main.ts

@@ -0,0 +1,9 @@
+import { getInput, info, setOutput } from '@actions/core'
+
+export async function run(): Promise<void> {
+  const myInput: string = getInput('myInput')
+
+  setOutput('myOutput', myInput)
+
+  info('TypeScript Action Succeeded!')
+}

__fixtures__/typescript/success-yaml/tsconfig.json

@@ -0,0 +1,20 @@
+{
+  "$schema": "https://json.schemastore.org/tsconfig",
+  "compilerOptions": {
+    "target": "ES2022",
+    "module": "NodeNext",
+    "rootDir": "./src",
+    "moduleResolution": "NodeNext",
+    "baseUrl": "./",
+    "sourceMap": true,
+    "outDir": "./dist",
+    "noImplicitAny": true,
+    "esModuleInterop": true,
+    "forceConsistentCasingInFileNames": true,
+    "strict": true,
+    "skipLibCheck": true,
+    "newLine": "lf"
+  },
+  "exclude": ["node_modules"],
+  "include": ["src"]
+}

__tests__/command.test.ts

@@ -37,7 +37,6 @@ describe('Commmand', () => {
   })
 
   afterEach(() => {
-    // Reset all spies
     jest.resetAllMocks()
   })
 
@@ -71,6 +70,24 @@ describe('Commmand', () => {
       expect(action).toHaveBeenCalled()
     })
 
+    it('Runs if all arguments are provided (action.yaml)', async () => {
+      await (
+        await makeProgram()
+      ).parseAsync(
+        [
+          './__fixtures__/typescript/success-yaml',
+          'src/main.ts',
+          './__fixtures__/typescript/success-yaml/.env.fixture'
+        ],
+        {
+          from: 'user'
+        }
+      )
+
+      expect(process_exitSpy).not.toHaveBeenCalled()
+      expect(action).toHaveBeenCalled()
+    })
+
     it('Exits if no path argument is provided', async () => {
       await (await makeProgram()).parseAsync([], { from: 'user' })
 

__tests__/stubs/artifact/internal/client.test.ts

@@ -1,5 +1,5 @@
 import { jest } from '@jest/globals'
-import * as core from '../../../../__fixtures__/core.js'
+import * as core from '../../../../__fixtures__/@actions/core.js'
 import { ResetCoreMetadata } from '../../../../src/stubs/core/core.js'
 import { EnvMeta, ResetEnvMetadata } from '../../../../src/stubs/env.js'
 
@@ -77,7 +77,6 @@ describe('DefaultArtifactClient', () => {
   })
 
   afterEach(() => {
-    // Reset all spies
     jest.resetAllMocks()
 
     // Unset environment variables