brighterscript 0.70.4 → 0.71.1

package/dist/bscPlugin/codeActions/CodeActionsProcessor.spec.js

@@ -6,6 +6,7 @@ const Program_1 = require("../../Program");
 const testHelpers_spec_1 = require("../../testHelpers.spec");
 const util_1 = require("../../util");
 const testHelpers_spec_2 = require("../../testHelpers.spec");
+const CodeActionsProcessor_1 = require("./CodeActionsProcessor");
 describe('CodeActionsProcessor', () => {
     let program;
     beforeEach(() => {
@@ -222,6 +223,42 @@ describe('CodeActionsProcessor', () => {
                 `import "pkg:/source/second.bs"`
             ]);
         });
+        it('clears suggestedImports after process() so the same instance could be reused in the future', () => {
+            program.setFile('source/lib.bs', `
+                function doSomething()
+                end function
+            `);
+            program.setFile('components/comp1.xml', (0, testHelpers_spec_1.trim) `
+                <?xml version="1.0" encoding="utf-8" ?>
+                <component name="comp1" extends="Scene">
+                    <script uri="comp1.bs" />
+                </component>
+            `);
+            const file = program.setFile('components/comp1.bs', `
+                sub init()
+                    doSomething()
+                end sub
+            `);
+            program.validate();
+            const range = util_1.util.createRange(2, 24, 2, 24);
+            const event = {
+                program: program,
+                file: file,
+                range: range,
+                scopes: program.getScopesForFile(file),
+                diagnostics: program.getDiagnostics().filter(d => d.file === file && util_1.util.rangesIntersectOrTouch(d.range, range)),
+                codeActions: []
+            };
+            const processor = new CodeActionsProcessor_1.CodeActionsProcessor(event);
+            processor.process();
+            const firstCallTitles = event.codeActions.map(x => x.title).sort();
+            // Reset codeActions and call process() again on the same instance.
+            // If suggestedImports were NOT cleared, the second call would return no import suggestions.
+            event.codeActions = [];
+            processor.process();
+            const secondCallTitles = event.codeActions.map(x => x.title).sort();
+            (0, chai_config_spec_1.expect)(secondCallTitles).to.eql(firstCallTitles);
+        });
         it('suggests files for second part of missing namespace', () => {
             program.setFile('source/first.bs', `
                 namespace alpha
@@ -246,6 +283,119 @@ describe('CodeActionsProcessor', () => {
             testGetCodeActions(file, util_1.util.createRange(3, 34, 3, 34), [`import "pkg:/source/second.bs"`]);
         });
     });
+    describe('Fix all: Auto fixable missing imports', () => {
+        it('offers fix-all when multiple unambiguous imports are missing', () => {
+            program.setFile('source/lib1.bs', `
+                function doSomething()
+                end function
+            `);
+            program.setFile('source/lib2.bs', `
+                function doSomethingElse()
+                end function
+            `);
+            program.setFile('components/MainScene.xml', (0, testHelpers_spec_1.trim) `<component name="MainScene"></component>`);
+            const file = program.setFile('components/MainScene.bs', `
+                sub init()
+                    doSomething()
+                    doSomethingElse()
+                end sub
+            `);
+            // doSome|thing()
+            testGetCodeActions(file, util_1.util.createRange(2, 26, 2, 26), [
+                `Fix all: Auto fixable missing imports`,
+                `import "pkg:/source/lib1.bs"`
+            ]);
+        });
+        it('does not offer fix-all when only one import is missing', () => {
+            program.setFile('source/lib1.bs', `
+                function doSomething()
+                end function
+            `);
+            program.setFile('components/MainScene.xml', (0, testHelpers_spec_1.trim) `<component name="MainScene"></component>`);
+            const file = program.setFile('components/MainScene.bs', `
+                sub init()
+                    doSomething()
+                end sub
+            `);
+            // doSome|thing()
+            testGetCodeActions(file, util_1.util.createRange(2, 26, 2, 26), [
+                `import "pkg:/source/lib1.bs"`
+            ]);
+        });
+        it('excludes ambiguous names from fix-all', () => {
+            program.setFile('source/lib1.bs', `
+                function doSomething()
+                end function
+            `);
+            program.setFile('source/lib2.bs', `
+                function doSomething()
+                end function
+            `);
+            program.setFile('source/lib3.bs', `
+                function doSomethingUnambiguous()
+                end function
+            `);
+            program.setFile('components/MainScene.xml', (0, testHelpers_spec_1.trim) `<component name="MainScene"></component>`);
+            const file = program.setFile('components/MainScene.bs', `
+                sub init()
+                    doSomething()
+                    doSomethingUnambiguous()
+                end sub
+            `);
+            program.validate();
+            const actions = program.getCodeActions(file.srcPath, util_1.util.createRange(2, 26, 2, 26));
+            // fix-all only has the unambiguous import, not doSomething (which has 2 options)
+            const fixAll = actions.find(a => a.title === 'Fix all: Auto fixable missing imports');
+            (0, chai_config_spec_1.expect)(fixAll).to.be.undefined;
+        });
+        it('includes class imports in fix-all', () => {
+            program.setFile('source/lib1.bs', `
+                function doSomething()
+                end function
+            `);
+            program.setFile('source/MyClass.bs', `
+                class MyClass
+                end class
+            `);
+            program.setFile('components/MainScene.xml', (0, testHelpers_spec_1.trim) `<component name="MainScene"></component>`);
+            const file = program.setFile('components/MainScene.bs', `
+                sub init()
+                    doSomething()
+                    obj = new MyClass()
+                end sub
+            `);
+            // doSome|thing()
+            testGetCodeActions(file, util_1.util.createRange(2, 26, 2, 26), [
+                `Fix all: Auto fixable missing imports`,
+                `import "pkg:/source/lib1.bs"`
+            ]);
+        });
+        it('deduplicates when multiple missing names resolve to the same file', () => {
+            program.setFile('source/lib1.bs', `
+                function doSomething()
+                end function
+                function doSomethingElse()
+                end function
+            `);
+            program.setFile('source/lib2.bs', `
+                function doThird()
+                end function
+            `);
+            program.setFile('components/MainScene.xml', (0, testHelpers_spec_1.trim) `<component name="MainScene"></component>`);
+            const file = program.setFile('components/MainScene.bs', `
+                sub init()
+                    doSomething()
+                    doSomethingElse()
+                    doThird()
+                end sub
+            `);
+            program.validate();
+            const actions = program.getCodeActions(file.srcPath, util_1.util.createRange(2, 26, 2, 26));
+            const fixAll = actions.find(a => a.title === 'Fix all: Auto fixable missing imports');
+            // lib1.bs and lib2.bs — only 2 changes despite 3 missing names
+            (0, chai_config_spec_1.expect)(Object.values(fixAll.edit.changes)[0]).to.have.lengthOf(2);
+        });
+    });
     it('suggests imports at very start and very end of diagnostic', () => {
         program.setFile('source/first.bs', `
             namespace alpha
@@ -275,6 +425,26 @@ describe('CodeActionsProcessor', () => {
             // return tr|ue
             testGetCodeActions(file, util_1.util.createRange(3, 29, 3, 29), [`Convert sub to function`, `Remove return value`]);
         });
+        it('suggests converting sub to function with inline body', () => {
+            const file = program.setFile('source/main.brs', `
+                sub test() : print "onItemContentChange"
+                    return true
+                end sub
+            `);
+            // return tr|ue
+            testGetCodeActions(file, util_1.util.createRange(2, 28, 2, 28), [`Convert sub to function`, `Remove return value`]);
+            // verify only the `sub` and `end sub` keywords are replaced, not the `: print ...` inline body
+            program.validate();
+            const actions = program.getCodeActions(file.srcPath, util_1.util.createRange(2, 28, 2, 28));
+            const convertAction = actions.find(a => a.title === 'Convert sub to function');
+            const changes = Object.values(convertAction.edit.changes)[0];
+            // change[0] replaces `sub` keyword only
+            (0, chai_config_spec_1.expect)(changes[0].range).to.eql(util_1.util.createRange(1, 16, 1, 19));
+            (0, chai_config_spec_1.expect)(changes[0].newText).to.eql('function');
+            // change[1] replaces `end sub` keyword only
+            (0, chai_config_spec_1.expect)(changes[1].range).to.eql(util_1.util.createRange(3, 16, 3, 23));
+            (0, chai_config_spec_1.expect)(changes[1].newText).to.eql('end function');
+        });
         it('suggests deleting the return type from void function', () => {
             const file = program.setFile('source/main.brs', `
                 function test() as void
@@ -285,6 +455,62 @@ describe('CodeActionsProcessor', () => {
             // return tr|ue
             testGetCodeActions(file, util_1.util.createRange(3, 29, 3, 29), [`Remove return type from function declaration`, `Remove return value`]);
         });
+        it('suggests deleting only the return type from void function with inline body', () => {
+            const file = program.setFile('source/main.brs', `
+                function test() as void : print "onItemContentChange"
+                    return "test"
+                end function
+            `);
+            // return |"test"
+            testGetCodeActions(file, util_1.util.createRange(2, 28, 2, 28), [`Remove return type from function declaration`, `Remove return value`]);
+            // verify only ` as void` is deleted, not the `: print ...` inline body
+            program.validate();
+            const actions = program.getCodeActions(file.srcPath, util_1.util.createRange(2, 28, 2, 28));
+            const removeTypeAction = actions.find(a => a.title === 'Remove return type from function declaration');
+            const changes = Object.values(removeTypeAction.edit.changes)[0];
+            // range should span ` as void` only, starting after `)` and ending at the close of `void`
+            (0, chai_config_spec_1.expect)(changes[0].range).to.eql(util_1.util.createRange(1, 31, 1, 39));
+        });
+        it('offers fix-all when multiple void-return violations exist in the file', () => {
+            const file = program.setFile('source/main.brs', `
+                sub test1()
+                    return true
+                end sub
+                sub test2()
+                    return false
+                end sub
+            `);
+            // return tr|ue (first violation)
+            testGetCodeActions(file, util_1.util.createRange(2, 27, 2, 27), [
+                `Convert sub to function`,
+                `Fix all: Remove void return values`,
+                `Remove return value`
+            ]);
+        });
+        it('does not offer fix-all when only one void-return violation exists', () => {
+            const file = program.setFile('source/main.brs', `
+                sub test1()
+                    return true
+                end sub
+                sub test2()
+                end sub
+            `);
+            // return tr|ue
+            testGetCodeActions(file, util_1.util.createRange(2, 27, 2, 27), [`Convert sub to function`, `Remove return value`]);
+        });
+        it('does not duplicate fix-all when multiple violations are at the cursor range', () => {
+            const file = program.setFile('source/main.brs', `
+                sub test1()
+                    return true
+                end sub
+                sub test2()
+                    return false
+                end sub
+            `);
+            program.validate();
+            const actions = program.getCodeActions(file.srcPath, util_1.util.createRange(2, 20, 4, 20));
+            (0, chai_config_spec_1.expect)(actions.filter(a => a.title === 'Fix all: Remove void return values')).to.have.lengthOf(1);
+        });
     });
     describe('typed function/sub empty return', () => {
         it('suggests deleting the return value and converting the sub to a function', () => {
@@ -297,6 +523,23 @@ describe('CodeActionsProcessor', () => {
             // ret|urn
             testGetCodeActions(file, util_1.util.createRange(3, 23, 3, 23), [`Add void return type to function declaration`, `Convert function to sub`]);
         });
+        it('suggests adding void return type to function with inline body', () => {
+            const file = program.setFile('source/main.brs', `
+                function test() : print "onItemContentChange"
+                    return
+                end function
+            `);
+            // ret|urn
+            testGetCodeActions(file, util_1.util.createRange(2, 23, 2, 23), [`Add void return type to function declaration`, `Convert function to sub`]);
+            // verify ` as void` is inserted after `)`, not after the inline body
+            program.validate();
+            const actions = program.getCodeActions(file.srcPath, util_1.util.createRange(2, 23, 2, 23));
+            const addVoidAction = actions.find(a => a.title === 'Add void return type to function declaration');
+            const changes = Object.values(addVoidAction.edit.changes)[0];
+            // insert position should be immediately after `)`, before ` : print ...`
+            (0, chai_config_spec_1.expect)(changes[0].range).to.eql(util_1.util.createRange(1, 31, 1, 31));
+            (0, chai_config_spec_1.expect)(changes[0].newText).to.eql(' as void');
+        });
         it('suggests deleting the return type from void function', () => {
             const file = program.setFile('source/main.brs', `
                 sub test() as integer
@@ -307,6 +550,338 @@ describe('CodeActionsProcessor', () => {
             // ret|urn
             testGetCodeActions(file, util_1.util.createRange(3, 23, 3, 23), [`Remove return type from sub declaration`]);
         });
+        it('suggests deleting only the return type from sub with inline body', () => {
+            const file = program.setFile('source/main.brs', `
+                sub test() as integer : print "onItemContentChange"
+                    return
+                end sub
+            `);
+            // ret|urn
+            testGetCodeActions(file, util_1.util.createRange(2, 23, 2, 23), [`Remove return type from sub declaration`]);
+            // verify only ` as integer` is deleted, not the `: print ...` inline body
+            program.validate();
+            const actions = program.getCodeActions(file.srcPath, util_1.util.createRange(2, 23, 2, 23));
+            const removeTypeAction = actions.find(a => a.title === 'Remove return type from sub declaration');
+            const changes = Object.values(removeTypeAction.edit.changes)[0];
+            // range should span ` as integer` only, starting after `)` and ending at the close of `integer`
+            (0, chai_config_spec_1.expect)(changes[0].range).to.eql(util_1.util.createRange(1, 26, 1, 37));
+        });
+        it('offers fix-all for multiple subs with return types', () => {
+            const file = program.setFile('source/main.brs', `
+                sub test1() as integer
+                    return
+                end sub
+                sub test2() as string
+                    return
+                end sub
+            `);
+            // ret|urn (first violation)
+            testGetCodeActions(file, util_1.util.createRange(2, 24, 2, 24), [
+                `Fix all: Remove return type from sub declarations`,
+                `Remove return type from sub declaration`
+            ]);
+        });
+        it('offers fix-all for multiple functions with missing return types', () => {
+            const file = program.setFile('source/main.brs', `
+                function test1()
+                    return
+                end function
+                function test2()
+                    return
+                end function
+            `);
+            // ret|urn (first violation)
+            testGetCodeActions(file, util_1.util.createRange(2, 24, 2, 24), [
+                `Add void return type to function declaration`,
+                `Convert function to sub`,
+                `Fix all: Add void return type to function declarations`
+            ]);
+        });
+        it('does not offer fix-all when only one non-void-return violation exists', () => {
+            const file = program.setFile('source/main.brs', `
+                sub test1() as integer
+                    return
+                end sub
+            `);
+            // ret|urn
+            testGetCodeActions(file, util_1.util.createRange(2, 24, 2, 24), [`Remove return type from sub declaration`]);
+        });
+        it('deduplicates fix-all changes when one function has multiple bare returns', () => {
+            const file = program.setFile('source/main.brs', `
+                sub test1() as integer
+                    return
+                    return
+                end sub
+                sub test2() as string
+                    return
+                end sub
+            `);
+            program.validate();
+            const actions = program.getCodeActions(file.srcPath, util_1.util.createRange(2, 24, 2, 24));
+            const fixAll = actions.find(a => a.title === 'Fix all: Remove return type from sub declarations');
+            // Two unique functions → two changes (not three)
+            (0, chai_config_spec_1.expect)(Object.values(fixAll.edit.changes)[0]).to.have.lengthOf(2);
+        });
+    });
+    describe('referencedFileDoesNotExist', () => {
+        it('offers to remove the script import line', () => {
+            const file = program.setFile('components/comp1.xml', (0, testHelpers_spec_1.trim) `
+                <?xml version="1.0" encoding="utf-8" ?>
+                <component name="comp1" extends="Scene">
+                    <script type="text/brightscript" uri="pkg:/source/missing.brs" />
+                </component>
+            `);
+            // uri="pkg:/source/missing.brs" is on line 2
+            testGetCodeActions(file, util_1.util.createRange(2, 50, 2, 50), ['Remove script import']);
+        });
+        it('deletes the entire script tag line', () => {
+            const file = program.setFile('components/comp1.xml', (0, testHelpers_spec_1.trim) `
+                <?xml version="1.0" encoding="utf-8" ?>
+                <component name="comp1" extends="Scene">
+                    <script type="text/brightscript" uri="pkg:/source/missing.brs" />
+                </component>
+            `);
+            program.validate();
+            (0, testHelpers_spec_1.expectCodeActions)(() => {
+                program.getCodeActions(file.srcPath, util_1.util.createRange(2, 50, 2, 50));
+            }, [{
+                    title: 'Remove script import',
+                    kind: 'quickfix',
+                    changes: [{
+                            type: 'delete',
+                            filePath: (0, util_1.standardizePath) `${testHelpers_spec_2.rootDir}/components/comp1.xml`,
+                            range: util_1.util.createRange(2, 0, 3, 0)
+                        }]
+                }]);
+        });
+        it('offers fix-all when multiple missing imports exist', () => {
+            const file = program.setFile('components/comp1.xml', (0, testHelpers_spec_1.trim) `
+                <?xml version="1.0" encoding="utf-8" ?>
+                <component name="comp1" extends="Scene">
+                    <script type="text/brightscript" uri="pkg:/source/missing1.brs" />
+                    <script type="text/brightscript" uri="pkg:/source/missing2.brs" />
+                </component>
+            `);
+            testGetCodeActions(file, util_1.util.createRange(2, 50, 2, 50), [
+                'Fix all: Remove script imports',
+                'Remove script import'
+            ]);
+        });
+    });
+    describe('unnecessaryScriptImportInChildFromParent', () => {
+        it('offers to remove the redundant script import', () => {
+            program.setFile('components/parent.xml', (0, testHelpers_spec_1.trim) `
+                <?xml version="1.0" encoding="utf-8" ?>
+                <component name="ParentScene" extends="Scene">
+                    <script type="text/brightscript" uri="pkg:/source/lib.brs" />
+                </component>
+            `);
+            program.setFile('source/lib.brs', '');
+            const file = program.setFile('components/child.xml', (0, testHelpers_spec_1.trim) `
+                <?xml version="1.0" encoding="utf-8" ?>
+                <component name="ChildScene" extends="ParentScene">
+                    <script type="text/brightscript" uri="pkg:/source/lib.brs" />
+                </component>
+            `);
+            // uri on line 2 of child.xml
+            testGetCodeActions(file, util_1.util.createRange(2, 50, 2, 50), ['Remove redundant script import']);
+        });
+        it('offers fix-all when multiple redundant imports exist', () => {
+            program.setFile('components/parent.xml', (0, testHelpers_spec_1.trim) `
+                <?xml version="1.0" encoding="utf-8" ?>
+                <component name="ParentScene" extends="Scene">
+                    <script type="text/brightscript" uri="pkg:/source/lib1.brs" />
+                    <script type="text/brightscript" uri="pkg:/source/lib2.brs" />
+                </component>
+            `);
+            program.setFile('source/lib1.brs', '');
+            program.setFile('source/lib2.brs', '');
+            const file = program.setFile('components/child.xml', (0, testHelpers_spec_1.trim) `
+                <?xml version="1.0" encoding="utf-8" ?>
+                <component name="ChildScene" extends="ParentScene">
+                    <script type="text/brightscript" uri="pkg:/source/lib1.brs" />
+                    <script type="text/brightscript" uri="pkg:/source/lib2.brs" />
+                </component>
+            `);
+            testGetCodeActions(file, util_1.util.createRange(2, 50, 2, 50), [
+                'Fix all: Remove redundant script imports',
+                'Remove redundant script import'
+            ]);
+        });
+    });
+    describe('unnecessaryCodebehindScriptImport', () => {
+        it('offers to remove the unnecessary codebehind import', () => {
+            const file = program.setFile('components/comp1.xml', (0, testHelpers_spec_1.trim) `
+                <?xml version="1.0" encoding="utf-8" ?>
+                <component name="comp1" extends="Scene">
+                    <script type="text/brightscript" uri="pkg:/components/comp1.brs" />
+                </component>
+            `);
+            program.setFile('components/comp1.brs', '');
+            // uri on line 2
+            testGetCodeActions(file, util_1.util.createRange(2, 50, 2, 50), ['Remove unnecessary codebehind import']);
+        });
+    });
+    describe('scriptImportCaseMismatch', () => {
+        it('offers to fix the casing of the script import path', () => {
+            const file = program.setFile('components/comp1.xml', (0, testHelpers_spec_1.trim) `
+                <?xml version="1.0" encoding="utf-8" ?>
+                <component name="comp1" extends="Scene">
+                    <script type="text/brightscript" uri="pkg:/SOURCE/lib.brs" />
+                </component>
+            `);
+            program.setFile('source/lib.brs', '');
+            testGetCodeActions(file, util_1.util.createRange(2, 50, 2, 50), ['Fix script import path casing']);
+        });
+        it('replaces the URI value with the correctly-cased path', () => {
+            const file = program.setFile('components/comp1.xml', (0, testHelpers_spec_1.trim) `
+                <?xml version="1.0" encoding="utf-8" ?>
+                <component name="comp1" extends="Scene">
+                    <script type="text/brightscript" uri="pkg:/SOURCE/lib.brs" />
+                </component>
+            `);
+            program.setFile('source/lib.brs', '');
+            program.validate();
+            const actions = program.getCodeActions(file.srcPath, util_1.util.createRange(2, 50, 2, 50));
+            const fix = actions.find(a => a.title === 'Fix script import path casing');
+            const changes = Object.values(fix.edit.changes)[0];
+            (0, chai_config_spec_1.expect)(changes[0].newText).to.equal('pkg:/source/lib.brs');
+        });
+        it('offers fix-all when multiple case-mismatched imports exist', () => {
+            const file = program.setFile('components/comp1.xml', (0, testHelpers_spec_1.trim) `
+                <?xml version="1.0" encoding="utf-8" ?>
+                <component name="comp1" extends="Scene">
+                    <script type="text/brightscript" uri="pkg:/SOURCE/lib1.brs" />
+                    <script type="text/brightscript" uri="pkg:/SOURCE/lib2.brs" />
+                </component>
+            `);
+            program.setFile('source/lib1.brs', '');
+            program.setFile('source/lib2.brs', '');
+            testGetCodeActions(file, util_1.util.createRange(2, 50, 2, 50), [
+                'Fix all: Fix script import path casing',
+                'Fix script import path casing'
+            ]);
+        });
+        it('offers to fix the casing of a relative script import path', () => {
+            const file = program.setFile('components/comp1.xml', (0, testHelpers_spec_1.trim) `
+                <?xml version="1.0" encoding="utf-8" ?>
+                <component name="comp1" extends="Scene">
+                    <script type="text/brightscript" uri="LIB.brs" />
+                </component>
+            `);
+            program.setFile('components/lib.brs', '');
+            testGetCodeActions(file, util_1.util.createRange(2, 45, 2, 45), ['Fix script import path casing']);
+        });
+        it('replaces a relative URI with the correctly-cased relative path', () => {
+            const file = program.setFile('components/comp1.xml', (0, testHelpers_spec_1.trim) `
+                <?xml version="1.0" encoding="utf-8" ?>
+                <component name="comp1" extends="Scene">
+                    <script type="text/brightscript" uri="LIB.brs" />
+                </component>
+            `);
+            program.setFile('components/lib.brs', '');
+            program.validate();
+            const actions = program.getCodeActions(file.srcPath, util_1.util.createRange(2, 45, 2, 45));
+            const fix = actions.find(a => a.title === 'Fix script import path casing');
+            const changes = Object.values(fix.edit.changes)[0];
+            (0, chai_config_spec_1.expect)(changes[0].newText).to.equal('lib.brs');
+        });
+        it('replaces a cross-directory relative URI with the correctly-cased relative path', () => {
+            const file = program.setFile('components/sub/comp1.xml', (0, testHelpers_spec_1.trim) `
+                <?xml version="1.0" encoding="utf-8" ?>
+                <component name="comp1" extends="Scene">
+                    <script type="text/brightscript" uri="../utils/HELPER.brs" />
+                </component>
+            `);
+            program.setFile('components/utils/helper.brs', '');
+            program.validate();
+            const actions = program.getCodeActions(file.srcPath, util_1.util.createRange(2, 50, 2, 50));
+            const fix = actions.find(a => a.title === 'Fix script import path casing');
+            const changes = Object.values(fix.edit.changes)[0];
+            (0, chai_config_spec_1.expect)(changes[0].newText).to.equal('../utils/helper.brs');
+        });
+    });
+    describe('missingOverrideKeyword', () => {
+        it('offers to add the override keyword', () => {
+            program.setFile('components/comp1.xml', (0, testHelpers_spec_1.trim) `<component name="comp1" extends="Scene"></component>`);
+            const file = program.setFile('components/comp1.bs', `
+                class Animal
+                    function speak()
+                    end function
+                end class
+                class Dog extends Animal
+                    function speak()
+                    end function
+                end class
+            `);
+            // "function speak()" in Dog — diagnostic starts at the function keyword on line 6
+            testGetCodeActions(file, util_1.util.createRange(6, 20, 6, 20), [`Add missing 'override' keyword`]);
+        });
+        it('inserts override before the function keyword', () => {
+            program.setFile('components/comp1.xml', (0, testHelpers_spec_1.trim) `<component name="comp1" extends="Scene"></component>`);
+            const file = program.setFile('components/comp1.bs', `
+                class Animal
+                    function speak()
+                    end function
+                end class
+                class Dog extends Animal
+                    function speak()
+                    end function
+                end class
+            `);
+            program.validate();
+            const actions = program.getCodeActions(file.srcPath, util_1.util.createRange(6, 20, 6, 20));
+            const fix = actions.find(a => a.title === `Add missing 'override' keyword`);
+            const changes = Object.values(fix.edit.changes)[0];
+            (0, chai_config_spec_1.expect)(changes[0].newText).to.equal('override ');
+        });
+        it('offers fix-all when multiple methods are missing override', () => {
+            program.setFile('components/comp1.xml', (0, testHelpers_spec_1.trim) `<component name="comp1" extends="Scene"></component>`);
+            const file = program.setFile('components/comp1.bs', `
+                class Animal
+                    function speak()
+                    end function
+                    function move()
+                    end function
+                end class
+                class Dog extends Animal
+                    function speak()
+                    end function
+                    function move()
+                    end function
+                end class
+            `);
+            testGetCodeActions(file, util_1.util.createRange(8, 20, 8, 20), [
+                `Add missing 'override' keyword`,
+                `Fix all: Add missing 'override' keywords`
+            ]);
+        });
+    });
+    describe('cannotUseOverrideKeywordOnConstructorFunction', () => {
+        it('offers to remove override from constructor', () => {
+            const file = program.setFile('source/main.bs', `
+                class Dog
+                    override function new()
+                    end function
+                end class
+            `);
+            // "override" is on line 2
+            testGetCodeActions(file, util_1.util.createRange(2, 20, 2, 20), [`Remove 'override' from constructor`]);
+        });
+        it('deletes the override keyword and trailing space', () => {
+            const file = program.setFile('source/main.bs', `
+                class Dog
+                    override function new()
+                    end function
+                end class
+            `);
+            program.validate();
+            const actions = program.getCodeActions(file.srcPath, util_1.util.createRange(2, 20, 2, 20));
+            const fix = actions.find(a => a.title === `Remove 'override' from constructor`);
+            const changes = Object.values(fix.edit.changes)[0];
+            // range covers "override " (8 chars + 1 space)
+            (0, chai_config_spec_1.expect)(changes[0].range.start.character).to.equal(changes[0].range.end.character - 9);
+        });
     });
 });
 //# sourceMappingURL=CodeActionsProcessor.spec.js.map