brighterscript 0.41.5 → 0.43.1

package/dist/files/BrsFile.spec.js

@@ -12,7 +12,8 @@ const IntegerType_1 = require("../types/IntegerType");
 const StringType_1 = require("../types/StringType");
 const BrsFile_1 = require("./BrsFile");
 const source_map_1 = require("source-map");
-const lexer_1 = require("../lexer");
+const Lexer_1 = require("../lexer/Lexer");
+const TokenKind_1 = require("../lexer/TokenKind");
 const DiagnosticMessages_1 = require("../DiagnosticMessages");
 const util_1 = require("../util");
 const PluginInterface_1 = require("../PluginInterface");
@@ -36,24 +37,22 @@ describe('BrsFile', () => {
         program.dispose();
     });
     it('supports the third parameter in CreateObject', () => {
-        var _a;
         program.addOrReplaceFile('source/main.brs', `
             sub main()
                 regexp = CreateObject("roRegex", "[a-z]+", "i")
             end sub
         `);
         program.validate();
-        (0, chai_1.expect)((_a = program.getDiagnostics()[0]) === null || _a === void 0 ? void 0 : _a.message).to.not.exist;
+        (0, testHelpers_spec_1.expectZeroDiagnostics)(program);
     });
     it('supports the 6 params in CreateObject for roRegion', () => {
-        var _a;
         program.addOrReplaceFile('source/main.brs', `
             sub createRegion(bitmap as object)
                 region = CreateObject("roRegion", bitmap, 20, 40, 100, 200)
             end sub
         `);
         program.validate();
-        (0, chai_1.expect)((_a = program.getDiagnostics()[0]) === null || _a === void 0 ? void 0 : _a.message).to.not.exist;
+        (0, testHelpers_spec_1.expectZeroDiagnostics)(program);
     });
     it('sets needsTranspiled to true for .bs files', () => {
         //BrightScript
@@ -68,8 +67,7 @@ describe('BrsFile', () => {
                 range: undefined
             }];
         file.addDiagnostics(expected);
-        const actual = file.getDiagnostics();
-        (0, chai_1.expect)(actual).deep.equal(expected);
+        (0, testHelpers_spec_1.expectDiagnostics)(file, expected);
     });
     describe('getPartialVariableName', () => {
         let entry = {
@@ -181,7 +179,7 @@ describe('BrsFile', () => {
 
                 end sub
             `);
-            let keywords = Object.keys(lexer_1.Keywords).filter(x => !x.includes(' '));
+            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 names = result.map(x => x.label);
@@ -258,7 +256,6 @@ describe('BrsFile', () => {
     describe('comment flags', () => {
         describe('bs:disable-next-line', () => {
             it('disables critical diagnostic issues', () => {
-                var _a, _b;
                 program.addOrReplaceFile('source/main.brs', `
                     sub main()
                         Dim requestData
@@ -266,19 +263,18 @@ describe('BrsFile', () => {
                 `);
                 //should have an error
                 program.validate();
-                (0, chai_1.expect)((_a = program.getDiagnostics()[0]) === null || _a === void 0 ? void 0 : _a.message).to.exist;
+                (0, testHelpers_spec_1.expectHasDiagnostics)(program);
                 program.addOrReplaceFile('source/main.brs', `
                     sub main()
                         'bs:disable-next-line
                         Dim requestData
                     end sub
                 `);
-                //should have an error
+                //should not have an error
                 program.validate();
-                (0, chai_1.expect)((_b = program.getDiagnostics()[0]) === null || _b === void 0 ? void 0 : _b.message).not.to.exist;
+                (0, testHelpers_spec_1.expectZeroDiagnostics)(program);
             });
             it('works with leading whitespace', () => {
-                var _a;
                 program.addOrReplaceFile('source/main.brs', `
                     sub main()
                         ' bs:disable-next-line
@@ -287,10 +283,9 @@ describe('BrsFile', () => {
                 `);
                 //should have an error
                 program.validate();
-                (0, chai_1.expect)((_a = program.getDiagnostics()[0]) === null || _a === void 0 ? void 0 : _a.message).not.to.exist;
+                (0, testHelpers_spec_1.expectZeroDiagnostics)(program);
             });
             it('works for all', () => {
-                var _a;
                 let file = program.addOrReplaceFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
                     sub Main()
                         'bs:disable-next-line
@@ -305,10 +300,9 @@ describe('BrsFile', () => {
                 });
                 program.validate();
                 //the "unterminated string" error should be filtered out
-                (0, chai_1.expect)((_a = program.getDiagnostics()[0]) === null || _a === void 0 ? void 0 : _a.message).not.to.exist;
+                (0, testHelpers_spec_1.expectZeroDiagnostics)(program);
             });
             it('works for specific codes', () => {
-                var _a;
                 let file = program.addOrReplaceFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
                     sub Main()
                         'bs:disable-next-line: 1083, 1001
@@ -322,10 +316,9 @@ describe('BrsFile', () => {
                     affectedRange: util_1.default.createRange(3, 0, 3, Number.MAX_SAFE_INTEGER)
                 });
                 //the "unterminated string" error should be filtered out
-                (0, chai_1.expect)((_a = program.getDiagnostics()[0]) === null || _a === void 0 ? void 0 : _a.message).to.not.exist;
+                (0, testHelpers_spec_1.expectZeroDiagnostics)(program);
             });
             it('recognizes non-numeric codes', () => {
-                var _a;
                 let file = program.addOrReplaceFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
                     sub Main()
                         'bs:disable-next-line: LINT9999
@@ -333,7 +326,7 @@ describe('BrsFile', () => {
                     end sub
                 `);
                 (0, chai_1.expect)(file.commentFlags[0]).to.exist;
-                (0, chai_1.expect)((_a = program.getDiagnostics()[0]) === null || _a === void 0 ? void 0 : _a.message).to.exist;
+                (0, testHelpers_spec_1.expectHasDiagnostics)(program);
             });
             it('supports disabling non-numeric error codes', () => {
                 const program = new Program_1.Program({});
@@ -348,7 +341,8 @@ describe('BrsFile', () => {
                         message: 'Something is not right',
                         range: util_1.default.createRange(2, 16, 2, 26)
                     }]);
-                (0, chai_1.expect)(program.getScopesForFile(file)[0].getDiagnostics()).to.be.empty;
+                const scope = program.getScopesForFile(file)[0];
+                (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' }, `
@@ -357,13 +351,7 @@ describe('BrsFile', () => {
                     end sub
                 `);
                 program.validate();
-                (0, chai_1.expect)(program.getDiagnostics()).to.be.lengthOf(2);
-                (0, chai_1.expect)(program.getDiagnostics()[0]).to.deep.include({
-                    range: vscode_languageserver_1.Range.create(2, 53, 2, 59)
-                });
-                (0, chai_1.expect)(program.getDiagnostics()[1]).to.deep.include({
-                    range: vscode_languageserver_1.Range.create(2, 60, 2, 66)
-                });
+                (0, testHelpers_spec_1.expectDiagnostics)(program, [Object.assign(Object.assign({}, DiagnosticMessages_1.DiagnosticMessages.unknownDiagnosticCode(123456)), { range: vscode_languageserver_1.Range.create(2, 53, 2, 59) }), Object.assign(Object.assign({}, DiagnosticMessages_1.DiagnosticMessages.unknownDiagnosticCode(999999)), { range: vscode_languageserver_1.Range.create(2, 60, 2, 66) })]);
             });
         });
         describe('bs:disable-line', () => {
@@ -381,7 +369,7 @@ describe('BrsFile', () => {
                 });
                 program.validate();
                 //the "unterminated string" error should be filtered out
-                (0, chai_1.expect)(program.getDiagnostics()).to.be.lengthOf(0);
+                (0, testHelpers_spec_1.expectZeroDiagnostics)(program);
             });
             it('works for specific codes', () => {
                 program.addOrReplaceFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
@@ -395,10 +383,9 @@ describe('BrsFile', () => {
                     end sub
                 `);
                 program.validate();
-                (0, chai_1.expect)(program.getDiagnostics()).to.be.lengthOf(1);
-                (0, chai_1.expect)(program.getDiagnostics()[0]).to.deep.include({
-                    range: vscode_languageserver_1.Range.create(5, 24, 5, 35)
-                });
+                (0, testHelpers_spec_1.expectDiagnostics)(program, [{
+                        range: vscode_languageserver_1.Range.create(5, 24, 5, 35)
+                    }]);
             });
             it('handles the erraneous `stop` keyword', () => {
                 //the current version of BRS causes parse errors after the `parse` keyword, showing error in comments
@@ -411,7 +398,7 @@ describe('BrsFile', () => {
                     end sub
                 `);
                 program.validate();
-                (0, chai_1.expect)(program.getDiagnostics()).to.be.lengthOf(0);
+                (0, testHelpers_spec_1.expectZeroDiagnostics)(program);
             });
         });
     });
@@ -447,7 +434,7 @@ describe('BrsFile', () => {
                     myLabel:
                 end sub
             `);
-            (0, chai_1.expect)(file.getDiagnostics()).to.be.lengthOf(0);
+            (0, testHelpers_spec_1.expectZeroDiagnostics)(file);
         });
         it('supports empty print statements', () => {
             let file = program.addOrReplaceFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
@@ -455,7 +442,7 @@ describe('BrsFile', () => {
                    print
                 end sub
             `);
-            (0, chai_1.expect)(file.getDiagnostics()).to.be.lengthOf(0);
+            (0, testHelpers_spec_1.expectZeroDiagnostics)(file);
         });
         describe('conditional compile', () => {
             it('works for upper case keywords', () => {
@@ -471,7 +458,7 @@ describe('BrsFile', () => {
                         #ENDIF
                     end sub
                 `);
-                (0, chai_1.expect)(file.getDiagnostics()).to.be.lengthOf(0);
+                (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' }, `
@@ -484,7 +471,7 @@ describe('BrsFile', () => {
                         #endif
                     end sub
                 `);
-                (0, chai_1.expect)(file.getDiagnostics()).to.be.lengthOf(0);
+                (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' }, `
@@ -497,7 +484,7 @@ describe('BrsFile', () => {
                         #end if
                     end sub
                 `);
-                (0, chai_1.expect)(file.getDiagnostics()).to.be.lengthOf(0);
+                (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' }, `
@@ -507,7 +494,7 @@ describe('BrsFile', () => {
                         #end if
                     end sub
                 `);
-                (0, chai_1.expect)(file.getDiagnostics()).to.be.lengthOf(0);
+                (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' }, `
@@ -517,7 +504,9 @@ describe('BrsFile', () => {
                         #end if
                     end sub
                 `);
-                (0, chai_1.expect)(file.getDiagnostics()[0]).to.exist.and.deep.include(Object.assign({}, DiagnosticMessages_1.DiagnosticMessages.invalidHashConstValue));
+                (0, testHelpers_spec_1.expectDiagnostics)(file, [
+                    DiagnosticMessages_1.DiagnosticMessages.referencedConstDoesNotExist()
+                ]);
             });
             it('detects syntax error in #const', () => {
                 let file = program.addOrReplaceFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
@@ -527,7 +516,10 @@ describe('BrsFile', () => {
                         #end if
                     end sub
                 `);
-                (0, chai_1.expect)(file.getDiagnostics()[0]).to.exist.and.deep.include(Object.assign({}, DiagnosticMessages_1.DiagnosticMessages.unexpectedCharacter('%')));
+                (0, testHelpers_spec_1.expectDiagnostics)(file, [
+                    DiagnosticMessages_1.DiagnosticMessages.unexpectedCharacter('%'),
+                    DiagnosticMessages_1.DiagnosticMessages.invalidHashIfValue()
+                ]);
             });
             it('detects #const name using reserved word', () => {
                 let file = program.addOrReplaceFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
@@ -535,7 +527,10 @@ describe('BrsFile', () => {
                         #const function = true
                     end sub
                 `);
-                (0, chai_1.expect)(file.getDiagnostics()[0]).to.exist.and.deep.include(Object.assign({}, DiagnosticMessages_1.DiagnosticMessages.constNameCannotBeReservedWord()));
+                (0, testHelpers_spec_1.expectDiagnostics)(file, [
+                    DiagnosticMessages_1.DiagnosticMessages.constNameCannotBeReservedWord(),
+                    DiagnosticMessages_1.DiagnosticMessages.unexpectedToken('#const')
+                ]);
             });
             it('detects syntax error in #const', () => {
                 let file = program.addOrReplaceFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
@@ -543,7 +538,9 @@ describe('BrsFile', () => {
                         #const someConst = 123
                     end sub
                 `);
-                (0, chai_1.expect)(file.getDiagnostics()[0]).to.exist.and.deep.include(Object.assign({}, DiagnosticMessages_1.DiagnosticMessages.invalidHashConstValue()));
+                (0, testHelpers_spec_1.expectDiagnostics)(file, [
+                    DiagnosticMessages_1.DiagnosticMessages.invalidHashConstValue()
+                ]);
             });
         });
         it('supports stop statement', () => {
@@ -552,7 +549,7 @@ describe('BrsFile', () => {
                    stop
                 end sub
             `);
-            (0, chai_1.expect)(file.getDiagnostics()).to.be.lengthOf(0);
+            (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' }, `
@@ -565,16 +562,15 @@ describe('BrsFile', () => {
                     if true then : test = sub() : print "yes" : end sub : end if
                 end sub
             `);
-            (0, chai_1.expect)(file.getDiagnostics()).to.be.lengthOf(0);
+            (0, testHelpers_spec_1.expectZeroDiagnostics)(file);
         });
         it('supports line_num as global variable', () => {
-            var _a;
             file.parse(`
                 sub Main()
                     print LINE_NUM
                 end sub
             `);
-            (0, chai_1.expect)((_a = file.getDiagnostics()[0]) === null || _a === void 0 ? void 0 : _a.message).not.to.exist;
+            (0, testHelpers_spec_1.expectZeroDiagnostics)(file);
         });
         it('supports many keywords as object property names', () => {
             file.parse(`
@@ -640,7 +636,7 @@ describe('BrsFile', () => {
                     person.new = true
                 end sub
             `);
-            (0, chai_1.expect)(file.getDiagnostics()).to.be.lengthOf(0);
+            (0, testHelpers_spec_1.expectZeroDiagnostics)(file);
         });
         it('does not error on numeric literal type designators', () => {
             file.parse(`
@@ -654,7 +650,7 @@ describe('BrsFile', () => {
                     print 9876543210&
                 end sub
             `);
-            (0, chai_1.expect)(file.getDiagnostics()).to.be.lengthOf(0);
+            (0, testHelpers_spec_1.expectZeroDiagnostics)(file);
         });
         it('does not error when encountering sub with return type', () => {
             file.parse(`
@@ -662,7 +658,7 @@ describe('BrsFile', () => {
                     return
                 end sub
             `);
-            (0, chai_1.expect)(file.getDiagnostics()).to.be.lengthOf(0);
+            (0, testHelpers_spec_1.expectZeroDiagnostics)(file);
         });
         it('does not lose function scopes when mismatched end sub', () => {
             file.parse(`
@@ -697,7 +693,7 @@ describe('BrsFile', () => {
                     foo.bar = true and false or 3 > 4
                 end sub
             `);
-            (0, chai_1.expect)(file.getDiagnostics()).to.be.lengthOf(0);
+            (0, testHelpers_spec_1.expectZeroDiagnostics)(file);
         });
         it('does not error with boolean in RHS of set statement', () => {
             file.parse(`
@@ -710,7 +706,7 @@ describe('BrsFile', () => {
                     m.isTrue = m.isTrue = m.isTrue
                 end sub
             `);
-            (0, chai_1.expect)(file.getDiagnostics()).to.be.lengthOf(0);
+            (0, testHelpers_spec_1.expectZeroDiagnostics)(file);
         });
         it('supports variable names ending with type designators', () => {
             file.parse(`
@@ -722,7 +718,7 @@ describe('BrsFile', () => {
                   someHex& = 13
                 end sub
             `);
-            (0, chai_1.expect)(file.getDiagnostics()).to.be.lengthOf(0);
+            (0, testHelpers_spec_1.expectZeroDiagnostics)(file);
         });
         it('supports multiple spaces between two-word keywords', () => {
             file.parse(`
@@ -734,7 +730,7 @@ describe('BrsFile', () => {
                     end if
                 end sub
             `);
-            (0, chai_1.expect)(file.getDiagnostics()).to.be.lengthOf(0);
+            (0, testHelpers_spec_1.expectZeroDiagnostics)(file);
         });
         it('does not error with `stop` as object key', () => {
             file.parse(`
@@ -747,7 +743,7 @@ describe('BrsFile', () => {
                     return obj
                 end function
             `);
-            (0, chai_1.expect)(file.getDiagnostics()).to.be.lengthOf(0);
+            (0, testHelpers_spec_1.expectZeroDiagnostics)(file);
         });
         it('does not error with `run` as object key', () => {
             file.parse(`
@@ -760,7 +756,7 @@ describe('BrsFile', () => {
                     return obj
                 end function
             `);
-            (0, chai_1.expect)(file.getDiagnostics()).to.be.lengthOf(0);
+            (0, testHelpers_spec_1.expectZeroDiagnostics)(file);
         });
         it('supports assignment operators', () => {
             file.parse(`
@@ -777,7 +773,7 @@ describe('BrsFile', () => {
                     print x
                 end function
             `);
-            (0, chai_1.expect)(file.getDiagnostics()).to.be.lengthOf(0);
+            (0, testHelpers_spec_1.expectZeroDiagnostics)(file);
         });
         it('supports `then` as object property', () => {
             file.parse(`
@@ -789,7 +785,7 @@ describe('BrsFile', () => {
                     promise.then()
                 end function
             `);
-            (0, chai_1.expect)(file.getDiagnostics()).to.be.lengthOf(0);
+            (0, testHelpers_spec_1.expectZeroDiagnostics)(file);
         });
         it('supports function as parameter type', () => {
             file.parse(`
@@ -798,7 +794,7 @@ describe('BrsFile', () => {
                     end function
                 end sub
             `);
-            (0, chai_1.expect)(file.getDiagnostics()).to.be.lengthOf(0);
+            (0, testHelpers_spec_1.expectZeroDiagnostics)(file);
         });
         it('supports increment operator', () => {
             file.parse(`
@@ -807,8 +803,7 @@ describe('BrsFile', () => {
                     x++
                 end function
             `);
-            file.getDiagnostics();
-            (0, chai_1.expect)(file.getDiagnostics()).to.be.lengthOf(0);
+            (0, testHelpers_spec_1.expectZeroDiagnostics)(file);
         });
         it('supports decrement operator', () => {
             file.parse(`
@@ -817,8 +812,7 @@ describe('BrsFile', () => {
                     x--
                 end function
             `);
-            file.getDiagnostics();
-            (0, chai_1.expect)(file.getDiagnostics()).to.be.lengthOf(0);
+            (0, testHelpers_spec_1.expectZeroDiagnostics)(file);
         });
         it('supports writing numbers with decimal but no trailing digit', () => {
             file.parse(`
@@ -827,7 +821,7 @@ describe('BrsFile', () => {
                     print x
                 end function
             `);
-            (0, chai_1.expect)(file.getDiagnostics()).to.be.lengthOf(0);
+            (0, testHelpers_spec_1.expectZeroDiagnostics)(file);
         });
         it('supports assignment operators against object properties', () => {
             file.parse(`
@@ -849,7 +843,7 @@ describe('BrsFile', () => {
                     print m.age
                 end function
             `);
-            (0, chai_1.expect)(file.getDiagnostics()).to.be.lengthOf(0);
+            (0, testHelpers_spec_1.expectZeroDiagnostics)(file);
         });
         //skipped until `brs` supports this
         it('supports bitshift assignment operators', () => {
@@ -861,7 +855,7 @@ describe('BrsFile', () => {
                     print x
                 end function
             `);
-            (0, chai_1.expect)(file.getDiagnostics()).to.be.lengthOf(0);
+            (0, testHelpers_spec_1.expectZeroDiagnostics)(file);
         });
         //skipped until `brs` supports this
         it('supports bitshift assignment operators on objects', () => {
@@ -873,7 +867,7 @@ describe('BrsFile', () => {
                         print m.x
                     end function
                 `);
-            (0, chai_1.expect)(file.getDiagnostics()).to.be.lengthOf(0);
+            (0, testHelpers_spec_1.expectZeroDiagnostics)(file);
         });
         it('supports leading and trailing periods for numeric literals', () => {
             file.parse(`
@@ -884,7 +878,7 @@ describe('BrsFile', () => {
                     print pointOne
                 end function
             `);
-            (0, chai_1.expect)(file.getDiagnostics()).to.be.lengthOf(0);
+            (0, testHelpers_spec_1.expectZeroDiagnostics)(file);
         });
         it('supports bitshift assignment operators on object properties accessed by array syntax', () => {
             file.parse(`
@@ -895,7 +889,7 @@ describe('BrsFile', () => {
                         print m.x
                     end function
                 `);
-            (0, chai_1.expect)(file.getDiagnostics()).to.be.lengthOf(0);
+            (0, testHelpers_spec_1.expectZeroDiagnostics)(file);
         });
         it('supports weird period AA accessor', () => {
             file.parse(`
@@ -904,7 +898,7 @@ describe('BrsFile', () => {
                     print m.["_uuid"]
                 end function
             `);
-            (0, chai_1.expect)(file.getDiagnostics()).to.be.lengthOf(0);
+            (0, testHelpers_spec_1.expectZeroDiagnostics)(file);
         });
         it('adds error for library statements NOT at top of file', () => {
             let file = program.addOrReplaceFile('source/main.bs', `
@@ -912,15 +906,15 @@ describe('BrsFile', () => {
                 end sub
                 import "file.brs"
             `);
-            (0, chai_1.expect)(file.getDiagnostics().map(x => x.message)).to.eql([
-                DiagnosticMessages_1.DiagnosticMessages.importStatementMustBeDeclaredAtTopOfFile().message
+            (0, testHelpers_spec_1.expectDiagnostics)(file, [
+                DiagnosticMessages_1.DiagnosticMessages.importStatementMustBeDeclaredAtTopOfFile()
             ]);
         });
         it('supports library imports', () => {
             file.parse(`
                 Library "v30/bslCore.brs"
             `);
-            (0, chai_1.expect)(file.getDiagnostics()).to.be.lengthOf(0);
+            (0, testHelpers_spec_1.expectZeroDiagnostics)(file);
         });
         it('adds error for library statements NOT at top of file', () => {
             let file = program.addOrReplaceFile('source/main.brs', `
@@ -928,8 +922,8 @@ describe('BrsFile', () => {
                 end sub
                 Library "v30/bslCore.brs"
             `);
-            (0, chai_1.expect)(file.getDiagnostics().map(x => x.message)).to.eql([
-                DiagnosticMessages_1.DiagnosticMessages.libraryStatementMustBeDeclaredAtTopOfFile().message
+            (0, testHelpers_spec_1.expectDiagnostics)(file, [
+                DiagnosticMessages_1.DiagnosticMessages.libraryStatementMustBeDeclaredAtTopOfFile()
             ]);
         });
         it('adds error for library statements inside of function body', () => {
@@ -938,8 +932,8 @@ describe('BrsFile', () => {
                     Library "v30/bslCore.brs"
                 end sub
             `);
-            (0, chai_1.expect)(file.getDiagnostics().map(x => x.message)).to.eql([
-                DiagnosticMessages_1.DiagnosticMessages.libraryStatementMustBeDeclaredAtTopOfFile().message
+            (0, testHelpers_spec_1.expectDiagnostics)(file, [
+                DiagnosticMessages_1.DiagnosticMessages.libraryStatementMustBeDeclaredAtTopOfFile()
             ]);
         });
         it('supports colons as separators in associative array properties', () => {
@@ -948,7 +942,7 @@ describe('BrsFile', () => {
                     obj = {x:0 : y: 1}
                 end sub
             `);
-            (0, chai_1.expect)(file.getDiagnostics()).to.be.lengthOf(0);
+            (0, testHelpers_spec_1.expectZeroDiagnostics)(file);
         });
         it('succeeds when finding variables with "sub" in them', () => {
             let file = program.addOrReplaceFile('source/main.brs', `
@@ -1034,10 +1028,8 @@ describe('BrsFile', () => {
                 function DoSomething
                 end function
             `);
-            (0, chai_1.expect)(file.getDiagnostics().length).to.be.greaterThan(0);
-            (0, chai_1.expect)(file.getDiagnostics()[0]).to.deep.include({
-                file: file
-            });
+            (0, testHelpers_spec_1.expectHasDiagnostics)(file);
+            (0, chai_1.expect)(file.getDiagnostics()[0].file).to.equal(file);
             (0, chai_1.expect)(file.getDiagnostics()[0].range.start.line).to.equal(1);
         });
         it('supports using the `next` keyword in a for loop', () => {
@@ -1049,7 +1041,7 @@ describe('BrsFile', () => {
                     next
                 end sub
             `);
-            (0, chai_1.expect)(file.getDiagnostics()).to.be.empty;
+            (0, testHelpers_spec_1.expectZeroDiagnostics)(file);
         });
         //test is not working yet, but will be enabled when brs supports this syntax
         it('supports assigning functions to objects', () => {
@@ -1062,7 +1054,7 @@ describe('BrsFile', () => {
                     end sub
                 end function
             `);
-            (0, chai_1.expect)(file.getDiagnostics().length).to.equal(0);
+            (0, testHelpers_spec_1.expectZeroDiagnostics)(file);
         });
     });
     describe('findCallables', () => {
@@ -1183,7 +1175,6 @@ describe('BrsFile', () => {
                 }]);
         });
         it('finds function calls nested inside statements', () => {
-            var _a;
             program.addOrReplaceFile(`source/main.brs`, `
                 sub main()
                     if true then
@@ -1192,7 +1183,9 @@ describe('BrsFile', () => {
                 end sub
             `);
             program.validate();
-            (0, chai_1.expect)((_a = program.getDiagnostics()[0]) === null || _a === void 0 ? void 0 : _a.message).to.equal(DiagnosticMessages_1.DiagnosticMessages.callToUnknownFunction('DoesNotExist', 'source').message);
+            (0, testHelpers_spec_1.expectDiagnostics)(program, [
+                DiagnosticMessages_1.DiagnosticMessages.callToUnknownFunction('DoesNotExist', 'source')
+            ]);
         });
         it('finds arguments with variable values', () => {
             let file = new BrsFile_1.BrsFile('absolute_path/file.brs', 'relative_path/file.brs', program);
@@ -1406,7 +1399,11 @@ describe('BrsFile', () => {
             let hover = file.getHover(vscode_languageserver_1.Position.create(1, 28));
             (0, chai_1.expect)(hover).to.exist;
             (0, chai_1.expect)(hover.range).to.eql(vscode_languageserver_1.Range.create(1, 25, 1, 29));
-            (0, chai_1.expect)(hover.contents).to.equal('function Main(count? as dynamic) as dynamic');
+            (0, chai_1.expect)(hover.contents).to.equal([
+                '```brightscript',
+                'function Main(count? as dynamic) as dynamic',
+                '```'
+            ].join('\n'));
         });
         it('finds variable function hover in same scope', () => {
             let file = program.addOrReplaceFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
@@ -1419,7 +1416,36 @@ describe('BrsFile', () => {
             `);
             let hover = file.getHover(vscode_languageserver_1.Position.create(5, 24));
             (0, chai_1.expect)(hover.range).to.eql(vscode_languageserver_1.Range.create(5, 20, 5, 29));
-            (0, chai_1.expect)(hover.contents).to.equal('sub sayMyName(name as string) as void');
+            (0, chai_1.expect)(hover.contents).to.equal([
+                '```brightscript',
+                'sub sayMyName(name as string) as void',
+                '```'
+            ].join('\n'));
+        });
+        it('does not crash when hovering on built-in functions', async () => {
+            let file = program.addOrReplaceFile('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.addOrReplaceFile('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' }, `
@@ -1433,7 +1459,11 @@ describe('BrsFile', () => {
             `);
             let hover = file.getHover(vscode_languageserver_1.Position.create(2, 25));
             (0, chai_1.expect)(hover.range).to.eql(vscode_languageserver_1.Range.create(2, 20, 2, 29));
-            (0, chai_1.expect)(hover.contents).to.equal('sub sayMyName() as void');
+            (0, chai_1.expect)(hover.contents).to.equal([
+                '```brightscript',
+                'sub sayMyName() as void',
+                '```'
+            ].join('\n'));
         });
         it('finds function hover in scope', () => {
             let rootDir = process.cwd();
@@ -1453,7 +1483,53 @@ describe('BrsFile', () => {
             let hover = mainFile.getHover(vscode_languageserver_1.Position.create(2, 25));
             (0, chai_1.expect)(hover).to.exist;
             (0, chai_1.expect)(hover.range).to.eql(vscode_languageserver_1.Range.create(2, 20, 2, 29));
-            (0, chai_1.expect)(hover.contents).to.equal('sub sayMyName(name as string) as void');
+            (0, chai_1.expect)(hover.contents).to.equal([
+                '```brightscript',
+                'sub sayMyName(name as string) as void',
+                '```'
+            ].join('\n'));
+        });
+        it('includes markdown comments in hover.', async () => {
+            let rootDir = process.cwd();
+            program = new Program_1.Program({
+                rootDir: rootDir
+            });
+            const file = program.addOrReplaceFile('source/lib.brs', `
+                '
+                ' The main function
+                '
+                sub main()
+                    log("hello")
+                end sub
+
+                '
+                ' Prints a message to the log.
+                ' Works with *markdown* **content**
+                '
+                sub log(message as string)
+                    print message
+                end sub
+            `);
+            //hover over log("hello")
+            (0, chai_1.expect)((await program.getHover(file.pathAbsolute, vscode_languageserver_1.Position.create(5, 22))).contents).to.equal([
+                '```brightscript',
+                'sub log(message as string) as void',
+                '```',
+                '***',
+                '',
+                ' Prints a message to the log.',
+                ' Works with *markdown* **content**',
+                ''
+            ].join('\n'));
+            //hover over sub ma|in()
+            (0, chai_1.expect)((await program.getHover(file.pathAbsolute, vscode_languageserver_1.Position.create(4, 22))).contents).to.equal((0, testHelpers_spec_1.trim) `
+                \`\`\`brightscript
+                sub main() as void
+                \`\`\`
+                ***
+
+                 The main function
+                `);
         });
         it('handles mixed case `then` partions of conditionals', () => {
             let mainFile = program.addOrReplaceFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
@@ -1463,7 +1539,7 @@ describe('BrsFile', () => {
                     end if
                 end sub
             `);
-            (0, chai_1.expect)(mainFile.getDiagnostics()).to.be.lengthOf(0);
+            (0, testHelpers_spec_1.expectZeroDiagnostics)(mainFile);
             mainFile = program.addOrReplaceFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
                 sub Main()
                     if true Then
@@ -1471,7 +1547,7 @@ describe('BrsFile', () => {
                     end if
                 end sub
             `);
-            (0, chai_1.expect)(mainFile.getDiagnostics()).to.be.lengthOf(0);
+            (0, testHelpers_spec_1.expectZeroDiagnostics)(mainFile);
             mainFile = program.addOrReplaceFile({ src: `${rootDir}/source/main.brs`, dest: 'source/main.brs' }, `
                 sub Main()
                     if true THEN
@@ -1479,7 +1555,7 @@ describe('BrsFile', () => {
                     end if
                 end sub
             `);
-            (0, chai_1.expect)(mainFile.getDiagnostics()).to.be.lengthOf(0);
+            (0, testHelpers_spec_1.expectZeroDiagnostics)(mainFile);
         });
     });
     it('does not throw when encountering incomplete import statement', () => {
@@ -1788,9 +1864,9 @@ describe('BrsFile', () => {
         });
         it('computes correct locations for sourcemap', async () => {
             let source = `function abc(name)\n    firstName = name\nend function`;
-            let tokens = lexer_1.Lexer.scan(source).tokens
+            let tokens = Lexer_1.Lexer.scan(source).tokens
                 //remove newlines and EOF
-                .filter(x => x.kind !== lexer_1.TokenKind.Eof && x.kind !== lexer_1.TokenKind.Newline);
+                .filter(x => x.kind !== TokenKind_1.TokenKind.Eof && x.kind !== TokenKind_1.TokenKind.Newline);
             program.options.sourceMap = true;
             let result = testTranspile(source, source, 'none');
             //load the source map
@@ -2048,14 +2124,13 @@ describe('BrsFile', () => {
     describe('callfunc operator', () => {
         describe('transpile', () => {
             it('does not produce diagnostics', () => {
-                var _a;
                 program.addOrReplaceFile('source/main.bs', `
                     sub main()
                         someObject@.someFunction(paramObject.value)
                     end sub
                 `);
                 program.validate();
-                (0, chai_1.expect)((_a = program.getDiagnostics()[0]) === null || _a === void 0 ? void 0 : _a.message).not.to.exist;
+                (0, testHelpers_spec_1.expectZeroDiagnostics)(program);
             });
             it('sets invalid on empty callfunc', () => {
                 testTranspile(`
@@ -2095,13 +2170,12 @@ describe('BrsFile', () => {
                 name: 'transform callback',
                 afterFileParse: onParsed
             });
-            const file = new BrsFile_1.BrsFile(`absolute_path/file${ext}`, `relative_path/file${ext}`, program);
-            (0, chai_1.expect)(file.extension).to.equal(ext);
-            file.parse(`
+            file = program.addOrReplaceFile({ src: `absolute_path/file${ext}`, dest: `relative_path/file${ext}` }, `
                 sub Sum()
                     print "hello world"
                 end sub
             `);
+            (0, chai_1.expect)(file.extension).to.equal(ext);
             return file;
         }
         it('called for BRS file', () => {