Skip to content

Commit

Permalink
mv gl3d_cone-* out of npm run test-image into noci_test.sh
Browse files Browse the repository at this point in the history
.. so that we test gl3d_cone-* mocks using orca, not
   our old nw.js imagetest container which doesn't generate
   cones properly for some reason.
  • Loading branch information
etpinard committed May 17, 2018
1 parent 3f3bfac commit aaf7249
Show file tree
Hide file tree
Showing 6 changed files with 8 additions and 2 deletions.
3 changes: 3 additions & 0 deletions tasks/noci_test.sh
Original file line number Diff line number Diff line change
Expand Up @@ -13,8 +13,11 @@ npm run test-jasmine -- --tags=noCI --nowatch || EXIT_STATE=$?
# since the update to mapbox-gl@0.44.0, we must use orca
# as mapbox-gl versions >0.22.1 aren't supported on nw.js@0.12 used in the
# 'old' image server
#
# cone traces don't render correctly in the imagetest container
$root/../orca/bin/orca.js graph \
$root/test/image/mocks/mapbox_* \
$root/test/image/mocks/gl3d_cone* \
--plotly $root/build/plotly.js \
--mapbox-access-token "pk.eyJ1IjoiZXRwaW5hcmQiLCJhIjoiY2luMHIzdHE0MGFxNXVubTRxczZ2YmUxaCJ9.hwWZful0U2CQxit4ItNsiQ" \
--output-dir $root/test/image/baselines/ \
Expand Down
Binary file modified test/image/baselines/gl3d_cone-simple.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file modified test/image/baselines/gl3d_cone-sparse.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file modified test/image/baselines/gl3d_cone-wind.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
4 changes: 3 additions & 1 deletion test/image/compare_pixels_test.js
Original file line number Diff line number Diff line change
Expand Up @@ -86,6 +86,7 @@ else {
* - font-wishlist
* - all gl2d
* - all mapbox
* - gl3d_cone-*
*
* don't behave consistently from run-to-run and/or
* machine-to-machine; skip over them for now.
Expand All @@ -95,7 +96,8 @@ function untestableFilter(mockName) {
var cond = !(
mockName === 'font-wishlist' ||
mockName.indexOf('gl2d_') !== -1 ||
mockName.indexOf('mapbox_') !== -1
mockName.indexOf('mapbox_') !== -1 ||
mockName.indexOf('gl3d_cone-') !== -1
);

if(!cond) console.log(' -', mockName);
Expand Down
3 changes: 2 additions & 1 deletion test/image/mocks/gl3d_cone-simple.json
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,7 @@
},
"scene2": {
"domain": {"x": [0.5, 1]}
}
},
"width": 800
}
}

0 comments on commit aaf7249

Please sign in to comment.