Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test(e2e): add steps to check media summary pages #407

Merged
merged 1 commit into from
Feb 4, 2025

Conversation

seferturan
Copy link
Collaborator

@seferturan seferturan commented Feb 4, 2025

🎶 Notes 🎶

  • Adds 2 simple scenarios to make sure the movie and show summary pages are visible.
  • Kept everything in a single scenario to reduce overhead.

👀 Example 👀

Reduced the speed for dramatic purposes:
https://github.com/user-attachments/assets/3b5d2948-191a-4828-8ca8-b88dbad763dd

Still runs fast:
Screenshot 2025-02-04 at 17 30 31

Copy link

deepsource-io bot commented Feb 4, 2025

Here's the code health analysis summary for commits fbff581..f04671b. View details on DeepSource ↗.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource JavaScript LogoJavaScript✅ SuccessView Check ↗
DeepSource Test coverage LogoTest coverage✅ SuccessView Check ↗

Code Coverage Report

MetricAggregateJavascript
Branch Coverage28.6%28.6%
Composite Coverage64.9% (down 0.1% from main)64.9% (down 0.1% from main)
Line Coverage77.2% (down 0.2% from main)77.2% (down 0.2% from main)

💡 If you’re a repository administrator, you can configure the quality gates from the settings.

@seferturan seferturan requested a review from vladjerca February 4, 2025 16:37
@vladjerca vladjerca merged commit c9a5d35 into main Feb 4, 2025
9 checks passed
@vladjerca vladjerca deleted the test/e2e_summary branch February 4, 2025 17:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants