Skip to content

Browser integration tests #1461

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

Merged
merged 14 commits into from
Aug 30, 2018
Merged

Browser integration tests #1461

merged 14 commits into from
Aug 30, 2018

Conversation

kamilogorek
Copy link
Contributor

@kamilogorek kamilogorek commented Aug 3, 2018

Tested on headless Chrome and headless Firefox.

screen shot 2018-08-29 at 17 45 03

@kamilogorek kamilogorek requested a review from HazAT August 3, 2018 16:46
Copy link
Member

@HazAT HazAT left a comment

Choose a reason for hiding this comment

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

Only one minor thing, besides that the tests are failing.

@@ -30,6 +30,32 @@ export default [
commonjs(),
],
},
{
Copy link
Member

Choose a reason for hiding this comment

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

We could de-dupe it by having this object top level, and for minified we inject uglify into plugins.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updated

@@ -202,8 +202,6 @@ function serializeValue<T>(value: T): T | string {

/** JSDoc */
function serializeObject<T>(value: T, depth: number): T | string | {} {
return value;
Copy link
Member

Choose a reason for hiding this comment

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

🙈

@kamilogorek kamilogorek force-pushed the browser-integration-tests branch from 2d1a110 to 6d373bb Compare August 28, 2018 10:52
@codecov-io
Copy link

codecov-io commented Aug 28, 2018

Codecov Report

Merging #1461 into master will decrease coverage by 7.13%.
The diff coverage is 33.33%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1461      +/-   ##
==========================================
- Coverage   79.68%   72.55%   -7.14%     
==========================================
  Files          51       41      -10     
  Lines        2063     1246     -817     
  Branches      474      238     -236     
==========================================
- Hits         1644      904     -740     
+ Misses        409      333      -76     
+ Partials       10        9       -1
Impacted Files Coverage Δ
packages/utils/src/object.ts 47.82% <ø> (+0.51%) ⬆️
packages/utils/src/misc.ts 8.69% <0%> (-0.26%) ⬇️
packages/hub/src/scope.ts 95.18% <100%> (+1.13%) ⬆️
packages/raven-node/lib/client.js
packages/raven-node/lib/utils.js
packages/raven-node/index.js
...ges/raven-node/lib/instrumentation/instrumentor.js
packages/raven-node/lib/parsers.js
packages/raven-node/lib/instrumentation/http.js
packages/raven-node/lib/transports.js
... and 4 more

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 61bd254...b617683. Read the comment docs.

@kamilogorek kamilogorek force-pushed the browser-integration-tests branch 11 times, most recently from b8cc8db to 64dc00e Compare August 28, 2018 16:04
@kamilogorek kamilogorek force-pushed the browser-integration-tests branch 2 times, most recently from 24107b1 to f3e2c36 Compare August 28, 2018 16:32
@kamilogorek kamilogorek force-pushed the browser-integration-tests branch from f3e2c36 to 1ce7c60 Compare August 28, 2018 20:12
Copy link
Member

@HazAT HazAT left a comment

Choose a reason for hiding this comment

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

This is super nice,
one more thing since we are changed quite a bit in travis now.
I wrote travis that we want to move to travis-ci.com to have a even better github integration.

@HazAT HazAT merged commit 1f3bdca into master Aug 30, 2018
@HazAT HazAT deleted the browser-integration-tests branch August 30, 2018 14:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants