fix tests

kubeflow · Nov 21, 2018 · 8cdbaf1 · 8cdbaf1
1 parent 23f6f36
commit 8cdbaf1
Show file tree

Hide file tree

Showing 2 changed files with 39 additions and 14 deletions.
diff --git a/frontend/src/components/Toolbar.test.tsx b/frontend/src/components/Toolbar.test.tsx
@@ -87,7 +87,7 @@ describe('Toolbar', () => {
   });
 
   it('renders without actions, one breadcrumb, and a page name', () => {
-    const tree = shallow(<Toolbar breadcrumbs={breadcrumbs} actions={[]} history={history}
+    const tree = shallow(<Toolbar breadcrumbs={[breadcrumbs[0]]} actions={[]} history={history}
       pageTitle='test page title' />);
     expect(tree).toMatchSnapshot();
   });

diff --git a/frontend/src/components/__snapshots__/Toolbar.test.tsx.snap b/frontend/src/components/__snapshots__/Toolbar.test.tsx.snap
@@ -118,6 +118,22 @@ exports[`Toolbar renders primary action buttons 1`] = `
           test display name
         </Link>
       </span>
+      <span
+        className="flex"
+        key="1"
+        title="test display name2"
+      >
+        <pure(ChevronRightIcon)
+          className="chevron"
+        />
+        <Link
+          className="unstyled ellipsis link"
+          replace={false}
+          to="/some/test/path2"
+        >
+          test display name2
+        </Link>
+      </span>
     </div>
     <div
       className="flex"
@@ -137,10 +153,7 @@ exports[`Toolbar renders primary action buttons 1`] = `
       </WithStyles(Tooltip)>
       <span
         className="pageName ellipsis"
-        title="test display name2"
-      >
-        test display name2
-      </span>
+      />
     </div>
   </div>
   <div
@@ -229,9 +242,7 @@ exports[`Toolbar renders primary action buttons without outline, even if outline
       </WithStyles(Tooltip)>
       <span
         className="pageName ellipsis"
-      >
-        test page title
-      </span>
+      />
     </div>
   </div>
   <div
@@ -285,6 +296,22 @@ exports[`Toolbar renders with two breadcrumbs and two actions 1`] = `
           test display name
         </Link>
       </span>
+      <span
+        className="flex"
+        key="1"
+        title="test display name2"
+      >
+        <pure(ChevronRightIcon)
+          className="chevron"
+        />
+        <Link
+          className="unstyled ellipsis link"
+          replace={false}
+          to="/some/test/path2"
+        >
+          test display name2
+        </Link>
+      </span>
     </div>
     <div
       className="flex"
@@ -304,10 +331,7 @@ exports[`Toolbar renders with two breadcrumbs and two actions 1`] = `
       </WithStyles(Tooltip)>
       <span
         className="pageName ellipsis"
-        title="test display name2"
-      >
-        test display name2
-      </span>
+      />
     </div>
   </div>
   <div
@@ -461,9 +485,8 @@ exports[`Toolbar renders without actions, one breadcrumb, and a page name 1`] =
       </WithStyles(Tooltip)>
       <span
         className="pageName ellipsis"
-        title="test display name2"
       >
-        test display name2
+        test page title
       </span>
     </div>
   </div>
@@ -512,6 +535,7 @@ exports[`Toolbar renders without breadcrumbs and a component page title 1`] = `
       <div>
         <BusyButton
           busy={false}
+          className=""
           color="secondary"
           icon={[Function]}
           id="test id"
@@ -560,6 +584,7 @@ exports[`Toolbar renders without breadcrumbs and a string page title 1`] = `
       <div>
         <BusyButton
           busy={false}
+          className=""
           color="secondary"
           icon={[Function]}
           id="test id"