brighterscript 0.43.0 → 0.45.1

package/dist/files/BrsFile.spec.js

@@ -37,7 +37,7 @@ describe('BrsFile', () => {
         program.dispose();
     });
     it('supports the third parameter in CreateObject', () => {
-        program.addOrReplaceFile('source/main.brs', `
+        program.setFile('source/main.brs', `
             sub main()
                 regexp = CreateObject("roRegex", "[a-z]+", "i")
             end sub
@@ -46,7 +46,7 @@ describe('BrsFile', () => {
         (0, testHelpers_spec_1.expectZeroDiagnostics)(program);
     });
     it('supports the 6 params in CreateObject for roRegion', () => {
-        program.addOrReplaceFile('source/main.brs', `
+        program.setFile('source/main.brs', `
             sub createRegion(bitmap as object)
                 region = CreateObject("roRegion", bitmap, 20, 40, 100, 200)
             end sub
@@ -75,7 +75,7 @@ describe('BrsFile', () => {
             dest: `source/lib.brs`
         };
         it('creates proper tokens', () => {
-            file = program.addOrReplaceFile(entry, `call(ModuleA.ModuleB.ModuleC.`);
+            file = program.setFile(entry, `call(ModuleA.ModuleB.ModuleC.`);
             (0, chai_1.expect)(file['getPartialVariableName'](file.parser.tokens[7])).to.equal('ModuleA.ModuleB.ModuleC.');
             (0, chai_1.expect)(file['getPartialVariableName'](file.parser.tokens[6])).to.equal('ModuleA.ModuleB.ModuleC');
             (0, chai_1.expect)(file['getPartialVariableName'](file.parser.tokens[5])).to.equal('ModuleA.ModuleB.');
@@ -87,13 +87,13 @@ describe('BrsFile', () => {
     describe('getScopesForFile', () => {
         it('finds the scope for the file', () => {
             var _a;
-            let file = program.addOrReplaceFile('source/main.brs', ``);
+            let file = program.setFile('source/main.brs', ``);
             (0, chai_1.expect)((_a = program.getScopesForFile(file)[0]) === null || _a === void 0 ? void 0 : _a.name).to.equal('source');
         });
     });
     describe('getCompletions', () => {
         it('does not crash for callfunc on a function call', () => {
-            const file = program.addOrReplaceFile('source/main.brs', `
+            const file = program.setFile('source/main.brs', `
                 sub main()
                     getManager()@.
                 end sub
@@ -103,7 +103,7 @@ describe('BrsFile', () => {
             }).not.to.throw;
         });
         it('suggests pkg paths in strings that match that criteria', () => {
-            program.addOrReplaceFile('source/main.brs', `
+            program.setFile('source/main.brs', `
                 sub main()
                     print "pkg:"
                 end sub
@@ -115,7 +115,7 @@ describe('BrsFile', () => {
             ]);
         });
         it('suggests libpkg paths in strings that match that criteria', () => {
-            program.addOrReplaceFile('source/main.brs', `
+            program.setFile('source/main.brs', `
                 sub main()
                     print "libpkg:"
                 end sub
@@ -127,7 +127,7 @@ describe('BrsFile', () => {
             ]);
         });
         it('suggests pkg paths in template strings', () => {
-            program.addOrReplaceFile('source/main.brs', `
+            program.setFile('source/main.brs', `
                 sub main()
                     print \`pkg:\`
                 end sub
@@ -140,7 +140,7 @@ describe('BrsFile', () => {
         });
         it('waits for the file to be processed before collecting completions', () => {
             //eslint-disable-next-line @typescript-eslint/no-floating-promises
-            program.addOrReplaceFile('source/main.brs', `
+            program.setFile('source/main.brs', `
                 sub Main()
                     print "hello"
                     Say
@@ -154,9 +154,79 @@ describe('BrsFile', () => {
             (0, chai_1.expect)(names).to.includes('Main');
             (0, chai_1.expect)(names).to.includes('SayHello');
         });
+        it('includes every type of item at base level', () => {
+            program.setFile('source/main.bs', `
+                sub main()
+                    print
+                end sub
+                sub speak()
+                end sub
+                namespace stuff
+                end namespace
+                class Person
+                end class
+                enum Direction
+                end enum
+            `);
+            (0, testHelpers_spec_1.expectCompletionsIncludes)(program.getCompletions('source/main.bs', util_1.default.createPosition(2, 26)), [{
+                    label: 'main',
+                    kind: vscode_languageserver_1.CompletionItemKind.Function
+                }, {
+                    label: 'speak',
+                    kind: vscode_languageserver_1.CompletionItemKind.Function
+                }, {
+                    label: 'stuff',
+                    kind: vscode_languageserver_1.CompletionItemKind.Module
+                }, {
+                    label: 'Person',
+                    kind: vscode_languageserver_1.CompletionItemKind.Class
+                }, {
+                    label: 'Direction',
+                    kind: vscode_languageserver_1.CompletionItemKind.Enum
+                }]);
+        });
+        describe('namespaces', () => {
+            it('gets full namespace completions at any point through the leading identifier', () => {
+                program.setFile('source/main.bs', `
+                    sub main()
+                        foo.bar
+                    end sub
+
+                    namespace foo.bar
+                    end namespace
+
+                    class Person
+                    end class
+                `);
+                const result = program.getCompletions(`${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');
+            });
+            it('gets namespace completions', () => {
+                program.setFile('source/main.bs', `
+                    namespace foo.bar
+                        function sayHello()
+                        end function
+                    end namespace
+
+                    sub Main()
+                        print "hello"
+                        foo.ba
+                        foo.bar.
+                    end sub
+                `);
+                let result = program.getCompletions(`${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));
+                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.addOrReplaceFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+            program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
                 sub Main()
 
                 end sub
@@ -167,14 +237,14 @@ describe('BrsFile', () => {
         });
         it('does not fail for missing previousToken', () => {
             //add a single character to the file, and get completions after it
-            program.addOrReplaceFile('source/main.brs', `i`);
+            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);
             }).not.to.throw;
         });
         it('includes all keywords`', () => {
             //eslint-disable-next-line @typescript-eslint/no-floating-promises
-            program.addOrReplaceFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+            program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
                 sub Main()
 
                 end sub
@@ -195,7 +265,7 @@ describe('BrsFile', () => {
         });
         it('does not provide completions within a comment', () => {
             //eslint-disable-next-line @typescript-eslint/no-floating-promises
-            program.addOrReplaceFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+            program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
                 sub Main()
                     'some comment
                 end sub
@@ -206,7 +276,7 @@ describe('BrsFile', () => {
         });
         it('does not provide duplicate entries for variables', () => {
             //eslint-disable-next-line @typescript-eslint/no-floating-promises
-            program.addOrReplaceFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+            program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
                 sub Main()
                     name = "bob"
                     age = 12
@@ -221,7 +291,7 @@ 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.addOrReplaceFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+            program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
                 sub Main(name as string)
 
                 end sub
@@ -232,7 +302,7 @@ describe('BrsFile', () => {
         });
         it('does not provide intellisense results when inside a comment', () => {
             //eslint-disable-next-line @typescript-eslint/no-floating-promises
-            program.addOrReplaceFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+            program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
                 sub Main(name as string)
                     'this is a comment
                 end sub
@@ -243,7 +313,7 @@ describe('BrsFile', () => {
         it('does provide intellisence for labels only after a goto keyword', () => {
             var _a;
             //eslint-disable-next-line @typescript-eslint/no-floating-promises
-            program.addOrReplaceFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+            program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
                 sub Main(name as string)
                     something:
                     goto \nend sub
@@ -256,7 +326,7 @@ describe('BrsFile', () => {
     describe('comment flags', () => {
         describe('bs:disable-next-line', () => {
             it('disables critical diagnostic issues', () => {
-                program.addOrReplaceFile('source/main.brs', `
+                program.setFile('source/main.brs', `
                     sub main()
                         Dim requestData
                     end sub
@@ -264,7 +334,7 @@ describe('BrsFile', () => {
                 //should have an error
                 program.validate();
                 (0, testHelpers_spec_1.expectHasDiagnostics)(program);
-                program.addOrReplaceFile('source/main.brs', `
+                program.setFile('source/main.brs', `
                     sub main()
                         'bs:disable-next-line
                         Dim requestData
@@ -275,7 +345,7 @@ describe('BrsFile', () => {
                 (0, testHelpers_spec_1.expectZeroDiagnostics)(program);
             });
             it('works with leading whitespace', () => {
-                program.addOrReplaceFile('source/main.brs', `
+                program.setFile('source/main.brs', `
                     sub main()
                         ' bs:disable-next-line
                         =asdf=sadf=
@@ -286,7 +356,7 @@ describe('BrsFile', () => {
                 (0, testHelpers_spec_1.expectZeroDiagnostics)(program);
             });
             it('works for all', () => {
-                let file = program.addOrReplaceFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+                let file = program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
                     sub Main()
                         'bs:disable-next-line
                         name = "bob
@@ -303,7 +373,7 @@ describe('BrsFile', () => {
                 (0, testHelpers_spec_1.expectZeroDiagnostics)(program);
             });
             it('works for specific codes', () => {
-                let file = program.addOrReplaceFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+                let file = program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
                     sub Main()
                         'bs:disable-next-line: 1083, 1001
                         name = "bob
@@ -319,7 +389,7 @@ describe('BrsFile', () => {
                 (0, testHelpers_spec_1.expectZeroDiagnostics)(program);
             });
             it('recognizes non-numeric codes', () => {
-                let file = program.addOrReplaceFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+                let file = program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
                     sub Main()
                         'bs:disable-next-line: LINT9999
                         name = "bob
@@ -330,7 +400,7 @@ describe('BrsFile', () => {
             });
             it('supports disabling non-numeric error codes', () => {
                 const program = new Program_1.Program({});
-                const file = program.addOrReplaceFile('source/main.brs', `
+                const file = program.setFile('source/main.brs', `
                     sub main()
                         something = true 'bs:disable-line: LINT1005
                     end sub
@@ -345,7 +415,7 @@ describe('BrsFile', () => {
                 (0, testHelpers_spec_1.expectZeroDiagnostics)(scope);
             });
             it('adds diagnostics for unknown numeric diagnostic codes', () => {
-                program.addOrReplaceFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+                program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
                     sub main()
                         print "hi" 'bs:disable-line: 123456 999999   aaaab
                     end sub
@@ -356,7 +426,7 @@ describe('BrsFile', () => {
         });
         describe('bs:disable-line', () => {
             it('works for all', () => {
-                let file = program.addOrReplaceFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+                let file = program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
                     sub Main()
                         z::;;%%%%%% 'bs:disable-line
                     end sub
@@ -372,7 +442,7 @@ describe('BrsFile', () => {
                 (0, testHelpers_spec_1.expectZeroDiagnostics)(program);
             });
             it('works for specific codes', () => {
-                program.addOrReplaceFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+                program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
                     sub main()
                         'should not have any errors
                         DoSomething(1) 'bs:disable-line:1002
@@ -391,7 +461,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.addOrReplaceFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+                program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
                     sub main()
                         stop 'bs:disable-line
                         print "need a valid line to fix stop error"
@@ -404,7 +474,7 @@ describe('BrsFile', () => {
     });
     describe('parse', () => {
         it('supports iife in assignment', () => {
-            program.addOrReplaceFile('source/main.brs', `
+            program.setFile('source/main.brs', `
                 sub main()
                     result = sub()
                     end sub()
@@ -416,7 +486,7 @@ describe('BrsFile', () => {
         });
         it('uses the proper parse mode based on file extension', () => {
             function testParseMode(destPath, expectedParseMode) {
-                const file = program.addOrReplaceFile(destPath, '');
+                const file = program.setFile(destPath, '');
                 (0, chai_1.expect)(file.parseMode).to.equal(expectedParseMode);
             }
             testParseMode('source/main.brs', Parser_1.ParseMode.BrightScript);
@@ -427,7 +497,7 @@ describe('BrsFile', () => {
             testParseMode('source/main.spec.bs', Parser_1.ParseMode.BrighterScript);
         });
         it('supports labels and goto statements', () => {
-            let file = program.addOrReplaceFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+            let file = program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
                 sub Main()
                     'multiple goto statements on one line
                     goto myLabel : goto myLabel
@@ -437,7 +507,7 @@ describe('BrsFile', () => {
             (0, testHelpers_spec_1.expectZeroDiagnostics)(file);
         });
         it('supports empty print statements', () => {
-            let file = program.addOrReplaceFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+            let file = program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
                 sub main()
                    print
                 end sub
@@ -446,7 +516,7 @@ describe('BrsFile', () => {
         });
         describe('conditional compile', () => {
             it('works for upper case keywords', () => {
-                let file = program.addOrReplaceFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+                let file = program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
                     sub main()
                         #CONST someFlag = true
                         #IF someFlag
@@ -461,7 +531,7 @@ describe('BrsFile', () => {
                 (0, testHelpers_spec_1.expectZeroDiagnostics)(file);
             });
             it('supports single-word #elseif and #endif', () => {
-                let file = program.addOrReplaceFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+                let file = program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
                     sub main()
                         #const someFlag = true
                         #if someFlag
@@ -474,7 +544,7 @@ describe('BrsFile', () => {
                 (0, testHelpers_spec_1.expectZeroDiagnostics)(file);
             });
             it('supports multi-word #else if and #end if', () => {
-                let file = program.addOrReplaceFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+                let file = program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
                     sub main()
                         #const someFlag = true
                         #if someFlag
@@ -487,7 +557,7 @@ describe('BrsFile', () => {
                 (0, testHelpers_spec_1.expectZeroDiagnostics)(file);
             });
             it('does not choke on invalid code inside a false conditional compile', () => {
-                let file = program.addOrReplaceFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+                let file = program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
                     sub main()
                         #if false
                             non-commented code here should not cause parse errors
@@ -497,7 +567,7 @@ describe('BrsFile', () => {
                 (0, testHelpers_spec_1.expectZeroDiagnostics)(file);
             });
             it('detects syntax error in #if', () => {
-                let file = program.addOrReplaceFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+                let file = program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
                     sub main()
                         #if true1
                             print "true"
@@ -509,7 +579,7 @@ describe('BrsFile', () => {
                 ]);
             });
             it('detects syntax error in #const', () => {
-                let file = program.addOrReplaceFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+                let file = program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
                     sub main()
                         #if %
                             print "true"
@@ -522,7 +592,7 @@ describe('BrsFile', () => {
                 ]);
             });
             it('detects #const name using reserved word', () => {
-                let file = program.addOrReplaceFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+                let file = program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
                     sub main()
                         #const function = true
                     end sub
@@ -533,7 +603,7 @@ describe('BrsFile', () => {
                 ]);
             });
             it('detects syntax error in #const', () => {
-                let file = program.addOrReplaceFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+                let file = program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
                     sub main()
                         #const someConst = 123
                     end sub
@@ -544,7 +614,7 @@ describe('BrsFile', () => {
             });
         });
         it('supports stop statement', () => {
-            let file = program.addOrReplaceFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+            let file = program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
                 sub main()
                    stop
                 end sub
@@ -552,7 +622,7 @@ describe('BrsFile', () => {
             (0, testHelpers_spec_1.expectZeroDiagnostics)(file);
         });
         it('supports single-line if statements', () => {
-            let file = program.addOrReplaceFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+            let file = program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
                 sub main()
                     if 1 < 2: return true: end if
                     if 1 < 2: return true
@@ -901,38 +971,42 @@ describe('BrsFile', () => {
             (0, testHelpers_spec_1.expectZeroDiagnostics)(file);
         });
         it('adds error for library statements NOT at top of file', () => {
-            let file = program.addOrReplaceFile('source/main.bs', `
+            program.setFile('source/file.brs', ``);
+            program.setFile('source/main.bs', `
                 sub main()
                 end sub
                 import "file.brs"
             `);
-            (0, testHelpers_spec_1.expectDiagnostics)(file, [
+            program.validate();
+            (0, testHelpers_spec_1.expectDiagnostics)(program, [
                 DiagnosticMessages_1.DiagnosticMessages.importStatementMustBeDeclaredAtTopOfFile()
             ]);
         });
         it('supports library imports', () => {
-            file.parse(`
+            program.setFile('source/main.brs', `
                 Library "v30/bslCore.brs"
             `);
-            (0, testHelpers_spec_1.expectZeroDiagnostics)(file);
+            (0, testHelpers_spec_1.expectZeroDiagnostics)(program);
         });
         it('adds error for library statements NOT at top of file', () => {
-            let file = program.addOrReplaceFile('source/main.brs', `
+            program.setFile('source/main.brs', `
                 sub main()
                 end sub
                 Library "v30/bslCore.brs"
             `);
-            (0, testHelpers_spec_1.expectDiagnostics)(file, [
+            program.validate();
+            (0, testHelpers_spec_1.expectDiagnostics)(program, [
                 DiagnosticMessages_1.DiagnosticMessages.libraryStatementMustBeDeclaredAtTopOfFile()
             ]);
         });
         it('adds error for library statements inside of function body', () => {
-            let file = program.addOrReplaceFile('source/main.brs', `
+            program.setFile('source/main.brs', `
                 sub main()
                     Library "v30/bslCore.brs"
                 end sub
             `);
-            (0, testHelpers_spec_1.expectDiagnostics)(file, [
+            program.validate();
+            (0, testHelpers_spec_1.expectDiagnostics)(program, [
                 DiagnosticMessages_1.DiagnosticMessages.libraryStatementMustBeDeclaredAtTopOfFile()
             ]);
         });
@@ -945,7 +1019,7 @@ describe('BrsFile', () => {
             (0, testHelpers_spec_1.expectZeroDiagnostics)(file);
         });
         it('succeeds when finding variables with "sub" in them', () => {
-            let file = program.addOrReplaceFile('source/main.brs', `
+            let file = program.setFile('source/main.brs', `
                 function DoSomething()
                     return value.subType()
                 end function
@@ -1175,7 +1249,7 @@ describe('BrsFile', () => {
                 }]);
         });
         it('finds function calls nested inside statements', () => {
-            program.addOrReplaceFile(`source/main.brs`, `
+            program.setFile(`source/main.brs`, `
                 sub main()
                     if true then
                         DoesNotExist(1, 2)
@@ -1220,7 +1294,7 @@ describe('BrsFile', () => {
             (0, chai_1.expect)(file.callables.length).to.equal(0);
         });
         it('finds return type', () => {
-            let file = program.addOrReplaceFile('source/main.brs', `
+            let file = program.setFile('source/main.brs', `
                 function DoSomething() as string
                 end function
             `);
@@ -1314,7 +1388,7 @@ describe('BrsFile', () => {
             (0, chai_1.expect)(scope.variableDeclarations[0].name).to.equal('theLength');
         });
         it('finds value from global return', () => {
-            let file = program.addOrReplaceFile('source/main.brs', `
+            let file = program.setFile('source/main.brs', `
                 sub Main()
                    myName = GetName()
                 end sub
@@ -1359,7 +1433,7 @@ describe('BrsFile', () => {
     });
     describe('getHover', () => {
         it('works for param types', () => {
-            let file = program.addOrReplaceFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+            let file = program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
                 sub DoSomething(name as string)
                     name = 1
                     sayMyName = function(name as string)
@@ -1377,7 +1451,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.addOrReplaceFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+            let file = program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
                 sub Main(name as string)
                 end sub
                 sub as()
@@ -1389,7 +1463,7 @@ describe('BrsFile', () => {
             (0, chai_1.expect)(file.getHover(vscode_languageserver_1.Position.create(1, 36))).not.to.exist;
         });
         it('finds declared function', () => {
-            let file = program.addOrReplaceFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+            let file = program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
                 function Main(count = 1)
                     firstName = "bob"
                     age = 21
@@ -1406,7 +1480,7 @@ describe('BrsFile', () => {
             ].join('\n'));
         });
         it('finds variable function hover in same scope', () => {
-            let file = program.addOrReplaceFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+            let file = program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
                 sub Main()
                     sayMyName = sub(name as string)
                     end sub
@@ -1422,8 +1496,33 @@ describe('BrsFile', () => {
                 '```'
             ].join('\n'));
         });
+        it('does not crash when hovering on built-in functions', async () => {
+            let file = program.setFile('source/main.brs', `
+                function doUcase(text)
+                    return ucase(text)
+                end function
+            `);
+            (0, chai_1.expect)((await program.getHover(file.pathAbsolute, vscode_languageserver_1.Position.create(2, 30))).contents).to.equal([
+                '```brightscript',
+                'function UCase(s as string) as string',
+                '```'
+            ].join('\n'));
+        });
+        it('does not crash when hovering on object method call', async () => {
+            let file = program.setFile('source/main.brs', `
+                function getInstr(url, text)
+                    return url.instr(text)
+                end function
+            `);
+            (0, chai_1.expect)((await program.getHover(file.pathAbsolute, vscode_languageserver_1.Position.create(2, 35))).contents).to.equal([
+                '```brightscript',
+                //TODO this really shouldn't be returning the global function, but it does...so make sure it doesn't crash right now.
+                'function Instr(start as integer, text as string, substring as string) as integer',
+                '```'
+            ].join('\n'));
+        });
         it('finds function hover in file scope', () => {
-            let file = program.addOrReplaceFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+            let file = program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
                 sub Main()
                     sayMyName()
                 end sub
@@ -1445,12 +1544,12 @@ describe('BrsFile', () => {
             program = new Program_1.Program({
                 rootDir: rootDir
             });
-            let mainFile = program.addOrReplaceFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+            let mainFile = program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
                 sub Main()
                     sayMyName()
                 end sub
             `);
-            program.addOrReplaceFile({ src: `${rootDir}/source/lib.brs`, dest: 'source/lib.brs' }, `
+            program.setFile({ src: `${rootDir}/source/lib.brs`, dest: 'source/lib.brs' }, `
                 sub sayMyName(name as string)
 
                 end sub
@@ -1469,7 +1568,7 @@ describe('BrsFile', () => {
             program = new Program_1.Program({
                 rootDir: rootDir
             });
-            const file = program.addOrReplaceFile('source/lib.brs', `
+            const file = program.setFile('source/lib.brs', `
                 '
                 ' The main function
                 '
@@ -1507,7 +1606,7 @@ describe('BrsFile', () => {
                 `);
         });
         it('handles mixed case `then` partions of conditionals', () => {
-            let mainFile = program.addOrReplaceFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+            let mainFile = program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
                 sub Main()
                     if true then
                         print "works"
@@ -1515,7 +1614,7 @@ describe('BrsFile', () => {
                 end sub
             `);
             (0, testHelpers_spec_1.expectZeroDiagnostics)(mainFile);
-            mainFile = program.addOrReplaceFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+            mainFile = program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
                 sub Main()
                     if true Then
                         print "works"
@@ -1523,7 +1622,7 @@ describe('BrsFile', () => {
                 end sub
             `);
             (0, testHelpers_spec_1.expectZeroDiagnostics)(mainFile);
-            mainFile = program.addOrReplaceFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+            mainFile = program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
                 sub Main()
                     if true THEN
                         print "works"
@@ -1534,7 +1633,7 @@ describe('BrsFile', () => {
         });
     });
     it('does not throw when encountering incomplete import statement', () => {
-        program.addOrReplaceFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
+        program.setFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
             import
             sub main()
             end sub
@@ -2057,7 +2156,7 @@ describe('BrsFile', () => {
             `);
         });
         it('simple mapped files include a reference to the source map', () => {
-            let file = program.addOrReplaceFile('source/logger.brs', (0, testHelpers_spec_1.trim) `
+            let file = program.setFile('source/logger.brs', (0, testHelpers_spec_1.trim) `
                 sub logInfo()
                 end sub
             `);
@@ -2066,7 +2165,7 @@ describe('BrsFile', () => {
             (0, chai_1.expect)(code.endsWith(`'//# sourceMappingURL=./logger.brs.map`)).to.be.true;
         });
         it('AST generated files include a reference to the source map', () => {
-            let file = program.addOrReplaceFile('source/logger.brs', (0, testHelpers_spec_1.trim) `
+            let file = program.setFile('source/logger.brs', (0, testHelpers_spec_1.trim) `
                 sub logInfo()
                 end sub
             `);
@@ -2075,7 +2174,7 @@ describe('BrsFile', () => {
             (0, chai_1.expect)(code.endsWith(`'//# sourceMappingURL=./logger.brs.map`)).to.be.true;
         });
         it('replaces custom types in parameter types and return types', () => {
-            program.addOrReplaceFile('source/SomeKlass.bs', `
+            program.setFile('source/SomeKlass.bs', `
                 class SomeKlass
                 end class
             `);
@@ -2099,7 +2198,7 @@ describe('BrsFile', () => {
     describe('callfunc operator', () => {
         describe('transpile', () => {
             it('does not produce diagnostics', () => {
-                program.addOrReplaceFile('source/main.bs', `
+                program.setFile('source/main.bs', `
                     sub main()
                         someObject@.someFunction(paramObject.value)
                     end sub
@@ -2145,7 +2244,7 @@ describe('BrsFile', () => {
                 name: 'transform callback',
                 afterFileParse: onParsed
             });
-            file = program.addOrReplaceFile({ src: `absolute_path/file${ext}`, dest: `relative_path/file${ext}` }, `
+            file = program.setFile({ src: `absolute_path/file${ext}`, dest: `relative_path/file${ext}` }, `
                 sub Sum()
                     print "hello world"
                 end sub
@@ -2166,25 +2265,25 @@ describe('BrsFile', () => {
     });
     describe('typedefKey', () => {
         it('works for .brs files', () => {
-            (0, chai_1.expect)((0, util_1.standardizePath)((program.addOrReplaceFile('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) `${rootDir.toLowerCase()}/source/main.d.bs`);
         });
         it('returns undefined for files that should not have a typedef', () => {
-            (0, chai_1.expect)((program.addOrReplaceFile('source/main.bs', '')).typedefKey).to.be.undefined;
-            (0, chai_1.expect)((program.addOrReplaceFile('source/main.d.bs', '')).typedefKey).to.be.undefined;
-            const xmlFile = program.addOrReplaceFile('components/comp.xml', '');
+            (0, chai_1.expect)((program.setFile('source/main.bs', '')).typedefKey).to.be.undefined;
+            (0, chai_1.expect)((program.setFile('source/main.d.bs', '')).typedefKey).to.be.undefined;
+            const xmlFile = program.setFile('components/comp.xml', '');
             (0, chai_1.expect)(xmlFile.typedefKey).to.be.undefined;
         });
     });
     describe('type definitions', () => {
         it('only exposes defined functions even if source has more', () => {
             //parse the .brs file first so it doesn't know about the typedef
-            program.addOrReplaceFile('source/main.brs', `
+            program.setFile('source/main.brs', `
                 sub main()
                 end sub
                 sub speak()
                 end sub
             `);
-            program.addOrReplaceFile('source/main.d.bs', `
+            program.setFile('source/main.d.bs', `
                 sub main()
                 end sub
             `);
@@ -2194,7 +2293,7 @@ describe('BrsFile', () => {
             (0, chai_1.expect)(functionNames).not.to.include('speak');
         });
         it('reacts to typedef file changes', () => {
-            let file = program.addOrReplaceFile('source/main.brs', `
+            let file = program.setFile('source/main.brs', `
                 sub main()
                 end sub
                 sub speak()
@@ -2202,14 +2301,14 @@ describe('BrsFile', () => {
             `);
             (0, chai_1.expect)(file.hasTypedef).to.be.false;
             (0, chai_1.expect)(file.typedefFile).not.to.exist;
-            program.addOrReplaceFile('source/main.d.bs', `
+            program.setFile('source/main.d.bs', `
                 sub main()
                 end sub
             `);
             (0, chai_1.expect)(file.hasTypedef).to.be.true;
             (0, chai_1.expect)(file.typedefFile).to.exist;
             //add replace file, does it still find the typedef
-            file = program.addOrReplaceFile('source/main.brs', `
+            file = program.setFile('source/main.brs', `
                 sub main()
                 end sub
                 sub speak()
@@ -2224,36 +2323,36 @@ describe('BrsFile', () => {
     });
     describe('typedef', () => {
         it('sets typedef path properly', () => {
-            (0, chai_1.expect)((program.addOrReplaceFile('source/main1.brs', '')).typedefKey).to.equal((0, util_1.standardizePath) `${rootDir}/source/main1.d.bs`.toLowerCase());
-            (0, chai_1.expect)((program.addOrReplaceFile('source/main2.d.bs', '')).typedefKey).to.equal(undefined);
-            (0, chai_1.expect)((program.addOrReplaceFile('source/main3.bs', '')).typedefKey).to.equal(undefined);
+            (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/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
-            (0, chai_1.expect)((program.addOrReplaceFile({ src: 'source/main4.bs', dest: 'source/main4.brs' }, '')).typedefKey).to.equal(undefined);
+            (0, chai_1.expect)((program.setFile({ src: 'source/main4.bs', dest: 'source/main4.brs' }, '')).typedefKey).to.equal(undefined);
         });
         it('does not link when missing from program', () => {
-            const file = program.addOrReplaceFile('source/main.brs', ``);
+            const file = program.setFile('source/main.brs', ``);
             (0, chai_1.expect)(file.typedefFile).not.to.exist;
         });
         it('links typedef when added BEFORE .brs file', () => {
-            const typedef = program.addOrReplaceFile('source/main.d.bs', ``);
-            const file = program.addOrReplaceFile('source/main.brs', ``);
+            const typedef = program.setFile('source/main.d.bs', ``);
+            const file = program.setFile('source/main.brs', ``);
             (0, chai_1.expect)(file.typedefFile).to.equal(typedef);
         });
         it('links typedef when added AFTER .brs file', () => {
-            const file = program.addOrReplaceFile('source/main.brs', ``);
-            const typedef = program.addOrReplaceFile('source/main.d.bs', ``);
+            const file = program.setFile('source/main.brs', ``);
+            const typedef = program.setFile('source/main.d.bs', ``);
             (0, chai_1.expect)(file.typedefFile).to.eql(typedef);
         });
         it('removes typedef link when typedef is removed', () => {
-            const typedef = program.addOrReplaceFile('source/main.d.bs', ``);
-            const file = program.addOrReplaceFile('source/main.brs', ``);
+            const typedef = program.setFile('source/main.d.bs', ``);
+            const file = program.setFile('source/main.brs', ``);
             program.removeFile(typedef.pathAbsolute);
             (0, chai_1.expect)(file.typedefFile).to.be.undefined;
         });
     });
     describe('getTypedef', () => {
         function testTypedef(original, expected) {
-            let file = program.addOrReplaceFile('source/main.brs', original);
+            let file = program.setFile('source/main.brs', original);
             (0, chai_1.expect)(file.getTypedef()).to.eql(expected);
         }
         it('includes namespace on extend class names', () => {
@@ -2504,7 +2603,7 @@ describe('BrsFile', () => {
     });
     describe('parser getter', () => {
         it('recreates the parser when missing', () => {
-            const file = program.addOrReplaceFile('source/main.brs', `
+            const file = program.setFile('source/main.brs', `
                 sub main()
                 end sub
             `);
@@ -2518,11 +2617,11 @@ describe('BrsFile', () => {
             (0, chai_1.expect)(file.parser).to.equal(newParser);
         });
         it('call parse when previously skipped', () => {
-            program.addOrReplaceFile('source/main.d.bs', `
+            program.setFile('source/main.d.bs', `'typedef
                 sub main()
                 end sub
             `);
-            const file = program.addOrReplaceFile('source/main.brs', `
+            const file = program.setFile('source/main.brs', `'source
                 sub main()
                 end sub
             `);