-
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 #3401: Introduced SplashScreenWelcomeMsg PlatformParameter #3441
Conversation
…e Time PlatformParameter
…ng Target annotation from Qualifiers
) | ||
} | ||
|
||
private fun processParameterLoadingStatus(loadingStatus: AsyncResult<Unit>): Boolean { |
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.
Maybe processPlatformParameters
since "loading status" seems like a UI thing.
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.
Done
} | ||
} | ||
|
||
/** Returns a [TypeSafeMatcher] which checks for any [Toast] message in the screen. */ |
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.
Write as // comment since we don't need a public KDoc.
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.
Done
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.
A few quick comments checking that this matches the original review, LGTM after those changes are made!
…or PlatformParameterIntegrationTest is made to run on Espresso only
app/src/sharedTest/java/org/oppia/android/app/testing/PlatformParameterIntegrationTest.kt
Show resolved
Hide resolved
Unassigning @anandwana001 since the review is done. |
Hi @Arjupta, it looks like some changes were requested on this pull request by @anandwana001. PTAL. 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.
Approving for code owner files.
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.
I tried few things like decor view and putting more delay, but it might take more time so we are going with the run only on espresso for this test testSplashTestActivity_updateEmptyDatabase_readDatabaseValues_checkWelcomeMsgIsVisible
.
@vinitamurthi assigning it back to you for merging |
Explanation
Fixes #3401
This PR introduces SplashScreenWelcomeMsg PlatformParameter for testing the implementation of Compile Time Platform Parameters. Also this PR is a replica of #3405
Screenshots
Checklist