-
Notifications
You must be signed in to change notification settings - Fork 14
Collected data seems suspicious; Canary mode doesn't report metrics #6
Comments
@jimfb did you modify your tests at all between samples? Seems as if test isn't running as expected. You may also want to start up a server and manually interact with the application, then inspect the console to see if any errors are reported. |
@jeffbcross Nope, didn't modify the tests. Clean checkout of this repository, only change is that on the second run I set the chromeOptions binary path (and saw that chrome canary did run, so the path is correct). I also saw the webdriver clicking the tests, so it appeared to be working, and it doesn't appear to be a complete failure because the renderTime column has data. I just tried starting the server manually and running the test myself in chrome and chrome-canary, both looked perfectly reasonable to me. |
Is it possible you could share the complete runnable app, benchmark, setup, etc? |
It's literally just a clean checkout for the first run, and a single line ( Other than that, the only relevant setup is my chrome&canary installations (both standard, afaik) and my OS (Yosemite 10.10.4 (14E46)) - I'm not really sure how to "share" those. What did you have in mind? |
Sounds like the same issue as reported here: angular/angular#3411 |
@jimfb do you want to take a stab at a PR to fix benchpress? |
I checked out the demo and ran it with the following results:
The GC times looked awfully suspicious to me (but I wasn't surprised, since the runner appeared to be running the default version of chrome). I modified chromeOptions to include
'binary': '/Applications/Google\ Chrome\ Canary.app/Contents/MacOS/Google\ Chrome\ Canary',
, which did successfully cause Canary to be launched, but then the results were empty:I'm new to angular / protractor / benchpress, so maybe I'm missing something obvious. Any ideas?
The text was updated successfully, but these errors were encountered: