Skip to content

Commit

Permalink
Add ticket to TODO
Browse files Browse the repository at this point in the history
  • Loading branch information
tagniam committed Dec 2, 2021
1 parent c207ad9 commit 50c7551
Show file tree
Hide file tree
Showing 7 changed files with 23 additions and 23 deletions.
2 changes: 1 addition & 1 deletion test/cmd_line/bang.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ import { getAndUpdateModeHandler } from '../../extension';
import { ModeHandler } from '../../src/mode/modeHandler';
import { assertEqualLines, cleanUpWorkspace, setupWorkspace } from './../testUtils';

// TODO: this fails on Windows
// TODO(#4844): this fails on Windows
suite('bang (!) cmd_line', () => {
if (process.platform === 'win32') {
return;
Expand Down
4 changes: 2 additions & 2 deletions test/completion/lineCompletion.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ suite('Provide line completions', () => {
};

suite('Line Completion Provider unit tests', () => {
// TODO: this fails on Windows
// TODO(#4844): this fails on Windows
test('Can complete lines in file, prioritizing above cursor, near cursor', async () => {
if (process.platform === 'win32') {
return;
Expand All @@ -46,7 +46,7 @@ suite('Provide line completions', () => {
assert.deepStrictEqual(topCompletions, expectedCompletions, 'Unexpected completions found');
});

// TODO: this fails on Windows
// TODO(#4844): this fails on Windows
test('Can complete lines in file with different indentation', async () => {
if (process.platform === 'win32') {
return;
Expand Down
2 changes: 1 addition & 1 deletion test/configuration/validators/neovimValidator.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ suite('Neovim Validator', () => {
assert.strictEqual(configuration.enableNeovim, false);
});

// TODO: this fails on Windows
// TODO(#4844): this fails on Windows
test('neovim enabled with nvim in path', async () => {
if (process.platform === 'win32') {
return;
Expand Down
18 changes: 9 additions & 9 deletions test/jumpTracker.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -317,7 +317,7 @@ suite('Record and navigate jumps', () => {
end: ['|start', '{', 'a1', 'b1', 'a2', 'b2', '}', 'end'],
jumps: ['{', '}'],
});
// TODO: this fails on Windows
// TODO(#4844): this fails on Windows
newJumpTestSkipOnWindows({
start: ['|start', '{', 'a1', 'b1', 'a2', 'b2', '}', 'end'],
keysPressed: '/^\nnnn<C-o><C-o><C-o><C-i>gg',
Expand Down Expand Up @@ -369,21 +369,21 @@ suite('Record and navigate jumps', () => {
});

suite('Can shifts jump lines up after deleting a line with Visual Line Mode', () => {
// TODO: this fails on Windows
// TODO(#4844): this fails on Windows
newJumpTestSkipOnWindows({
start: ['|start', 'a1', 'a2', 'a3', 'a4', 'a5', 'end'],
keysPressed: '/^\nnnnkkdd',
end: ['start', 'a1', '|a3', 'a4', 'a5', 'end'],
jumps: ['start', 'a1', 'a3'],
});
// TODO: this fails on Windows
// TODO(#4844): this fails on Windows
newJumpTestSkipOnWindows({
start: ['|start', 'a1', 'a2', 'a3', 'a4', 'a5', 'end'],
keysPressed: '/^\nnnnkdd',
end: ['start', 'a1', 'a2', '|a4', 'a5', 'end'],
jumps: ['start', 'a1', 'a2', 'a4'],
});
// TODO: this fails on Windows
// TODO(#4844): this fails on Windows
newJumpTestSkipOnWindows({
start: ['|start', 'a1', 'a2', 'a3', 'a4', 'a5', 'end'],
keysPressed: '/^\nnnnnn<C-o><C-o><C-o><C-o>dd',
Expand All @@ -399,7 +399,7 @@ suite('Record and navigate jumps', () => {
});

suite('Can shifts jump lines up after deleting a line with Visual Mode', () => {
// TODO: this fails on Windows
// TODO(#4844): this fails on Windows
newJumpTestSkipOnWindows({
start: ['|start', 'a1', 'a2', 'a3', 'a4', 'a5', 'end'],
keysPressed: '/^\nnnnkklvjjhx',
Expand All @@ -409,21 +409,21 @@ suite('Record and navigate jumps', () => {
});

suite('Can shift jump lines down after inserting a line', () => {
// TODO: this fails on Windows
// TODO(#4844): this fails on Windows
newJumpTestSkipOnWindows({
start: ['|start', 'a1', 'a2', 'a3', 'a4', 'a5', 'end'],
keysPressed: '/^\nnnnkkoINSERTED<Esc>0',
end: ['start', 'a1', 'a2', '|INSERTED', 'a3', 'a4', 'a5', 'end'],
jumps: ['start', 'a1', 'a2', 'a3'],
});
// TODO: this fails on Windows
// TODO(#4844): this fails on Windows
newJumpTestSkipOnWindows({
start: ['|start', 'a1', 'a2', 'a3', 'a4', 'a5', 'end'],
keysPressed: '/^\nnnnkoINSERTED<Esc>0',
end: ['start', 'a1', 'a2', 'a3', '|INSERTED', 'a4', 'a5', 'end'],
jumps: ['start', 'a1', 'a2', 'a3'],
});
// TODO: this fails on Windows
// TODO(#4844): this fails on Windows
newJumpTestSkipOnWindows({
start: ['|start', 'a1', 'a2', 'a3', 'a4', 'a5', 'end'],
keysPressed: '/^\nnnnkOINSERTED<Esc>0',
Expand All @@ -448,7 +448,7 @@ suite('Record and navigate jumps', () => {
});

suite('Can track jumps from macros', () => {
// TODO: this fails on Windows
// TODO(#4844): this fails on Windows
newJumpTestSkipOnWindows({
start: ['|start', 'a1', 'a2', 'a3', 'a4', 'a5', 'a6', 'a7', 'a8', 'a9', 'end'],
keysPressed: 'qq/^\nnq@q@q<C-o><C-o>',
Expand Down
16 changes: 8 additions & 8 deletions test/mode/modeNormal.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -1674,7 +1674,7 @@ suite('Mode Normal', () => {
],
});

// TODO: this fails on Windows
// TODO(#4844): this fails on Windows
newTestSkip(
{
title: 'gq work correctly with cursor in the middle of a line',
Expand All @@ -1699,7 +1699,7 @@ suite('Mode Normal', () => {
end: ['|abc', '', '', '', 'def'],
});

// TODO: this fails on Windows
// TODO(#4844): this fails on Windows
newTestSkip(
{
title: 'gq handles single-line comments',
Expand All @@ -1710,7 +1710,7 @@ suite('Mode Normal', () => {
process.platform === 'win32'
);

// TODO: this fails on Windows
// TODO(#4844): this fails on Windows
newTestSkip(
{
title: 'gq handles multiline comments',
Expand All @@ -1721,7 +1721,7 @@ suite('Mode Normal', () => {
process.platform === 'win32'
);

// TODO: this fails on Windows
// TODO(#4844): this fails on Windows
newTestSkip(
{
title: 'gq handles multiline comments with inner and final on same line',
Expand All @@ -1732,7 +1732,7 @@ suite('Mode Normal', () => {
process.platform === 'win32'
);

// TODO: this fails on Windows
// TODO(#4844): this fails on Windows
newTestSkip(
{
title: 'gq handles multiline comments with content on start line',
Expand All @@ -1757,7 +1757,7 @@ suite('Mode Normal', () => {
end: ['|/* abc', ' *', ' *', ' * def */'],
});

// TODO: this fails on Windows
// TODO(#4844): this fails on Windows
newTestSkip(
{
title: 'gq does not merge adjacent multiline comments',
Expand All @@ -1768,7 +1768,7 @@ suite('Mode Normal', () => {
process.platform === 'win32'
);

// TODO: this fails on Windows
// TODO(#4844): this fails on Windows
newTestSkip(
{
title: 'gq does not merge adjacent multiline comments',
Expand All @@ -1779,7 +1779,7 @@ suite('Mode Normal', () => {
process.platform === 'win32'
);

// TODO: this fails on Windows
// TODO(#4844): this fails on Windows
newTestSkip(
{
title: 'gq leaves alone whitespace within a line',
Expand Down
2 changes: 1 addition & 1 deletion test/mode/normalModeTests/motions.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -936,7 +936,7 @@ suite('Motions in Normal Mode', () => {
end: ['ab|c', 'def', 'ghi'],
});

// TODO: this fails on Windows due to \r\n
// TODO(#4844): this fails on Windows due to \r\n
newTestSkip(
{
title: '`[count]go` goes to offset <count>, newlines disregarded',
Expand Down
2 changes: 1 addition & 1 deletion test/operator/filter.test.ts
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
import { newTest } from '../testSimplifier';
import { cleanUpWorkspace, setupWorkspace } from '../testUtils';

// TODO: this fails on Windows
// TODO(#4844): this fails on Windows
suite('filter operator', () => {
if (process.platform === 'win32') {
return;
Expand Down

0 comments on commit 50c7551

Please sign in to comment.