Skip to content
This repository has been archived by the owner on Nov 22, 2020. It is now read-only.

Visual testing views with Cypress #94

Open
wants to merge 30 commits into
base: develop
Choose a base branch
from
Open

Conversation

gaelreyrol
Copy link
Member

@gaelreyrol gaelreyrol commented Mar 31, 2020

This PR is still under heavy development.

It provides visual testing for the views served by the api.

ToDo :

  • Form test suite
  • Track test suite
  • Continuous Integration

We should go further on track test suite but I think we might want to merge this one before.

PS: Github Actions is cool for simple things but it's actually crap when you need more, like postgres service. I can't use an newer version than 10.8 otherwise the container fails...

@gaelreyrol gaelreyrol added the WIP label Mar 31, 2020
@gaelreyrol gaelreyrol self-assigned this Mar 31, 2020
Copy link
Member

@alexisjanvier alexisjanvier left a comment

Choose a reason for hiding this comment

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

Great 👍
Just a small question : is it complicated to move the cypress folder outside the api one ? Maybe in a /tests folder at the root of the project ?

@gaelreyrol
Copy link
Member Author

Just a small question : is it complicated to move the cypress folder outside the api one ? Maybe in a /tests folder at the root of the project ?

No it is not, it just needs a few changes 😄

I'll do it tonight !

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants