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.

Sign up for GitHub

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

Jump to bottom

Create an integration test for flow.start #6143

Closed
vlukashov opened this issue Jul 31, 2019 · 0 comments
Closed

Create an integration test for flow.start #6143

vlukashov opened this issue Jul 31, 2019 · 0 comments
Assignees
@manolo
Labels
hilla Issues related to Hilla
Milestone
CCDM MVP

Comments

@vlukashov
Copy link

vlukashov commented Jul 31, 2019 •
edited by manolo

Run the existing Vaadin tests verifying push, error messages, stats, etc added to the page by BootstrapHandler, but now in the case that the app is client-side bootstrapped through flow.start().

The additional tests should be run together with all other Flow integration tests (see flow-tests). That will likely be a separate module in the Flow project because it requires a different configuration. This module could be taken as reference https://github.com/vaadin/flow/tree/master/flow-tests/test-mixed

The idea is to reuse the existing Flow tests, but apply them to the app that is started in a different way.
@project-bot project-bot bot added this to Inbox - needs triage in OLD Vaadin Flow ongoing work (Vaadin 10+) Jul 31, 2019
@pleku pleku removed this from Inbox - needs triage in OLD Vaadin Flow ongoing work (Vaadin 10+) Aug 5, 2019
@manolo manolo added the hilla Issues related to Hilla label Aug 5, 2019
@manolo manolo self-assigned this Aug 9, 2019
@manolo manolo mentioned this issue Aug 9, 2019
Merged
@vlukashov vlukashov added this to the CCDM MVP milestone Aug 12, 2019
@manolo manolo closed this as completed Aug 13, 2019
manolo added a commit that referenced this issue Aug 13, 2019
@manolo
CCDM: Add ITs for Flow client API (#6215)
f218ec2 
Fixes #6143

- Added couple of tests in the test-ccdm module that tests
  the asyncronous bootstraping of flow from javascript,
  and the connection of client and server UIs
- Added support for typescript in webpack generated file.
@vlukashov vlukashov mentioned this issue Feb 27, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@manolo manolo

Labels
hilla Issues related to Hilla
Projects
None yet
Milestone
CCDM MVP
Development

No branches or pull requests
2 participants
@manolo @vlukashov