Apps-android-commons: Delete Old UI Tests?

Created on 6 Dec 2020  路  6Comments  路  Source: commons-app/apps-android-commons

Summary:
Capture

assigned discussion tests
Source
picture madhurgupta10

Most helpful comment

I think it's OK to delete for now. We should probably make a separate issue for writing UI tests that actually work.

picture misaochan on 2 Feb 2021
👍2

All 6 comments

I support removing the tests if they don't even compile. We haven't been running these instrumentated tests in so long it'd be great to see them fixed or removed so we can get them running in Travis again.

domdomegg picture domdomegg on 6 Dec 2020

Agreed, we need to either fix or remove these.

misaochan picture misaochan on 7 Dec 2020

Can I fix this issue?
My Approach: Deleting the old UI tests from the file.
Any suggestions?

Roshan13046 picture Roshan13046 on 29 Jan 2021

I think it's OK to delete for now. We should probably make a separate issue for writing UI tests that actually work.

misaochan picture misaochan on 2 Feb 2021
👍2

@Roshan13046 Feel free to make a PR for this issue!

madhurgupta10 picture madhurgupta10 on 3 Feb 2021
👍1

@Roshan13046 Feel free to make a PR for this issue!

Please, suggest to me which UI tests are considered old UI tests?

Roshan13046 picture Roshan13046 on 3 Feb 2021
Was this page helpful?
0 / 5 - 0 ratings

Related issues

misaochan picture misaochan  路  3Comments
Saral-code picture Saral-code  路  3Comments
misaochan picture misaochan  路  4Comments
nicolas-raoul picture nicolas-raoul  路  3Comments
jidanni picture jidanni  路  3Comments