-
Notifications
You must be signed in to change notification settings - Fork 1
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
explore ui test #55
explore ui test #55
Conversation
Quality Gate passedIssues Measures |
tint = Color.Unspecified | ||
) | ||
|
||
Column( | ||
modifier = Modifier | ||
.clearAndSetSemantics { }, | ||
modifier = Modifier, |
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.
μ΄ Modifierλ νμνκ°μ?
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.
accessibility ν₯μμ μν΄ contentDecriptionμ λ¬Άμ΄μ£Όλ €κ³ semanticsλ₯Ό μ΄κΈ°ν νμλλ°μ
κ·Έλ κ² λλκΉ ν
μ€νΈ μ½λμμ onNodeWithText μ°Ύμ§λ₯Ό λͺ»νλλΌκ΅¬μ, λ€λ₯Έ ν΄κ²°λ°©λ²μ μ°Ύμ§ λͺ»ν΄ μ§μ λλ° Modifierλ§μλ κ²λ μ§μ°κ² μ΅λλ€
verticalArrangement = Arrangement.spacedBy(4.dp) | ||
) { | ||
Text( | ||
modifier = Modifier | ||
.clearAndSetSemantics { }, | ||
modifier = Modifier, |
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
text = name, | ||
style = Theme.typography.body2sb, | ||
color = Theme.colors.text | ||
) | ||
Text( | ||
modifier = Modifier | ||
.clearAndSetSemantics { }, | ||
modifier = Modifier, |
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
@get: Rule | ||
val composeTestRule = createComposeRule() | ||
private val fakeUiState: MutableState<ExploreUiState> = | ||
mutableStateOf(ExploreUiState.INIT) |
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.
μ€λ°κΏν μ΄μ κ° μμκΉμ?
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.
λ무 κΈΈμ΄ λ³΄μμ΅λλ€
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.
λ΅, μ€λ°κΏ λ±μ μΈμ΄λ§λ€ μ 곡λλ κΈ°λ³Έ 컨벀μ μ΄λ IDEμμ μ 곡νλ μ½λ μ λ ¬ λꡬλ₯Ό μ¬μ©νλ©΄ μΌκ΄μ μΈ μ€νμΌμ μ μ§ν μ μμ΅λλ€. λ°κΎΈμ€ νμλ μκ³ μ°Έκ³ λ§ ν΄μ£ΌμΈμ.
mutableStateOf(ExploreUiState.INIT) | ||
|
||
private val fakeHandleIntent: MutableState<(ExploreUiIntent) -> Unit> = | ||
mutableStateOf({}) |
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
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,
μ½λ©νΈ νμΈνμκ³ λ¨Έμ§ν΄μ£ΌμΈμ.
πμμ λ΄μ©