fix(menu): make menu open idempotent

Author: kara

src/lib/menu/menu-trigger.ts

@@ -66,9 +66,11 @@ export class MdMenuTrigger implements AfterViewInit, OnDestroy {
   }
 
   openMenu(): void {
-    this._createOverlay();
-    this._overlayRef.attach(this._portal);
-    this._initMenu();
+    if (!this._menuOpen) {
+      this._createOverlay();
+      this._overlayRef.attach(this._portal);
+      this._initMenu();
+    }
   }
 
   closeMenu(): void {

src/lib/menu/menu.spec.ts

@@ -1,6 +1,6 @@
 import {TestBed, async} from '@angular/core/testing';
-import {Component} from '@angular/core';
-import {MdMenuModule} from './menu';
+import {Component, ViewChild} from '@angular/core';
+import {MdMenuModule, MdMenuTrigger} from './menu';
 
 
 describe('MdMenu', () => {
@@ -14,11 +14,24 @@ describe('MdMenu', () => {
     TestBed.compileComponents();
   }));
 
-  it('should add and remove focus class on focus/blur', () => {
+  it('should open the menu as an idempotent operation', () => {
     let fixture = TestBed.createComponent(TestMenu);
-    expect(fixture).toBeTruthy();
+    fixture.detectChanges();
+    expect(() => {
+      fixture.componentInstance.trigger.openMenu();
+      fixture.componentInstance.trigger.openMenu();
+    }).not.toThrowError();
   });
 });
 
-@Component({template: ``})
-class TestMenu {}
+@Component({
+  template: `
+    <button [md-menu-trigger-for]="menu">Toggle menu</button>
+    <md-menu #menu="mdMenu">
+      Content
+    </md-menu>
+  `
+})
+class TestMenu {
+  @ViewChild(MdMenuTrigger) trigger: MdMenuTrigger;
+}