Cesium: Intermittent GlobeSurfaceTile test failures on CI

Created on 9 Feb 2017 · 10Comments · Source: CesiumGS/cesium

@shehzan10 do you want to look at this at the bug bash?

These tests fail Intermittently. Perhaps there is a way to simplify them and make them more consistent.

  Scene/GlobeSurfaceTile
    processStateMachine
      ✗ upsampled terrainData is copied to the tile once it is available
    Error: Timeout - Async callback was not invoked within timeout specified by jasmine.DEFAULT_TIMEOUT_INTERVAL.
      ✗ loaded terrain data replaces upsampled terrain data
    Failed: promise rejected: TypeError: Cannot read property 'state' of undefined
        at Specs/customizeJasmine.js:79:26
        at Object.then (Source/ThirdParty/when.js:217:33)
        at Source/ThirdParty/when.js:297:13
        at processQueue (Source/ThirdParty/when.js:647:4)

category - terrain and imagery onramping priority - high test failure

Source

pjcozzi

Most helpful comment

Sure.

shehzan10 on 9 Feb 2017

👍2

All 10 comments

Sure.

shehzan10 on 9 Feb 2017

👍2

@shehzan10 did you get a chance to look into this yet? This test failure keeps making greenkeeper throw up errors

hpinkos on 7 Mar 2017

I did look into it around the February bug bash, but it was incredibly hard to reproduce as it was only happening intermittently. I don't really have any update on why this is happening because of that.

shehzan10 on 7 Mar 2017

Does this test use external data? (If so that could certainly be part of the problem).

We should either fix this ASAP (preferred) or exclude the test in master (via xit) because it's making our unit tests too unstable.

mramato on 7 Mar 2017

I agree with @mramato. Given that we are focused elsewhere, perhaps just xit for now.

pjcozzi on 8 Mar 2017

Ignored for now in #5085.

pjcozzi on 8 Mar 2017

We are still receiving frequent travis failures because of the below two related tests. I would recommend we xit these as well, but there's a large problem here that we really need to look into sooner rather than later.

1) loaded terrain replacing upsampled terrain triggers re-upsampling and re-loading of children
     Scene/GlobeSurfaceTile processStateMachine
2) improved upsampled terrain triggers re-upsampling of children
     Scene/GlobeSurfaceTile processStateMachine

mramato on 20 Mar 2017

👍1

Agreed. @rahwang do you want to make this small change: https://github.com/AnalyticalGraphicsInc/cesium/issues/4968#issuecomment-287766749

pjcozzi on 20 Mar 2017

Sure thing!