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

Delete drawboard function from WebSocketsTest #15278

Merged
merged 1 commit into from Jul 26, 2022

Conversation

rfan1
Copy link
Contributor

@rfan1 rfan1 commented Jul 25, 2022

Due to poo#114532, in the sake of stability, that we can safely
delete the function that tests the drawboard example.
We already cover the rest of Tomcat's WebSocket examples,
and, in reality, this drawboard example is not really tested
with multiple connections as it would ideally be the case.

At the same time, increase the timeout value for some tomcat
test modules to reduce the impact by performance.

Due to poo#114532, in the sake of stability, that we can safely
delete the function that tests the drawboard example.
We already cover the rest of Tomcat's WebSocket examples,
and, in reality, this drawboard example is not really tested
with multiple connections as it would ideally be the case.

At the same time, increase the timeout value for some tomcat
test modules to reduce the impact by performance.
@rfan1 rfan1 force-pushed the tomcat_multiplayer_drawboard branch from 83a6bcb to 654fe81 Compare July 25, 2022 13:25
@Zaoliang
Copy link
Contributor

you can remove from commit msg:
[type description here, PLEASE, REMOVE THIS LINE, PLACEHOLDER, BEFORE SUBMITTING THIS PULL REQUEST]

@a-kpappas a-kpappas self-requested a review July 26, 2022 08:52
@a-kpappas a-kpappas merged commit f6a7450 into os-autoinst:master Jul 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants