Skip to content
This repository has been archived by the owner on Dec 8, 2022. It is now read-only.

Expose Karma browser errors during skyux test; upgrade @angular-devkit/build-optimizer #309

Merged
merged 7 commits into from
Jul 30, 2020

Conversation

Blackbaud-SteveBrush
Copy link
Member

No description provided.

@Blackbaud-SteveBrush Blackbaud-SteveBrush changed the title Add karma logs Expose Karma browser errors during skyux test Jul 29, 2020
@Blackbaud-SteveBrush Blackbaud-SteveBrush changed the title Expose Karma browser errors during skyux test Expose Karma browser errors during skyux test; upgrade @angular-devkit/build-optimizer Jul 29, 2020
@codecov
Copy link

codecov bot commented Jul 29, 2020

Codecov Report

Merging #309 into master will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##            master      #309   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           54        54           
  Lines         1634      1635    +1     
  Branches       252       252           
=========================================
+ Hits          1634      1635    +1     
Flag Coverage Δ
#builder 100.00% <100.00%> (ø)
#runtime 100.00% <ø> (ø)
#srcapp 100.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
cli/utils/karma-utils.js 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8f820c3...a764a49. Read the comment docs.

@Blackbaud-SteveBrush Blackbaud-SteveBrush marked this pull request as ready for review July 29, 2020 19:54
server.on('browser_error', () => {
logger.warn('Experienced a browser disconnect error. Karma will retry up to 3 times.');
server.on('browser_error', (browser, error) => {
logger.warn(`[SKY UX Builder] Karma encountered a browser error: ${error.message}`);

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Any benefit in spitting out the browser in this message too?

Copy link

@Blackbaud-BobbyEarl Blackbaud-BobbyEarl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Other than the one comment I left, this LGTM Steve.

@Blackbaud-SteveBrush Blackbaud-SteveBrush merged commit 13c53eb into master Jul 30, 2020
@Blackbaud-SteveBrush Blackbaud-SteveBrush deleted the add-karma-logs branch July 30, 2020 16:12
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