Skip to content
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.

Sign up for GitHub

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

Jump to bottom

add widget tests #6

Closed
AyushBherwani1998 opened this issue Mar 23, 2020 · 1 comment · Fixed by #7
Closed

add widget tests #6

AyushBherwani1998 opened this issue Mar 23, 2020 · 1 comment · Fixed by #7
Labels
enhancement New feature or request
Projects
VelocityX Project

Comments

@AyushBherwani1998
Copy link
Contributor

Add widget tests to ensure that the widgets are rendered correctly. The tests can also be used in CI/CD.

I would like to work on this issue.

I need a suggestion about writing the test. Should I write a different widget test for the various parameter or should I write a single widget test and include all the parameters of the widget?

IMO, having a separate widget test for parameters can ensure that the widgets are correctly responding to the values.
@iampawan
Copy link
Owner

Yes, I would suggest writing separate tests.

@iampawan iampawan added the enhancement New feature or request label Mar 23, 2020
@AyushBherwani1998 AyushBherwani1998 mentioned this issue Mar 24, 2020
Merged
@iampawan iampawan reopened this Mar 24, 2020
@iampawan iampawan added this to To do in VelocityX Project via automation Mar 24, 2020
@iampawan iampawan moved this from To do to In progress in VelocityX Project Mar 24, 2020
VelocityX Project automation moved this from In progress to Done May 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
VelocityX Project
  
Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

adds widget test AyushBherwani1998/VelocityX
2 participants
@iampawan @AyushBherwani1998