fix: second argument to shell.writeShortcutLink is optional (#49501)

fix: second argument to shell.writeShortcutLink is optional

Co-authored-by: trop[bot] <37223003+trop[bot]@users.noreply.github.com>
Co-authored-by: Shelley Vohr <shelley.vohr@gmail.com>
This commit is contained in:
trop[bot]
2026-01-26 14:29:56 +01:00
committed by GitHub
parent db6b22a8e5
commit ac4efed3d1
2 changed files with 24 additions and 5 deletions

View File

@@ -172,6 +172,16 @@ describe('shell module', () => {
expect(fs.existsSync(tmpShortcut)).to.be.true();
});
it('writes the shortcut with omitted operation (defaults to create)', () => {
expect(shell.writeShortcutLink(tmpShortcut, shortcutOptions)).to.be.true();
expect(fs.existsSync(tmpShortcut)).to.be.true();
expect(shell.readShortcutLink(tmpShortcut)).to.deep.equal(shortcutOptions);
const newOptions = { ...shortcutOptions, description: 'new description' };
expect(shell.writeShortcutLink(tmpShortcut, newOptions)).to.be.true();
expect(shell.readShortcutLink(tmpShortcut)).to.deep.equal(newOptions);
});
it('correctly sets the fields', () => {
expect(shell.writeShortcutLink(tmpShortcut, shortcutOptions)).to.be.true();
expect(shell.readShortcutLink(tmpShortcut)).to.deep.equal(shortcutOptions);