Skip to content
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

Jump to bottom

Feature/api tests #68

Open
wants to merge 44 commits into
base: master
Choose a base branch
from
Open

Feature/api tests #68

wants to merge 44 commits into from

Conversation

JMLX42
Copy link
Owner

@JMLX42 JMLX42 commented Oct 15, 2016

No description provided.

@JMLX42
Implement a first REST API test using the Jest framework.
e460fd4
@JMLX42
Add missing 'superagent' dev dependency.
73709a7
@JMLX42
Bootstrap the tests for the /vote API endpoints.
409382f 
Use promises for the API tests.
@JMLX42
Make a dummy test fail to make sure it will result in a failed Travis…
ba4130d 
… build.
@JMLX42
Fix dummy test.
025579e
@JMLX42
Add the /script/add-app.js script to create an App using the command …
bed18a7 
…line.

Minor coding style fixes.
@JMLX42
Create a test App during the provisioning of the "local" machine.
42e398e
@JMLX42
Make the test App settings available in the API config.json file.
5a07ca9
@JMLX42
Add tests for the JWT authentication on /api/user.
58803d2
@JMLX42
Add tests for the POST /api/oauth API endpoint.
6c29681
@JMLX42
Fix API config.json generation to be done in two steps.
94c2e66 
The first step creates the initial config with all that is required for the API to work.
The second "update" step will update the config with the test App settings.
We need to do this because the config.json file is required to run the add-app.js script.
@JMLX42
Dump the database before each test suite, and restore it after each t…
edc5b29 
…est suite.
@JMLX42
Add more tests for /api/vote.
da12b9b 
Refactor all tests to use async/await.
@JMLX42
Fix the blockchain-miner upstart to properly exec testrpc.
f0bc7c3
@JMLX42
Update the "local" machine conf to use testrpc when the provider is "…
036d061 
…travis".
@JMLX42
Add tests for Vote.voteContractAddress and Vote.voteContractABI.
653d45e
@JMLX42
Fix empty /api/npm-shrinkwrap.json.
2da9274
@JMLX42
Fix broken comparison in local.yml.
9969425
@JMLX42
Add the /ping API endpoint and the corresponding test.
a32631f
@JMLX42
Update run-tests.sh to show the relevant logs when the API tests fail.
2e7e430
@JMLX42
Set +e in run-tests.sh.
e154f4c 
We need this to make sure the tests will continue after an error.
@JMLX42
Fix Vote.schema.pre('validate') to handle the metafetch error.
caa924d
@JMLX42
Refactor all API tests to use a common getAPIURL() function.
0a2b919
@JMLX42
Fix run-tests.sh to have return code != 0 when an error occurred.
3497c97
@JMLX42
Use bunyan to the API logs.
b68c4bd
@JMLX42
Add JSON pretty-printing for the test logs.
eeea380
@JMLX42
Add JSON logs for the API.
a06e403
@JMLX42
Add the --verbose option for run-tests.sh.
551d0e1
@JMLX42
Update npm-shrinkwrap.json for /api.
e793b57
@JMLX42
Fix Vote.pre('validate') to instanciate an actual Error object.
4ef09f5
@JMLX42
Use meetup.com the API test URL.
16d8dc4
@JMLX42
Disable JSON pretty printing in run-tests.sh.
1aa772e
@JMLX42
Fix development blockchain upstart scripts to read the private key fr…
caed7c7 
…om the keystore.
@JMLX42
Fix vote.create() to respond with an {error} object.
5072641
@JMLX42
Bootstrap the /ballot API endpoint tests.
aa58965
@JMLX42
Mark the "unknown ballot" error as "no retry".
88486d1 
When we run the tests, ballots are likely to be mined after we restore the database. So this error will never end if we don't set it as "no retry".
@JMLX42
Reset the queues and restart the blockchain miner after a test suite.
58796e7 
It's the same idea as why we dump/restore database after each test suite: tests are easier to code if we don't have to worry about the current state of the queue/blockchain.
@JMLX42
Add a test to make sure a user cannot vote twice.
fbdc588 
getVote() will now return the same Vote object for the whole test suite.
@JMLX42
Add support for --gasLimit when running testrpc.
4deb061
@JMLX42
Merge remote-tracking branch 'origin/master' into feature/api-tests
6c72767
@JMLX42
Fix broken reference to voteConsumer.run.
945c4d8
@JMLX42
Set vote_consumer_count to 2.
5f7bb9f
@JMLX42
Fix vote and ballot tests.
84d62f1 
In order to have the tests working without an Internet connection, we now use https://localhost as a test URL (instead of https://meetup.com).
The best thing to do would be to actually run a dedicated local server with test assets HTML pages.
@JMLX42
Raise the POST /vote test timeout delay to 15sec to avoid false negat…
e2f00f4 
…ives.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@JMLX42