Skip to content

test(ActivityLandscape): add component tests#1960

Merged
JhaSourav07 merged 1 commit into
JhaSourav07:mainfrom
Shikhartiwari967:test-activity-landscape
May 31, 2026
Merged

test(ActivityLandscape): add component tests#1960
JhaSourav07 merged 1 commit into
JhaSourav07:mainfrom
Shikhartiwari967:test-activity-landscape

Conversation

@Shikhartiwari967
Copy link
Copy Markdown
Contributor

Description

Fixes #677

Added comprehensive tests for the ActivityLandscape component.

Changes

  • Created components/dashboard/ActivityLandscape.test.tsx
  • Mocked framer-motion for stable test execution
  • Added test for rendering the Activity Landscape heading
  • Added test for rendering all time-range tabs (1W, 1M, 3M, 1Y)
  • Added test to verify 3M is active by default
  • Added test to verify tab switching functionality
  • Added test to verify activity bars are rendered correctly
  • Added test to ensure the component handles empty datasets without crashing

Pillar

  • 🛠️ Other (Bug fix, refactoring, docs)

Visual Preview

N/A (Test-only change)

Checklist before requesting a review:

  • I have read the CONTRIBUTING.md file.
  • I have run npm run lint and resolved all errors.
  • My commit follows the Conventional Commits format.
  • I have made sure that I have only one commit to merge in this PR.
  • This PR only adds automated tests and does not modify component behavior.

@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented May 31, 2026

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

A member of the Team first needs to authorize it.

@github-actions github-actions Bot added the status:blocked This PR is blocked due to a failing CI check. label May 31, 2026
@github-actions
Copy link
Copy Markdown

🚨 Hey @Shikhartiwari967, the CI Pipeline is failing on this PR and it has been marked as status:blocked.

Please fix the issues before this can be reviewed. Here's how:

1. Run checks locally before pushing:

npm run format:check   # Check Prettier formatting
npm run lint           # Run ESLint
npm run typecheck      # TypeScript type check
npm run test           # Run unit tests (Vitest)
npm run build          # Verify production build passes

2. Auto-fix common issues:

npm run format         # Auto-fix formatting with Prettier
npm run lint -- --fix  # Auto-fix lint errors where possible

3. Check the full failure log here:
👉 View CI Run

Once you push a fix and the CI passes, the status:blocked label will be removed automatically. 💪

@Shikhartiwari967 Shikhartiwari967 force-pushed the test-activity-landscape branch 2 times, most recently from 4ff4011 to 4b5467a Compare May 31, 2026 06:09
@Aamod007
Copy link
Copy Markdown
Collaborator

@Shikhartiwari967 RUN TESTS

@Shikhartiwari967 Shikhartiwari967 force-pushed the test-activity-landscape branch 2 times, most recently from 9904c3a to e70cda1 Compare May 31, 2026 06:25
@Shikhartiwari967 Shikhartiwari967 force-pushed the test-activity-landscape branch from e70cda1 to c6b27f2 Compare May 31, 2026 06:37
@github-actions github-actions Bot removed the status:blocked This PR is blocked due to a failing CI check. label May 31, 2026
@Shikhartiwari967
Copy link
Copy Markdown
Contributor Author

Hi @JhaSourav07 please merge this PR.

@JhaSourav07 JhaSourav07 merged commit e85dcd5 into JhaSourav07:main May 31, 2026
4 of 5 checks passed
@JhaSourav07 JhaSourav07 added GSSoC 2026 level:beginner Small changes Usually isolated fixes or simple UI/text updates. quality:exceptional Outstanding contribution with exceptional implementation quality, testing. gssoc:approved PR has been reviewed and accepted for valid contribution points mentor:Aamod007 labels May 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

gssoc:approved PR has been reviewed and accepted for valid contribution points GSSoC 2026 level:beginner Small changes Usually isolated fixes or simple UI/text updates. mentor:Aamod007 quality:exceptional Outstanding contribution with exceptional implementation quality, testing.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

test(ActivityLandscape): create components/dashboard/ActivityLandscape.test.tsx

3 participants