-
Notifications
You must be signed in to change notification settings - Fork 8
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
refactor: separate revert test suite into its own file
- Loading branch information
Showing
3 changed files
with
139 additions
and
133 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,138 @@ | ||
| import { Uri, window, workspace, commands } from 'vscode'; | ||
| import * as sinon from 'sinon'; | ||
| import { add, fakeFossilStatus, getExecStub, getRepository } from './common'; | ||
| import * as assert from 'assert/strict'; | ||
| import * as fs from 'fs/promises'; | ||
| import { Suite } from 'mocha'; | ||
| import { FossilResourceGroup } from '../../resourceGroups'; | ||
|
|
||
| export function RevertSuite(this: Suite): void { | ||
| test('Single source', async () => { | ||
| const url = await add( | ||
| 'revert_me.txt', | ||
| 'Some original text\n', | ||
| 'add revert_me.txt' | ||
| ); | ||
| await fs.writeFile(url.fsPath, 'something new'); | ||
|
|
||
| const repository = getRepository(); | ||
| await repository.updateModelState(); | ||
| const resource = repository.workingGroup.getResource(url); | ||
| assert.ok(resource); | ||
|
|
||
| const showWarningMessage: sinon.SinonStub = this.ctx.sandbox.stub( | ||
| window, | ||
| 'showWarningMessage' | ||
| ); | ||
| showWarningMessage.onFirstCall().resolves('&&Discard Changes'); | ||
|
|
||
| await commands.executeCommand('fossil.revert', resource); | ||
| const newContext = await fs.readFile(url.fsPath); | ||
| assert.equal(newContext.toString('utf-8'), 'Some original text\n'); | ||
| }); | ||
|
|
||
| test('Dialog has no typos', async () => { | ||
| const repository = getRepository(); | ||
| const rootUri = workspace.workspaceFolders![0].uri; | ||
| const fake_status = []; | ||
| const execStub = getExecStub(this.ctx.sandbox); | ||
| const fileUris: Uri[] = []; | ||
| for (const filename of 'abcdefghijklmn') { | ||
| const fileUri = Uri.joinPath(rootUri, 'added', filename); | ||
| fake_status.push(`EDITED added/${filename}`); | ||
| fileUris.push(fileUri); | ||
| } | ||
| const statusCall = fakeFossilStatus(execStub, fake_status.join('\n')); | ||
| await repository.updateModelState(); | ||
| sinon.assert.calledOnce(statusCall); | ||
| const resources = fileUris.map(uri => { | ||
| const resource = repository.workingGroup.getResource(uri); | ||
| assert.ok(resource); | ||
| return resource; | ||
| }); | ||
| const swm: sinon.SinonStub = this.ctx.sandbox.stub( | ||
| window, | ||
| 'showWarningMessage' | ||
| ); | ||
| await commands.executeCommand('fossil.revert', ...resources); | ||
| assert.ok( | ||
| swm.firstCall.calledWith( | ||
| 'Are you sure you want to discard changes to 14 files?\n\n • a\n • b\n • c\n • d\n • e\n • f\n • g\n • h\nand 6 others' | ||
| ) | ||
| ); | ||
| await commands.executeCommand( | ||
| 'fossil.revert', | ||
| ...resources.slice(0, 3) | ||
| ); | ||
| assert.ok( | ||
| swm.secondCall.calledWith( | ||
| 'Are you sure you want to discard changes to 3 files?\n\n • a\n • b\n • c' | ||
| ) | ||
| ); | ||
| }); | ||
|
|
||
| test('Revert (Nothing)', async () => { | ||
| await commands.executeCommand('fossil.revert'); | ||
| }); | ||
|
|
||
| async function revertAllTest( | ||
| sandbox: sinon.SinonSandbox, | ||
| groups: FossilResourceGroup[], | ||
| message: string, | ||
| files: string[] | ||
| ): Promise<void> { | ||
| const swm: sinon.SinonStub = sandbox.stub(window, 'showWarningMessage'); | ||
| swm.onFirstCall().resolves('&&Discard Changes'); | ||
|
|
||
| const repository = getRepository(); | ||
| const execStub = getExecStub(sandbox); | ||
| const statusStub = fakeFossilStatus( | ||
| execStub, | ||
| 'EDITED a.txt\nEDITED b.txt\nCONFLICT c.txt\nCONFLICT d.txt' | ||
| ); | ||
| const revertStub = execStub | ||
| .withArgs(sinon.match.array.startsWith(['revert'])) | ||
| .resolves(); | ||
| await repository.updateModelState(); | ||
| sinon.assert.calledOnce(statusStub); | ||
| await commands.executeCommand('fossil.revertAll', ...groups); | ||
| sinon.assert.calledOnceWithExactly( | ||
| swm, | ||
| message, | ||
| { modal: true }, | ||
| '&&Discard Changes' | ||
| ); | ||
| sinon.assert.calledOnceWithExactly(revertStub, ['revert', ...files]); | ||
| } | ||
|
|
||
| test('Revert all (no groups)', async () => { | ||
| await revertAllTest( | ||
| this.ctx.sandbox, | ||
| [], | ||
| 'Are you sure you want to discard changes in ' + | ||
| '"Changes" and "Unresolved Conflicts" group?', | ||
| ['a.txt', 'b.txt', 'c.txt', 'd.txt'] | ||
| ); | ||
| }); | ||
|
|
||
| test('Revert all (changes group)', async () => { | ||
| const repository = getRepository(); | ||
| await revertAllTest( | ||
| this.ctx.sandbox, | ||
| [repository.workingGroup], | ||
| 'Are you sure you want to discard changes in "Changes" group?', | ||
| ['a.txt', 'b.txt'] | ||
| ); | ||
| }); | ||
|
|
||
| test('Revert all (conflict group)', async () => { | ||
| const repository = getRepository(); | ||
| await revertAllTest( | ||
| this.ctx.sandbox, | ||
| [repository.conflictGroup], | ||
| 'Are you sure you want to discard changes ' + | ||
| 'in "Unresolved Conflicts" group?', | ||
| ['c.txt', 'd.txt'] | ||
| ); | ||
| }); | ||
| } |