Skip to content

fix: update tsc test data to fix e2e tests#5339

Merged
princerajpoot20 merged 3 commits intoasyncapi:masterfrom
princerajpoot20:fix-e2e-tests
Apr 23, 2026
Merged

fix: update tsc test data to fix e2e tests#5339
princerajpoot20 merged 3 commits intoasyncapi:masterfrom
princerajpoot20:fix-e2e-tests

Conversation

@princerajpoot20
Copy link
Copy Markdown
Member

@princerajpoot20 princerajpoot20 commented Apr 18, 2026

Uodated tsc test data
Resloves #5281

Summary by CodeRabbit

  • Tests
    • Updated TSC member test data and associated social media profile information in test fixtures.

@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 18, 2026

Deploy Preview for asyncapi-website ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 8f277f6
🔍 Latest deploy log https://app.netlify.com/projects/asyncapi-website/deploys/69ea5c150d7859000823a611
😎 Deploy Preview https://deploy-preview-5339--asyncapi-website.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 18, 2026

📝 Walkthrough

Walkthrough

Test fixture data in a Cypress test file was updated: one TSC member entry was replaced with another, along with corresponding GitHub, Twitter, and LinkedIn URLs. No test logic or control flow was modified.

Changes

Cohort / File(s) Summary
Test Fixture Data
cypress/tscpage.cy.js
Updated member test data by replacing "Khuda Dad Nomani" with "Thulisile Sibanda" and updating associated social media URLs (GitHub, Twitter, LinkedIn).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰✨ A roster shift in tests so fine,
Old names out, new names align,
URLs dance in four-line grace,
Fresh faces take their rightful place!

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The PR title 'fix: update tsc test data to fix e2e tests' accurately describes the main change—updating test fixture data for TSC members to fix failing e2e tests. It is concise, specific, and directly relevant to the changeset.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 18, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (c0593fb) to head (8f277f6).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##            master     #5339   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           22        22           
  Lines          830       830           
  Branches       159       159           
=========================================
  Hits           830       830           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@princerajpoot20 princerajpoot20 changed the title fix: e2e tests fix: update tsc test data to fix e2e tests Apr 18, 2026
@asyncapi-bot
Copy link
Copy Markdown
Contributor

asyncapi-bot commented Apr 18, 2026

⚡️ Lighthouse report for the changes in this PR:

Category Score
🔴 Performance 45
🟢 Accessibility 98
🟢 Best practices 92
🟢 SEO 100
🔴 PWA 33

Lighthouse ran on https://deploy-preview-5339--asyncapi-website.netlify.app/

@princerajpoot20 princerajpoot20 moved this from To Be Triaged to Need review in Website - Kanban Apr 18, 2026
@princerajpoot20
Copy link
Copy Markdown
Member Author

@anshgoyalevil will be a quick one

@sonarqubecloud
Copy link
Copy Markdown

@princerajpoot20 princerajpoot20 merged commit 6e238ff into asyncapi:master Apr 23, 2026
21 checks passed
@github-project-automation github-project-automation Bot moved this from Need review to Done in Website - Kanban Apr 23, 2026
codxbrexx pushed a commit to codxbrexx/website-asyncapi that referenced this pull request May 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants