Skip to content

Remove travis allowed failure - #1096

Open
nfcampos wants to merge 1 commit into
masterfrom
nuno/no-travis-allowed-failures
Open

Remove travis allowed failure#1096
nfcampos wants to merge 1 commit into
masterfrom
nuno/no-travis-allowed-failures

Conversation

@nfcampos

Copy link
Copy Markdown
Collaborator

It passes now. When was this failing?

@nfcampos
nfcampos requested a review from ljharb August 29, 2017 18:50
@ljharb

ljharb commented Aug 29, 2017

Copy link
Copy Markdown
Member

It failed consistently on my fork. I think it's flaky.

@nfcampos

Copy link
Copy Markdown
Collaborator Author

@ljharb hmm, do you have a link to a travis build where it failed? if it does indeed still fail it might also affect users

@ljharb

ljharb commented Sep 1, 2017

Copy link
Copy Markdown
Member

@ljharb

ljharb commented Sep 1, 2017

Copy link
Copy Markdown
Member

https://travis-ci.org/airbnb/enzyme/jobs/270638186 is another one (i'm going to rerun it, so i'll paste the failure output here)

Details
> karma start "--single-run"
01 09 2017 00:17:57.417:INFO [karma]: Karma v1.7.1 server started at http://0.0.0.0:9876/
01 09 2017 00:17:57.422:INFO [launcher]: Launching browsers Chrome_travis, Firefox with unlimited concurrency
01 09 2017 00:17:57.435:INFO [launcher]: Starting browser Chrome
01 09 2017 00:17:57.499:INFO [launcher]: Starting browser Firefox
01 09 2017 00:18:05.292:INFO [Chromium 59.0.3071 (Ubuntu 0.0.0)]: Connected on socket wS3NuuVAfWAGMpxeAAAA with id 30455731
01 09 2017 00:18:08.434:INFO [Firefox 50.0.0 (Linux 0.0.0)]: Connected on socket zA4jW4zNrl80EpZwAAAB with id 7583193
................................................................................
................................................................................
................................................................................
..............................................
ERROR: 'Warning: render(): Rendering components directly into document.body is discouraged, since its children are often manipulated by third-party scripts and browser extensions. This may lead to subtle reconciliation issues. Try rendering into a container element created for your app.'
......
ERROR: 'Warning: render(): Rendering components directly into document.body is discouraged, since its children are often manipulated by third-party scripts and browser extensions. This may lead to subtle reconciliation issues. Try rendering into a container element created for your app.'
................................................................................
................................................................................
................................................................................
................................................................................
......................................
Chromium 59.0.3071 (Ubuntu 0.0.0): Executed 650 of 656 (skipped 6) SUCCESS (2.119 secs / 1.607 secs)
Chromium 59.0.3071 (Ubuntu 0.0.0) ERROR
  Some of your tests did a full page reload!
Chromium 59.0.3071 (Ubuntu 0.0.0): Executed 650 of 656 (skipped 6) ERROR (2.149 secs / 1.607 secs)
................................................................................
................................................................................
................................................................................
..............................................
Firefox 50.0.0 (Linux 0.0.0) ERROR: 'Warning: render(): Rendering components directly into document.body is discouraged, since its children are often manipulated by third-party scripts and browser extensions. This may lead to subtle reconciliation issues. Try rendering into a container element created for your app.'
......
Firefox 50.0.0 (Linux 0.0.0) ERROR: 'Warning: render(): Rendering components directly into document.body is discouraged, since its children are often manipulated by third-party scripts and browser extensions. This may lead to subtle reconciliation issues. Try rendering into a container element created for your app.'
................................................................................
................................................................................
................................................................................
................................................................................
......................................
Firefox 50.0.0 (Linux 0.0.0): Executed 650 of 656 (skipped 6) SUCCESS (5.344 secs / 3.556 secs)
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR! enzyme@0.0.1 test:karma: `karma start "--single-run"`
npm ERR! Exit status 1
npm ERR! 
npm ERR! Failed at the enzyme@0.0.1 test:karma script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR! A complete log of this run can be found in:
npm ERR!     /home/travis/.npm/_logs/2017-09-01T00_18_20_995Z-debug.log

@nfcampos

nfcampos commented Sep 4, 2017

Copy link
Copy Markdown
Collaborator Author

@ljharb hey, so some of them failed with ReferenceError: Set is not defined, because they ran on travis workers with an older version of Chrome installed (v37) which does not have Set. For that reason I think we should add babel-polypill to the karma tests, I'll add a commit here doing that.

This one that you posted the output of here failed because some of the tests did a full page reload, that I don't understand how it could happen only sometimes...

@ljharb

ljharb commented Sep 5, 2017

Copy link
Copy Markdown
Member

Let's please not use babel-polyfill for anything; we should be using airbnb-browser-shims. Thanks for the research tho, that helps!

@ljharb

ljharb commented Aug 7, 2018

Copy link
Copy Markdown
Member

I've rebased this; I suspect all the karma tests still fail.

I've rebased #1119 on top of this, as well.

@ljharb
ljharb force-pushed the master branch 3 times, most recently from 40cc703 to 0a17404 Compare April 12, 2019 23:05
@ljharb
ljharb force-pushed the master branch 4 times, most recently from 2227326 to 0d5ead7 Compare December 21, 2020 07:46
@ljharb
ljharb force-pushed the master branch 3 times, most recently from 43eb75e to 39e6b1f Compare November 3, 2022 21:47
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.

2 participants