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.

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

CCDM: test SessionCloseLogoutIT fails in bootstrapping v15 #7600

Closed
manolo opened this issue Feb 14, 2020 · 5 comments
Closed

CCDM: test SessionCloseLogoutIT fails in bootstrapping v15 #7600

manolo opened this issue Feb 14, 2020 · 5 comments

Comments

@manolo
Copy link
Member

manolo commented Feb 14, 2020

There is an exception when running the test in TC that does not happen in local.

Caused by: java.lang.AssertionError: Received error message in browser log console right after opening the page, message: [2020-02-14T18:53:31+0000] [SEVERE] http://172.16.1.163:8888/view/VAADIN/build/vaadin-4-2ecfa5ca1a31d0bd1c7b.cache.js 0:8701 "Push connection using long-polling failed!"

It might be due to the @Push annotation that is in multiple test views (as shown below) instead of in an AppShell class, but needs further investigation.

[main] ERROR com.vaadin.flow.server.startup.VaadinAppShellInitializer -

Found app shell configuration annotations in non `AppShellConfigurator` classes.
Please create a custom class implementing `AppShellConfigurator` and move the following annotations to it:
    - @Push from com.vaadin.flow.uitest.ui.PushSettingsView
    - @Push from com.vaadin.flow.uitest.ui.LongPollingPushView
    - @Push from com.vaadin.flow.uitest.ui.PushLongPollingUpdateDivView
    - @Push from com.vaadin.flow.uitest.ui.LongPollingMultipleThreadsView
    - @Push from com.vaadin.flow.uitest.ui.SessionCloseLogoutView
    - @Push from com.vaadin.flow.uitest.ui.routing.PushLayout
    - @Push from com.vaadin.flow.uitest.ui.TrackMessageSizeView
    - @Push from com.vaadin.flow.uitest.ui.PushWSUpdateDivView
@denis-anisimov
Copy link
Contributor

There is an exception when running the test in TC that does not happen in local.

Yes and no:

  • if I run the IT locally then it doesn't fail
  • if I open run the server with view locally and open a view manually then I see the error in the console:
vaadin-5-f776035f32111fb70854.cache.js:1 Push connection using long-polling failed!

So there is an issue and this is might be quite serious issue.
It doesn't look like Long polling Push is broken but this needs to be investigated. The message at least says that push
connection failed which is serious.

@denis-anisimov
Copy link
Contributor

Screen Shot 2021-03-24 at 3 17 06 PM

@denis-anisimov
Copy link
Contributor

I'm going to enable the IT via workaround : ignore the error message in the console.
The IT tests totally different thing.
But this ticket:

  • has to be fixed
  • it should be done ASAP
  • can't be closed because test doesn't fail: as I said I will change the IT so it doesn't fail but this ticket describes a bug which is unrelated to the SessionCloseLogoutIT

@denis-anisimov
Copy link
Contributor

Looks like this is a generic Push issue which can be reproduced without fusion, in Flow 14: #10103

@ZheSun88
Copy link
Contributor

the issue is still going on and the related bug has been reported to atmosphere Atmosphere/atmosphere#2444.

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

No branches or pull requests

5 participants