Skip to content

test(cypress): don't expect assistant modal input field to be visible#8523

Open
mejo- wants to merge 1 commit intomainfrom
fix/cypress_assistant_modal_input
Open

test(cypress): don't expect assistant modal input field to be visible#8523
mejo- wants to merge 1 commit intomainfrom
fix/cypress_assistant_modal_input

Conversation

@mejo-
Copy link
Copy Markdown
Member

@mejo- mejo- commented Apr 28, 2026

Apparently it's mostly covered by a "Choose file" button on small screens with latest assistant app version, but the test still should not fail.

🏁 Checklist

  • Code is properly formatted (npm run lint / npm run stylelint / composer run cs:check)
  • Sign-off message is added to all commits
  • Tests (unit, integration and/or end-to-end) passing and the changes are covered with tests

@mejo- mejo- self-assigned this Apr 28, 2026
@mejo- mejo- added tests If you write them we ♥ you 3. to review regression labels Apr 28, 2026
@mejo- mejo- moved this from 🧭 Planning evaluation (don't pick) to 👀 In review in 📝 Productivity team Apr 28, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 28, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Apparently it's mostly covered by a "Choose file" button on small
screens with latest assistant app version, but the test still should not
fail.

Signed-off-by: Jonas <jonas@freesources.org>
@mejo- mejo- force-pushed the fix/cypress_assistant_modal_input branch from dcde7db to 40ad677 Compare April 28, 2026 16:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review regression tests If you write them we ♥ you

Projects

Status: 👀 In review

Development

Successfully merging this pull request may close these issues.

1 participant