Skip to content

CircleCI with Ubuntu 14.04, 2x parallelism and WebGL jasmine tests #1455

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.

Sign up for GitHub

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 12 commits into from
Mar 9, 2017
Merged

CircleCI with Ubuntu 14.04, 2x parallelism and WebGL jasmine tests #1455

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
0e3c61f
merge karma ciconf with (regular) conf
etpinard Mar 9, 2017
aa1aaef
comment & lint in karma conf
etpinard Mar 9, 2017
62d33b9
rm hasWebGLSupport early return in jasmine specs
etpinard Mar 9, 2017
100b993
rm unnecessary quotes in yml
etpinard Mar 9, 2017
14621e1
revamp circle.yml for ubuntu 14.04 with 2x parallel containers !!!
etpinard Mar 9, 2017
b42f021
use `karma-jasmine-spec-tags` to skip a few test on CI
etpinard Mar 9, 2017
8d7171f
add karma-verbose-reporter in dev deps
etpinard Mar 9, 2017
b86bf06
add `karma-spec-reporter` to log info about skipped specs
etpinard Mar 9, 2017
631533d
tweak precision to make tests pass on CI w/ Chrome 54
etpinard Mar 9, 2017
dbc6ca1
lock uglify-js to 2.7.x
etpinard Mar 9, 2017
e52f020
add no_citest script
etpinard Mar 9, 2017
33ae963
use (plain) npm run test-jasmine on CI
etpinard Mar 9, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 16 additions & 0 deletions tasks/noci_test.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,16 @@
#! /bin/bash

EXIT_STATE=0

# tests that aren't run on CI

# jasmine specs with @noCI tag
npm run citest-jasmine -- tests/*_test.js --tags noCI || EXIT_STATE=$?
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'm very pleased with karma-jasmine-spec-tags. This ⏫ allow us to run allow the specs with the @noCI tag. This will make the pre-release process a lot less painful.

Copy link
Contributor Author

@etpinard etpinard Mar 9, 2017 •
edited
Loading

Choose a reason for hiding this comment

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

TODO:

  • update dev docs.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done in https://github.com/plotly/dev-docs/commit/17302734858a6bc967944450490ea4ffef1757bd

Copy link
Contributor

Choose a reason for hiding this comment

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

The link ⬆️ is broken.

See usage here: b42f021


# mapbox image tests take too much resources on CI
npm run test-image -- mapbox_* || EXIT_STATE=$?

# run gl2d image test again (some mocks are skipped on CI)
npm run test-image-gl2d || EXIT_STATE=$?

exit $EXIT_STATE