mirror of
https://github.com/JamesIves/github-pages-deploy-action.git
synced 2023-12-15 20:03:39 +08:00
95f8a2cd05
* Return early from dry run Determining whether to create a merge commit would elicit a nested conditional, which could be hard to parse for a human reader. This is avoided by returning early as soon as possible for a dry run. This also resolves the erroneous 'changes committed' message when no changes were actually committed because of the dry run. A message specific to dry-run is logged instead. * Add force parameter to action interface Existing behaviour is equivalent to force=true, so the default value is true. * Implement pull+rebase procedure * Declare force parameter in action * Detect both rejection syntaxes * Return both stdout and stderr from execute * Ignore non-zero exit status on push * Remove unnecessary error catch * Fetch and rebase in separate steps * Explicitly bind incoming branch I think the fetch will update the origin/gh-pages branch but not the gh-pages branch, despite requesting gh-pages. This means that when I later attempt to rebase the temp branch on top of the gh-pages branch, there will be nothing to do, because that's already where it is. * Implement attempt limit I don't expect this to ever require more than one attempt in production, but in theory it's possible that this procedure could loop forever. We would need to keep fetching and rebasing if changes keep being added to the remote. In practice, I believe this would only happen if there are lots of workflows simultaneously deploying to the same branch, all using this action. In this case only one would be able to secure a lock at a time, leading to the total number of attempts being equal to the number of simultaneous deployments, assuming each deployment makes each attempt at the exact same time. The limit may need to be increased or even be configurable, but 3 should cover most uses. * Update tests for execute output split * Document 'force' parameter * Create integration test for rebase procedure This test is composed of 3 jobs. The first two jobs run simultaneously, and as such both depend on the previous integration test only. The final job cleans up afterwards, and depends on both of the prior jobs. The two jobs are identical except that they both create a temporary file in a different location. This is to ensure that they conflict. Correctly resolving this conflict by rebasing one deployment over the other, resulting in a deployment containing both files, indicates a successful test.
142 lines
3.5 KiB
TypeScript
142 lines
3.5 KiB
TypeScript
import {exportVariable} from '@actions/core'
|
|
import {mkdirP} from '@actions/io'
|
|
import child_process, {execFileSync, execSync} from 'child_process'
|
|
import {appendFileSync} from 'fs'
|
|
import {action, TestFlag} from '../src/constants'
|
|
import {execute} from '../src/execute'
|
|
import {configureSSH} from '../src/ssh'
|
|
|
|
const originalAction = JSON.stringify(action)
|
|
|
|
jest.mock('fs', () => ({
|
|
appendFileSync: jest.fn(),
|
|
existsSync: jest.fn()
|
|
}))
|
|
|
|
jest.mock('child_process', () => ({
|
|
execFileSync: jest.fn(),
|
|
execSync: jest.fn()
|
|
}))
|
|
|
|
jest.mock('@actions/io', () => ({
|
|
rmRF: jest.fn(),
|
|
mkdirP: jest.fn()
|
|
}))
|
|
|
|
jest.mock('@actions/core', () => ({
|
|
setFailed: jest.fn(),
|
|
getInput: jest.fn(),
|
|
setOutput: jest.fn(),
|
|
isDebug: jest.fn(),
|
|
info: jest.fn(),
|
|
exportVariable: jest.fn()
|
|
}))
|
|
|
|
jest.mock('../src/execute', () => ({
|
|
execute: jest.fn(() => ({stdout: '', stderr: ''}))
|
|
}))
|
|
|
|
describe('configureSSH', () => {
|
|
afterEach(() => {
|
|
Object.assign(action, JSON.parse(originalAction))
|
|
})
|
|
|
|
it('should skip client configuration if sshKey is set to true', async () => {
|
|
Object.assign(action, {
|
|
hostname: 'github.com',
|
|
silent: false,
|
|
folder: 'assets',
|
|
branch: 'branch',
|
|
sshKey: true,
|
|
pusher: {
|
|
name: 'asd',
|
|
email: 'as@cat'
|
|
},
|
|
isTest: TestFlag.HAS_CHANGED_FILES
|
|
})
|
|
|
|
await configureSSH(action)
|
|
|
|
expect(execute).toBeCalledTimes(0)
|
|
expect(mkdirP).toBeCalledTimes(0)
|
|
expect(appendFileSync).toBeCalledTimes(0)
|
|
})
|
|
|
|
it('should configure the ssh client if a key is defined', async () => {
|
|
;(child_process.execFileSync as jest.Mock).mockImplementationOnce(() => {
|
|
return 'SSH_AUTH_SOCK=/some/random/folder/agent.123; export SSH_AUTH_SOCK;\nSSH_AGENT_PID=123; export SSH_AGENT_PID;'
|
|
})
|
|
|
|
Object.assign(action, {
|
|
hostname: 'github.com',
|
|
silent: false,
|
|
folder: 'assets',
|
|
branch: 'branch',
|
|
sshKey: '?=-----BEGIN 123 456\n 789',
|
|
pusher: {
|
|
name: 'asd',
|
|
email: 'as@cat'
|
|
},
|
|
isTest: TestFlag.HAS_CHANGED_FILES
|
|
})
|
|
|
|
await configureSSH(action)
|
|
|
|
expect(execFileSync).toBeCalledTimes(1)
|
|
expect(exportVariable).toBeCalledTimes(2)
|
|
expect(execSync).toBeCalledTimes(3)
|
|
})
|
|
|
|
it('should not export variables if the return from ssh-agent is skewed', async () => {
|
|
;(child_process.execFileSync as jest.Mock).mockImplementationOnce(() => {
|
|
return 'useless nonsense here;'
|
|
})
|
|
|
|
Object.assign(action, {
|
|
hostname: 'github.com',
|
|
silent: false,
|
|
folder: 'assets',
|
|
branch: 'branch',
|
|
sshKey: '?=-----BEGIN 123 456\n 789',
|
|
pusher: {
|
|
name: 'asd',
|
|
email: 'as@cat'
|
|
},
|
|
isTest: TestFlag.HAS_CHANGED_FILES
|
|
})
|
|
|
|
await configureSSH(action)
|
|
|
|
expect(execFileSync).toBeCalledTimes(1)
|
|
expect(exportVariable).toBeCalledTimes(0)
|
|
expect(execSync).toBeCalledTimes(3)
|
|
})
|
|
|
|
it('should throw if something errors', async () => {
|
|
;(child_process.execFileSync as jest.Mock).mockImplementationOnce(() => {
|
|
throw new Error('Mocked throw')
|
|
})
|
|
|
|
Object.assign(action, {
|
|
hostname: 'github.com',
|
|
silent: false,
|
|
folder: 'assets',
|
|
branch: 'branch',
|
|
sshKey: 'real_key',
|
|
pusher: {
|
|
name: 'asd',
|
|
email: 'as@cat'
|
|
},
|
|
isTest: TestFlag.HAS_CHANGED_FILES
|
|
})
|
|
|
|
try {
|
|
await configureSSH(action)
|
|
} catch (error) {
|
|
expect(error instanceof Error && error.message).toBe(
|
|
'The ssh client configuration encountered an error: Mocked throw ❌'
|
|
)
|
|
}
|
|
})
|
|
})
|