Skip to content
This repository was archived by the owner on May 21, 2024. It is now read-only.

WIP: Maven simple E2E test added #184

Open
4rokis wants to merge 5 commits into
base: master
Choose a base branch
from
Open

WIP: Maven simple E2E test added #184

4rokis wants to merge 5 commits into from

Conversation

@4rokis
Copy link
Contributor

@4rokis 4rokis commented Sep 23, 2020

Closes #110

@4rokis 4rokis requested a review from jvorcak September 23, 2020 20:46
@4rokis
Copy link
Contributor Author

4rokis commented Sep 23, 2020

@jvorcak
There are 2 options to do it. This one. Real requests, real run. Negatives are that it is slow and unstable, but it is real. If we gonna go with this we need somehow split this and normal tests

@4rokis
Copy link
Contributor Author

4rokis commented Sep 23, 2020

Other option is to mock everything that is possible, that is fast but might not reflect reality

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@jvorcak jvorcak Awaiting requested review from jvorcak

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Testsuite

2 participants

@4rokis