Fixed a bug in capabilities parsing in the runner for multiple quoted capabilities

Author: David Haeffner

packages/selenium-side-runner/src/__test__/capabilities.spec.js

@@ -51,6 +51,19 @@ describe('capabilities string parser', () => {
       binary: '/Applications/Google Chrome.app/Contents/MacOS/Google Chrome',
     })
   })
+  it('should parse multiple multiword capability keys', () => {
+    const expected = {
+      browserName: 'internet explorer',
+      version: 10,
+      platform: 'Windows 8.1',
+    }
+    const capabilitiesSingleQuotes =
+      "browserName='internet explorer' version=10.0 platform='Windows 8.1'"
+    expect(Capabilities.parseString(capabilitiesSingleQuotes)).toEqual(expected)
+    const capabilitiesDoubleQuotes =
+      'browserName="internet explorer" version=10.0 platform="Windows 8.1"'
+    expect(Capabilities.parseString(capabilitiesDoubleQuotes)).toEqual(expected)
+  })
   it('should parse boolean capability key', () => {
     const capabilities = 'javascriptEnabled=false databaseEnabled=true'
     expect(Capabilities.parseString(capabilities)).toEqual({

packages/selenium-side-runner/src/capabilities.js

@@ -30,7 +30,7 @@ export default {
 }
 
 function matchStringPairs(input) {
-  const regex = /([^\s=]*)\s?=\s?(".*"|'.*'|\[.*\]|[^\s]*)/g
+  const regex = /([^\s=]*)\s?=\s?(".*?"|'.*?'|\[.*\]|[^\s]*)/g
   let result
   const splitCapabilities = []
   while ((result = regex.exec(input)) !== null) {