Skip to content
This repository has been archived by the owner on Jul 28, 2023. It is now read-only.

Karma BrowserStack reporting enhancements #170

Open
wants to merge 36 commits into
base: master
Choose a base branch
from

Conversation

samirans89
Copy link

@samirans89 samirans89 commented Feb 5, 2020

Current reporting ability of the Karma Launcher is only to mark status = 'failed' for a session. Does not handle success, timeouts, disconnect cases. Also, there is no provision added to mark failure reasons. 
This PR implements the following changes:

  • Handle additional Karma events for ‘spec complete’, ‘spec success’, ‘spec failure’, ‘spec skipped’, ‘browser complete’, ‘browser error’, ‘run complete’ for session reporting requirements 
  • Update ‘Session Name’ with Karma ‘Test Suite name’ which got executed in the sessionUpdate session ‘Status’ field to ‘Passed’ or ‘Failed’ based on the actual session run
  • Update the session ‘Reason’ field (255 chars) with the ‘specs completed’ within the session for ‘Passed’ runs and the first failure reason for ‘Failed’ runs
  • Update ‘Build Name’ post batch execution of Karma tests with the number of specs passed, failed (if any) and also report whether the session encountered any disconnects / browser errors.

@googlebot
Copy link

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here with @googlebot I signed it! and we'll verify it.


What to do if you already signed the CLA

Individual signers
Corporate signers

ℹ️ Googlers: Go here for more info.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants