Add basic e2e tests

[yubathom]

Add basic e2e tests with cypress

Related to #65.

preview of npm run dev:e2e

this should be running in ci too - https://travis-ci.org/yubathom/postwoman/builds/593324131
at before_scripts

wich should give some reports at every pull requests

New scripts:

• npm run tests - run tests once in cli mode. Now just these e2e. In future we should run unit tests too.
• npm run dev:e2e- will start localhos:3000 and run e2e tests within cypress app browser. This is the default mode to tdd.

Notes:

• I'm not mounting and testing the docker container. Just testing the dev server (run dev) mounted at the travis-ci.
• Though would be a better approach (running on docker) the dev server is the same enviroment that is deploying the front end. So I think this should be enough right now.
• Cypress is awesome and we can add fake backend responses to run tests (see tests/e2e/fixtures/ and tests/e2e/support/commands.js). We should create some fixtures of firebase. Something like this.
• We should write more tests. Any ideas?

[ghost]

DeepCode Report (#ade773)

DeepCode analyzed this pull request.
There is 1 new info report.

[TravisBuddy]

Hey @yubathom,
Your changes look good to me!

View build log

TravisBuddy Request Identifier: bf9ad560-e648-11e9-9927-991e8c29daea

[liyasthomas]

This is AWESOME ✨

We'll write more tests 👌

[liyasthomas]

Tiny suggestions for clean code
https://www.deepcode.ai/app/gh/liyasthomas/postwoman/3ad814ebb71019ca1e9707e4338bacda1af7c0ad/liyasthomas/postwoman/ade7733d4751402cb46def4b4c3cf293b02706bf/_/%2Ftests%2Fe2e%2Fintegration%2Ffeature.url-queries.spec.js/javascript%2Fdc%2FPROMISE_NOT_CATCHED_GENERAL/code/

[liyasthomas]

🆗👌

[liyasthomas]

@NBTX seems to be busy for review. @yubathom can merge this on your requirements.

After merge, we could add more tests.

[yubathom]

Tiny suggestions for clean code
https://www.deepcode.ai/app/gh/liyasthomas/postwoman/3ad814ebb71019ca1e9707e4338bacda1af7c0ad/liyasthomas/postwoman/ade7733d4751402cb46def4b4c3cf293b02706bf/_/%2Ftests%2Fe2e%2Fintegration%2Ffeature.url-queries.spec.js/javascript%2Fdc%2FPROMISE_NOT_CATCHED_GENERAL/code/

Hey @liyasthomas, thank you for reviewing and merging. And sorry co's I've got sick and was out theese days.

I tested the deepcodecheck changes and that broke the tests. I think is because I am not able to catch in cypress promises.

I will check at cypress docs and look for workarounds to keep the tests code cleaner in future tests.

[yubathom]

And just deleted at travis the envs SURGE_LOGINand SURGE_TOKEN.

Those were required to already closed #151 and #148.