diff --git a/__tests__/logtask/index.test.ts b/__tests__/logtask/index.test.ts index 693186ba..3f9c1802 100644 --- a/__tests__/logtask/index.test.ts +++ b/__tests__/logtask/index.test.ts @@ -140,7 +140,7 @@ describe('LogTask', () => { it('should log a failure message', () => { vi.stubEnv('GITHUB_ACTIONS', 'false'); logTask.fail(testMessage); - expect(core.setFailed).toHaveBeenCalledWith( + expect(core.error).toHaveBeenCalledWith( chalk.bgRedBright(`[FAILURE][TestTask ] ❌: ${chalk.yellow.bold(testMessage)}`), ); }); @@ -148,7 +148,7 @@ describe('LogTask', () => { it('should log a failure message without ingroup', () => { vi.stubEnv('GITHUB_ACTIONS', 'false'); logTask.fail(testMessage, false); - expect(core.setFailed).toHaveBeenCalledWith( + expect(core.error).toHaveBeenCalledWith( chalk.bgRedBright(`[FAILURE][TestTask ] ❌: ${chalk.yellow.bold(testMessage)}`), ); }); @@ -158,6 +158,7 @@ describe('LogTask', () => { logTask.ingroup = true; logTask.fail(testMessage); expect(core.endGroup).toHaveBeenCalled(); + expect(core.setFailed).toHaveBeenCalled(); }); }); diff --git a/src/logtask/index.ts b/src/logtask/index.ts index 79f54ad6..8a8dbbc8 100644 --- a/src/logtask/index.ts +++ b/src/logtask/index.ts @@ -264,7 +264,7 @@ export default class LogTask { const desc = message === '' ? `Completed ${this.name}.` : message; if (ingroup) { this.ingroup = false; - if (process.env.GITHUB_ACTIONS) { + if (inGitHubActions()) { core.endGroup(); } } @@ -281,11 +281,11 @@ export default class LogTask { const desc = message === '' ? `Failed ${this.name}.` : message; if (ingroup) { this.ingroup = false; - if (process.env.GITHUB_ACTIONS) { + if (inGitHubActions()) { core.endGroup(); } } - const msgtype = process.env.GITHUB_ACTIONS ? LogGroup.IS_FAILED : LogGroup.IS_ERROR; + const msgtype = inGitHubActions() ? LogGroup.IS_FAILED : LogGroup.IS_ERROR; this.logStep('❌', 'FAILURE', desc, msgtype); }