Skip to content

Button tests with test kit #1785

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 8 commits into from
Jan 18, 2022
Merged

Button tests with test kit #1785

merged 8 commits into from
Jan 18, 2022

Conversation

Dan-Kovalsky
Copy link
Contributor

Description

Create the first version of test kit for some basic components using react-native/testing-library.
Use it to write tests for the Button component

Changelog

expose testkits for: Text, Image, Button

Copy link
Collaborator

@ethanshar ethanshar left a comment

Choose a reason for hiding this comment

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

Looks great!
I wrote 2 small comments..
Anyway, Im still not sure why there's that one test that fail.. maybe we can skip it for now and get back to it later on.

@@ -0,0 +1,142 @@
import React, {useState} from 'react';
Copy link
Collaborator

Choose a reason for hiding this comment

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

This is the spec that checks the driver? maybe it's better to name the file index.driver.spec.js?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I don't think it right to say that it checks the driver.
more like, it checks the button using the driver.
I still not sure what is good name for this, so I will change like you say

@ethanshar ethanshar merged commit 78a3ea1 into master Jan 18, 2022
@lidord-wix
Copy link
Contributor

@Dan-Kovalsky
A recommendation for next time - run npm run prepush before pushing to uilib, it'll generate typing files (which were missing this time) and run the test :)

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.

3 participants