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

Tests for each functionality #8

Open
xgleich1 opened this Issue May 19, 2018 · 8 comments

Comments

Projects
None yet
3 participants
@xgleich1

xgleich1 commented May 19, 2018

First of all a big thanks for your work on this lib, it looks really helpful!

Bit I am hesitant to use it since there are almost no tests in this project (the simple one in the sample project is not enough) and thus no way to verify that your code works as intendet.

This would also serve as a great way to showcase your complete api.

@Zhuinden

This comment has been minimized.

Show comment
Hide comment
@Zhuinden

Zhuinden May 19, 2018

Owner

Completely agree! Lot of the code is based on agoda/kakao, but tests would be nice nonetheless 👍

Owner

Zhuinden commented May 19, 2018

Completely agree! Lot of the code is based on agoda/kakao, but tests would be nice nonetheless 👍

@aumarbello

This comment has been minimized.

Show comment
Hide comment
@aumarbello

aumarbello Sep 12, 2018

Interested in taking this up, but could explain the nature of the tests that should be written?

aumarbello commented Sep 12, 2018

Interested in taking this up, but could explain the nature of the tests that should be written?

@Zhuinden

This comment has been minimized.

Show comment
Hide comment
@Zhuinden

Zhuinden Sep 12, 2018

Owner

@aumarbello technically there are a bunch of assertion types that start with check___, technically this block of code

For example, for checkContainsText, you'd need a text view that has a given text, and you would need to verify that if the TextView contains the right text then there is no assertion error, but if the TextView contains the wrong text then an assertion error is thrown.

This would be required for each check___ assertion type, but just adding any of such tests would be helpful.

Evidently, this would have to be done in instrumentation test.

Owner

Zhuinden commented Sep 12, 2018

@aumarbello technically there are a bunch of assertion types that start with check___, technically this block of code

For example, for checkContainsText, you'd need a text view that has a given text, and you would need to verify that if the TextView contains the right text then there is no assertion error, but if the TextView contains the wrong text then an assertion error is thrown.

This would be required for each check___ assertion type, but just adding any of such tests would be helpful.

Evidently, this would have to be done in instrumentation test.

@aumarbello

This comment has been minimized.

Show comment
Hide comment
@aumarbello

aumarbello Sep 12, 2018

Okay, so for example if I want to test all the assertions that have to do with texts I will need an activity with TextView and in the activity's instrumentation test after using the assertions, verify the behavior (happy and sad path) of the assertions I used right?

aumarbello commented Sep 12, 2018

Okay, so for example if I want to test all the assertions that have to do with texts I will need an activity with TextView and in the activity's instrumentation test after using the assertions, verify the behavior (happy and sad path) of the assertions I used right?

@Zhuinden

This comment has been minimized.

Show comment
Hide comment
@Zhuinden

Zhuinden Sep 12, 2018

Owner

Yep!

Owner

Zhuinden commented Sep 12, 2018

Yep!

@aumarbello

This comment has been minimized.

Show comment
Hide comment
@aumarbello

aumarbello Sep 12, 2018

Okay, would love to work on the full suites of tests but would it be okay to do so gradually?

aumarbello commented Sep 12, 2018

Okay, would love to work on the full suites of tests but would it be okay to do so gradually?

@Zhuinden

This comment has been minimized.

Show comment
Hide comment
@Zhuinden

Zhuinden Sep 12, 2018

Owner

I mean, it'd be crazy if I expected immediate 100% completion, I'm already glad someone volunteers for any of it in the first place.

Totally ok to do it gradually, of course. Happy to have help in general. 😄

Owner

Zhuinden commented Sep 12, 2018

I mean, it'd be crazy if I expected immediate 100% completion, I'm already glad someone volunteers for any of it in the first place.

Totally ok to do it gradually, of course. Happy to have help in general. 😄

@aumarbello

This comment has been minimized.

Show comment
Hide comment
@aumarbello

aumarbello Sep 12, 2018

Okay, would get started on it.

aumarbello commented Sep 12, 2018

Okay, would get started on it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment