mirror of
https://github.com/openobserve/openobserve.git
synced 2026-11-06 08:09:24 +00:00
Shrinath Rao
9b9473ea64
### **PR Type** Tests, Enhancement ___ ### **Description** - Add streaming toggle helper and usage - Add afterEach flip across logs specs - Clean up duplicate imports in tests - Increase Playwright workers on CI ___ ### Diagram Walkthrough ```mermaid flowchart LR CA["CommonActions: streaming helpers"] -- "flipStreaming()" --> Specs["Logs specs afterEach hooks"] Specs -- "stabilize UI by dismissing overlays" --> Stable["More reliable test runs"] CI["Playwright config"] -- "workers 2 -> 5" --> Faster["Faster CI"] ``` <details> <summary><h3> File Walkthrough</h3></summary> <table><thead><tr><th></th><th align="left">Relevant files</th></tr></thead><tbody><tr><td><strong>Enhancement</strong></td><td><details><summary>1 files</summary><table> <tr> <td><strong>commonActions.js</strong><dd><code>Add streaming toggle helpers and overlay dismissal</code> </dd></td> <td><a href="https://github.com/openobserve/openobserve/pull/7907/files#diff-4d726dd2bdb34b2c90a31798ff49b6df5bc952bd049b7c38bebb18b6db0076e2">+41/-0</a> </td> </tr> </table></details></td></tr><tr><td><strong>Tests</strong></td><td><details><summary>13 files</summary><table> <tr> <td><strong>indexquery.spec.js</strong><dd><code>Initialize PageManager and flip streaming post-test</code> </dd></td> <td><a href="https://github.com/openobserve/openobserve/pull/7907/files#diff-34747033f2191057e144ed1892b682d355e11788603a61aa98fbf991eed50c5a">+7/-0</a> </td> </tr> <tr> <td><strong>join.spec.js</strong><dd><code>Normalize imports and add afterEach flip</code> </dd></td> <td><a href="https://github.com/openobserve/openobserve/pull/7907/files#diff-b161f6e50affa381ff0ea27463a79c061139041f44ecaef5a64e4aa88ef97d45">+5/-1</a> </td> </tr> <tr> <td><strong>logshistogram.spec.js</strong><dd><code>Remove duplicate import and add afterEach flip</code> </dd></td> <td><a href="https://github.com/openobserve/openobserve/pull/7907/files#diff-d01da1c5a03b1d1c1c74e11f6e4ebe879def1ce87e407aa9a1afc8473040f4bc">+6/-1</a> </td> </tr> <tr> <td><strong>logspage.spec.js</strong><dd><code>Parallel note, minor cleanups; no afterEach flip</code> </dd></td> <td><a href="https://github.com/openobserve/openobserve/pull/7907/files#diff-b1a64252adae01c5e8f7e758e3c820f5476260f60c44a4a57b4f15a05a59a85a">+21/-33</a> </td> </tr> <tr> <td><strong>logsqueries.cte.spec.js</strong><dd><code>Remove duplicate import; add afterEach flip</code> </dd></td> <td><a href="https://github.com/openobserve/openobserve/pull/7907/files#diff-153f79d86888d9e3a82f42a15242cacb3a126c263aee91cada88ae04dffacd36">+6/-1</a> </td> </tr> <tr> <td><strong>logsqueries.matchall.spec.js</strong><dd><code>Remove duplicate import; add afterEach flip</code> </dd></td> <td><a href="https://github.com/openobserve/openobserve/pull/7907/files#diff-99c81de942a418ed93cb748d23740c18a68417844f033c94d6636685a4c3cc68">+6/-1</a> </td> </tr> <tr> <td><strong>logsqueries.spec.js</strong><dd><code>Remove duplicate import; add afterEach flip</code> </dd></td> <td><a href="https://github.com/openobserve/openobserve/pull/7907/files#diff-c2055f6cef1580b15c5f94f5c8c2e496a6e921332b48bad943bb7c0630846a65">+6/-1</a> </td> </tr> <tr> <td><strong>logsquickmode.spec.js</strong><dd><code>Remove duplicate import; add afterEach flip</code> </dd></td> <td><a href="https://github.com/openobserve/openobserve/pull/7907/files#diff-81f2f36450c07435f1b0f3c33462c7dac98c0798250913ae5ed928f57ad82cdf">+6/-1</a> </td> </tr> <tr> <td><strong>pagination.spec.js</strong><dd><code>Normalize import and add afterEach flip</code> </dd></td> <td><a href="https://github.com/openobserve/openobserve/pull/7907/files#diff-638d825c69799170731ba602def4b274a04a5444e8252f24fd97cdfe194049f0">+6/-2</a> </td> </tr> <tr> <td><strong>region.spec.js</strong><dd><code>Remove duplicate import; add afterEach flip</code> </dd></td> <td><a href="https://github.com/openobserve/openobserve/pull/7907/files#diff-19ef2f7fe7e4063bd458370eca8951079965c22f91120d249dacf917c9b6da90">+6/-1</a> </td> </tr> <tr> <td><strong>searchpartition.spec.js</strong><dd><code>Remove duplicate import; add afterEach flip</code> </dd></td> <td><a href="https://github.com/openobserve/openobserve/pull/7907/files#diff-967859311f28b32020e4c9cbb0fb7a290dbbe542fc125054afad9a712d299517">+6/-1</a> </td> </tr> <tr> <td><strong>secondsPrecisionAdded.spec.js</strong><dd><code>Normalize import; no afterEach hook added</code> </dd></td> <td><a href="https://github.com/openobserve/openobserve/pull/7907/files#diff-862c0bb7d64fc0e09f8e0dd4a3d0571de4d4f08dc57d48b06581461413352f26">+4/-1</a> </td> </tr> <tr> <td><strong>unflattened.spec.js</strong><dd><code>Comment cleanup and add afterEach flip</code> </dd></td> <td><a href="https://github.com/openobserve/openobserve/pull/7907/files#diff-35703a04047bd3dc257e8e606360e196f9ead45fc44f3857a6e04023590f1a1e">+6/-0</a> </td> </tr> </table></details></td></tr><tr><td><strong>Configuration changes</strong></td><td><details><summary>2 files</summary><table> <tr> <td><strong>playwright.config.js</strong><dd><code>Increase CI workers from 2 to 5</code> </dd></td> <td><a href="https://github.com/openobserve/openobserve/pull/7907/files#diff-91cb9615068914b2e4f5d66fcbed50d3e9771ea2d882f8e8048bbe54eab3c414">+1/-1</a> </td> </tr> <tr> <td><strong>playwright.yml</strong><dd><code>Rename UI test job to e2e</code> </dd></td> <td><a href="https://github.com/openobserve/openobserve/pull/7907/files#diff-7afcd2d8f7b49bda74843f209eefb7b2da45f7e7803bf2e4bd636699b76aa2d3">+1/-1</a> </td> </tr> </table></details></td></tr></tr></tbody></table> </details> ___