chore: update test

[jackton1]

No description provided.

[cursor]

Bug: Mocked Tests Incorrectly Skipped

The shouldSkip condition is incorrectly applied to tests within the getPreviousGitTag suite that mock exec.getExecOutput. As these tests do not depend on actual Git repository state, skipping them is unnecessary and reduces test coverage. This also creates an inconsistency with other similar mocked tests in the same suite that correctly lack the shouldSkip condition.

src/__tests__/utils.test.ts#L730-L753

changed-files/src/__tests__/utils.test.ts

Lines 730 to 753 in 8ab20b1

	it('should return empty values when only one tag is available', async () => {
	if (shouldSkip) {
	return
	}
	jest.spyOn(exec, 'getExecOutput').mockResolvedValueOnce({
	stdout:
	'v1.0.1|f0751de6af436d4e79016e2041cf6400e0833653|2021-01-01T00:00:00Z',
	stderr: '',
	exitCode: 0
	})
	const result = await getPreviousGitTag({
	cwd: '.',
	tagsPattern: '*',
	tagsIgnorePattern: '',
	currentBranch: 'v1.0.1'
	})
	expect(result).toEqual({tag: '', sha: ''})
	})
	// Git commands fail and throw errors
	it('should throw an error when git commands fail', async () => {
	if (shouldSkip) {
	return
	}

Fix in Cursor

---

Was this report helpful? Give feedback by reacting with 👍 or 👎