saltsthlm / kamel-graduation-project

Kamel's Graduation Project
MIT License
0 stars 0 forks source link

merge e2e tests into main repo #32

Closed worstpractice closed 4 years ago

worstpractice commented 4 years ago

For the time being, our e2e tests reside outside of our main project repo. This arrangement comes with certain benefits, as well as a number of obvious weaknesses (which I suspect may prove outstanding).

As I see it, in short, the main benefits of this arrangement is:

However, the main drawbacks are:

Either way, the tests need to be transparent, available, team property. And sooner rather than later, in my opinion.

Suggestions welcome (as in all matters)!

moritzploss commented 4 years ago

How would we go about merging the tests?

worstpractice commented 4 years ago

How would we go about merging the tests?

That is a very good question.

I think we should consider the issue of Github/Cypress integration and the issue of merging the e2e tests into our main repo, into a single issue on monday.

I fear the added complexity of combining them all... There are Jest extensions that trick Cypress into accepting Jest, despite the fact Cypress wants Mocha to run it...

It's important to state again that the e2e tests work just as well from outside the main repo.

I think the issue of rising complexity might weigh in favor of keeping the repos separate, just in order to stave off an absolute combinatorial explosion of complexity inside the main repo.

We'll revisit the issue on monday. Either way, I think it is critical that the team has full insight into (and easy access to) our e2e test suite.

I hope to have a (perhaps temporary) public repo available before the weekend is over, containing our (currently embryonic) e2e test suite, but mostly the files and folders (and configuration) cypress brings.

worstpractice commented 4 years ago

I've grown convinced that keeping the e2e tests in a separate repo is actually a pure benefit to the current project layout. With the secondary repo in place (link for those who missed it), the experience right now is nothing but convenient. As such, I suggest we do not merge the e2e tests into the main repo -- especially considering the risk in doing so (incompatible test runners and all that jazz).