Make the operator section navigable (#1570)

vmware-tanzu · Mar 10, 2020 · e77620e · e77620e
1 parent f278440
commit e77620e
Show file tree

Hide file tree

Showing 2 changed files with 25 additions and 3 deletions.
diff --git a/dashboard/src/components/Header/Header.test.tsx b/dashboard/src/components/Header/Header.test.tsx
@@ -51,7 +51,7 @@ describe("settings", () => {
 
   it("renders settings with reposPerNamespace", () => {
     const wrapper = shallow(
-      <Header {...defaultProps} featureFlags={{ reposPerNamespace: true }} />,
+      <Header {...defaultProps} featureFlags={{ reposPerNamespace: true, operators: false }} />,
     );
     const settingsbar = wrapper.find(".header__nav__submenu").first();
     const items = settingsbar.find("NavLink").map(p => p.props());
@@ -64,6 +64,23 @@ describe("settings", () => {
       expect(item.to).toBe(expectedItems[index].to);
     });
   });
+
+  it("renders operators link", () => {
+    const wrapper = shallow(
+      <Header {...defaultProps} featureFlags={{ reposPerNamespace: true, operators: true }} />,
+    );
+    const settingsbar = wrapper.find(".header__nav__submenu").first();
+    const items = settingsbar.find("NavLink").map(p => p.props());
+    const expectedItems = [
+      { children: "App Repositories", to: "/config/ns/default/repos" },
+      { children: "Service Brokers", to: "/config/brokers" },
+      { children: "Operators", to: "/operators/ns/default" },
+    ];
+    items.forEach((item, index) => {
+      expect(item.children).toBe(expectedItems[index].children);
+      expect(item.to).toBe(expectedItems[index].to);
+    });
+  });
 });
 
 it("updates state when the path changes", () => {

diff --git a/dashboard/src/components/Header/Header.tsx b/dashboard/src/components/Header/Header.tsx
@@ -24,7 +24,7 @@ interface IHeaderProps {
   setNamespace: (ns: string) => void;
   createNamespace: (ns: string) => Promise<boolean>;
   getNamespace: (ns: string) => void;
-  featureFlags: { reposPerNamespace: boolean };
+  featureFlags: { reposPerNamespace: boolean; operators: boolean };
 }
 
 interface IHeaderState {
@@ -34,7 +34,7 @@ interface IHeaderState {
 
 class Header extends React.Component<IHeaderProps, IHeaderState> {
   public static defaultProps = {
-    featureFlags: { reposPerNamespace: false },
+    featureFlags: { reposPerNamespace: false, operators: false },
   };
 
   // Defines the route
@@ -148,6 +148,11 @@ class Header extends React.Component<IHeaderProps, IHeaderState> {
                       <li role="none">
                         <NavLink to="/config/brokers">Service Brokers</NavLink>
                       </li>
+                      {this.props.featureFlags.operators && (
+                        <li role="none">
+                          <NavLink to={`/operators/ns/${namespace.current}`}>Operators</NavLink>
+                        </li>
+                      )}
                     </ul>
                   </li>
                   <li>