fix flaky. sharing test

[ScharfViktor]

Description

await page.locator(closeSidebarSubPanelBtn).isVisible() --> is FLAKY don't use it https://playwright.dev/docs/api/class-locator#locator-is-visible

Related Issue

• Fixes [QA] e2e test failure in Opencloud Nightly CI build qa#67

How Has This Been Tested?

• test environment:
• test case 1:
• test case 2:
• ...

Types of changes

• Bugfix
• Enhancement (a change that doesn't break existing code or deployments)
• Breaking change (a modification that affects current functionality)
• Technical debt (addressing code that needs refactoring or improvements)
• Tests (adding or improving tests)
• Documentation (updates or additions to documentation)
• Maintenance (like dependency updates or tooling adjustments)

[Copilot]

Pull Request Overview

This PR fixes a flaky test in the sidebar closing functionality by replacing unstable visibility checks with Playwright's built-in expect assertions. The change addresses test flakiness related to timing issues when determining which sidebar button is visible.

• Replaces manual isVisible() check with expect().toBeVisible() for more reliable element detection
• Uses Playwright's or() locator to handle either sub-panel or root panel button visibility
• Removes arbitrary timeout that was previously used to wait for sidebar transitions

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[ScharfViktor]

I restarted the pipeline several times. It seems the flaky is gone.