Resolving unused settings bug when plugin is disabled

src/server/config/__tests__/config.js

@@ -238,24 +238,5 @@ describe('lib/config/config', function () {
       });
 
     });
-
-    describe('#removeSchema(key)', function () {
-      it('should completely remove the key', function () {
-        const config = new Config(Joi.object().keys({
-          a: Joi.number().default(1)
-        }));
-
-        expect(config.get('a')).to.be(1);
-        config.removeSchema('a');
-        expect(() => config.get('a')).to.throwException('Unknown config key');
-      });
-
-      it('only removes existing keys', function () {
-        const config = new Config(Joi.object());
-
-        expect(() => config.removeSchema('b')).to.throwException('Unknown schema');
-      });
-    });
-
   });
 });

src/server/config/config.js

@@ -42,16 +42,6 @@ module.exports = class Config {
     this.set(key, settings);
   }
 
-  removeSchema(key) {
-    if (!_.has(this[schemaExts], key)) {
-      throw new TypeError(`Unknown schema key: ${key}`);
-    }
-
-    this[schema] = null;
-    unset(this[schemaExts], key);
-    unset(this[vals], key);
-  }
-
   resetTo(obj) {
     this._commit(obj);
   }

src/server/plugins/plugin_collection.js

@@ -24,11 +24,6 @@ async function addPluginConfig(pluginCollection, plugin) {
   config.extendSchema(configSchema, transformedPluginSettings, plugin.configPrefix);
 }
 
-function removePluginConfig(pluginCollection, plugin) {
-  const { config } = pluginCollection.kbnServer;
-  config.removeSchema(plugin.configPrefix);
-}
-
 module.exports = class Plugins extends Collection {
 
   constructor(kbnServer) {
@@ -60,7 +55,6 @@ module.exports = class Plugins extends Collection {
   }
 
   async disable(plugin) {
-    removePluginConfig(this, plugin);
     this.delete(plugin);
   }