handle multiple selenium protocol versions for session-id (#164)

handle multiple selenium protocol versions for session-id

ui/src/containers/Capabilities/index.js

@@ -97,6 +97,10 @@ defer driver.Quit()
     }
 };
 
+export const sessionIdFrom = (res) => {
+    return res.response.sessionId || (res.response.value && res.response.value.sessionId) || '';
+};
+
 class Capabilities extends Component {
     static propTypes = {
         state: PropTypes.object,
@@ -147,7 +151,7 @@ class Capabilities extends Component {
             session.subscribe(
                 res => {
                   if (res.status === 200) {
-                    this.props.history.push(`/sessions/${res.response.value.sessionId}`)
+                    this.props.history.push(`/sessions/${sessionIdFrom(res)}`)
                   }
                 },
                 err => {

ui/src/containers/Capabilities/index.test.js

@@ -0,0 +1,26 @@
+import { sessionIdFrom } from "./index";
+
+it('handles old selenium protocol versions', () => {
+    expect(sessionIdFrom({
+        response: {
+            sessionId: 'session-1'
+        }
+    })).toBe('session-1')
+});
+
+it('handles new selenium protocol versions', () => {
+    expect(sessionIdFrom({
+        response: {
+            value: {
+                sessionId: 'session-2'
+            }
+        }
+    })).toBe('session-2')
+});
+
+it('handles wrong response as empty', () => {
+    expect(sessionIdFrom({
+        response: {
+        }
+    })).toBe('')
+});