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

Add System tests #12

Open
rmpestano opened this issue Jun 19, 2019 · 3 comments
Open

Add System tests #12

rmpestano opened this issue Jun 19, 2019 · 3 comments
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@rmpestano
Copy link
Contributor

I think we should have some system tests in order to guarantee we not break the extension on future changes (on the extension itself, myfaces or quakus-core update)

I have done something on quarkus-omnifaces (see here), it uses htmlunit and works quite good.

I also have tried to use selenium with drone/graphene but quarkus team said that users should not rely on quarkus-arquillian extension, see discussion here.

I think we have two options which is:

1 - htmlunit - simpler but less reliable (I think as it emulates a browser)
2 - pure selenium without arquillian - more reliable but a bit more complex and harder to use in CI (travis)

WDYT?

@tandraschko
Copy link
Owner

+1 for tests in generell
Just do it as you like :)

We also need think where we host the project later, see: https://groups.google.com/forum/#!topic/quarkus-dev/yCGtMdnPMV8

@rmpestano
Copy link
Contributor Author

We also need think where we host the project later

I'm following the discussion on ggroups, don't know if we can host on apache repo, as child/module of myfaces.

@tandraschko
Copy link
Owner

tandraschko commented Jun 19, 2019

Not sure...
I would like to see it in quarkus - we also don't have e.g. TomEE or WebSphere integration code in the myfaces repo

@tandraschko tandraschko added enhancement New feature or request help wanted Extra attention is needed labels Jun 20, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants