-
Notifications
You must be signed in to change notification settings - Fork 499
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
Fix #3419: Separate ViewEventLogsActivityTest into activity and fragment test files #3468
Fix #3419: Separate ViewEventLogsActivityTest into activity and fragment test files #3468
Conversation
Hi! @yashraj-01 Welcome to Oppia! Could you please follow the instructions here and sign the CLA Sheet to get started? You'll need to do this before we can accept your PR. Thanks! |
@rt4914 PTAL. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ditto for this PR--same comments as #3471 apply here. PTAL.
I have updated this PR as per comments on #3471. PTAL. Thanks. |
Deferring to @Sarthak2601 for the initial main reviews per #3471. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks.
Unassigning @BenHenning since they have already approved the PR. |
@yashraj-01 it looks like the static checks are failing & the exemption list may need to be updated. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Requesting changes due to failing CI.
@BenHenning are we having activity labels for test activities as well? |
Hi, @yashraj-01, the LGTM Label has been removed because the changes were requested on this PR. Thanks!. |
Added test activities to exemption list |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @yashraj-01! LGTM, just one comment--PTAL.
app/src/sharedTest/java/org/oppia/android/app/devoptions/ViewEventLogsFragmentTest.kt
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @yashraj-01. LGTM!
Explanation
Fixes #3419. Separated ViewEventLogsActivityTest into activity and fragment test files.
Screenshots
Expresso test results for
ViewEventLogsActivityTest
Expresso test results for
ViewEventLogsFragmentTest
Checklist