Add test for open logs

readmeio · Sep 28, 2018 · 67a5384 · 67a5384
1 parent ea7cd5b
commit 67a5384
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/packages/api-logs/__tests__/index.test.jsx b/packages/api-logs/__tests__/index.test.jsx
@@ -105,6 +105,14 @@ describe('Logs', () => {
     expect(instance.state.group).toBe('1230');
   });
 
+  test('open window on log visit', () => {
+    const comp = shallow(<Logs {...props} />);
+    global.open = jest.fn();
+    const instance = comp.instance();
+    instance.visitLogItem(requestmodel);
+    expect(global.open).toBeCalled();
+  });
+
   test('when parsed agent is not Other', () => {
     const comp = shallow(<LogTest {...props} />);
     requestmodel.request.log.entries[0].request.headers[0].value = 'IE4.0';

diff --git a/packages/api-logs/index.jsx b/packages/api-logs/index.jsx
@@ -118,7 +118,7 @@ class Logs extends React.Component {
 
   visitLogItem(log) {
     const { baseUrl } = this.props;
-    window.location = `${baseUrl}logs/${log._id}`;
+    window.open(`${baseUrl}logs/${log._id}`);
   }
 
   renderLogs() {