Skip to content

test: add massive scaling tests for BrandParticles#3634

Open
riyakandwal wants to merge 5 commits into
JhaSourav07:mainfrom
riyakandwal:statscard-type-compiler-tests
Open

test: add massive scaling tests for BrandParticles#3634
riyakandwal wants to merge 5 commits into
JhaSourav07:mainfrom
riyakandwal:statscard-type-compiler-tests

Conversation

@riyakandwal
Copy link
Copy Markdown
Contributor

Description

Fixes #2644

Pillar

  • Other (Bug fix, refactoring, docs)

Changes

  • Added massive scaling tests for BrandParticles
  • Verified stability during repeated renders
  • Verified component behavior under heavy rerender load
  • Verified DOM structure remains stable during scaling scenarios

Visual Preview

N/A (test-only changes)

Checklist before requesting a review:

  • I have read the CONTRIBUTING.md file.
  • I have tested these changes locally.
  • I have made sure that I have only one commit to merge in this PR.

@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented Jun 4, 2026

Someone is attempting to deploy a commit to the jhasourav07's projects Team on Vercel.

A member of the Team first needs to authorize it.

Aamod007
Aamod007 previously approved these changes Jun 4, 2026
@Aamod007 Aamod007 added gssoc26 GSSoC 2026 mentor:Aamod007 quality:clean PR follows clean coding practices, proper formatting, documentation, and maintainability standards. level:beginner Small changes Usually isolated fixes or simple UI/text updates. type:testing Adding, updating, or fixing tests testing labels Jun 4, 2026
@Aamod007 Aamod007 dismissed their stale review June 4, 2026 07:42

Dismissing to re-review

Copy link
Copy Markdown
Collaborator

@Aamod007 Aamod007 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi! 👋 Thanks for your contribution! I noticed the PR title says est: add massive scaling tests for BrandParticles, but this PR also includes tests for CherryBlossom, type compiler tests for StatsCard, and even refactors the StatsCard.tsx implementation by removing the exported �uildMiniChart function.

To keep PRs focused and easier to review, we prefer sticking to one feature or scope per PR (as mentioned in the CONTRIBUTING guidelines). Could you please separate these changes, and just keep the BrandParticles tests in this one as per the title? Let me know if you need any help splitting this up!

@Aamod007 Aamod007 added the status:blocked This PR is blocked due to a failing CI check. label Jun 4, 2026
@github-actions github-actions Bot removed the status:blocked This PR is blocked due to a failing CI check. label Jun 4, 2026
Copy link
Copy Markdown
Collaborator

@Aamod007 Aamod007 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! BrandParticles scaling tests are straightforward and effective.

@Aamod007 Aamod007 removed the gssoc26 label Jun 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

GSSoC 2026 level:beginner Small changes Usually isolated fixes or simple UI/text updates. mentor:Aamod007 quality:clean PR follows clean coding practices, proper formatting, documentation, and maintainability standards. testing type:testing Adding, updating, or fixing tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

test(BrandParticles-massive-scaling): verify Massive Data Sets and Extreme High Bounds Scaling (Variation 2)

2 participants