Merge pull request #9 from browserstack/args

args

Author: tr4n2uil

src/main/java/com/browserstack/local/Local.java

@@ -21,19 +21,19 @@ public class Local {
 
     public Local() {
         parameters = new HashMap<String, String>();
-        parameters.put("v", "-vvv");
-        parameters.put("f", "-f");
-        parameters.put("force", "-force");
-        parameters.put("only", "-only");
-        parameters.put("forcelocal", "-forcelocal");
-        parameters.put("localIdentifier", "-localIdentifier");
-        parameters.put("onlyAutomate", "-onlyAutomate");
-        parameters.put("proxyHost", "-proxyHost");
-        parameters.put("proxyPort", "-proxyPort");
-        parameters.put("proxyUser", "-proxyUser");
-        parameters.put("proxyPass", "-proxyPass");
-        parameters.put("forceproxy", "-forceproxy");
-        parameters.put("hosts", "-hosts");
+        parameters.put("-v", "-vvv");
+        parameters.put("-f", "-f");
+        parameters.put("-force", "-force");
+        parameters.put("-only", "-only");
+        parameters.put("-forcelocal", "-forcelocal");
+        parameters.put("-localIdentifier", "-localIdentifier");
+        parameters.put("-onlyAutomate", "-onlyAutomate");
+        parameters.put("-proxyHost", "-proxyHost");
+        parameters.put("-proxyPort", "-proxyPort");
+        parameters.put("-proxyUser", "-proxyUser");
+        parameters.put("-proxyPass", "-proxyPass");
+        parameters.put("-forceproxy", "-forceproxy");
+        parameters.put("-hosts", "-hosts");
     }
 
     /**
@@ -57,7 +57,7 @@ public void start(Map<String, String> options) throws Exception {
         command.add("-logFile");
         command.add(logFilePath);
 
-        command.add(options.get("key"));
+        command.add(options.get("-key"));
         makeCommand(options);
 
         if (options.get("onlyCommand") != null) return;

src/test/java/com/browserstack/local/BrowserStackLocalTest.java

@@ -18,7 +18,7 @@ public class BrowserStackLocalTest {
     public void setUp() throws Exception {
         l = new Local();
         options = new HashMap<String, String>();
-        options.put("key", System.getenv("BROWSERSTACK_ACCESS_KEY"));
+        options.put("-key", System.getenv("BROWSERSTACK_ACCESS_KEY"));
     }
 
     @Test
@@ -42,15 +42,15 @@ public void testMultipleBinary() throws Exception {
 
     @Test
     public void testEnableVerbose() throws Exception {
-        options.put("v", "");
+        options.put("-v", "");
         options.put("onlyCommand", "true");
         l.start(options);
         assertTrue(l.command.contains("-vvv"));
     }
 
     @Test
     public void testSetFolder() throws Exception {
-        options.put("f", "/var/html");
+        options.put("-f", "/var/html");
         options.put("onlyCommand", "true");
         l.start(options);
         assertTrue(l.command.contains("-f"));
@@ -59,47 +59,47 @@ public void testSetFolder() throws Exception {
 
     @Test
     public void testEnableForce() throws Exception {
-        options.put("force", "");
+        options.put("-force", "");
         options.put("onlyCommand", "true");
         l.start(options);
         assertTrue(l.command.contains("-force"));
     }
 
     @Test
     public void testEnableOnly() throws Exception {
-        options.put("only", "");
+        options.put("-only", "");
         options.put("onlyCommand", "true");
         l.start(options);
         assertTrue(l.command.contains("-only"));
     }
 
     @Test
     public void testEnableOnlyAutomate() throws Exception {
-        options.put("onlyAutomate", "");
+        options.put("-onlyAutomate", "");
         options.put("onlyCommand", "true");
         l.start(options);
         assertTrue(l.command.contains("-onlyAutomate"));
     }
 
     @Test
     public void testEnableForceLocal() throws Exception {
-        options.put("forcelocal", "");
+        options.put("-forcelocal", "");
         options.put("onlyCommand", "true");
         l.start(options);
         assertTrue(l.command.contains("-forcelocal"));
     }
 
     @Test
     public void testEnableForceProxy() throws Exception {
-        options.put("forceproxy", "");
+        options.put("-forceproxy", "");
         options.put("onlyCommand", "true");
         l.start(options);
         assertTrue(l.command.contains("-forceproxy"));
     }
 
     @Test
     public void testSetLocalIdentifier() throws Exception {
-        options.put("localIdentifier", "abcdef");
+        options.put("-localIdentifier", "abcdef");
         options.put("onlyCommand", "true");
         l.start(options);
         assertTrue(l.command.contains("-localIdentifier"));
@@ -108,10 +108,10 @@ public void testSetLocalIdentifier() throws Exception {
 
     @Test
     public void testSetProxy() throws Exception {
-        options.put("proxyHost", "localhost");
-        options.put("proxyPort", "8080");
-        options.put("proxyUser", "user");
-        options.put("proxyPass", "pass");
+        options.put("-proxyHost", "localhost");
+        options.put("-proxyPort", "8080");
+        options.put("-proxyUser", "user");
+        options.put("-proxyPass", "pass");
         options.put("onlyCommand", "true");
         l.start(options);
         assertTrue(l.command.contains("-proxyHost"));
@@ -126,7 +126,7 @@ public void testSetProxy() throws Exception {
 
     @Test
     public void testSetHosts() throws Exception {
-        options.put("hosts", "localhost,8000,0");
+        options.put("-hosts", "localhost,8000,0");
         options.put("onlyCommand", "true");
         l.start(options);
         assertTrue(l.command.contains("localhost,8000,0"));