Merge pull request #653 from fortran-lang/gnikit/issue652

test: add requirements.txt for Python deps

Author: gnikit

CHANGELOG.md

@@ -28,6 +28,8 @@ and this project adheres to [Semantic Versioning](http://semver.org/).
 
 ### Changed
 
+- Changed how Python packages are installed for unittesting for performance
+  ([#652](https://github.com/fortran-lang/vscode-fortran-support/issues/652))
 - Changed the linter to be asynchronous, should imprpove performance
   ([#623](https://github.com/fortran-lang/vscode-fortran-support/pull/623))
 - Changed native `SymbolProvider` to use non-deprecated constructor

test/formatting-provider.test.ts

@@ -3,7 +3,6 @@ import { FortranFormattingProvider } from '../src/features/formatting-provider';
 import * as vscode from 'vscode';
 import * as path from 'path';
 import { Logger } from '../src/services/logging';
-import { spawnSync } from 'child_process';
 
 suite('Formatting tests', () => {
   let doc: vscode.TextDocument;
@@ -13,8 +12,6 @@ suite('Formatting tests', () => {
   );
 
   suiteSetup(async function (): Promise<void> {
-    spawnSync('pip', ['install', '--user', '--upgrade', 'findent']);
-    spawnSync('pip', ['install', '--user', '--upgrade', 'fprettify']);
     doc = await vscode.workspace.openTextDocument(fileUri);
     await vscode.window.showTextDocument(doc);
     console.log('Open file: ' + fileUri.toString());

test/lsp-client.test.ts

@@ -1,7 +1,6 @@
 import * as vscode from 'vscode';
 import * as path from 'path';
 import { strictEqual } from 'assert';
-import { spawnSync } from 'child_process';
 import { Logger } from '../src/services/logging';
 import { FortlsClient } from '../src/lsp/client';
 import { delay } from '../src/lib/helper';
@@ -14,8 +13,6 @@ suite('Language Server integration tests', () => {
   );
 
   suiteSetup(async function (): Promise<void> {
-    console.log('Installing fortls Language Server');
-    spawnSync('pip', ['install', '--user', '--upgrade', 'fortls']);
     await server.activate();
   });
 

test/requirements.txt

@@ -0,0 +1,3 @@
+fortls
+findent
+fprettify

test/runTest.ts

@@ -1,9 +1,23 @@
 import * as path from 'path';
 
 import { runTests } from '@vscode/test-electron';
+import { spawnSync } from 'child_process';
 
 async function main() {
   try {
+    // Install PyPi dependencies
+    const results = spawnSync(`pip`, [
+      'install',
+      '-r',
+      path.resolve(__dirname, '../../test/requirements.txt'),
+      '--force',
+      '--upgrade',
+    ]);
+    if (results.status !== 0) {
+      console.error(results.stderr.toString());
+      process.exit(1);
+    }
+    console.log(results.stdout.toString().trim());
     // The folder containing the Extension Manifest package.json
     // Passed to `--extensionDevelopmentPath`
     const extensionDevelopmentPath = path.resolve(__dirname, '../../');