allow smoke test on windows

anthonykim1 · Oct 29, 2024 · 3c3bfc9 · 3c3bfc9
1 parent 7dd3f83
commit 3c3bfc9
Showing 1 changed file with 66 additions and 69 deletions.
diff --git a/src/test/smoke/smartSend.smoke.test.ts b/src/test/smoke/smartSend.smoke.test.ts
@@ -6,84 +6,81 @@ import { EXTENSION_ROOT_DIR_FOR_TESTS, IS_SMOKE_TEST } from '../constants';
 import { closeActiveWindows, initialize, initializeTest } from '../initialize';
 import { openFile, waitForCondition } from '../common';
 
-// TODO: This test is being flaky for windows, need to investigate why only fails on windows
-if (process.platform !== 'win32') {
-    suite('Smoke Test: Run Smart Selection and Advance Cursor', async () => {
-        suiteSetup(async function () {
-            if (!IS_SMOKE_TEST) {
-                return this.skip();
-            }
-            await initialize();
-            return undefined;
-        });
+suite('Smoke Test: Run Smart Selection and Advance Cursor', async () => {
+    suiteSetup(async function () {
+        if (!IS_SMOKE_TEST) {
+            return this.skip();
+        }
+        await initialize();
+        return undefined;
+    });
 
-        setup(initializeTest);
-        suiteTeardown(closeActiveWindows);
-        teardown(closeActiveWindows);
+    setup(initializeTest);
+    suiteTeardown(closeActiveWindows);
+    teardown(closeActiveWindows);
 
-        test('Smart Send', async () => {
-            const configuration = vscode.workspace.getConfiguration('python');
-            await configuration.update('REPL.sendToNativeREPL', false, vscode.ConfigurationTarget.Global);
+    test('Smart Send', async () => {
+        const configuration = vscode.workspace.getConfiguration('python');
+        await configuration.update('REPL.sendToNativeREPL', false, vscode.ConfigurationTarget.Global);
 
-            const file = path.join(
-                EXTENSION_ROOT_DIR_FOR_TESTS,
-                'src',
-                'testMultiRootWkspc',
-                'smokeTests',
-                'create_delete_file.py',
-            );
-            const outputFile = path.join(
-                EXTENSION_ROOT_DIR_FOR_TESTS,
-                'src',
-                'testMultiRootWkspc',
-                'smokeTests',
-                'smart_send_smoke.txt',
-            );
+        const file = path.join(
+            EXTENSION_ROOT_DIR_FOR_TESTS,
+            'src',
+            'testMultiRootWkspc',
+            'smokeTests',
+            'create_delete_file.py',
+        );
+        const outputFile = path.join(
+            EXTENSION_ROOT_DIR_FOR_TESTS,
+            'src',
+            'testMultiRootWkspc',
+            'smokeTests',
+            'smart_send_smoke.txt',
+        );
 
-            await fs.remove(outputFile);
+        await fs.remove(outputFile);
 
-            const textDocument = await openFile(file);
+        const textDocument = await openFile(file);
 
-            if (vscode.window.activeTextEditor) {
-                const myPos = new vscode.Position(0, 0);
-                vscode.window.activeTextEditor!.selections = [new vscode.Selection(myPos, myPos)];
-            }
-            await vscode.commands
-                .executeCommand<void>('python.execSelectionInTerminal', textDocument.uri)
-                .then(undefined, (err) => {
-                    assert.fail(`Something went wrong running the Python file in the terminal: ${err}`);
-                });
+        if (vscode.window.activeTextEditor) {
+            const myPos = new vscode.Position(0, 0);
+            vscode.window.activeTextEditor!.selections = [new vscode.Selection(myPos, myPos)];
+        }
+        await vscode.commands
+            .executeCommand<void>('python.execSelectionInTerminal', textDocument.uri)
+            .then(undefined, (err) => {
+                assert.fail(`Something went wrong running the Python file in the terminal: ${err}`);
+            });
 
-            const checkIfFileHasBeenCreated = () => fs.pathExists(outputFile);
-            await waitForCondition(checkIfFileHasBeenCreated, 10_000, `"${outputFile}" file not created`);
+        const checkIfFileHasBeenCreated = () => fs.pathExists(outputFile);
+        await waitForCondition(checkIfFileHasBeenCreated, 10_000, `"${outputFile}" file not created`);
 
-            await vscode.commands
-                .executeCommand<void>('python.execSelectionInTerminal', textDocument.uri)
-                .then(undefined, (err) => {
-                    assert.fail(`Something went wrong running the Python file in the terminal: ${err}`);
-                });
-            await vscode.commands
-                .executeCommand<void>('python.execSelectionInTerminal', textDocument.uri)
-                .then(undefined, (err) => {
-                    assert.fail(`Something went wrong running the Python file in the terminal: ${err}`);
-                });
+        await vscode.commands
+            .executeCommand<void>('python.execSelectionInTerminal', textDocument.uri)
+            .then(undefined, (err) => {
+                assert.fail(`Something went wrong running the Python file in the terminal: ${err}`);
+            });
+        await vscode.commands
+            .executeCommand<void>('python.execSelectionInTerminal', textDocument.uri)
+            .then(undefined, (err) => {
+                assert.fail(`Something went wrong running the Python file in the terminal: ${err}`);
+            });
 
-            async function wait() {
-                return new Promise<void>((resolve) => {
-                    setTimeout(() => {
-                        resolve();
-                    }, 5000);
-                });
-            }
+        async function wait() {
+            return new Promise<void>((resolve) => {
+                setTimeout(() => {
+                    resolve();
+                }, 5000);
+            });
+        }
 
-            await wait();
+        await wait();
 
-            const deletedFile = !(await fs.pathExists(outputFile));
-            if (deletedFile) {
-                assert.ok(true, `"${outputFile}" file has been deleted`);
-            } else {
-                assert.fail(`"${outputFile}" file still exists`);
-            }
-        });
+        const deletedFile = !(await fs.pathExists(outputFile));
+        if (deletedFile) {
+            assert.ok(true, `"${outputFile}" file has been deleted`);
+        } else {
+            assert.fail(`"${outputFile}" file still exists`);
+        }
     });
-}
+});