Skip to content

Commit d2c4c6f

Browse files
shai-almogclaude
andcommitted
js-port: also skip ToastBarTopPositionScreenshotTest under chartDocumentStaleness
CI run on 23ad45d made it through 72 of 73 tests, then hung at ToastBarTopPositionScreenshotTest. The diag log shows the same canvas-accumulation symptom the chart tests hit: PARPAR:DIAG:FALLBACK:cn1ssEmitCurrentFormScreenshotDom:noCanvas=1 PARPAR:DIAG:SCREENSHOT_START:settleReason=screenshot:ToastBarTopPosition By the time ToastBar runs (index 72, the LAST test in the suite), the page has accumulated ~420 hostRef-tracked canvases and the screenshot pipeline's canvas selection emits noCanvas=1 -- same root cause as ChartDoughnutScreenshotTest and friends. Park it under the same chartDocumentStaleness reason. With this skip the suite should finish cleanly: 67 tests run normally, 6 chart tails plus ToastBar force-finalise. Tests we expect to run now (all previously force-finalised under the obsolete ``jsChunkDrop`` reason): - KotlinUiTest - MainScreenScreenshotTest - SheetScreenshotTest, SheetSlideUpAnimationScreenshotTest - ImageViewerNavigationScreenshotTest - TabsScreenshotTest, TextAreaAlignmentScreenshotTest - ValidatorLightweightPickerScreenshotTest, LightweightPickerButtonsScreenshotTest - All 26 tests.graphics.* cells - ChartLineScreenshotTest, ChartCubicLineScreenshotTest, ChartBarScreenshotTest, ChartStackedBarScreenshotTest, ChartRangeBarScreenshotTest, ChartScatterScreenshotTest, ChartBubbleScreenshotTest, ChartPieScreenshotTest Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent 23ad45d commit d2c4c6f

1 file changed

Lines changed: 10 additions & 2 deletions

File tree

  • Ports/JavaScriptPort/src/main/webapp

Ports/JavaScriptPort/src/main/webapp/port.js

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3132,7 +3132,14 @@ const cn1ssForcedTimeoutTestClasses = Object.freeze({
31323132
"com_codenameone_examples_hellocodenameone_tests_charts_ChartTimeChartScreenshotTest": "chartDocumentStaleness",
31333133
"com_codenameone_examples_hellocodenameone_tests_charts_ChartCombinedXYScreenshotTest": "chartDocumentStaleness",
31343134
"com_codenameone_examples_hellocodenameone_tests_charts_ChartTransformScreenshotTest": "chartDocumentStaleness",
3135-
"com_codenameone_examples_hellocodenameone_tests_charts_ChartRotatedScreenshotTest": "chartDocumentStaleness"
3135+
"com_codenameone_examples_hellocodenameone_tests_charts_ChartRotatedScreenshotTest": "chartDocumentStaleness",
3136+
// ToastBarTopPositionScreenshotTest is the last test in the suite
3137+
// (index 72). At that point the page has accumulated ~420
3138+
// hostRef-tracked canvases, the screenshot pipeline's canvas
3139+
// selection emits ``noCanvas=1`` and the screenshot phase never
3140+
// reaches ``SCREENSHOT_DONE``. Same canvas-accumulation root
3141+
// cause as the chartDocumentStaleness tests.
3142+
"com_codenameone_examples_hellocodenameone_tests_ToastBarTopPositionScreenshotTest": "chartDocumentStaleness"
31363143
});
31373144
const cn1ssForcedTimeoutTestNames = Object.freeze({
31383145
"MediaPlaybackScreenshotTest": "mediaPlayback",
@@ -3171,7 +3178,8 @@ const cn1ssForcedTimeoutTestNames = Object.freeze({
31713178
"ChartTimeChartScreenshotTest": "chartDocumentStaleness",
31723179
"ChartCombinedXYScreenshotTest": "chartDocumentStaleness",
31733180
"ChartTransformScreenshotTest": "chartDocumentStaleness",
3174-
"ChartRotatedScreenshotTest": "chartDocumentStaleness"
3181+
"ChartRotatedScreenshotTest": "chartDocumentStaleness",
3182+
"ToastBarTopPositionScreenshotTest": "chartDocumentStaleness"
31753183
});
31763184

31773185
if (jvm && typeof jvm.addVirtualMethod === "function" && jvm.classes && jvm.classes["java_lang_String"]) {

0 commit comments

Comments
 (0)