Skip to content

🧪 feat : added tests for molecules components#27

Merged
dev-madhurendra merged 2 commits intomainfrom
FE_Molecule_TestCase
Aug 19, 2025
Merged

🧪 feat : added tests for molecules components#27
dev-madhurendra merged 2 commits intomainfrom
FE_Molecule_TestCase

Conversation

@somnath-more
Copy link
Copy Markdown
Collaborator

📦 Pull Request

📌 Description

Added tests for molecules ,organisms and template components

🧩 Type of Change

Please delete options that are not relevant.

  • 🐛 Bug fix
  • ✨ New feature
  • ♻️ Code refactor
  • 🧪 Test addition/changes
  • 📝 Documentation update
  • 🚀 Performance improvement
  • 🔧 Build/deployment changes
  • ⚠️ Breaking change

✅ Checklist

  • I have followed the project's coding standards
  • I have tested the changes locally
  • All dependent packages are updated (if needed)
  • I have added or updated relevant documentation/comments
  • I have added tests that prove my fix is effective or that my feature works
  • All new and existing tests passed
  • I have linked the related issue (if applicable)

🧪 How to Test

  1. Start backend server
  2. Run frontend with npm run dev
  3. Test /api/barber/token endpoint manually or using Postman
  4. [Optional] Check console logs or database

📸 Screenshots / Video

🔍 Review Notes

  • Focus on validation logic in BarberTokenController
  • Ensure role-based access is working correctly

🚨 Breaking Changes?

  • Yes (please describe below)
  • No

If yes, please describe:

📚 Additional Context

@dev-madhurendra dev-madhurendra self-requested a review August 17, 2025 16:28
Copy link
Copy Markdown
Owner

@dev-madhurendra dev-madhurendra left a comment

Choose a reason for hiding this comment

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

Pls fix the build.

@dev-madhurendra dev-madhurendra added the invalid This doesn't seem right label Aug 17, 2025
@dev-madhurendra dev-madhurendra changed the title feat: Added tests for molecules components 🧪 feat : added tests for molecules components Aug 17, 2025
@sonarqubecloud
Copy link
Copy Markdown

Copy link
Copy Markdown
Owner

@dev-madhurendra dev-madhurendra left a comment

Choose a reason for hiding this comment

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

Please now onward attach the screenshots as well.

@dev-madhurendra dev-madhurendra merged commit 156ce56 into main Aug 19, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

invalid This doesn't seem right

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants