Merge pull request elastic#7586 from ycombinator/management/add-dereg…

…ister-method

Adding method to deregister a (child) section

src/ui/public/management/__tests__/section.js

@@ -58,6 +58,27 @@ describe('ManagementSection', () => {
     });
   });
 
+  describe('deregister', () => {
+    let section;
+
+    beforeEach(() => {
+      section = new ManagementSection('kibana');
+      section.register('about');
+    });
+
+    it ('deregisters an existing section', () => {
+      section.deregister('about');
+      expect(section.items).to.have.length(0);
+    });
+
+    it ('allows deregistering a section more than once', () => {
+      section.deregister('about');
+      section.deregister('about');
+      expect(section.items).to.have.length(0);
+    });
+
+  });
+
   describe('getSection', () => {
     let section;
 

src/ui/public/management/section.js

@@ -44,6 +44,15 @@ export default class ManagementSection {
     return item;
   }
 
+  /**
+  * Deregisters a section
+  *
+  * @param {string} id
+  */
+  deregister(id) {
+    this.items.remove(item => item.id === id);
+  }
+
   /**
    * Determine if an id is already registered
    *