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.

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

Test cypress integration #346

Closed
wants to merge 1 commit into from
Closed

Conversation

tbille
Copy link
Contributor

@tbille tbille commented Mar 2, 2018

THIS IS A TEST PR DON'T MERGE!!!

@webteam-app
Copy link

Starting demo at: http://snapcraft.io-pr-346.run.demo.haus/

WTF_CSRF_SECRET_KEY=this_is_a_random_string
SNAPCRAFT_IO_API=https://api.staging.snapcraft.io/api/v1/
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This won't be merged it's just to test for now

@@ -1,7 +1,20 @@
sudo: required
language: node_js
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This comes from the basic configuration for Cypress: https://docs.cypress.io/guides/guides/continuous-integration.html#Travis

.travis.yml Outdated

services:
- docker

script:
- ./run test
- ./run &
- wait 10
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 just want to let time to the run script to start (should find a better way)

@@ -0,0 +1,3 @@
{
"chromeWebSecurity": false
Copy link
Contributor Author

@tbille tbille Mar 2, 2018

Choose a reason for hiding this comment

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

This is needed for redirections from login SSO (in HTTPS) to our local s.io (not in HTTPS)

@@ -0,0 +1,5 @@
{
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Generated file: to be removed

@@ -0,0 +1,5 @@
{
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Generated file: to be removed

@@ -0,0 +1,232 @@
[
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Generated file: to be removed

@@ -0,0 +1,17 @@
// ***********************************************************
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Generated file: to be removed

@@ -0,0 +1,25 @@
// ***********************************************
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Generated file: to be removed

@@ -0,0 +1,20 @@
// ***********************************************************
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Generated file: to be removed

@tbille tbille force-pushed the test-cypress branch 3 times, most recently from 38fbf55 to f99d99e Compare March 2, 2018 16:11
.travis.yml Outdated

services:
- docker

script:
- ./run test
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 removed this line, because the API calls are hardcoded, and I want to use staging to run cypress (for now)

@tbille tbille force-pushed the test-cypress branch 2 times, most recently from 29d827b to 4809367 Compare March 2, 2018 16:39
@Lukewh
Copy link
Contributor

Lukewh commented Mar 14, 2018

Is this PR still relevant? Is the test complete? If not can we move it to a feature branch to clean up the PR list pleeeease?

@tbille
Copy link
Contributor Author

tbille commented Mar 15, 2018

This was a test PR. I am closing it.

@tbille tbille closed this Mar 15, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants