Fix #153/#154 : MultipleChoice/ItemSelection Interaction View

app/build.gradle

@@ -77,6 +77,7 @@ dependencies {
       'androidx.test.ext:junit:1.1.1',
       'com.google.truth:truth:0.43',
       'org.robolectric:robolectric:4.3',
+      'androidx.test.espresso:espresso-contrib:3.1.0',
   )
   androidTestImplementation(
       'androidx.test:core:1.2.0',
@@ -85,6 +86,8 @@ dependencies {
       'androidx.test.ext:junit:1.1.1',
       'androidx.test:runner:1.2.0',
       'com.google.truth:truth:0.43',
+      'androidx.test.espresso:espresso-intents:3.1.0',
+      'androidx.test.espresso:espresso-contrib:3.1.0',
   )
   androidTestUtil(
       'androidx.test:orchestrator:1.2.0',

app/src/main/java/org/oppia/app/player/state/InteractionAdapter.kt

@@ -0,0 +1,131 @@
+package org.oppia.app.player.state;
+
+import android.content.Context
+import android.text.Html
+import android.text.Spannable
+import android.view.LayoutInflater
+import android.view.ViewGroup
+import android.widget.TextView
+import android.widget.Toast
+import androidx.databinding.DataBindingUtil
+import androidx.databinding.ViewDataBinding
+import androidx.databinding.library.baseAdapters.BR
+import androidx.recyclerview.widget.RecyclerView
+import kotlinx.android.synthetic.main.item_selection_interaction_items.view.*
+import kotlinx.android.synthetic.main.multiple_choice_interaction_items.view.*
+import org.oppia.app.R
+import org.oppia.app.databinding.ItemSelectionInteractionItemsBinding
+import org.oppia.app.databinding.MultipleChoiceInteractionItemsBinding
+import org.oppia.util.data.HtmlParser
+import org.oppia.util.data.UrlImageParser
+
+const val VIEW_TYPE_MULTIPLE_CHOICE = 1
+const val VIEW_TYPE_ITEM_SELECTION = 2
+
+/** Adapter to bind the interactions to the [RecyclerView]. It handles MultipleChoiceInput and ItemSelectionInput interaction views. */
+class InteractionAdapter(
+  private val context: Context,
+  private val entity_type: String,
+  private val entity_id: String,
+  val itemList: Array<String>?,
+  val interactionInstanceId: String?
+) : RecyclerView.Adapter<RecyclerView.ViewHolder>() {
+
+  private var mSelectedItem = -1
+
+  override fun onCreateViewHolder(parent: ViewGroup, viewType: Int): RecyclerView.ViewHolder {
+    return when (viewType) {
+      VIEW_TYPE_MULTIPLE_CHOICE -> {
+        val inflater = LayoutInflater.from(parent.getContext())
+        val binding =
+          DataBindingUtil.inflate<MultipleChoiceInteractionItemsBinding>(
+            inflater,
+            R.layout.multiple_choice_interaction_items,
+            parent,
+            false
+          )
+        MultipleChoiceViewHolder(binding)
+      }
+      VIEW_TYPE_ITEM_SELECTION -> {
+        val inflater = LayoutInflater.from(parent.getContext())
+        val binding =
+          DataBindingUtil.inflate<ItemSelectionInteractionItemsBinding>(
+            inflater,
+            R.layout.item_selection_interaction_items,
+            parent,
+            false
+          )
+        ItemSelectionViewHolder(binding)
+      }
+      else -> throw IllegalArgumentException("Invalid view type")
+    }
+  }
+
+  override fun onBindViewHolder(holder: RecyclerView.ViewHolder, position: Int) {
+    when (holder.itemViewType) {
+      VIEW_TYPE_MULTIPLE_CHOICE -> (holder as MultipleChoiceViewHolder).bind(
+        itemList!!.get(position),
+        position,
+        mSelectedItem
+      )
+      VIEW_TYPE_ITEM_SELECTION -> (holder as ItemSelectionViewHolder).bind(
+        itemList!!.get(position),
+        position,
+        mSelectedItem
+      )
+    }
+  }
+
+  // Determines the appropriate ViewType according to the interaction type.
+  override fun getItemViewType(position: Int): Int {
+    return if (interactionInstanceId.equals("ItemSelectionInput")) {
+      VIEW_TYPE_ITEM_SELECTION
+    } else {
+      VIEW_TYPE_MULTIPLE_CHOICE
+    }
+  }
+
+  override fun getItemCount(): Int {
+    return itemList!!.size
+  }
+
+  private inner class ItemSelectionViewHolder(val binding: ViewDataBinding) : RecyclerView.ViewHolder(binding.root) {
+   internal fun bind(rawString: String?, position: Int, selectedPosition: Int) {
+      binding.setVariable(BR.htmlContent, rawString)
+      binding.executePendingBindings();
+      val html: Spannable = HtmlParser(context,entity_type,entity_id).parseHtml(rawString, binding.root.tv_item_selection_contents)
+      binding.root.tv_item_selection_contents.text = html
+
+      binding.root.rl_checkbox_container.setOnClickListener {
+        if (binding.root.cb_item_selection.isChecked)
+          binding.root.cb_item_selection.setChecked(false)
+        else
+          binding.root.cb_item_selection.setChecked(true)
+        Toast.makeText(context, "" + binding.root.tv_item_selection_contents.text, Toast.LENGTH_LONG).show()
+        notifyDataSetChanged()
+      }
+    }
+  }
+
+  private inner class MultipleChoiceViewHolder(val binding: ViewDataBinding) : RecyclerView.ViewHolder(binding.root) {
+   internal fun bind(rawString: String?, position: Int, selectedPosition: Int) {
+      binding.setVariable(BR.htmlContent, rawString)
+      binding.executePendingBindings();
+      val html: Spannable =  HtmlParser(context,entity_type,entity_id).parseHtml(rawString, binding.root.tv_multiple_choice_contents)
+      binding.root.tv_multiple_choice_contents.text = html
+
+      if (selectedPosition == position)
+        binding.root.rb_multiple_choice.setChecked(true)
+      else
+        binding.root.rb_multiple_choice.setChecked(false)
+
+      binding.root.rl_radio_container.setOnClickListener {
+        Toast.makeText(context, "" + binding.root.tv_multiple_choice_contents.text, Toast.LENGTH_LONG).show()
+        mSelectedItem = getAdapterPosition()
+        notifyDataSetChanged()
+      }
+    }
+  }
+
+
+}

app/src/main/java/org/oppia/app/player/state/StateFragmentPresenter.kt

@@ -1,5 +1,6 @@
 package org.oppia.app.player.state
 
+import android.content.Context
 import android.view.LayoutInflater
 import android.view.View
 import android.view.ViewGroup
@@ -11,6 +12,7 @@ import org.oppia.app.databinding.StateFragmentBinding
 import org.oppia.app.fragment.FragmentScope
 import org.oppia.app.model.CellularDataPreference
 import org.oppia.app.model.EphemeralState
+import org.oppia.app.model.InteractionObject
 import org.oppia.app.player.audio.CellularDataDialogFragment
 import org.oppia.app.viewmodel.ViewModelProvider
 import org.oppia.domain.audio.CellularDialogController
@@ -24,6 +26,7 @@ private const val TAG_CELLULAR_DATA_DIALOG = "CELLULAR_DATA_DIALOG"
 /** The presenter for [StateFragment]. */
 @FragmentScope
 class StateFragmentPresenter @Inject constructor(
+  private val context: Context,
   private val fragment: Fragment,
   private val cellularDialogController: CellularDialogController,
   private val viewModelProvider: ViewModelProvider<StateViewModel>,
@@ -34,6 +37,14 @@ class StateFragmentPresenter @Inject constructor(
   private var showCellularDataDialog = true
   private var useCellularData = false
 
+  private var items: Array<String>? = null
+  var customizationArgsMap = HashMap<String, InteractionObject>()
+  var interactionInstanceId: String? = null
+  private var entity_type: String = ""
+  private var entity_id: String = ""
+
+  var interactionAdapter: InteractionAdapter? = null
+
   fun handleCreateView(inflater: LayoutInflater, container: ViewGroup?): View? {
     cellularDialogController.getCellularDataPreference()
       .observe(fragment, Observer<AsyncResult<CellularDataPreference>> {
@@ -95,6 +106,20 @@ class StateFragmentPresenter @Inject constructor(
   private fun subscribeToCurrentState() {
     ephemeralStateLiveData.observe(fragment, Observer<EphemeralState> { result ->
       logger.d("StateFragment", "getCurrentState: ${result.state.name}")
+      entity_type = "exploration"
+      // TODO replace exploration Id from result. Exploration id is missing in proto
+      entity_id = "DIWZiVgs0km-"
+      val customizationArgsMap: Map<String, InteractionObject> = result.state.interaction.customizationArgsMap
+      logger.d("TAG", "getCurrentState: " + result.state)
+      val allKeys: Set<String> = customizationArgsMap.keys
+
+      for (key in allKeys) {
+        logger.d("StateFragment", key)
+      }
+      if (customizationArgsMap.contains("choices")) {
+        val customizationArgs: InteractionObject? = customizationArgsMap["choices"]
+        logger.d("StateFragment", "value: ${customizationArgs}")
+      }
     })
   }
 
@@ -112,4 +137,22 @@ class StateFragmentPresenter @Inject constructor(
     }
     return ephemeralStateResult.getOrDefault(EphemeralState.getDefaultInstance())
   }
+
+  private fun showInputInteractions(binding: StateFragmentBinding) {
+    val gaeCustomizationArgs: Any? = customizationArgsMap.values
+    if (interactionInstanceId.equals("MultipleChoiceInput")) {
+      val gaeCustomArgsInString: String = gaeCustomizationArgs.toString().replace("[", "").replace("]", "")
+      items = gaeCustomArgsInString.split(",").toTypedArray()
+      interactionAdapter = InteractionAdapter(context, entity_type, entity_id, items, interactionInstanceId);
+      binding.rvInteractions.adapter = interactionAdapter
+
+    } else if (interactionInstanceId.equals("ItemSelectionInput") || interactionInstanceId.equals("SingleChoiceInput")) {
+      val gaeCustomArgsInString: String = gaeCustomizationArgs.toString().replace("[", "").replace("]", "")
+      items = gaeCustomArgsInString.split(",").toTypedArray()
+      interactionAdapter = InteractionAdapter(context, entity_type, entity_id, items, interactionInstanceId);
+      binding.rvInteractions.adapter = interactionAdapter
+    } else {
+      //Do no show any view
+    }
+  }
 }

app/src/main/res/layout/item_selection_interaction_items.xml

@@ -0,0 +1,28 @@
+<layout xmlns:android="http://schemas.android.com/apk/res/android" xmlns:app="http://schemas.android.com/apk/res-auto">
+  <data>
+    <variable
+      name="htmlContent"
+      type="String"/>
+  </data>
+  <RelativeLayout
+    android:id="@+id/rl_checkbox_container"
+    android:layout_width="match_parent"
+    android:layout_height="wrap_content"
+    android:layout_margin="4dp"
+    android:orientation="vertical">
+    <androidx.appcompat.widget.AppCompatCheckBox
+      android:id="@+id/cb_item_selection"
+      android:layout_width="wrap_content"
+      android:layout_height="wrap_content"
+      android:padding="4dp"
+      app:buttonTint="@color/oppiaDarkBlue"/>
+    <TextView
+      android:id="@+id/tv_item_selection_contents"
+      android:layout_width="match_parent"
+      android:layout_height="wrap_content"
+      android:layout_toRightOf="@+id/cb_item_selection"
+      android:padding="4dp"
+      android:text="@{htmlContent}"
+      android:textColor="@color/oppiaDarkBlue"/>
+  </RelativeLayout>
+</layout>

app/src/main/res/layout/multiple_choice_interaction_items.xml

@@ -0,0 +1,28 @@
+<layout xmlns:android="http://schemas.android.com/apk/res/android" xmlns:app="http://schemas.android.com/apk/res-auto">
+  <data>
+    <variable
+      name="htmlContent"
+      type="String"/>
+  </data>
+  <RelativeLayout
+    android:id="@+id/rl_radio_container"
+    android:layout_width="match_parent"
+    android:layout_height="wrap_content"
+    android:layout_margin="4dp"
+    android:orientation="vertical">
+    <androidx.appcompat.widget.AppCompatRadioButton
+      android:id="@+id/rb_multiple_choice"
+      android:layout_width="wrap_content"
+      android:layout_height="wrap_content"
+      android:padding="4dp"
+      app:buttonTint="@color/oppiaDarkBlue"/>
+    <TextView
+      android:id="@+id/tv_multiple_choice_contents"
+      android:layout_width="match_parent"
+      android:layout_height="wrap_content"
+      android:layout_toRightOf="@+id/rb_multiple_choice"
+      android:padding="4dp"
+      android:text="@{htmlContent}"
+      android:textColor="@color/oppiaDarkBlue"/>
+  </RelativeLayout>
+</layout>

app/src/main/res/layout/state_fragment.xml

@@ -1,7 +1,7 @@
 <?xml version="1.0" encoding="utf-8"?>
 <layout
   xmlns:android="http://schemas.android.com/apk/res/android"
-  xmlns:app="http://schemas.android.com/apk/res-auto">
+  xmlns:app="http://schemas.android.com/apk/res-auto" xmlns:tools="http://schemas.android.com/tools">
   <data>
     <import type="android.view.View"/>
     <variable
@@ -15,6 +15,7 @@
     android:layout_width="match_parent"
     android:layout_height="match_parent">
     <LinearLayout
+      android:id="@+id/audio_container_linear_layout"
       android:layout_width="0dp"
       android:layout_height="wrap_content"
       android:layout_marginStart="24dp"
@@ -29,6 +30,30 @@
         android:layout_width="match_parent"
         android:layout_height="wrap_content"/>
     </LinearLayout>
+
+    <LinearLayout
+      android:id="@+id/interaction_container_linear_layout"
+      android:layout_width="match_parent"
+      android:layout_height="wrap_content"
+      android:layout_margin="8dp"
+      android:background="@color/white"
+      android:orientation="vertical"
+      android:padding="8dp"
+      app:layout_constraintBottom_toTopOf="@+id/dummy_audio_button"
+      app:layout_constraintTop_toBottomOf="@+id/audio_container_linear_layout"
+      tools:layout_editor_absoluteX="8dp">
+      <androidx.recyclerview.widget.RecyclerView
+        android:id="@+id/rvInteractions"
+        android:layout_width="match_parent"
+        android:layout_height="wrap_content"
+        android:layout_alignParentTop="true"
+        android:layout_marginStart="8dp"
+        android:layout_marginTop="4dp"
+        android:layout_marginEnd="8dp"
+        android:divider="@android:color/transparent"
+        android:dividerHeight="8dp"
+        app:layoutManager="androidx.recyclerview.widget.LinearLayoutManager"/>
+    </LinearLayout>
     <Button
       android:id="@+id/dummy_audio_button"
       android:layout_width="wrap_content"

app/src/sharedTest/java/org/oppia/app/player/state/StateFragmentTest.kt

@@ -2,12 +2,15 @@ package org.oppia.app.player.state
 
 import android.app.Application
 import android.content.Context
+import androidx.recyclerview.widget.RecyclerView
 import androidx.test.core.app.ActivityScenario
 import androidx.test.espresso.Espresso
+import androidx.test.espresso.Espresso.onView
 import androidx.test.ext.junit.runners.AndroidJUnit4
 import androidx.test.espresso.action.ViewActions.click
 import androidx.test.espresso.assertion.ViewAssertions.doesNotExist
 import androidx.test.espresso.assertion.ViewAssertions.matches
+import androidx.test.espresso.contrib.RecyclerViewActions
 import androidx.test.espresso.matcher.RootMatchers.isDialog
 import androidx.test.espresso.matcher.ViewMatchers.isDisplayed
 import androidx.test.espresso.matcher.ViewMatchers.withId
@@ -16,6 +19,7 @@ import dagger.BindsInstance
 import dagger.Component
 import dagger.Module
 import dagger.Provides
+import junit.framework.TestCase.assertEquals
 import kotlinx.coroutines.CoroutineDispatcher
 import org.hamcrest.CoreMatchers.not
 import org.junit.Test
@@ -159,6 +163,27 @@ class StateFragmentTest {
     }
   }
 
+  var interactionInstanceId: String? = null
+
+  @Test
+  fun testMultipleChoiceInput_showsRadioButtons_onMultipleChoiceInputInteractionInstanceId_userSelectsDesiredOption() {
+    ActivityScenario.launch(StateFragmentTestActivity::class.java).use {
+      interactionInstanceId = "MultipleChoiceInput"
+      assertEquals(interactionInstanceId, "MultipleChoiceInput")
+      onView(withId(R.id.rvInteractions)).perform(
+        RecyclerViewActions.actionOnItemAtPosition<RecyclerView.ViewHolder>(2,click()))
+    }
+  }
+
+  @Test
+  fun tesItemSelectionInput_showsCheckbox_onItemSelectionInputInteractionInstanceId_userCanSelectMoreThanOneCorrectOption() {
+    ActivityScenario.launch(StateFragmentTestActivity::class.java).use {
+      interactionInstanceId = "ItemSelectionInput"
+      assertEquals(interactionInstanceId, "ItemSelectionInput")
+      onView(withId(R.id.rvInteractions)).perform(
+        RecyclerViewActions.actionOnItemAtPosition<RecyclerView.ViewHolder>(1,click()))
+    }
+  }
   @Module
   class TestModule {
     @Provides