From 910ae249399fd3d879084161118ab4e861ab4038 Mon Sep 17 00:00:00 2001 From: Harish Ved Date: Thu, 28 Apr 2016 21:40:19 +0530 Subject: [PATCH 1/4] Readme changes and allow custom caps --- README.md | 136 +++++++++++++++++++++++++++++++++++++++++++++--------- 1 file changed, 113 insertions(+), 23 deletions(-) diff --git a/README.md b/README.md index 7f55f54..ffa1094 100644 --- a/README.md +++ b/README.md @@ -2,40 +2,130 @@ [![Build Status](https://travis-ci.org/browserstack/browserstack-local-csharp.svg?branch=master)](https://travis-ci.org/browserstack/browserstack-local-csharp) +A simple C-sharp wrapper for BrowserStack Local Binary. + ## Setup Open the solution file `BrowserStack/BrowserStack.sln` in `Visual Studio`. The projects are `Visual Studio 2015` compatible. You will need to resolve the references from the `Solution Explorer`. `Visual Studio` with automatically download the references from NuGet. -## API +## Example + +``` +using BrowserStack; + +# creates an instance of Local +Local local = new Local(); + +# replace with your key. You can also set an environment variable - "BROWSERSTACK_ACCESS_KEY". +List> bsLocalArgs = new List>() { + new KeyValuePair("key", ""), +} + +# starts the Local instance with the required arguments +local.start(bsLocalArgs); + +# check if BrowserStack local instance is running +Console.WriteLine(local.isRunning()); + +# stop the Local instance +local.stop(); +``` + +## Arguments + +Apart from the key, all other BrowserStack Local modifiers are optional. For the full list of modifiers, refer [BrowserStack Local modifiers](https://www.browserstack.com/local-testing#modifiers). For examples, refer below - + +#### Verbose Logging +To enable verbose logging - +``` +bsLocalArgs.Add(new KeyValuePair("v", "true")); +``` + +#### Folder Testing +To test local folder rather internal server, provide path to folder as value of this option - +``` +bsLocalArgs.Add(new KeyValuePair("f", "/my/awesome/folder")); +``` + +#### Force Start +To kill other running Browserstack Local instances - +``` +bsLocalArgs.Add(new KeyValuePair("force", "true")); +``` + +#### Only Automate +To disable local testing for Live and Screenshots, and enable only Automate - +``` +bsLocalArgs.Add(new KeyValuePair("onlyAutomate", "true")); +``` + +#### Force Local +To route all traffic via local(your) machine - +``` +bsLocalArgs.Add(new KeyValuePair("forcelocal", "true")); +``` + +#### Proxy +To use a proxy for local testing - + +* proxyHost: Hostname/IP of proxy, remaining proxy options are ignored if this option is absent +* proxyPort: Port for the proxy, defaults to 3128 when -proxyHost is used +* proxyUser: Username for connecting to proxy (Basic Auth Only) +* proxyPass: Password for USERNAME, will be ignored if USERNAME is empty or not specified + +``` +bsLocalArgs.Add(new KeyValuePair("proxyHost", "127.0.0.1")); +bsLocalArgs.Add(new KeyValuePair("proxyPort", "8000")); +bsLocalArgs.Add(new KeyValuePair("proxyUser", "user")); +bsLocalArgs.Add(new KeyValuePair("proxyPass", "password")); +``` + +#### Local Identifier +If doing simultaneous multiple local testing connections, set this uniquely for different processes - +``` +bsLocalArgs.Add(new KeyValuePair("localIdentifier", "randomstring")); +``` + +## Additional Arguments + +#### Binary Path + +By default, BrowserStack local wrappers try downloading and executing the latest version of BrowserStack binary in ~/.browserstack or the present working directory or the tmp folder by order. But you can override these by passing the -binarypath argument. +Path to specify local Binary path - +``` +bsLocalArgs.Add(new KeyValuePair("binarypath", "/browserstack/BrowserStackLocal")); +``` + +#### Logfile +To save the logs to the file while running with the '-v' argument, you can specify the path of the file. By default the logs are saved in the local.log file in the present woring directory. +To specify the path to file where the logs will be saved - +``` +bsLocalArgs.Add(new KeyValuePair("v", "true")); +bsLocalArgs.Add(new KeyValuePair("logfile", "/browserstack/logs.txt")); +``` + +## Contribute + +### Build Instructions + +To run the test suite run the nunit tests from Visual Studio. + +### Reporting bugs -### Constructor +You can submit bug reports either in the Github issue tracker. -* `new Local()`: creates an instance of Local +Before submitting an issue please check if there is already an existing issue. If there is, please add any additional information give it a "+1" in the comments. -### Methods +When submitting an issue please describe the issue clearly, including how to reproduce the bug, which situations it appears in, what you expect to happen, what actually happens, and what platform (operating system and version) you are using. -* `start(options)`: starts Local instance with options. The options available are detailed below. -* `stop()`: stops the Local instance -* `isRunning()`: checks if Local instance is running and returns a corresponding boolean value +### Pull Requests -### Options +We love pull requests! We are very happy to work with you to get your changes merged in, however, please keep the following in mind. -* `key`: BrowserStack Access Key -* `v`: Provides verbose logging -* `f`: If you want to test local folder rather internal server, provide path to folder as value of this option -* `force`: Kill other running Browserstack Local -* `only`: Restricts Local Testing access to specified local servers and/or folders -* `forcelocal`: Route all traffic via local machine -* `onlyAutomate`: Disable Live Testing and Screenshots, just test Automate -* `proxyHost`: Hostname/IP of proxy, remaining proxy options are ignored if this option is absent -* `proxyPort`: Port for the proxy, defaults to 3128 when -proxyHost is used -* `proxyUser`: Username for connecting to proxy (Basic Auth Only) -* `proxyPass`: Password for USERNAME, will be ignored if USERNAME is empty or not specified -* `localIdentifier`: If doing simultaneous multiple local testing connections, set this uniquely for different processes -* `hosts`: List of hosts and ports where Local must be enabled for eg. localhost,3000,1,localhost,3001,0 -* `logfile`: Path to file where Local logs be saved to -* `binarypath`: Optional path to Local binary +* Adhere to the coding conventions you see in the surrounding code. +* Include tests, and make sure all tests pass. +* Before submitting a pull-request, clean up the git history by going over your commits and squashing together minor changes and fixes into the corresponding commits. You can do this using the interactive rebase command. ## Example From 4e7f797371aac0bf0b8b12e403e542e33f435674 Mon Sep 17 00:00:00 2001 From: Harish Ved Date: Sat, 30 Apr 2016 22:33:22 +0530 Subject: [PATCH 2/4] custom args --- .../BrowserStack Unit Tests/LocalTests.cs | 25 ++++++++++++++++++- BrowserStack/BrowserStack/Local.cs | 17 ++++++++++++- 2 files changed, 40 insertions(+), 2 deletions(-) diff --git a/BrowserStack/BrowserStack Unit Tests/LocalTests.cs b/BrowserStack/BrowserStack Unit Tests/LocalTests.cs index b3ac466..2a90666 100644 --- a/BrowserStack/BrowserStack Unit Tests/LocalTests.cs +++ b/BrowserStack/BrowserStack Unit Tests/LocalTests.cs @@ -130,7 +130,7 @@ public void TestWorksWithBooleanOptions() local.setTunnel(tunnelMock.Object); local.start(options); tunnelMock.Verify(mock => mock.addBinaryPath(""), Times.Once); - tunnelMock.Verify(mock => mock.addBinaryArguments(It.IsRegex("-vvv.*-force.*-forcelocal*-forceproxy.*-onlyAutomate")), Times.Once()); + tunnelMock.Verify(mock => mock.addBinaryArguments(It.IsRegex("-vvv.*-force.*-forcelocal.*-forceproxy.*-onlyAutomate")), Times.Once()); tunnelMock.Verify(mock => mock.Run("dummyKey", "", logAbsolute), Times.Once()); local.stop(); } @@ -160,6 +160,29 @@ public void TestWorksWithValueOptions() local.stop(); } + [TestMethod] + public void TestWorksWithCustomOptions() + { + options = new List>(); + options.Add(new KeyValuePair("key", "dummyKey")); + options.Add(new KeyValuePair("customBoolKey1", "true")); + options.Add(new KeyValuePair("customBoolKey2", "false")); + options.Add(new KeyValuePair("customKey1", "customValue1")); + options.Add(new KeyValuePair("customKey2", "customValue2")); + + local = new LocalClass(); + Mock tunnelMock = new Mock(); + tunnelMock.Setup(mock => mock.Run("dummyKey", "", logAbsolute)); + local.setTunnel(tunnelMock.Object); + local.start(options); + tunnelMock.Verify(mock => mock.addBinaryPath(""), Times.Once); + tunnelMock.Verify(mock => mock.addBinaryArguments( + It.IsRegex("-customBoolKey1.*customBoolKey2.*-customKey1.*'customValue1'.*-customKey2.*'customValue2'") + ), Times.Once()); + tunnelMock.Verify(mock => mock.Run("dummyKey", "", logAbsolute), Times.Once()); + local.stop(); + } + [TestMethod] public void TestCallsFallbackOnFailure() { diff --git a/BrowserStack/BrowserStack/Local.cs b/BrowserStack/BrowserStack/Local.cs index a032ad8..54d59a2 100644 --- a/BrowserStack/BrowserStack/Local.cs +++ b/BrowserStack/BrowserStack/Local.cs @@ -68,13 +68,18 @@ private void addArgs(string key, string value) else if (key.Equals("logfile")) { customLogPath = value; + } + else if (key.Equals("verbose")) + { + } else { result = valueCommands.Find(pair => pair.Key == key); if (!result.Equals(emptyStringPair)) { - argumentString += result.Value + " " + value + " "; + argumentString += result.Value + " '" + value + "' "; + return; } result = booleanCommands.Find(pair => pair.Key == key); @@ -83,8 +88,18 @@ private void addArgs(string key, string value) if (value.Trim().ToLower() == "true") { argumentString += result.Value + " "; + return; } } + + if (value.Trim().ToLower() == "true") + { + argumentString += "-" + key + " "; + } + else + { + argumentString += "-" + key + " '" + value + "' "; + } } } private void setupLogging() From cdddbb1a73f806f54ce5c9606c6e377c5a2fcf3e Mon Sep 17 00:00:00 2001 From: Harish Ved Date: Sun, 1 May 2016 11:14:00 +0530 Subject: [PATCH 3/4] Use File Watcher --- BrowserStack.dll | Bin 17408 -> 17920 bytes .../BrowserStack/BrowserStackTunnel.cs | 44 ++++++++++++++---- BrowserStack/BrowserStack/Local.cs | 2 +- .../BrowserStackExample/Example.cs | 1 + 4 files changed, 38 insertions(+), 9 deletions(-) diff --git a/BrowserStack.dll b/BrowserStack.dll index f9dbf463e4c48fd5ce9bedb65cfc125d64a4640e..01a56bd1787be8cd8febba566504ec5c1e3a3c99 100644 GIT binary patch delta 7901 zcma)Bdw5humOoYZ=JxH|-M1g1^B@m8fh3)fAjmVkB)k+9M}nYe6e2_j0ylIA77U&4 zh@hw_u@wc^RTv#dM_1j9k8hk=WSntz9M>26Eecxo0`2z3>VX~Pm3BcGkZi-7C!_J{G5lM zq&(Vu7SYA^L?T-;d3^RFxm3){J}F0e5`w5B70Mow<=O9Kwe%j zC5bemaB?Vuv<#V=PNW-osU|e4w@? zs@z^q1JfR71X0?mDs(V;ZjfqjM!C(|ZZn#j0%r^wzf6urlj2a_fK6u{1TJ0ZjA#0| zfvFnF37`Y6!s|2+7;2M|x#JR*jMV8B1g4|vwBo|z_FN&SIc*83Q(zRX<5~ z1Gc<*Xy+?qGvQ1acD}>`r;Gn(Sdj zz`^q~ZXA_WfW;JP!2&`R6HSzb4rXM?_*HOLS1ui#Rz@A_ES*Q|*#b}E1q__=Ae6*W zrcgb4@eXWNQB{z*RilphGIUP-f++-S?W-z`FIR0*yp7uo3R6#B4YH~z@ea(2k}JSC zD_K&U(Dly^0oRY*=^NVcmEDtn5O_x+^%fWVm9i30q(hAyqWKh)^|? zGd)=|SLJF%oKZ6_uM#*@qBB>N%A-4^TRHAgHvvJ*&0J8bXrwC3lc(B|Qp9j&_6hG< zY0hYFmFq5(U04mhK^CA$m~wn7-$-~KeJF^K<#4hJGRx3XI1F^7s*4+SPP+odCoabL z?lv@~o+^Y%o#AR)PSX@1kprrwr>O>?qSyEh&YyN)@$L4OV zweX(Uh0x*-sQPjgKf7b!tho;tQ5-Q^DK*Fsc1qV*=6WUWhofBfRL_oqo)VX$D)fQy zmGQo+BG-N&>h}Tk-d8mkb1O+a!gL%9m>ZdoY)Ma}Cb4V6GQJL)PN+pH$_pct2{y%n zgTt9I@dO+=NeEru=>(a?PP8nedjlHh?*K+lxshpS6EJx#z~@O6<)M2QXiuTDnYA7# z#Z9sss7Tp`QyAwfP2j#6hzBIN#(UW8x&dLKgY9JD5aEjlds;o&x&F$u`vSt*l`hHE zB-Or83_}p=BuG^#AvUxn;~3VENe!6d%CF<=VhKtLheKF%u7`~^JceDf6%_B>ZD{?u z1r12k8;2X*n(-STZ^*vw&rfsJ*GFU*w947WA$5D5EJSzD%P!89 zgQ;U3ITdbgIXIY&f_!kUjAtdg5F1bpj%{TeiB`Mk&#w%MbU4^GR$;z=1m|Nt-fBv- z-v)-Ik#y1Qdf^L&z2%N|UPqk@Ffu=?SmHOqMFv-ip5`5VJj$Clc{7AniJ>0kVg?|2 z3wURT61K4r9!?XkW8qfU5^oMU(|cI@xGNnH{+5M@&JZ4Dp=O@yG{YxNE|6kq5@n~U zs}eOV9DRnc$)t2g;*v9@4x4U0LwGL>pE*sKiD67t?K<6x3|VT=s`BjyZw-oL<&0D% ztRmX+C=P2G(PD&Nsd~t~Rs;!5W2fVd!$^&WD6GHEZ2UI#j_(BcJn>#M{##gV7Q}A{ zBfHq+5U=Tp;|wqw1}EM^Sn(`Ww=BhC#_s^-R%sn>>^R2jzp<~DT|?2gUesyTGG4)o z<%KC0)!~w3w{fg3-bq#bD)PE=S~7tl{prg&|n zq314~GZXEz;Qv1RYN)1oDLCAKGy^{f2?66pi)dFi5Ux#+H&JJJ47w+;(r=JyFutb0 z=?l_yjBU_2`juXTy7L&9C_Jk0IfVxm+6q6?HS|58aJrJz=DlSY^niJ{Y0v_j@de2k z3+{t_rZp`cvgp@Vjj-tUz?(jU?xv-nyFy>V(PiOzrbYi@UuIhrwBGbt^zRCv@z)50 zc9=C{2|a+B8MGa}a1?mi^-uIlzeRif?CpU6ZquemLU)C1`eX1{L7Rqo+HHdz*xQt` z7F%hXCin_{Kh8NCrn3vv!??%8!+afLj3GpWTJYg#&_U~Ck3so5qod7(-h>Zo?vm#%9NjHiEX|`!YvvfVEHmgY`V*8w{41qPaq#RdpRHJ znAxV=bgg%>Wz+9fimH(ho8~Hf(BjmsLZWQi<2?~F=oK}CMJi8!RBQRZhgW?A2H-k+ zoPh-YLt$m~=RHj>hw7B(^PX9F{bFs1-t~Y>Q3&*(FjgMzhM}B7B%HKF6~(;{3K!1QGbzclB9{;T&w;C`F6PbhpxF#Q|GTuLW-o3%xmFHOe$ z0ORiz?N#zhg&#s1O>w^g7X|lkit8-V2Ps96*VtZp>$OpA@x^x8d?tnw+zZB=reh7)PSl2^5`yo`KxNVd`W@0oZ z$K4iqM2pdMms3kvL30(ig&xGRRP^i8IpxZnaw0vBO7G_ug5$*%*no@&aS8G!;tQ-9 zN2G?5V#!;0;}x_M5=pz2r7OY53ncdg1qGEQj{N}}rqlf$PE};p0GBF9Gu4n6V7p2> zpoY0b(amtG(HH!BN{=YlElR#e**C*=|Byk|s{F34DIMB$F-@gxcBC%``iqVxeIkAj{g{|Q(H`~YFG zWUzb|_+)TD@SuDF_zeyp^a6e&o;B0|g62*8S^7eCI0k%`j)^CO2F`)Q+J5p%PLx-^ z8ajw?%b#O!JdD9YR1Eo}puZOH0(U9oP(Gm*9!~z}{tqzMhfNYqvP%TT9QmPL2>frO zOzfqla;WgrUfF?xWndTs>%(zgS+-gk#=7nXguJSH;YAAsrAGKxPNnDn$Xah5*^ z4AJwzLG)W-5&aJMF}()-Ov(RKNxnicne;6k0{)GT07-l#N8mmC7upCM-!A}5=tJO8 z`V=^vz6Xw0I7#6QB|kq+nm&RSDADB#mr)5M*U&6rlA84yq6=XyQ;0vyfsQGhrf@Lw zwTb>h39(B&BEAs~a+RzzYsD1NES?rGh{NI{sf)1`wsAU%@l=6!nrNg()lMSuMYgvz zI_G(uU^vhQo(JTGus>7gzTpX+gu$HUW3nA59lmC87W&vj_Vd!AqA?pTi(??hsxdpgY_w?X+fWt~7EG4RSlPOEMO*gz@`YZ-CfZs%``#(P zKtw<-Z(7Z`>7qnivTbc=wk&q3xuUIe$?Wy3S7$q7f9+dZaX=18`bvi_lZCU}SGUbs zd$mJMEZCInY+J)@w!8Wvk<1RMsVraZthgGwOIkaZt!zu2Vy7h8sWoR=C;Rp-t$9qG z%4@YZYvv`1_Rh8@1hKt-laC{s*`8=y*6CzV)ZbyuTGP?FiTZADcu;1Sjg0l(H*&qm zZW&pc9d!2Db-@kPhQC-}g)fmLDDrlKThEl%Kq4=O4>gfjLopJ47oELdtB+m{%LLxy z+o%JrLmRQ`RMBy>;vOQrrRRzIMyPW1GXL$ zqBJiig>CCN=)hB?%?m!9c{P%GITFi*+1ugFp>XECaOQ(>=FbLtL^5yWant7Y=#jV` zSrb{8N5U?{7nNq?k%pZ|>=T~Ba+h%CbJb-HL}^i4q?mQCvoG+9Ao0WB(4`yb82l9-ZBqHG7qcC&C!iW&wMvYxn{A;UBUV5X>*zOB2OM%KGr{0&iyPb zl~-lpF^ug$)k4+By;wj2jpfM;mzqW4(r`=~MYzf$J?q0gn_y#t%9b{ki0`u#8qc4j z8D+L+MCRc)AHRk84Z?4+*WkQ!+7WN&PKMo(`1bJ&T-loF$R;N> zNA78XS?+m?=~!^9Gjk?0A@7GnLc$~jB!Gl>pokJskf(y8Mu-#zPM9HHFrCay zRV0FKD6G5g>d89a z{$Kmwd;k02k2A?`7ul`iM_X$;w|%sf^3O4aPb}Yh^Q$$1w(#j+^=0%HbfF>MA6hxbj~tl zdODGA_|mQJ0=+^sx$E@BnF@%K_X+Bh-9eCvY9h-S4v+~SYRT>>q)kDubP`bXs4Az< zs#11Z(C3$P_zcz=ULKEQ&O@ z@3roC#zElGCC+(F4;VH&nHmo|>?s0H%W0#4GvPE1(+HbRgv3G6^M0JkAnRqS61sGO zYAWLu+^&OZwVXI~_4xoy{ifzLCW}iFey9_z%nTT)nr{N3wu}Lzh6OV~r@jM7%w)6f zFuOhR=uxWu^t{1MA&G7JY%x8ByuO3D%!f(pjD ziZTnOiXaG^{$eF_kS|t!Akl`x@5})@b6L_8<5n*MS*VX#5z`YFGwF-@67xU=93-pd zTmr~A3xG|NWj6w*L%$I0s6u%LuSPBN$}}4WoJFv!Kiu{8*I&c?VlX3rdU&+*rLd7h@@d6W?VN)-f`sWKS6e;al6W1i z4d*f#d-+=h>c~vbb@`}pJ{sj-9LiM$ygr>kMfp9Bh#Iw2f&FF;yxm9w$;MB8JypdK z1oWR~&Lrdp(Ii=taj;DoGO>&YQ!O1t#%QF%(LU;83p~l7Mx9v`O(a{GLI&z(w__oR zy5i)UkR+C)Zc^%?FzeRO>q-)rtFkn41((?XYNS?xtSe1c!>lxQB^c)_mSoD37o91} za_TgqVGieOD6!K;b?0hWPka%q?8b!Yp5W|cvnJdGix{bN3&w=Xw!A54o0n|Yh z5N;LkMS=)Xj;3lMvkWbbRjC_w-CQ_G7t0eUKAA<#oOTGC{8CS2N$QP;r`dXjW=Q5+ zs_kc}O24J~`5BsG*7P6;b?hm_(3{jXXvtX#Kumj(Q2KUEO2t4iqq7R~krQM^J||>% z2@H@0h+qe~R|cyxEa<^r&;IKEK7Zr9F+117dGepoy~JuzO;uP5>`nQG=3X2OiF#<# zEak+NvrgoB z0XUY}*m0P4I)SMTfS{Qi=0o))XtTsgvDS1pa*@K?u@ou0(Qu>g+`wYzAAm~Y33@iM z+2+&266Z#@I$RbR!W~Q^9Xi#_x}(E29coD>u`Insmc}-s%LFWObfO7ErZixhlS*y# zSD`wRE2=r&u(3wOk&xd5#fu`1GL(;CSo#CJYz^pp+}FY-8BV6AnC#|K<(-4W$r?FV zCg*YMNGMBn%d_}DEQ84&7^aX#){UM~a)&{kYCwrBq&VI<#keY$-)f{p_GEasDEVn} z7pkcJB+WMYX-_^0saNZpS@)Kb?B;}aFl@{hhrvq}2m75-TnNsS@u+0Cim=x>QrI|B zl;7fCF(}rlE+i#|`Vnk9O*rVw-IMmPOdK2DLeC3dP}J(>wT;z_tkX%LWqwprByt#L z7Kf;w;ngxw#0w?$eF*E4=bDJcd_W3&mF3*5gd1204`&F|EbQ|vGp9`-V(DH_dRlmZ zh3}sw{4W+p{NHw(UNF6lEhBMT4$+0YKJxh48pE8}vm1jx2+4RR}2(!G%2P5(W zL<14RI#g-ea|_rZ3Cxm_j>9ab4HRL&lQAt$;M1t42do%^g~cmlx{+>%B&z?G*+gaw zYA0?51kJ=&6rt@b_ZKI6!N_j*cuv4?Ca_eEk>$z9(XYg9P;JLqBof;|c^TP=Lyr&0 z!87k#**z3>n?$2lFB4c+hAc_5q!Gsidre>(jrvQZ>c8rF<9JV_C6EOMPE9qy#v2mb z(MpMPyXw$%RF&PW>;R)Q8s|m4g(uZEomlN|iYBI(hKOo=h;BxCKlb?AMH`cA*0081 zLX^N%@e;0GvWPg@p{Cjkm$aRaasv3QdpJ7045OnU-WhnYwGsw^y9_RCZdI%ybJ7%` z(L8ndwgwG)L*Wj6LNG!Pqnbf4>JDZ3;t5{)N8N zFsR+X#Bb08Va8ub#-zO#@(%-dWFi*57n*=-chl9vq8B0;gbeB{u>2NHw;u^x^h97n z(4tiq+syM17Y5A%;!swd>qRYtMh4hvr@j@Irt(%8TH?3qMC9#=P0f)A@Ka-N*dPb? zHq}`J0h{iz(pKi)yrVy9?BWfB@lunUX)0iRIlwp)-!KL(upTfC`h&(eU9(`hPBMKm z#Pl0M#&;F|MQMA~Fgi5udZ)#nuUC=yA2s@I0x`g%)i@Lx$qf{$=3fmleU8qUHts-# zqUHr5iw*}kHa=zlG_kum_7cRYH^lOlkl1vdwKr^2Tfz7On`Q@i+=tBu-KLfRkGNVz zVmrod(_;!VGc68LLIn=?EP z`S-k;HfQ0E!rB7*iQ)<=s<_Jp>*7>Oeo;Wp;C@Lp85*kCW5jGC&q<|h2FFe4s)-hG z8Y<$pL9?he&vEk3LTcBNVN4YV<GXzaX~i_hvb8AfvkHN8gB8GKz@Zu1i@pdd(mH_0L_#wt zXO95hsqkTi&neum@VLT1gh#>VUlg7UHiJ$W|+W$LQLPP=zR*0D!dnHPzlCr(0L&X^g%@8k~rps z_KB%x0p|TDd9E;AfL-Si#ce4qLOyBq-*H^L*mOAZSEEaYf}t&>Aqh1~)G)VCYzxfP zG@8xRPC2U73a}-u<$Vm?U1E-A(%p*NLLZ7f*bVPj+&9KQc-$5&YFwE3q$6*&&yZV5O(X9U(2V||!$KfXdG z)(rm&RDvT9*^$AWRM2vr6{~=P4yg)@P(jkl|Cc_f+|?+XyKp=Tx(M5Wq+68FOBG!V zpBmkTCUsh^n!G~EKUVhlhcW+yO@4(#QP7)~hZ?ovE4>YAQWO-}!8l{ffEG0Xv41I? zsBnhDHiZimE>qYJEW%F+Y`;O_W`zO4-rB_#{^|@DW(}hri6Da~N(e47SR`e$p zODuN*g|&G z!@%RB%l~WOF#A!gYp(D)@Fk!YG3b=|nYIVxzSjQ%9jEK`6N>%{bhj|XP&v-FfiDFY`JaX3!{i~@=p9>xkIv_q2i=nD0j&+t4wZ| z@0zE?E_uit5Rc2<0^f2?2u_Qfg5az`k2nTX)acvfopO_ShB{;~wae+&#j;6_Zyc~! zG+}&;D}Df6EQ|2z?WYH& zA3s0;B7>rU9s>@cCxNB(EbtgT4}4F_|D+@z(@T(iMy~)*(thCI=y!4yF1Gh*qp*8E z1gxNaz*>48ID(D?$10qxaHf*ar9VQxNJ%aSW=7%S^fg4+Q-eN=Qdj{qMK^A#D;2(= z@R&mUtOR*E;CjHCHjWuxmY2%`NUb)ni3VfAKj5!W|P|)ZZ6x z6Yl3j7r6^+-q$WouIV3J`;zR}>#vmVjfn@{;SH7Us)nZihZ=q+`Y#`m)7%e7ZFH|c zx36CteZOoRF%v%nCh^xn>uDp!@pge)4}LAU6!><$9TayT7*pd;8CTi=n=xI&y=>eV zE3gsux}c-}C&oRmxw|IZgx?u1`0bN>(am0! zTN7L7BVkwBxN6vV;1I}`(d>uHat*gpS*qxD8PCTj_z;zNOMUnWG5fhE{A;;s#5Tmb zs(i5pMbXVN>W{4`il$ZR$7gDn9nA?du8At2A2)Q}6GU@?Xf6z~pT+Di7e!lkl@AzO zVEc<=%ajW%PDRnoW?4vu!awL=3DbizuPp?kP@qe(ToDJQkN~j-K9Y82rO!kJaylV& z0Nch1X`39wXs&@vb{;e3p!>n@Tc(UPF_S^C;|O-JD3%*3SYk7s-8o3d;X>&O$?(){ zUy0UX>4%%_u2}W~HO%vLBbJ-0VmDaK^0+o1dYoI}F-wX~Ur{u>TeWbJZF^pATLw5F zkQY+$v>sE(*|7ujD4?)RU$oL+8m)}RrBUjKD%T#(aR&K8xzA0m5U1ROljmL_4O=s+ z@Ww92y8!PHyh{ScVD!<~Y#+mqArCRJ!^{sb1S2m1?%FAJ2@Ye_ANBL)gXzjxwl`ni zZlhPHjbEHnV+If$49NXx%7;yZ|G&fagKoh)eHq@#L<<+4zv!Lt`lc^O|Hu4aHS}He z-ht`Q-Z8C0-`J7b=nkE>(EbF5gO9n~%}pzprk4<}|-=6-Tnyj4E MA3gts=o@YRFSneJ!2kdN diff --git a/BrowserStack/BrowserStack/BrowserStackTunnel.cs b/BrowserStack/BrowserStack/BrowserStackTunnel.cs index ce5983a..b7f2440 100644 --- a/BrowserStack/BrowserStack/BrowserStackTunnel.cs +++ b/BrowserStack/BrowserStack/BrowserStackTunnel.cs @@ -30,6 +30,8 @@ public class BrowserStackTunnel : IDisposable protected StringBuilder output; public LocalState localState; + protected string logFilePath = ""; + protected FileSystemWatcher logfileWatcher; Job job = null; Process process = null; @@ -120,6 +122,10 @@ public virtual void Run(string accessKey, string folder, string logFilePath) process.Close(); } + if (File.Exists(logFilePath)) + { + File.WriteAllText(logFilePath, string.Empty); + } Local.logger.Info("BrowserStackLocal binary is located at " + binaryAbsolute); Local.logger.Info("Starting Binary with arguments " + arguments.Replace(accessKey, "")); ProcessStartInfo processStartInfo = new ProcessStartInfo() @@ -190,8 +196,30 @@ public virtual void Run(string accessKey, string folder, string logFilePath) private void readFile(string filename) { - using (Stream fileStream = File.Open(filename, FileMode.Create)) - fileStream.Write(new Byte[1], 0, 1); + logFilePath = filename; + if (File.Exists(filename)) + { + readAlreadyPresentFile(filename); + } + else + { + logfileWatcher = new FileSystemWatcher(new FileInfo(filename).Directory.FullName); + logfileWatcher.Created += readAlreadyPresentFile; + logfileWatcher.Changed += readAlreadyPresentFile; + logfileWatcher.EnableRaisingEvents = true; + } + } + + private void readAlreadyPresentFile(object sender, FileSystemEventArgs e) + { + if (e.FullPath.Equals(logFilePath)) + { + readAlreadyPresentFile(e.FullPath); + } + } + + private void readAlreadyPresentFile(string filename) + { using (FileStream fs = File.Open(filename, FileMode.Open, FileAccess.Read, FileShare.ReadWrite)) { byte[] bytes = new byte[1024]; @@ -224,7 +252,6 @@ private void TunnelStateChanged(LocalState prevState, LocalState state) { connectingEvent.Set(); } - Local.logger.Info("Current tunnel state " + state); } public bool IsConnected() @@ -236,10 +263,12 @@ public virtual void Kill() { try { - this.process.Kill(); - this.process = null; - this.job.Close(); - this.job = null; + if (process != null) + process.Kill(); + process = null; + if (job != null) + job.Close(); + job = null; } catch (Exception e) { @@ -251,7 +280,6 @@ public virtual void Kill() TunnelStateChanged(localState, LocalState.Disconnected); localState = LocalState.Disconnected; - Local.logger.Info("TunnelState: " + localState.ToString()); } } diff --git a/BrowserStack/BrowserStack/Local.cs b/BrowserStack/BrowserStack/Local.cs index 54d59a2..b27a033 100644 --- a/BrowserStack/BrowserStack/Local.cs +++ b/BrowserStack/BrowserStack/Local.cs @@ -78,7 +78,7 @@ private void addArgs(string key, string value) result = valueCommands.Find(pair => pair.Key == key); if (!result.Equals(emptyStringPair)) { - argumentString += result.Value + " '" + value + "' "; + argumentString += result.Value + " " + value + " "; return; } diff --git a/BrowserStackExample/BrowserStackExample/Example.cs b/BrowserStackExample/BrowserStackExample/Example.cs index 5447f22..f3cf910 100644 --- a/BrowserStackExample/BrowserStackExample/Example.cs +++ b/BrowserStackExample/BrowserStackExample/Example.cs @@ -46,6 +46,7 @@ static void Main(string[] args) driver.Quit(); local.stop(); + Console.WriteLine("Test Completed."); Console.ReadLine(); } } From bd47eb50d26a148d9ea3f96b794174fa6883fb29 Mon Sep 17 00:00:00 2001 From: Harish Ved Date: Fri, 29 Apr 2016 11:46:38 +0530 Subject: [PATCH 4/4] Remove quotes from custom arguments --- BrowserStack/BrowserStack/Local.cs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/BrowserStack/BrowserStack/Local.cs b/BrowserStack/BrowserStack/Local.cs index b27a033..130221e 100644 --- a/BrowserStack/BrowserStack/Local.cs +++ b/BrowserStack/BrowserStack/Local.cs @@ -98,7 +98,7 @@ private void addArgs(string key, string value) } else { - argumentString += "-" + key + " '" + value + "' "; + argumentString += "-" + key + " " + value + " "; } } }