corrections

Author: Kartik Raj

src/client/providers/symbolProvider.ts

@@ -147,23 +147,23 @@ export class JediSymbolProvider implements DocumentSymbolProvider {
 
     // This does not appear to be used anywhere currently...
     // tslint:disable-next-line:no-unused-variable
-    private provideDocumentSymbolsUnthrottled(document: TextDocument, token: CancellationToken): Thenable<SymbolInformation[]> {
-        const filename = document.fileName;
-
-        const cmd: proxy.ICommand<proxy.ISymbolResult> = {
-            command: proxy.CommandType.Symbols,
-            fileName: filename,
-            columnIndex: 0,
-            lineIndex: 0
-        };
-
-        if (document.isDirty) {
-            cmd.source = document.getText();
-        }
-
-        return this.jediFactory.getJediProxyHandler<proxy.ISymbolResult>(document.uri).sendCommandNonCancellableCommand(cmd, token)
-            .then(data => this.parseData(document, data));
-    }
+    // private provideDocumentSymbolsUnthrottled(document: TextDocument, token: CancellationToken): Thenable<SymbolInformation[]> {
+    //     const filename = document.fileName;
+
+    //     const cmd: proxy.ICommand<proxy.ISymbolResult> = {
+    //         command: proxy.CommandType.Symbols,
+    //         fileName: filename,
+    //         columnIndex: 0,
+    //         lineIndex: 0
+    //     };
+
+    //     if (document.isDirty) {
+    //         cmd.source = document.getText();
+    //     }
+
+    //     return this.jediFactory.getJediProxyHandler<proxy.ISymbolResult>(document.uri).sendCommandNonCancellableCommand(cmd, token)
+    //         .then(data => this.parseData(document, data));
+    // }
 
     private parseData(document: TextDocument, data?: proxy.ISymbolResult): SymbolInformation[] {
         if (data) {

src/client/unittests/common/xUnitParser.ts

@@ -57,7 +57,7 @@ export function updateResultsFromXmlLogFile(tests: Tests, outputXmlFile: string,
             }
             // tslint:disable-next-line:no-require-imports
             const xml2js = require('xml2js');
-            xml2js.parseString(data, (error, parserResult) => {
+            xml2js.parseString(data, (error: Error, parserResult: { testsuite: TestSuiteResult }) => {
                 if (error) {
                     return reject(error);
                 }

src/client/unittests/display/main.ts

@@ -17,7 +17,7 @@ export class TestResultDisplay implements ITestResultDisplay {
     private statusBar: StatusBarItem;
     private discoverCounter = 0;
     private ticker = ['|', '/', '-', '|', '/', '-', '\\'];
-    private progressTimeout;
+    private progressTimeout: NodeJS.Timer | null = null;
     private _enabled: boolean = false;
     private progressPrefix!: string;
     private readonly didChange = new EventEmitter<void>();

src/client/unittests/pytest/services/testMessageService.ts

@@ -25,7 +25,7 @@ export class TestMessageService implements ITestMessageService {
      * @param testResults Details about all known tests.
      */
     public async getFilteredTestMessages(rootDirectory: string, testResults: Tests): Promise<IPythonUnitTestMessage[]> {
-        const testFuncs: FlattenedTestFunction[] = testResults.testFunctions.reduce<FlattenedTestFunction[]>((filtered, test) => {
+        const testFuncs = testResults.testFunctions.reduce<FlattenedTestFunction[]>((filtered, test) => {
             if (test.testFunction.passed !== undefined || test.testFunction.status === TestStatus.Skipped) {
                 filtered.push(test);
             }

src/client/unittests/types.ts

@@ -130,7 +130,7 @@ export interface IPythonUnitTestMessage {
     code: string | undefined;
     message?: string;
     severity: PythonUnitTestMessageSeverity;
-    provider: string;
+    provider: string | undefined;
     traceback?: string;
     testTime: number;
     status: TestStatus;