Skip to content

test: update HeroBanner tests for current UI#117

Open
Harshi06-code wants to merge 1 commit into
asheesh109:mainfrom
Harshi06-code:fix/herobanner-tests
Open

test: update HeroBanner tests for current UI#117
Harshi06-code wants to merge 1 commit into
asheesh109:mainfrom
Harshi06-code:fix/herobanner-tests

Conversation

@Harshi06-code

@Harshi06-code Harshi06-code commented Jun 22, 2026

Copy link
Copy Markdown

Summary

  • Brief description of changes

Related issue

  • Fixes: #ISSUE

Implementation notes

  • What changed and why

Checklist

  • Tests added/updated
  • Documentation updated
  • CHANGELOG updated (if applicable)

Screenshots (if applicable)

Summary

Provide a short description of the change and the motivation.

Related issues

  • Closes #

Type of change

  • Bug fix
  • New feature
  • Documentation update
  • Refactor

Checklist

  • Tests added/updated
  • Lint and type checks pass
  • Docs updated (if applicable)
  • PR linked to an issue (if applicable)

How to test

Provide steps to reproduce or verify the changes locally.

Screenshots (if applicable)


Mention reviewers and any special notes for maintainers.

📝 Description

Brief description of what this PR does.

🔗 Related Issues

Closes #(issue number)

Related to: #(issue number)

🎯 Type of Change

  • 🐛 Bug fix (non-breaking)
  • ✨ New feature (non-breaking)
  • 🔄 Breaking change
  • 📖 Documentation update
  • 🔧 Configuration change
  • ♻️ Refactoring

✅ Testing

What testing was done?

  • Unit tests added/updated
  • Integration tests added/updated
  • Manual testing completed
  • Cross-browser tested (if UI change)
  • Mobile tested (if UI change)

Test Coverage

  • New code has tests
  • Existing tests pass
  • Coverage maintained or improved

How to Test

# Commands to test this PR
npm run test
npm run dev

📸 Screenshots (if applicable)

Before

After

🔍 Code Review Checklist

  • Code follows project style
  • No console logs (except errors)
  • No commented-out code
  • TypeScript types are complete
  • Error handling is proper
  • Security considerations reviewed
  • Performance implications considered

📝 Documentation

  • Updated relevant documentation
  • Updated README if needed
  • Updated API docs if applicable
  • Added code comments for complex logic
  • Updated CHANGELOG.md

🚀 Deployment

  • This is production-ready
  • No database migrations needed
  • No environment variables added
  • No breaking changes

📋 Additional Notes

Any additional context or information that reviewers should know?


Reviewers Please Note

Important points:

  • This PR fixes [specific issue]
  • Changes affect [specific areas]
  • Special attention needed for [specific code]

🎫 PR Checklist

  • All CI checks pass
  • No conflicts with base branch
  • Branch is up-to-date with main
  • Commit messages are clear
  • No unrelated changes

Thank you for contributing to KisanAI! 🌾

@vercel

vercel Bot commented Jun 22, 2026

Copy link
Copy Markdown

@Harshi06-code is attempting to deploy a commit to the ashishparab03-gmailcom's projects Team on Vercel.

A member of the Team first needs to authorize it.

@Harshi06-code Harshi06-code left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Updated HeroBanner tests to match the current UI implementation.

Changes:

  • Added LanguageProvider wrapper for tests

  • Removed outdated assertions

  • Added tests for current HeroBanner content:

    • Empowering Farmers heading
    • Hero tagline
    • Explore Schemes button
    • Voice Assistant button

Verified all HeroBanner tests pass locally.

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.

1 participant