brighterscript 0.57.2 → 0.59.0

package/dist/files/BrsFile.spec.js

@@ -24,18 +24,17 @@ const creators_1 = require("../astUtils/creators");
 const fsExtra = require("fs-extra");
 const vscode_uri_1 = require("vscode-uri");
 const undent_1 = require("undent");
+const testHelpers_spec_2 = require("../testHelpers.spec");
 let sinon = sinonImport.createSandbox();
 describe('BrsFile', () => {
-    let tempDir = (0, util_1.standardizePath) `${process.cwd()}/.tmp`;
-    let rootDir = (0, util_1.standardizePath) `${tempDir}/rootDir`;
     let program;
-    let srcPath = (0, util_1.standardizePath) `${rootDir}/source/main.brs`;
+    let srcPath = (0, util_1.standardizePath) `${testHelpers_spec_2.rootDir}/source/main.brs`;
     let destPath = 'source/main.brs';
     let file;
-    let testTranspile = (0, testHelpers_spec_1.getTestTranspile)(() => [program, rootDir]);
+    let testTranspile = (0, testHelpers_spec_1.getTestTranspile)(() => [program, testHelpers_spec_2.rootDir]);
     beforeEach(() => {
-        fsExtra.emptyDirSync(tempDir);
-        program = new Program_1.Program({ rootDir: rootDir, sourceMap: true });
+        fsExtra.emptyDirSync(testHelpers_spec_2.tempDir);
+        program = new Program_1.Program({ rootDir: testHelpers_spec_2.rootDir, sourceMap: true });
         file = new BrsFile_1.BrsFile(srcPath, destPath, program);
     });
     afterEach(() => {
@@ -81,9 +80,9 @@ describe('BrsFile', () => {
     });
     it('sets needsTranspiled to true for .bs files', () => {
         //BrightScript
-        (0, chai_1.expect)(new BrsFile_1.BrsFile(`${rootDir}/source/main.brs`, 'source/main.brs', program).needsTranspiled).to.be.false;
+        (0, chai_1.expect)(new BrsFile_1.BrsFile(`${testHelpers_spec_2.rootDir}/source/main.brs`, 'source/main.brs', program).needsTranspiled).to.be.false;
         //BrighterScript
-        (0, chai_1.expect)(new BrsFile_1.BrsFile(`${rootDir}/source/main.bs`, 'source/main.bs', program).needsTranspiled).to.be.true;
+        (0, chai_1.expect)(new BrsFile_1.BrsFile(`${testHelpers_spec_2.rootDir}/source/main.bs`, 'source/main.bs', program).needsTranspiled).to.be.true;
     });
     it('computes new import statements after clearing parser references', () => {
         const file = program.setFile('source/main.bs', ``);
@@ -104,7 +103,7 @@ describe('BrsFile', () => {
     });
     describe('getPartialVariableName', () => {
         let entry = {
-            src: `${rootDir}/source/lib.brs`,
+            src: `${testHelpers_spec_2.rootDir}/source/lib.brs`,
             dest: `source/lib.brs`
         };
         it('creates proper tokens', () => {
@@ -141,7 +140,7 @@ describe('BrsFile', () => {
                     print "pkg:"
                 end sub
             `);
-            const result = program.getCompletions(`${rootDir}/source/main.brs`, vscode_languageserver_1.Position.create(2, 31));
+            const result = program.getCompletions(`${testHelpers_spec_2.rootDir}/source/main.brs`, vscode_languageserver_1.Position.create(2, 31));
             const names = result.map(x => x.label);
             (0, chai_1.expect)(names.sort()).to.eql([
                 'pkg:/source/main.brs'
@@ -153,7 +152,7 @@ describe('BrsFile', () => {
                     print "libpkg:"
                 end sub
             `);
-            const result = program.getCompletions(`${rootDir}/source/main.brs`, vscode_languageserver_1.Position.create(2, 31));
+            const result = program.getCompletions(`${testHelpers_spec_2.rootDir}/source/main.brs`, vscode_languageserver_1.Position.create(2, 31));
             const names = result.map(x => x.label);
             (0, chai_1.expect)(names.sort()).to.eql([
                 'libpkg:/source/main.brs'
@@ -165,7 +164,7 @@ describe('BrsFile', () => {
                     print \`pkg:\`
                 end sub
             `);
-            const result = program.getCompletions(`${rootDir}/source/main.brs`, vscode_languageserver_1.Position.create(2, 31));
+            const result = program.getCompletions(`${testHelpers_spec_2.rootDir}/source/main.brs`, vscode_languageserver_1.Position.create(2, 31));
             const names = result.map(x => x.label);
             (0, chai_1.expect)(names.sort()).to.eql([
                 'pkg:/source/main.brs'
@@ -182,7 +181,7 @@ describe('BrsFile', () => {
                 sub SayHello()
                 end sub
             `);
-            let result = program.getCompletions(`${rootDir}/source/main.brs`, vscode_languageserver_1.Position.create(3, 23));
+            let result = program.getCompletions(`${testHelpers_spec_2.rootDir}/source/main.brs`, vscode_languageserver_1.Position.create(3, 23));
             let names = result.map(x => x.label);
             (0, chai_1.expect)(names).to.includes('Main');
             (0, chai_1.expect)(names).to.includes('SayHello');
@@ -231,7 +230,7 @@ describe('BrsFile', () => {
                     class Person
                     end class
                 `);
-                const result = program.getCompletions(`${rootDir}/source/main.bs`, vscode_languageserver_1.Position.create(2, 24)).map(x => x.label);
+                const result = program.getCompletions(`${testHelpers_spec_2.rootDir}/source/main.bs`, vscode_languageserver_1.Position.create(2, 24)).map(x => x.label);
                 (0, chai_1.expect)(result).includes('main');
                 (0, chai_1.expect)(result).includes('foo');
                 (0, chai_1.expect)(result).includes('Person');
@@ -249,22 +248,22 @@ describe('BrsFile', () => {
                         foo.bar.
                     end sub
                 `);
-                let result = program.getCompletions(`${rootDir}/source/main.bs`, vscode_languageserver_1.Position.create(8, 30));
+                let result = program.getCompletions(`${testHelpers_spec_2.rootDir}/source/main.bs`, vscode_languageserver_1.Position.create(8, 30));
                 let names = result.map(x => x.label);
                 (0, chai_1.expect)(names).to.includes('bar');
-                result = program.getCompletions(`${rootDir}/source/main.bs`, vscode_languageserver_1.Position.create(9, 32));
+                result = program.getCompletions(`${testHelpers_spec_2.rootDir}/source/main.bs`, vscode_languageserver_1.Position.create(9, 32));
                 names = result.map(x => x.label);
                 (0, chai_1.expect)(names).to.includes('sayHello');
             });
         });
         it('always includes `m`', () => {
             //eslint-disable-next-line @typescript-eslint/no-floating-promises
-            program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+            program.setFile('source/main.brs', `
                 sub Main()
 
                 end sub
             `);
-            let result = program.getCompletions(`${rootDir}/source/main.brs`, vscode_languageserver_1.Position.create(2, 23));
+            let result = program.getCompletions(`${testHelpers_spec_2.rootDir}/source/main.brs`, vscode_languageserver_1.Position.create(2, 23));
             let names = result.map(x => x.label);
             (0, chai_1.expect)(names).to.contain('m');
         });
@@ -272,25 +271,25 @@ describe('BrsFile', () => {
             //add a single character to the file, and get completions after it
             program.setFile('source/main.brs', `i`);
             (0, chai_1.expect)(() => {
-                program.getCompletions(`${rootDir}/source/main.brs`, vscode_languageserver_1.Position.create(0, 1)).map(x => x.label);
+                program.getCompletions(`${testHelpers_spec_2.rootDir}/source/main.brs`, vscode_languageserver_1.Position.create(0, 1)).map(x => x.label);
             }).not.to.throw;
         });
         it('includes all keywords`', () => {
             //eslint-disable-next-line @typescript-eslint/no-floating-promises
-            program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+            program.setFile('source/main.brs', `
                 sub Main()
 
                 end sub
             `);
             let keywords = Object.keys(TokenKind_1.Keywords).filter(x => !x.includes(' '));
             //inside the function
-            let result = program.getCompletions(`${rootDir}/source/main.brs`, vscode_languageserver_1.Position.create(2, 23));
+            let result = program.getCompletions(`${testHelpers_spec_2.rootDir}/source/main.brs`, vscode_languageserver_1.Position.create(2, 23));
             let names = result.map(x => x.label);
             for (let keyword of keywords) {
                 (0, chai_1.expect)(names).to.include(keyword);
             }
             //outside the function
-            result = program.getCompletions(`${rootDir}/source/main.brs`, vscode_languageserver_1.Position.create(4, 8));
+            result = program.getCompletions(`${testHelpers_spec_2.rootDir}/source/main.brs`, vscode_languageserver_1.Position.create(4, 8));
             names = result.map(x => x.label);
             for (let keyword of keywords) {
                 (0, chai_1.expect)(names).to.include(keyword);
@@ -298,25 +297,25 @@ describe('BrsFile', () => {
         });
         it('does not provide completions within a comment', () => {
             //eslint-disable-next-line @typescript-eslint/no-floating-promises
-            program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+            program.setFile('source/main.brs', `
                 sub Main()
                     'some comment
                 end sub
             `);
             //inside the function
-            let result = program.getCompletions(`${rootDir}/source/main.brs`, vscode_languageserver_1.Position.create(2, 33));
+            let result = program.getCompletions(`${testHelpers_spec_2.rootDir}/source/main.brs`, vscode_languageserver_1.Position.create(2, 33));
             (0, chai_1.expect)(result).to.be.lengthOf(0);
         });
         it('does not provide duplicate entries for variables', () => {
             //eslint-disable-next-line @typescript-eslint/no-floating-promises
-            program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+            program.setFile('source/main.brs', `
                 sub Main()
                     name = "bob"
                     age = 12
                     name = "john"
                 end sub
             `);
-            let result = program.getCompletions(`${rootDir}/source/main.brs`, vscode_languageserver_1.Position.create(3, 23));
+            let result = program.getCompletions(`${testHelpers_spec_2.rootDir}/source/main.brs`, vscode_languageserver_1.Position.create(3, 23));
             let count = result.reduce((total, x) => {
                 return x.label === 'name' ? total + 1 : total;
             }, 0);
@@ -324,34 +323,34 @@ describe('BrsFile', () => {
         });
         it('does not include `as` and `string` text options when used in function params', () => {
             //eslint-disable-next-line @typescript-eslint/no-floating-promises
-            program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+            program.setFile('source/main.brs', `
                 sub Main(name as string)
 
                 end sub
             `);
-            let result = program.getCompletions(`${rootDir}/source/main.brs`, vscode_languageserver_1.Position.create(2, 23));
+            let result = program.getCompletions(`${testHelpers_spec_2.rootDir}/source/main.brs`, vscode_languageserver_1.Position.create(2, 23));
             (0, chai_1.expect)(result.filter(x => x.kind === vscode_languageserver_1.CompletionItemKind.Text)).not.to.contain('as');
             (0, chai_1.expect)(result.filter(x => x.kind === vscode_languageserver_1.CompletionItemKind.Text)).not.to.contain('string');
         });
         it('does not provide intellisense results when inside a comment', () => {
             //eslint-disable-next-line @typescript-eslint/no-floating-promises
-            program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+            program.setFile('source/main.brs', `
                 sub Main(name as string)
                     'this is a comment
                 end sub
             `);
-            let results = program.getCompletions(`${rootDir}/source/main.brs`, vscode_languageserver_1.Position.create(2, 30));
+            let results = program.getCompletions(`${testHelpers_spec_2.rootDir}/source/main.brs`, vscode_languageserver_1.Position.create(2, 30));
             (0, chai_1.expect)(results).to.be.empty;
         });
         it('does provide intellisence for labels only after a goto keyword', () => {
             var _a;
             //eslint-disable-next-line @typescript-eslint/no-floating-promises
-            program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+            program.setFile('source/main.brs', `
                 sub Main(name as string)
                     something:
                     goto \nend sub
             `);
-            let results = program.getCompletions(`${rootDir}/source/main.brs`, vscode_languageserver_1.Position.create(3, 25));
+            let results = program.getCompletions(`${testHelpers_spec_2.rootDir}/source/main.brs`, vscode_languageserver_1.Position.create(3, 25));
             (0, chai_1.expect)(results.length).to.equal(1);
             (0, chai_1.expect)((_a = results[0]) === null || _a === void 0 ? void 0 : _a.label).to.equal('something');
         });
@@ -389,7 +388,7 @@ describe('BrsFile', () => {
                 (0, testHelpers_spec_1.expectZeroDiagnostics)(program);
             });
             it('works for all', () => {
-                let file = program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+                let file = program.setFile({ src: `${testHelpers_spec_2.rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
                     sub Main()
                         'bs:disable-next-line
                         name = "bob
@@ -406,7 +405,7 @@ describe('BrsFile', () => {
                 (0, testHelpers_spec_1.expectZeroDiagnostics)(program);
             });
             it('works for specific codes', () => {
-                let file = program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+                let file = program.setFile({ src: `${testHelpers_spec_2.rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
                     sub Main()
                         'bs:disable-next-line: 1083, 1001
                         name = "bob
@@ -422,7 +421,7 @@ describe('BrsFile', () => {
                 (0, testHelpers_spec_1.expectZeroDiagnostics)(program);
             });
             it('recognizes non-numeric codes', () => {
-                let file = program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+                let file = program.setFile({ src: `${testHelpers_spec_2.rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
                     sub Main()
                         'bs:disable-next-line: LINT9999
                         name = "bob
@@ -448,7 +447,7 @@ describe('BrsFile', () => {
                 (0, testHelpers_spec_1.expectZeroDiagnostics)(scope);
             });
             it('adds diagnostics for unknown numeric diagnostic codes', () => {
-                program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+                program.setFile('source/main.brs', `
                     sub main()
                         print "hi" 'bs:disable-line: 123456 999999   aaaab
                     end sub
@@ -459,7 +458,7 @@ describe('BrsFile', () => {
         });
         describe('bs:disable-line', () => {
             it('works for all', () => {
-                let file = program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+                let file = program.setFile({ src: `${testHelpers_spec_2.rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
                     sub Main()
                         z::;;%%%%%% 'bs:disable-line
                     end sub
@@ -475,7 +474,7 @@ describe('BrsFile', () => {
                 (0, testHelpers_spec_1.expectZeroDiagnostics)(program);
             });
             it('works for specific codes', () => {
-                program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+                program.setFile('source/main.brs', `
                     sub main()
                         'should not have any errors
                         DoSomething(1) 'bs:disable-line:1002
@@ -494,7 +493,7 @@ describe('BrsFile', () => {
                 //the current version of BRS causes parse errors after the `parse` keyword, showing error in comments
                 //the program should ignore all diagnostics found in brs:* comment lines EXCEPT
                 //for the diagnostics about using unknown error codes
-                program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+                program.setFile('source/main.brs', `
                     sub main()
                         stop 'bs:disable-line
                         print "need a valid line to fix stop error"
@@ -567,7 +566,7 @@ describe('BrsFile', () => {
             testParseMode('source/main.spec.bs', Parser_1.ParseMode.BrighterScript);
         });
         it('supports labels and goto statements', () => {
-            let file = program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+            let file = program.setFile('source/main.brs', `
                 sub Main()
                     'multiple goto statements on one line
                     goto myLabel : goto myLabel
@@ -577,7 +576,7 @@ describe('BrsFile', () => {
             (0, testHelpers_spec_1.expectZeroDiagnostics)(file);
         });
         it('supports empty print statements', () => {
-            let file = program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+            let file = program.setFile('source/main.brs', `
                 sub main()
                    print
                 end sub
@@ -586,7 +585,7 @@ describe('BrsFile', () => {
         });
         describe('conditional compile', () => {
             it('supports case-insensitive bs_const variables', () => {
-                fsExtra.outputFileSync(`${rootDir}/manifest`, (0, undent_1.default) `
+                fsExtra.outputFileSync(`${testHelpers_spec_2.rootDir}/manifest`, (0, undent_1.default) `
                     bs_const=SomeKey=true
                 `);
                 program.setFile('source/main.brs', `
@@ -606,7 +605,7 @@ describe('BrsFile', () => {
                 (0, testHelpers_spec_1.expectZeroDiagnostics)(program);
             });
             it('works for upper case keywords', () => {
-                let file = program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+                let file = program.setFile('source/main.brs', `
                     sub main()
                         #CONST someFlag = true
                         #IF someFlag
@@ -621,7 +620,7 @@ describe('BrsFile', () => {
                 (0, testHelpers_spec_1.expectZeroDiagnostics)(file);
             });
             it('supports single-word #elseif and #endif', () => {
-                let file = program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+                let file = program.setFile('source/main.brs', `
                     sub main()
                         #const someFlag = true
                         #if someFlag
@@ -634,7 +633,7 @@ describe('BrsFile', () => {
                 (0, testHelpers_spec_1.expectZeroDiagnostics)(file);
             });
             it('supports multi-word #else if and #end if', () => {
-                let file = program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+                let file = program.setFile('source/main.brs', `
                     sub main()
                         #const someFlag = true
                         #if someFlag
@@ -647,7 +646,7 @@ describe('BrsFile', () => {
                 (0, testHelpers_spec_1.expectZeroDiagnostics)(file);
             });
             it('does not choke on invalid code inside a false conditional compile', () => {
-                let file = program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+                let file = program.setFile('source/main.brs', `
                     sub main()
                         #if false
                             non-commented code here should not cause parse errors
@@ -657,7 +656,7 @@ describe('BrsFile', () => {
                 (0, testHelpers_spec_1.expectZeroDiagnostics)(file);
             });
             it('detects syntax error in #if', () => {
-                let file = program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+                let file = program.setFile('source/main.brs', `
                     sub main()
                         #if true1
                             print "true"
@@ -669,7 +668,7 @@ describe('BrsFile', () => {
                 ]);
             });
             it('detects syntax error in #const', () => {
-                let file = program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+                let file = program.setFile('source/main.brs', `
                     sub main()
                         #if %
                             print "true"
@@ -682,7 +681,7 @@ describe('BrsFile', () => {
                 ]);
             });
             it('detects #const name using reserved word', () => {
-                let file = program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+                let file = program.setFile('source/main.brs', `
                     sub main()
                         #const function = true
                     end sub
@@ -693,7 +692,7 @@ describe('BrsFile', () => {
                 ]);
             });
             it('detects syntax error in #const', () => {
-                let file = program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+                let file = program.setFile('source/main.brs', `
                     sub main()
                         #const someConst = 123
                     end sub
@@ -704,7 +703,7 @@ describe('BrsFile', () => {
             });
         });
         it('supports stop statement', () => {
-            let file = program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+            let file = program.setFile('source/main.brs', `
                 sub main()
                    stop
                 end sub
@@ -712,7 +711,7 @@ describe('BrsFile', () => {
             (0, testHelpers_spec_1.expectZeroDiagnostics)(file);
         });
         it('supports single-line if statements', () => {
-            let file = program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+            let file = program.setFile('source/main.brs', `
                 sub main()
                     if 1 < 2: return true: end if
                     if 1 < 2: return true
@@ -1186,6 +1185,29 @@ describe('BrsFile', () => {
             (0, chai_1.expect)(file.functionCalls[1].range).to.eql(vscode_languageserver_1.Range.create(5, 20, 5, 25));
             (0, chai_1.expect)(file.functionCalls[1].nameRange).to.eql(vscode_languageserver_1.Range.create(5, 20, 5, 23));
         });
+        it('finds function calls that are unfinished', () => {
+            let file = new BrsFile_1.BrsFile('absolute_path/file.brs', 'relative_path/file.brs', program);
+            file.parse(`
+                function DoA()
+                    DoB("a"
+                end function
+                function DoB(a as string)
+                    DoC(
+                end function
+            `);
+            (0, testHelpers_spec_1.expectDiagnostics)(file.parser.diagnostics, [
+                DiagnosticMessages_1.DiagnosticMessages.expectedRightParenAfterFunctionCallArguments(),
+                DiagnosticMessages_1.DiagnosticMessages.expectedNewlineOrColon(),
+                DiagnosticMessages_1.DiagnosticMessages.unexpectedToken('end function'),
+                DiagnosticMessages_1.DiagnosticMessages.expectedRightParenAfterFunctionCallArguments(),
+                DiagnosticMessages_1.DiagnosticMessages.expectedNewlineOrColon()
+            ]);
+            (0, chai_1.expect)(file.functionCalls.length).to.equal(2);
+            (0, chai_1.expect)(file.functionCalls[0].range).to.eql(vscode_languageserver_1.Range.create(2, 20, 2, 27));
+            (0, chai_1.expect)(file.functionCalls[0].nameRange).to.eql(vscode_languageserver_1.Range.create(2, 20, 2, 23));
+            (0, chai_1.expect)(file.functionCalls[1].range).to.eql(vscode_languageserver_1.Range.create(5, 20, 5, 24));
+            (0, chai_1.expect)(file.functionCalls[1].nameRange).to.eql(vscode_languageserver_1.Range.create(5, 20, 5, 23));
+        });
         it('sanitizes brs errors', () => {
             let file = new BrsFile_1.BrsFile('absolute_path/file.brs', 'relative_path/file.brs', program);
             file.parse(`
@@ -1526,7 +1548,7 @@ describe('BrsFile', () => {
         });
     });
     it('handles mixed case `then` partions of conditionals', () => {
-        let mainFile = program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+        let mainFile = program.setFile('source/main.brs', `
             sub Main()
                 if true then
                     print "works"
@@ -1534,7 +1556,7 @@ describe('BrsFile', () => {
             end sub
         `);
         (0, chai_1.expect)(mainFile.getDiagnostics()).to.be.lengthOf(0);
-        mainFile = program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+        mainFile = program.setFile('source/main.brs', `
             sub Main()
                 if true Then
                     print "works"
@@ -1542,7 +1564,7 @@ describe('BrsFile', () => {
             end sub
         `);
         (0, chai_1.expect)(mainFile.getDiagnostics()).to.be.lengthOf(0);
-        mainFile = program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+        mainFile = program.setFile('source/main.brs', `
             sub Main()
                 if true THEN
                     print "works"
@@ -1553,7 +1575,7 @@ describe('BrsFile', () => {
     });
     describe('getHover', () => {
         it('works for param types', () => {
-            let file = program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+            let file = program.setFile('source/main.brs', `
                 sub DoSomething(name as string)
                     name = 1
                     sayMyName = function(name as string)
@@ -1571,7 +1593,7 @@ describe('BrsFile', () => {
         });
         //ignore this for now...it's not a huge deal
         it('does not match on keywords or data types', () => {
-            let file = program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+            let file = program.setFile('source/main.brs', `
                 sub Main(name as string)
                 end sub
                 sub as()
@@ -1583,7 +1605,7 @@ describe('BrsFile', () => {
             (0, chai_1.expect)(program.getHover(file.srcPath, vscode_languageserver_1.Position.create(1, 36))).to.be.empty;
         });
         it('finds declared function', () => {
-            let file = program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+            let file = program.setFile('source/main.brs', `
                 function Main(count = 1)
                     firstName = "bob"
                     age = 21
@@ -1600,7 +1622,7 @@ describe('BrsFile', () => {
             ].join('\n'));
         });
         it('finds declared namespace function', () => {
-            let file = program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+            let file = program.setFile('source/main.brs', `
             namespace mySpace
                 function Main(count = 1)
                     firstName = "bob"
@@ -1619,7 +1641,7 @@ describe('BrsFile', () => {
             ].join('\n'));
         });
         it('finds variable function hover in same scope', () => {
-            let file = program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+            let file = program.setFile('source/main.brs', `
                 sub Main()
                     sayMyName = sub(name as string)
                     end sub
@@ -1661,7 +1683,7 @@ describe('BrsFile', () => {
             ].join('\n'));
         });
         it('finds function hover in file scope', () => {
-            let file = program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+            let file = program.setFile('source/main.brs', `
                 sub Main()
                     sayMyName()
                 end sub
@@ -1679,7 +1701,7 @@ describe('BrsFile', () => {
             ].join('\n'));
         });
         it('finds namespace function hover in file scope', () => {
-            let file = program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+            let file = program.setFile('source/main.brs', `
                 namespace mySpace
                 sub Main()
                     sayMyName()
@@ -1703,12 +1725,12 @@ describe('BrsFile', () => {
             program = new Program_1.Program({
                 rootDir: rootDir
             });
-            let mainFile = program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+            let mainFile = program.setFile('source/main.brs', `
                 sub Main()
                     sayMyName()
                 end sub
             `);
-            program.setFile({ src: `${rootDir}/source/lib.brs`, dest: 'source/lib.brs' }, `
+            program.setFile('source/lib.brs', `
                 sub sayMyName(name as string)
 
                 end sub
@@ -1727,12 +1749,12 @@ describe('BrsFile', () => {
             program = new Program_1.Program({
                 rootDir: rootDir
             });
-            let mainFile = program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+            let mainFile = program.setFile('source/main.brs', `
                 sub Main()
                     mySpace.sayMyName()
                 end sub
             `);
-            program.setFile({ src: `${rootDir}/source/lib.brs`, dest: 'source/lib.brs' }, `
+            program.setFile('source/lib.brs', `
                 namespace mySpace
                     sub sayMyName(name as string)
                     end sub
@@ -1790,7 +1812,7 @@ describe('BrsFile', () => {
                 `);
         });
         it('handles mixed case `then` partions of conditionals', () => {
-            let mainFile = program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+            let mainFile = program.setFile('source/main.brs', `
                 sub Main()
                     if true then
                         print "works"
@@ -1798,7 +1820,7 @@ describe('BrsFile', () => {
                 end sub
             `);
             (0, testHelpers_spec_1.expectZeroDiagnostics)(mainFile);
-            mainFile = program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+            mainFile = program.setFile('source/main.brs', `
                 sub Main()
                     if true Then
                         print "works"
@@ -1806,7 +1828,7 @@ describe('BrsFile', () => {
                 end sub
             `);
             (0, testHelpers_spec_1.expectZeroDiagnostics)(mainFile);
-            mainFile = program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+            mainFile = program.setFile('source/main.brs', `
                 sub Main()
                     if true THEN
                         print "works"
@@ -1817,7 +1839,7 @@ describe('BrsFile', () => {
         });
     });
     it('does not throw when encountering incomplete import statement', () => {
-        program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+        program.setFile('source/main.brs', `
             import
             sub main()
             end sub
@@ -2241,7 +2263,7 @@ describe('BrsFile', () => {
                 return consumer.generatedPositionFor({
                     line: 3,
                     column: 0,
-                    source: (0, util_1.standardizePath) `${rootDir}/source/main.bs`,
+                    source: (0, util_1.standardizePath) `${testHelpers_spec_2.rootDir}/source/main.bs`,
                     bias: source_map_1.SourceMapConsumer.LEAST_UPPER_BOUND
                 });
             });
@@ -2528,12 +2550,14 @@ describe('BrsFile', () => {
             it('sets invalid on empty callfunc', () => {
                 testTranspile(`
                     sub main()
+                        node = invalid
                         node@.doSomething()
                         m.top.node@.doSomething()
                         m.top.node@.doSomething(1)
                     end sub
                 `, `
                     sub main()
+                        node = invalid
                         node.callfunc("doSomething", invalid)
                         m.top.node.callfunc("doSomething", invalid)
                         m.top.node.callfunc("doSomething", 1)
@@ -2543,10 +2567,12 @@ describe('BrsFile', () => {
             it('includes original arguments', () => {
                 testTranspile(`
                     sub main()
+                        node = invalid
                         node@.doSomething(1, true, m.top.someVal)
                     end sub
                 `, `
                     sub main()
+                        node = invalid
                         node.callfunc("doSomething", 1, true, m.top.someVal)
                     end sub
                 `);
@@ -2563,7 +2589,7 @@ describe('BrsFile', () => {
                 name: 'transform callback',
                 afterFileParse: onParsed
             });
-            file = program.setFile({ src: `absolute_path/file${ext}`, dest: `relative_path/file${ext}` }, `
+            file = program.setFile(`source/file${ext}`, `
                 sub Sum()
                     print "hello world"
                 end sub
@@ -2584,7 +2610,7 @@ describe('BrsFile', () => {
     });
     describe('typedefKey', () => {
         it('works for .brs files', () => {
-            (0, chai_1.expect)((0, util_1.standardizePath)((program.setFile('source/main.brs', '')).typedefKey)).to.equal((0, util_1.standardizePath) `${rootDir.toLowerCase()}/source/main.d.bs`);
+            (0, chai_1.expect)((0, util_1.standardizePath)((program.setFile('source/main.brs', '')).typedefKey)).to.equal((0, util_1.standardizePath) `${testHelpers_spec_2.rootDir.toLowerCase()}/source/main.d.bs`);
         });
         it('returns undefined for files that should not have a typedef', () => {
             (0, chai_1.expect)((program.setFile('source/main.bs', '')).typedefKey).to.be.undefined;
@@ -2635,14 +2661,14 @@ describe('BrsFile', () => {
             `);
             (0, chai_1.expect)(file.hasTypedef).to.be.true;
             (0, chai_1.expect)(file.typedefFile).to.exist;
-            program.removeFile((0, util_1.standardizePath) `${rootDir}/source/main.d.bs`);
+            program.removeFile((0, util_1.standardizePath) `${testHelpers_spec_2.rootDir}/source/main.d.bs`);
             (0, chai_1.expect)(file.hasTypedef).to.be.false;
             (0, chai_1.expect)(file.typedefFile).not.to.exist;
         });
     });
     describe('typedef', () => {
         it('sets typedef path properly', () => {
-            (0, chai_1.expect)((program.setFile('source/main1.brs', '')).typedefKey).to.equal((0, util_1.standardizePath) `${rootDir}/source/main1.d.bs`.toLowerCase());
+            (0, chai_1.expect)((program.setFile('source/main1.brs', '')).typedefKey).to.equal((0, util_1.standardizePath) `${testHelpers_spec_2.rootDir}/source/main1.d.bs`.toLowerCase());
             (0, chai_1.expect)((program.setFile('source/main2.d.bs', '')).typedefKey).to.equal(undefined);
             (0, chai_1.expect)((program.setFile('source/main3.bs', '')).typedefKey).to.equal(undefined);
             //works for dest with `.brs` extension
@@ -2672,6 +2698,7 @@ describe('BrsFile', () => {
     describe('getTypedef', () => {
         function testTypedef(original, expected) {
             let file = program.setFile('source/main.brs', original);
+            program.validate();
             (0, chai_1.expect)(file.getTypedef().trimEnd()).to.eql(expected);
         }
         it('includes namespace on extend class names', () => {
@@ -2988,7 +3015,7 @@ describe('BrsFile', () => {
         let idx = 1;
         beforeEach(() => {
             pluginFileName = `plugin-${idx++}.js`;
-            fsExtra.outputFileSync((0, util_1.standardizePath) `${tempDir}/plugins/${pluginFileName}`, `
+            fsExtra.outputFileSync((0, util_1.standardizePath) `${testHelpers_spec_2.tempDir}/plugins/${pluginFileName}`, `
                 function plugin() {
                     return {
                         name: 'lower-file-name',
@@ -3001,14 +3028,14 @@ describe('BrsFile', () => {
             `);
         });
         it('can load an absolute plugin which receives callbacks', () => {
-            program.plugins = new PluginInterface_1.default(util_1.default.loadPlugins(tempDir, [
-                (0, util_1.standardizePath) `${tempDir}/plugins/${pluginFileName}`
+            program.plugins = new PluginInterface_1.default(util_1.default.loadPlugins(testHelpers_spec_2.tempDir, [
+                (0, util_1.standardizePath) `${testHelpers_spec_2.tempDir}/plugins/${pluginFileName}`
             ]), new Logger_1.Logger());
             const file = program.setFile('source/MAIN.brs', '');
             (0, chai_1.expect)(file._customProp).to.exist;
         });
         it('can load a relative plugin which receives callbacks', () => {
-            program.plugins = new PluginInterface_1.default(util_1.default.loadPlugins(tempDir, [
+            program.plugins = new PluginInterface_1.default(util_1.default.loadPlugins(testHelpers_spec_2.tempDir, [
                 `./plugins/${pluginFileName}`
             ]), new Logger_1.Logger());
             const file = program.setFile('source/MAIN.brs', '');
@@ -3025,6 +3052,7 @@ describe('BrsFile', () => {
                     const CHARLIE = true
                 end namespace
             `);
+            program.validate();
             //print alpha.beta.char|lie
             (0, chai_1.expect)(program.getDefinition(file.srcPath, vscode_languageserver_1.Position.create(2, 41))).to.eql([{
                     uri: vscode_uri_1.URI.file(file.srcPath).toString(),