Skip to content

Unit tests for playground-web repository #71

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 5 commits into from
Oct 16, 2024

Conversation

ViragJain3010
Copy link
Contributor

Created tests for playground-web/components as discussed in #893

  1. Tests cover wide areas
  2. Some tests are dynamic (Search/Search.test.tsx) so as the change in DiceCmds don't hinder the functionality of tests.
  3. Mocking of clipboard.navigator directly is not allowed (details here) so had to create setUpClipboard() and tearDownClipboard()
  4. Changed the file-name of command.tsx -> CommandPage.tsx for improved clarity and consistency in the codebase.
  5. Added data-testid attributes to button.tsx (packages/ui/src/button.tsx) to ensure proper rendering and accessibility for testing purposes.

@lucifercr07 lucifercr07 requested a review from KaviiSuri October 16, 2024 17:04
Copy link
Contributor

@KaviiSuri KaviiSuri left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks for the contribution

@KaviiSuri KaviiSuri merged commit a5bf2d9 into DiceDB:master Oct 16, 2024
1 check passed
KaviiSuri added a commit to KaviiSuri/playground-web that referenced this pull request Nov 2, 2024
…setup-ui-packages-and-tailwind-for-console

* 'master' of github.com:DiceDB/playground-web:
  Fix docker compose file (DiceDB#77)
  integrate cleanup time from playground-mono (DiceDB#76)
  Changes to integrate rate-limit headers for `Commands left` component UI (DiceDB#67)
  Adding support for JSON.SET command (DiceDB#74)
  Unit tests for playground-web repository  (DiceDB#71)
  fix: docker compose contexts to ensure docker compose runs
KaviiSuri added a commit to KaviiSuri/playground-web that referenced this pull request Nov 2, 2024
…setup-ui-packages-and-tailwind-for-console

* 'master' of github.com:DiceDB/playground-web:
  Fix docker compose file (DiceDB#77)
  integrate cleanup time from playground-mono (DiceDB#76)
  Changes to integrate rate-limit headers for `Commands left` component UI (DiceDB#67)
  Adding support for JSON.SET command (DiceDB#74)
  Unit tests for playground-web repository  (DiceDB#71)
  fix: docker compose contexts to ensure docker compose runs
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.

2 participants