Skip to content

Commit

Permalink
Fix #2631: A11y Options Language screen (#2784)
Browse files Browse the repository at this point in the history
* added labels for options language screen

* a11y update

* Added Tests

* fixed spacing issues

* fix

* unused imports deleted

* Added tests for labels

* Fixed formatting issues

* Tests updated

* Removed unused imports

* merged develop

* Update app/src/sharedTest/java/org/oppia/android/app/options/AudioLanguageActivityTest.kt

Co-authored-by: Ben Henning <henning.benmax@gmail.com>

Co-authored-by: Ben Henning <henning.benmax@gmail.com>
  • Loading branch information
abhinav12k and BenHenning authored Mar 25, 2021
1 parent d802f57 commit 96ad91f
Show file tree
Hide file tree
Showing 10 changed files with 323 additions and 8 deletions.
6 changes: 4 additions & 2 deletions app/src/main/AndroidManifest.xml
Original file line number Diff line number Diff line change
Expand Up @@ -56,10 +56,12 @@
android:theme="@style/OppiaThemeWithoutActionBar" />
<activity
android:name=".app.options.AppLanguageActivity"
android:theme="@style/OppiaThemeWithoutActionBar" />
android:theme="@style/OppiaThemeWithoutActionBar"
android:label="@string/app_language_activity_title" />
<activity
android:name=".app.options.AudioLanguageActivity"
android:theme="@style/OppiaThemeWithoutActionBar" />
android:theme="@style/OppiaThemeWithoutActionBar"
android:label="@string/audio_language_activity_title" />
<activity
android:name=".app.options.OptionsActivity"
android:theme="@style/OppiaThemeWithoutActionBar" />
Expand Down
2 changes: 1 addition & 1 deletion app/src/main/res/layout-land/app_language_fragment.xml
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@
android:minHeight="?attr/actionBarSize"
app:navigationContentDescription="@string/navigate_up"
app:navigationIcon="?attr/homeAsUpIndicator"
app:title="@string/title_app_language"
app:title="@string/app_language_activity_title"
app:titleTextAppearance="@style/ToolbarTextAppearance"
app:titleTextColor="@color/white" />
</com.google.android.material.appbar.AppBarLayout>
Expand Down
2 changes: 1 addition & 1 deletion app/src/main/res/layout-land/audio_language_fragment.xml
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@
android:minHeight="?attr/actionBarSize"
app:navigationContentDescription="@string/navigate_up"
app:navigationIcon="?attr/homeAsUpIndicator"
app:title="@string/title_default_audio"
app:title="@string/audio_language_activity_title"
app:titleTextAppearance="@style/ToolbarTextAppearance"
app:titleTextColor="@color/white" />
</com.google.android.material.appbar.AppBarLayout>
Expand Down
3 changes: 3 additions & 0 deletions app/src/main/res/layout-land/language_items.xml
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@
android:gravity="center_vertical"
android:minHeight="48dp"
android:onClick="@{() -> viewModel.languageRadioButtonListener.onLanguageSelected(viewModel.language)}"
android:importantForAccessibility="yes"
android:orientation="horizontal">

<androidx.appcompat.widget.AppCompatRadioButton
Expand All @@ -26,6 +27,8 @@
android:layout_marginEnd="24dp"
android:checked="@{viewModel.isLanguageSelected}"
android:clickable="false"
android:focusable="false"
android:labelFor="@id/language_text_view"
android:minWidth="20dp"
android:minHeight="20dp" />

Expand Down
2 changes: 1 addition & 1 deletion app/src/main/res/layout/app_language_fragment.xml
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@
android:minHeight="?attr/actionBarSize"
app:navigationContentDescription="@string/navigate_up"
app:navigationIcon="?attr/homeAsUpIndicator"
app:title="@string/title_app_language"
app:title="@string/app_language_activity_title"
app:titleTextAppearance="@style/ToolbarTextAppearance"
app:titleTextColor="@color/white" />
</com.google.android.material.appbar.AppBarLayout>
Expand Down
2 changes: 1 addition & 1 deletion app/src/main/res/layout/audio_language_fragment.xml
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@
android:minHeight="?attr/actionBarSize"
app:navigationContentDescription="@string/navigate_up"
app:navigationIcon="?attr/homeAsUpIndicator"
app:title="@string/title_default_audio"
app:title="@string/audio_language_activity_title"
app:titleTextAppearance="@style/ToolbarTextAppearance"
app:titleTextColor="@color/white" />
</com.google.android.material.appbar.AppBarLayout>
Expand Down
3 changes: 3 additions & 0 deletions app/src/main/res/layout/language_items.xml
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@
android:gravity="center_vertical"
android:minHeight="48dp"
android:onClick="@{() -> viewModel.languageRadioButtonListener.onLanguageSelected(viewModel.language)}"
android:importantForAccessibility="yes"
android:orientation="horizontal">

<androidx.appcompat.widget.AppCompatRadioButton
Expand All @@ -26,6 +27,8 @@
android:layout_marginEnd="24dp"
android:checked="@{viewModel.isLanguageSelected}"
android:clickable="false"
android:focusable="false"
android:labelFor="@id/language_text_view"
android:minWidth="20dp"
android:minHeight="20dp" />

Expand Down
4 changes: 2 additions & 2 deletions app/src/main/res/values/strings.xml
Original file line number Diff line number Diff line change
Expand Up @@ -337,9 +337,9 @@
<string name="app_version_name">App Version %s</string>
<string name="app_last_update_date">The last update was installed on %s. Use the above version number to send feedback about bugs.</string>
<!-- OptionsActivity -->
<string name="app_language_activity_title">App Language</string>
<string name="audio_language_activity_title">Default Audio Language</string>
<string name="title_reading_text_size">Reading Text Size</string>
<string name="title_app_language">App Language</string>
<string name="title_default_audio">Default Audio Language</string>
<string name="story_text_will_look_like_this">Story text will look like this.</string>
<string name="constant_string_for_text_size">A</string>
<string name="audio_language">Default Audio</string>
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,154 @@
package org.oppia.android.app.options

import android.app.Application
import android.content.Context
import android.content.Intent
import androidx.appcompat.app.AppCompatActivity
import androidx.test.core.app.ApplicationProvider
import androidx.test.ext.junit.runners.AndroidJUnit4
import androidx.test.rule.ActivityTestRule
import com.google.common.truth.Truth.assertThat
import dagger.Component
import org.junit.Before
import org.junit.Rule
import org.junit.Test
import org.junit.runner.RunWith
import org.oppia.android.R
import org.oppia.android.app.activity.ActivityComponent
import org.oppia.android.app.application.ActivityComponentFactory
import org.oppia.android.app.application.ApplicationComponent
import org.oppia.android.app.application.ApplicationInjector
import org.oppia.android.app.application.ApplicationInjectorProvider
import org.oppia.android.app.application.ApplicationModule
import org.oppia.android.app.application.ApplicationStartupListenerModule
import org.oppia.android.app.player.state.hintsandsolution.HintsAndSolutionConfigModule
import org.oppia.android.app.shim.ViewBindingShimModule
import org.oppia.android.domain.classify.InteractionsModule
import org.oppia.android.domain.classify.rules.continueinteraction.ContinueModule
import org.oppia.android.domain.classify.rules.dragAndDropSortInput.DragDropSortInputModule
import org.oppia.android.domain.classify.rules.fractioninput.FractionInputModule
import org.oppia.android.domain.classify.rules.imageClickInput.ImageClickInputModule
import org.oppia.android.domain.classify.rules.itemselectioninput.ItemSelectionInputModule
import org.oppia.android.domain.classify.rules.multiplechoiceinput.MultipleChoiceInputModule
import org.oppia.android.domain.classify.rules.numberwithunits.NumberWithUnitsRuleModule
import org.oppia.android.domain.classify.rules.numericinput.NumericInputRuleModule
import org.oppia.android.domain.classify.rules.ratioinput.RatioInputModule
import org.oppia.android.domain.classify.rules.textinput.TextInputRuleModule
import org.oppia.android.domain.onboarding.ExpirationMetaDataRetrieverModule
import org.oppia.android.domain.oppialogger.LogStorageModule
import org.oppia.android.domain.oppialogger.loguploader.LogUploadWorkerModule
import org.oppia.android.domain.oppialogger.loguploader.WorkManagerConfigurationModule
import org.oppia.android.domain.question.QuestionModule
import org.oppia.android.domain.topic.PrimeTopicAssetsControllerModule
import org.oppia.android.testing.RobolectricModule
import org.oppia.android.testing.TestDispatcherModule
import org.oppia.android.testing.TestLogReportingModule
import org.oppia.android.testing.time.FakeOppiaClockModule
import org.oppia.android.util.accessibility.AccessibilityTestModule
import org.oppia.android.util.caching.testing.CachingTestModule
import org.oppia.android.util.gcsresource.GcsResourceModule
import org.oppia.android.util.logging.LoggerModule
import org.oppia.android.util.logging.firebase.FirebaseLogUploaderModule
import org.oppia.android.util.parser.GlideImageLoaderModule
import org.oppia.android.util.parser.HtmlParserEntityTypeModule
import org.oppia.android.util.parser.ImageParsingModule
import org.robolectric.annotation.Config
import org.robolectric.annotation.LooperMode
import javax.inject.Inject
import javax.inject.Singleton

/** Tests for [AppLanguageActivity]. */
@RunWith(AndroidJUnit4::class)
@LooperMode(LooperMode.Mode.PAUSED)
@Config(
application = AppLanguageActivityTest.TestApplication::class,
qualifiers = "port-xxhdpi"
)
class AppLanguageActivityTest {

@get:Rule
val activityTestRule: ActivityTestRule<AppLanguageActivity> = ActivityTestRule(
AppLanguageActivity::class.java, /* initialTouchMode= */ true, /* launchActivity= */ false
)

@Inject
lateinit var context: Context

private val summaryValue = "English"

@Before
fun setUp() {
setUpTestApplicationComponent()
}

private fun setUpTestApplicationComponent() {
ApplicationProvider.getApplicationContext<TestApplication>().inject(this)
}

@Test
fun testAppLanguageActivity_hasCorrectActivityLabel() {
activityTestRule.launchActivity(
createAppLanguageActivityIntent(
summaryValue
)
)
val title = activityTestRule.activity.title
// Verify that the activity label is correct as a proxy to verify TalkBack will announce the
// correct string when it's read out.
assertThat(title).isEqualTo(context.getString(R.string.app_language_activity_title))
}

private fun createAppLanguageActivityIntent(summaryValue: String): Intent {
return AppLanguageActivity.createAppLanguageActivityIntent(
ApplicationProvider.getApplicationContext(),
APP_LANGUAGE,
summaryValue
)
}

// TODO(#59): Figure out a way to reuse modules instead of needing to re-declare them.
// TODO(#1675): Add NetworkModule once data module is migrated off of Moshi.
@Singleton
@Component(
modules = [
RobolectricModule::class,
TestDispatcherModule::class, ApplicationModule::class,
LoggerModule::class, ContinueModule::class, FractionInputModule::class,
ItemSelectionInputModule::class, MultipleChoiceInputModule::class,
NumberWithUnitsRuleModule::class, NumericInputRuleModule::class, TextInputRuleModule::class,
DragDropSortInputModule::class, ImageClickInputModule::class, InteractionsModule::class,
GcsResourceModule::class, GlideImageLoaderModule::class, ImageParsingModule::class,
HtmlParserEntityTypeModule::class, QuestionModule::class, TestLogReportingModule::class,
AccessibilityTestModule::class, LogStorageModule::class, CachingTestModule::class,
PrimeTopicAssetsControllerModule::class, ExpirationMetaDataRetrieverModule::class,
ViewBindingShimModule::class, RatioInputModule::class,
ApplicationStartupListenerModule::class, LogUploadWorkerModule::class,
WorkManagerConfigurationModule::class, HintsAndSolutionConfigModule::class,
FirebaseLogUploaderModule::class, FakeOppiaClockModule::class
]
)
interface TestApplicationComponent : ApplicationComponent {
@Component.Builder
interface Builder : ApplicationComponent.Builder

fun inject(appLanguageActivityTest: AppLanguageActivityTest)
}

class TestApplication : Application(), ActivityComponentFactory, ApplicationInjectorProvider {
private val component: TestApplicationComponent by lazy {
DaggerAppLanguageActivityTest_TestApplicationComponent.builder()
.setApplication(this)
.build() as TestApplicationComponent
}

fun inject(appLanguageActivityTest: AppLanguageActivityTest) {
component.inject(appLanguageActivityTest)
}

override fun createActivityComponent(activity: AppCompatActivity): ActivityComponent {
return component.getActivityComponentBuilderProvider().get().setActivity(activity).build()
}

override fun getApplicationInjector(): ApplicationInjector = component
}
}
Loading

0 comments on commit 96ad91f

Please sign in to comment.