Delete unregister function, add unit test for getProvidersForPath.

Author: Mark-Simulacrum

src/language/CodeInspection.js

@@ -468,30 +468,6 @@ define(function (require, exports, module) {
         run();  // in case a file of this type is open currently
     }
 
-    /**
-     * Deregisters a provider, given a languageId and the provider.
-     * Allows code to remove providers which provide the same functionality
-     * as the code provides
-     *
-     * @param {string} languageId
-     * @param {{name:string, scanFileAsync:?function(string, string):!{$.Promise},
-     *         scanFile:?function(string, string):?{errors:!Array, aborted:boolean}}} provider
-     *
-     * @return {boolean} Returns true if at least one provider was removed.
-     */
-    function unregister(languageId, providerName) {
-        var isRemoved = false;
-        if (_providers[languageId]) {
-            _.remove(_providers[languageId], function (registeredProvider) {
-                if (registeredProvider.name === providerName) {
-                    isRemoved = true;
-                    return true;
-                }
-            });
-        }
-        return isRemoved;
-    }
-
     /**
      * Update DocumentManager listeners.
      */
@@ -668,7 +644,6 @@ define(function (require, exports, module) {
 
     // Public API
     exports.register            = register;
-    exports.unregister          = unregister;
     exports.Type                = Type;
     exports.toggleEnabled       = toggleEnabled;
     exports.inspectFile         = inspectFile;

test/spec/CodeInspection-test.js

@@ -167,6 +167,19 @@ define(function (require, exports, module) {
                 });
             });
 
+            it("should get the correct linter given a file path", function () {
+                var codeInspector1 = createCodeInspector("text linter 1", successfulLintResult());
+                var codeInspector2 = createCodeInspector("text linter 2", successfulLintResult());
+
+                CodeInspection.register("javascript", codeInspector1);
+                CodeInspection.register("javascript", codeInspector2);
+
+                var providers = CodeInspection.getProvidersForPath("test.js");
+                expect(providers.length).toBe(2);
+                expect(providers[0]).toBe(codeInspector1);
+                expect(providers[1]).toBe(codeInspector2);
+            });
+
             it("should run two linters", function () {
                 var codeInspector1 = createCodeInspector("text linter 1", successfulLintResult());
                 var codeInspector2 = createCodeInspector("text linter 2", successfulLintResult());