-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
Running query tests in browser #8584
Merged
Merged
Changes from 1 commit
Commits
Show all changes
62 commits
Select commit
Hold shift + click to select a range
a6f18fc
WIP investigating migrating to tape
f286372
staging work with using rollup to build tests for the browser
2143ed6
working prototype of tests in browser with browserify
4cf2312
remove beforeEach and afterEach temporarily
6d3df51
stage
42013e3
refactor tape build process, working with rollupnow :party:
62923a7
revert change to util/test
a6ae97e
working build script for bundling fixtures into browser bundle
f54c2fc
working style json preprocessing
1b19da4
first working protoype of in-browser query tests
8705ef6
add __ENVIRONMENT__ flag and add request proxies
6f88052
working query tests!!
c6a5b9a
further improments
e02bc60
attempting switch to non esm node
f5b5995
migrate to using hook in testem
95ca5c9
switch to using testem hooks
6619e8b
fix most linking errors
c2c0404
Merge branch 'master' of github.com:mapbox/mapbox-gl-js into devops/t…
dd71bb1
fix error in suite_implementation
964a698
change environment variable style
36a0042
fix failing query tests
925848c
first attempt at running query tests in-browser on circle-ci
c47b4ac
switch to curl instead of wget
bf03603
Using apt-get -f instead
0077659
attempt auto install of dependencies
3f4f8e4
switch to using public node image
de1cde3
de-derp
580a0cd
uncache node_modules
b3c79a0
try node 10.16
cf81bcd
remove xvfb
a7e4cd6
remove the need for multi-entry
64caacd
force invalidate eslintcache
5593011
upuse GL_JS_ENV environment variable instead
8ebd56f
better test result collection on circle-ci
142335e
address PR comments and simplify build folders
4f10a25
fix flow errors and fix a typo
18da05b
remove eslint and flow annotations from testem
9c56096
switch to using custom json diff with epsilon checking
d80bd42
actually add epsilon checking
9f97f16
cleanup and remove unused packages
8c59c36
add comments
481c27e
Merge branch 'master' of github.com:mapbox/mapbox-gl-js into devops/t…
7b4ec5b
Merge branch 'master' of github.com:mapbox/mapbox-gl-js into devops/t…
1415b4c
Remove unecessary `mapbox-gl-test.js` build, using `gl-dev` build dir…
7352d3b
add build watchers for convenient local debugging
a41873c
fix circle integration
758822a
Merge branch 'master' of github.com:mapbox/mapbox-gl-js into devops/t…
ebcc44b
Fix linting
a70055b
Revert whitespace change in rollup_plugins.js
3d49d95
Remove unused env-var flow definition
62a5a2a
Add prettified HTML reporter for tap output
657f211
revert intentionally broken test
f926ef2
Add persisting of testem test-page to disk for reporting on CI
614e795
Lock circleci to use `debian-stretch`
688f177
address CR comments: move testem config file and update circle yml
fc0feec
Merge branch 'master' of github.com:mapbox/mapbox-gl-js into devops/t…
d1acb15
Add env var to disable build notifications
f3e2574
- Move tap-parser to devDependencies
a631a9c
Add documentation to the readme
53f87a6
fix aggresive fixture watcher, and silence warnings from rollup build…
bc7e46b
Merge branch 'master' of github.com:mapbox/mapbox-gl-js into devops/t…
521363d
Merge branch 'master' of github.com:mapbox/mapbox-gl-js into devops/t…
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
needs a
&&
to serialize the steps. A Single&
will fork and try to run it in the background