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

test Jetty12 #17412

Closed
wants to merge 2 commits into from
Closed

test Jetty12 #17412

wants to merge 2 commits into from

Conversation

Artur-
Copy link
Member

@Artur- Artur- commented Aug 10, 2023

No description provided.

@sonatype-lift
Copy link

sonatype-lift bot commented Aug 10, 2023

Sonatype Lift is retiring

Sonatype Lift will be retiring on Sep 12, 2023, with its analysis stopping on Aug 12, 2023. We understand that this news may come as a disappointment, and Sonatype is committed to helping you transition off it seamlessly. If you’d like to retain your data, please export your issues from the web console.
We are extremely grateful and thank you for your support over the years.

📖 Read about the impacts and timeline

@github-actions
Copy link

github-actions bot commented Aug 10, 2023

Test Results

   805 files  +   289     805 suites  +289   56m 3s ⏱️ + 5m 0s
6 074 tests +5 174  6 033 ✔️ +5 166  40 💤 +7  1 +1 
6 129 runs  +4 986  6 087 ✔️ +4 984  41 💤 +1  1 +1 

For more details on these failures, see this check.

Results for commit e65d19b. ± Comparison against base commit a108dae.

This pull request removes 128 and adds 5302 tests. Note that renamed tests count towards both.
com.vaadin.flow.server.StaticFileServerTests ‑ serveJarResourceTwice
com.vaadin.flow.spring.flowsecurity.AppViewIT ‑ access_restricted_to_admin
com.vaadin.flow.spring.flowsecurity.AppViewIT ‑ access_restricted_to_logged_in_users
com.vaadin.flow.spring.flowsecurity.AppViewIT ‑ adminAppliesForHugeLoan_accessGranted
com.vaadin.flow.spring.flowsecurity.AppViewIT ‑ adminAppliesForLoan_accessDenied
com.vaadin.flow.spring.flowsecurity.AppViewIT ‑ logout_redirects_to_root_page
com.vaadin.flow.spring.flowsecurity.AppViewIT ‑ menu_correct_for_admin
com.vaadin.flow.spring.flowsecurity.AppViewIT ‑ menu_correct_for_anonymous
com.vaadin.flow.spring.flowsecurity.AppViewIT ‑ menu_correct_for_user
com.vaadin.flow.spring.flowsecurity.AppViewIT ‑ navigate_in_thread_with_access
…
com.vaadin.AssertionTest ‑ testAssertionsAreEnabled
com.vaadin.base.devserver.AbstractDevServerRunnerTest ‑ shouldPassEncodedUrlToDevServer
com.vaadin.base.devserver.AbstractDevServerRunnerTest ‑ updateServerStartupEnvironment_preferIpv4_LocalhostIpAddressAddedToProcessEnvironment
com.vaadin.base.devserver.AbstractDevServerRunnerTest ‑ updateServerStartupEnvironment_preferIpv6_LocalhostIpAddressAddedToProcessEnvironment
com.vaadin.base.devserver.BrowserLiveReloadAccessorImplTest ‑ getLiveReload_devMode_contextHasNoReloadInstance_instanceIsCreated
com.vaadin.base.devserver.BrowserLiveReloadAccessorImplTest ‑ getLiveReload_devMode_contextHasReloadInstance_instanceIsReturned
com.vaadin.base.devserver.BrowserLiveReloadAccessorImplTest ‑ getLiveReload_liveReloadDisabled_instanceIsCreated
com.vaadin.base.devserver.BrowserLiveReloadAccessorImplTest ‑ getLiveReload_productionMode_nullIsReturned
com.vaadin.base.devserver.DebugWindowConnectionTest ‑ backwardsCompatibilityClassExists
com.vaadin.base.devserver.DebugWindowConnectionTest ‑ getBackend_HotSwapVaadinIntegrationClassLoaded_returnsHOTSWAP_AGENT
…
This pull request removes 1 skipped test and adds 8 skipped tests. Note that renamed tests count towards both.
com.vaadin.flow.spring.test.UploadIT ‑ multiFileUpload
com.vaadin.base.devserver.startup.DevModeInitializerClassLoaderTest ‑ should_loadResources_from_customClassLoader
com.vaadin.flow.component.UITest ‑ navigateWithParameters_delegateToRouter
com.vaadin.flow.server.communication.WebComponentBootstrapHandlerViteTest ‑ writeBootstrapPage_noExportChunk
com.vaadin.flow.server.communication.WebComponentBootstrapHandlerViteTest ‑ writeBootstrapPage_withExportChunk
com.vaadin.flow.server.frontend.BundleValidationTest ‑ bundleMissesSomeEntries_devMode_skipBundleBuildSet_noBundleRebuild[0]
com.vaadin.flow.server.frontend.BundleValidationTest ‑ forceProductionBundle_bundleRequired[1]
com.vaadin.flow.server.frontend.BundleValidationTest ‑ loadDependenciesOnStartup_annotatedClassInProject_compilationRequiredForProduction[1]
com.vaadin.flow.server.frontend.FileIOUtilsTest ‑ projectFolderOnWindows

♻️ This comment has been updated with latest results.

@sonarcloud
Copy link

sonarcloud bot commented Aug 11, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@mshabarov
Copy link
Contributor

@Artur- do you have something in mind where we should look into, to migrate to Jetty 12?

@mshabarov mshabarov self-requested a review August 21, 2023 11:04
@Artur-
Copy link
Member Author

Artur- commented Aug 21, 2023

When Spring Boot has integrated it, tracked in spring-projects/spring-boot#36073

@Artur- Artur- closed this Aug 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants