-
-
Notifications
You must be signed in to change notification settings - Fork 409
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: add pushing and installation profiling logic #1027
Conversation
try { | ||
const bundlePathOnPhone = await this.pushAppBundle(app); | ||
await this.installApplcation(bundlePathOnPhone); | ||
log.info(`Installation is successful after ${new Date() - start}ms`); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I usually prefer to use process.hrtime for performance measurement
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we don't know if the installation is successful, since this.installApplcation
(btw there is a type in the method name) continues even after a timeout happens
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That is successful for us. The phones are something flaky and don't notify you back this happens really rarely so I let that slide and call it also successful
try { | ||
const bundlePathOnPhone = await this.pushAppBundle(app); | ||
await this.installApplcation(bundlePathOnPhone); | ||
await this.installApplication(bundlePathOnPhone); | ||
log.info(`Installation is successful after ${new Date() - start}ms`); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we already have a metrics tracking system, so you don't have to do this manually. see BaseDriver::logEvent
. You could log an install start and install finish event.
No description provided.