Fix oppia#1587 : Resolve test cases error. Made intent keys consistent.

The-Pascal · Aug 27, 2020 · e488310 · e488310
1 parent ad6738d
commit e488310
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/app/src/main/java/org/oppia/app/administratorcontrols/AdministratorControlsActivity.kt b/app/src/main/java/org/oppia/app/administratorcontrols/AdministratorControlsActivity.kt
@@ -12,7 +12,8 @@ import org.oppia.app.settings.profile.ProfileListActivity
 import javax.inject.Inject
 
 const val SELECTED_CONTROLS_TITLE_KEY = "SELECTED_CONTROLS_TITLE_KEY"
-const val LAST_LOADED_FRAGMENT_KEY = "LAST_LOADED_FRAGMENT_KEY"
+const val LAST_LOADED_FRAGMENT_SAVED_KEY =
+ "AdministratorControlsActivity.last_loaded_fragment"
 const val PROFILE_LIST_FRAGMENT = "PROFILE_LIST_FRAGMENT"
 const val APP_VERSION_FRAGMENT = "APP_VERSION_FRAGMENT"
 
@@ -32,7 +33,7 @@ class AdministratorControlsActivity :
  activityComponent.inject(this)
  val extraControlsTitle = savedInstanceState?.getString(SELECTED_CONTROLS_TITLE_KEY)
  lastLoadedFragment = if (savedInstanceState != null) {
- savedInstanceState.get(LAST_LOADED_FRAGMENT_KEY) as String
+ savedInstanceState.get(LAST_LOADED_FRAGMENT_SAVED_KEY) as String
  } else {
  // TODO(#661): Change the default fragment in the right hand side to be EditAccount fragment in the case of multipane controls.
  PROFILE_LIST_FRAGMENT

diff --git a/...c/main/java/org/oppia/app/administratorcontrols/AdministratorControlsActivityPresenter.kt b/...c/main/java/org/oppia/app/administratorcontrols/AdministratorControlsActivityPresenter.kt
@@ -97,6 +97,6 @@ class AdministratorControlsActivityPresenter @Inject constructor(
  if (titleTextView != null) {
  outState.putString(SELECTED_CONTROLS_TITLE_KEY, titleTextView.text.toString())
  }
- outState.putString(LAST_LOADED_FRAGMENT_KEY, lastLoadedFragment)
+ outState.putString(LAST_LOADED_FRAGMENT_SAVED_KEY, lastLoadedFragment)
  }
 }