Prevent @actions/core from writting to files

An implementation detail, but since the unittests cover multiple platforms the action trying to write to a file will fail since the path to the file is in the hosts format. Signed-off-by: Andrei Horodniceanu <[email protected]>
the-horo · Jul 12, 2024 · 4d45c00 · 4d45c00
1 parent 2e80267
commit 4d45c00
Show file tree

Hide file tree

Showing 5 changed files with 13 additions and 0 deletions.
diff --git a/__tests__/d-compiler.test.ts b/__tests__/d-compiler.test.ts
@@ -1,5 +1,8 @@
 import { Compiler } from '../src/d'
 import fs from 'fs'
+import * as testUtils from './test-helpers.test'
+
+testUtils.disableGithubPaths()
 
 describe('Test Compiler class', () => {
     const logSpy = jest.spyOn(console, 'log').mockReturnValue(undefined)

diff --git a/__tests__/d-dmd.test.ts b/__tests__/d-dmd.test.ts
@@ -7,6 +7,7 @@ import fs from 'fs'
 testUtils.hideConsoleLogs()
 testUtils.saveProcessRestorePoint()
 testUtils.disableNetwork()
+testUtils.disableGithubPaths()
 function init(version: string) { return DMD.initialize(version, '') }
 
 describe('amd64', () => {

diff --git a/__tests__/d-gdc.test.ts b/__tests__/d-gdc.test.ts
@@ -5,6 +5,7 @@ import * as tc from '@actions/tool-cache'
 
 testUtils.saveProcessRestorePoint()
 testUtils.hideConsoleLogs()
+testUtils.disableGithubPaths()
 const origEnv = process.env
 beforeEach(() => {
     process.env = origEnv

diff --git a/__tests__/d-ldc.test.ts b/__tests__/d-ldc.test.ts
@@ -6,6 +6,7 @@ import fs from 'fs'
 
 testUtils.saveProcessRestorePoint()
 testUtils.disableNetwork()
+testUtils.disableGithubPaths()
 testUtils.hideConsoleLogs()
 
 async function doTest(input: string, url: string, sig: string | undefined,

diff --git a/__tests__/test-helpers.test.ts b/__tests__/test-helpers.test.ts
@@ -1,4 +1,5 @@
 import * as utils from '../src/utils'
+import * as coreInternals from '@actions/core/lib/file-command'
 
 export function mockTags (expectedUrl: string | RegExp, pages: string[][]) {
     async function mocker (url: string, token: string,
@@ -56,3 +57,9 @@ export function hideConsoleLogs () {
     beforeEach(() => jest.spyOn(console, 'log').mockReturnValue(undefined))
     beforeEach(() => jest.spyOn(process.stdout, 'write').mockReturnValue(true))
 }
+
+export function disableGithubPaths () {
+    beforeAll(() => {
+	jest.spyOn(coreInternals, 'issueFileCommand').mockReturnValue(undefined)
+    })
+}