(texera) branch main updated: test(amber): cover WebSocket payload tuning (#7220)

[github-bot]

This is an automated email from the ASF dual-hosted git repository.

github-merge-queue[bot] pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/texera.git


The following commit(s) were added to refs/heads/main by this push:
     new 268f098fe6 test(amber): cover WebSocket payload tuning (#7220)
268f098fe6 is described below

commit 268f098fe648e93c16eb03a61edd47634c6bfe51
Author: Xinyuan Lin <xinyu...@uci.edu>
AuthorDate: Sat Aug 1 02:10:55 2026 -0700

    test(amber): cover WebSocket payload tuning (#7220)
    
    ### What changes were proposed in this PR?
    
    Adds focused unit coverage for servlet WebSocket container lookup and
    the configured text and binary payload limits, including exact one-KiB
    byte conversion.
    
    ### Any related issues, documentation, discussions?
    
    Closes #7217
    
    ### How was this PR tested?
    
    - `WorkflowExecutionService/testOnly
    org.apache.texera.web.resource.WebsocketPayloadSizeTunerSpec` — 2
    passed.
    - `WorkflowExecutionService/Test/scalafmtCheck`
    - `WorkflowExecutionService/Test/scalafix --check`
    
    Mutation proof (each production mutation was reverted):
    
    | Production mutation | Focused test result |
    | --- | --- |
    | Changed the text-buffer multiplier from `1024` to `1000` | 0 passed, 2
    failed |
    | Removed the binary-buffer setter | 0 passed, 2 failed |
    
    ### Was this PR authored or co-authored using generative AI tooling?
    
    Generated-by: Codex (GPT-5)
    
    ---------
    
    Signed-off-by: Xinyuan Lin <xinyu...@uci.edu>
    Co-authored-by: Copilot Autofix powered by AI 
<175728472+copi...@users.noreply.github.com>
---
 .../resource/WebsocketPayloadSizeTunerSpec.scala   | 57 ++++++++++++++++++++++
 1 file changed, 57 insertions(+)

diff --git 
a/amber/src/test/scala/org/apache/texera/web/resource/WebsocketPayloadSizeTunerSpec.scala
 
b/amber/src/test/scala/org/apache/texera/web/resource/WebsocketPayloadSizeTunerSpec.scala
new file mode 100644
index 0000000000..dc4d7abda4
--- /dev/null
+++ 
b/amber/src/test/scala/org/apache/texera/web/resource/WebsocketPayloadSizeTunerSpec.scala
@@ -0,0 +1,57 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ *   http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+
+package org.apache.texera.web.resource
+
+import org.scalamock.scalatest.MockFactory
+import org.scalatest.flatspec.AnyFlatSpec
+import org.scalatest.matchers.should.Matchers
+
+import javax.servlet.{ServletContext, ServletContextEvent}
+import javax.websocket.server.ServerContainer
+
+class WebsocketPayloadSizeTunerSpec extends AnyFlatSpec with Matchers with 
MockFactory {
+
+  private def eventWith(container: ServerContainer): ServletContextEvent = {
+    val servletContext = mock[ServletContext]
+    (servletContext
+      .getAttribute(_: String))
+      .expects(classOf[ServerContainer].getName)
+      .returning(container)
+      .once()
+    new ServletContextEvent(servletContext)
+  }
+
+  "contextInitialized" should "look up the server container by its class name 
and tune both limits" in {
+    val maxKiB = 64
+    val container = mock[ServerContainer]
+    (container.setDefaultMaxTextMessageBufferSize(_: Int)).expects(maxKiB * 
1024).once()
+    (container.setDefaultMaxBinaryMessageBufferSize(_: Int)).expects(maxKiB * 
1024).once()
+
+    new 
WebsocketPayloadSizeTuner(maxKiB).contextInitialized(eventWith(container))
+  }
+
+  it should "convert a one-KiB payload limit to exactly 1024 bytes" in {
+    val container = mock[ServerContainer]
+    (container.setDefaultMaxTextMessageBufferSize(_: Int)).expects(1024).once()
+    (container.setDefaultMaxBinaryMessageBufferSize(_: 
Int)).expects(1024).once()
+
+    new WebsocketPayloadSizeTuner(1).contextInitialized(eventWith(container))
+  }
+}