[New Designs] Update Run button from dashboard to display new flow for selecting desired tests

app/src/main/AndroidManifest.xml

@@ -38,6 +38,12 @@
 			android:name=".activity.OnboardingActivity"
 			android:screenOrientation="userPortrait"
 			android:theme="@style/Theme.MaterialComponents.NoActionBar.Onboarding" />
+		<activity
+			android:label="Run tests"
+			android:name=".activity.runtests.RunTestsActivity"
+			android:parentActivityName=".activity.MainActivity"
+			android:exported="false"
+			android:theme="@style/Theme.MaterialComponents.Light.DarkActionBar.App.NoActionBar"/>
 		<activity
 			android:name=".activity.PreferenceActivity"
 			android:launchMode="singleTop"

app/src/main/java/org/openobservatory/ooniprobe/activity/runtests/RunTestsActivity.kt

@@ -0,0 +1,189 @@
+package org.openobservatory.ooniprobe.activity.runtests
+
+import android.content.Context
+import android.content.Intent
+import android.os.Bundle
+import android.text.TextUtils
+import android.view.*
+import org.openobservatory.ooniprobe.R
+import org.openobservatory.ooniprobe.activity.AbstractActivity
+import org.openobservatory.ooniprobe.activity.RunningActivity
+import org.openobservatory.ooniprobe.activity.runtests.RunTestsViewModel.Companion.SELECT_ALL
+import org.openobservatory.ooniprobe.activity.runtests.RunTestsViewModel.Companion.SELECT_NONE
+import org.openobservatory.ooniprobe.activity.runtests.RunTestsViewModel.Companion.SELECT_SOME
+import org.openobservatory.ooniprobe.activity.runtests.adapter.RunTestsExpandableListViewAdapter
+import org.openobservatory.ooniprobe.activity.runtests.models.ChildItem
+import org.openobservatory.ooniprobe.activity.runtests.models.GroupItem
+import org.openobservatory.ooniprobe.common.PreferenceManager
+import org.openobservatory.ooniprobe.databinding.ActivityRunTestsBinding
+import org.openobservatory.ooniprobe.test.suite.*
+import java.io.Serializable
+import javax.inject.Inject
+
+class RunTestsActivity : AbstractActivity() {
+	lateinit var binding: ActivityRunTestsBinding
+
+	private lateinit var mAdapter: RunTestsExpandableListViewAdapter
+
+	@Inject
+	lateinit var preferenceManager: PreferenceManager
+
+	@Inject
+	lateinit var viewModel: RunTestsViewModel
+
+	companion object {
+		const val TESTS: String = "tests"
+
+		@JvmStatic
+		fun newIntent(context: Context, testSuites: List<AbstractSuite>): Intent {
+			return Intent(context, RunTestsActivity::class.java).putExtras(Bundle().apply {
+				putSerializable(TESTS, testSuites as Serializable)
+			})
+		}
+	}
+
+	override fun onCreate(savedInstanceState: Bundle?) {
+		super.onCreate(savedInstanceState)
+		binding = ActivityRunTestsBinding.inflate(layoutInflater)
+		setContentView(binding.getRoot())
+
+		setSupportActionBar(binding.toolbar)
+		supportActionBar?.setDisplayHomeAsUpEnabled(true)
+
+		activityComponent?.inject(this)
+
+		val testSuites: List<AbstractSuite>? = intent.extras?.getParcelableArrayList(TESTS)
+		testSuites?.let { ts ->
+			val tsGroups: List<GroupItem> = ts.map { testSuite ->
+				return@map testSuite.runTestsGroupItem(preferenceManager)
+			}
+
+			mAdapter = RunTestsExpandableListViewAdapter(this, tsGroups, viewModel)
+
+			binding.expandableListView.setAdapter(mAdapter)
+			for (i in 0 until mAdapter.groupCount) {
+				binding.expandableListView.expandGroup(i)
+			}
+			binding.selectAll.setOnClickListener { onSelectAllClickListener() }
+
+			binding.selectNone.setOnClickListener { onSelectNoneClickListener() }
+
+			viewModel.selectedAllBtnStatus.observe(this, this::selectAllBtnStatusObserver)
+
+			binding.bottomBar.setOnMenuItemClickListener { menuItem ->
+				onMenuItemClickListener(menuItem)
+			}
+		} ?: run {
+			finish()
+		}
+
+	}
+
+	override fun onCreateOptionsMenu(menu: Menu): Boolean {
+		val inflater: MenuInflater = menuInflater
+		inflater.inflate(R.menu.close, menu)
+		return true
+	}
+
+	override fun onOptionsItemSelected(item: MenuItem): Boolean {
+		return when (item.itemId) {
+			R.id.close_button -> {
+				finish()
+				true
+			}
+
+			else -> super.onOptionsItemSelected(item)
+		}
+	}
+
+	private fun onMenuItemClickListener(menuItem: MenuItem): Boolean {
+		return when (menuItem.itemId) {
+			R.id.runButton -> {
+				val selectedChildItems: List<String> = getChildItemsSelectedIdList()
+				if (selectedChildItems.isNotEmpty()) {
+					val testSuitesToRun = getGroupItemsAtLeastOneChildEnabled().map { groupItem ->
+						return@map AbstractSuite.getTestSuiteByName(groupItem.name).dynamicTestSuite(
+							nettests = groupItem.nettests.filter { nattest -> nattest.selected }
+						)
+					}
+					RunningActivity.runAsForegroundService(
+						this@RunTestsActivity,
+						java.util.ArrayList(testSuitesToRun),
+						{ finish() },
+						preferenceManager
+					)
+
+				}
+				true
+			}
+
+			else -> false
+		}
+	}
+
+	// TODO(aanorbel) Update button color from theme
+	private fun selectAllBtnStatusObserver(selectAllBtnStatus: String?) {
+		if (!TextUtils.isEmpty(selectAllBtnStatus)) {
+			when (selectAllBtnStatus) {
+				SELECT_ALL -> {
+					binding.selectNone.isActivated = true
+					binding.selectAll.isActivated = false
+				}
+
+				SELECT_NONE -> {
+					binding.selectNone.isActivated = true
+					binding.selectAll.isActivated = false
+				}
+
+				SELECT_SOME -> {
+					binding.selectNone.isActivated = true
+					binding.selectAll.isActivated = true
+				}
+			}
+			mAdapter.notifyDataSetChanged()
+			updateStatusIndicator()
+		}
+	}
+
+	private fun onSelectNoneClickListener() {
+		viewModel.setSelectedAllBtnStatus(SELECT_NONE)
+		mAdapter.notifyDataSetChanged()
+		updateStatusIndicator()
+	}
+
+	private fun onSelectAllClickListener() {
+		viewModel.setSelectedAllBtnStatus(SELECT_ALL)
+		mAdapter.notifyDataSetChanged()
+		updateStatusIndicator()
+	}
+
+
+	private fun updateStatusIndicator() {
+		//TODO(aanorbel): translate status indicator
+		binding.bottomBar.setTitle("${getChildItemsSelectedIdList().size} Tests")
+	}
+
+	private fun getChildItemsSelectedIdList(): List<String> {
+		val childItemSelectedIdList: MutableList<String> = ArrayList()
+		for (i in 0 until mAdapter.groupCount) {
+			val secondLevelItemList: List<ChildItem> = mAdapter.getGroup(i).nettests
+			secondLevelItemList
+				.asSequence()
+				.filter { it.selected }
+				.mapTo(childItemSelectedIdList) { it.name }
+		}
+		return childItemSelectedIdList
+	}
+
+	private fun getGroupItemsAtLeastOneChildEnabled(): List<GroupItem> {
+		val items: MutableList<GroupItem> = ArrayList()
+		for (i in 0 until mAdapter.groupCount) {
+			if (mAdapter.getGroup(i).nettests.any { it.selected }) {
+				items.add(mAdapter.getGroup(i).apply {
+					nettests = nettests.filter { it.selected }
+				})
+			}
+		}
+		return items
+	}
+}

app/src/main/java/org/openobservatory/ooniprobe/activity/runtests/RunTestsViewModel.kt

@@ -0,0 +1,73 @@
+package org.openobservatory.ooniprobe.activity.runtests
+
+import androidx.lifecycle.MutableLiveData
+import androidx.lifecycle.ViewModel
+import org.openobservatory.ooniprobe.common.PreferenceManager
+import org.openobservatory.ooniprobe.common.disableTest
+import org.openobservatory.ooniprobe.common.enableTest
+import org.openobservatory.ooniprobe.test.suite.CircumventionSuite
+import org.openobservatory.ooniprobe.test.suite.ExperimentalSuite
+import org.openobservatory.ooniprobe.test.suite.InstantMessagingSuite
+import org.openobservatory.ooniprobe.test.suite.PerformanceSuite
+import javax.inject.Inject
+
+
+class RunTestsViewModel() : ViewModel() {
+	lateinit var preferenceManager: PreferenceManager
+
+	@Inject
+	constructor(preferenceManager: PreferenceManager) : this() {
+		this.preferenceManager = preferenceManager
+	}
+
+	val selectedAllBtnStatus: MutableLiveData<String> = MutableLiveData()
+
+	init {
+		selectedAllBtnStatus.postValue(SELECT_SOME)
+	}
+
+	fun setSelectedAllBtnStatus(selectedStatus: String) {
+		selectedAllBtnStatus.postValue(selectedStatus)
+		when (selectedStatus) {
+			SELECT_ALL -> {
+				InstantMessagingSuite().getTestList(preferenceManager).forEach {
+					enableTest(it.name)
+				}
+				CircumventionSuite().getTestList(preferenceManager).forEach {
+					enableTest(it.name)
+				}
+				PerformanceSuite().getTestList(preferenceManager).forEach {
+					enableTest(it.name)
+				}
+				enableTest(ExperimentalSuite.NAME)
+			}
+
+			SELECT_NONE -> {
+				InstantMessagingSuite().getTestList(preferenceManager).forEach {
+					disableTest(it.name)
+				}
+				CircumventionSuite().getTestList(preferenceManager).forEach {
+					disableTest(it.name)
+				}
+				PerformanceSuite().getTestList(preferenceManager).forEach {
+					disableTest(it.name)
+				}
+				disableTest(ExperimentalSuite.NAME)
+			}
+		}
+	}
+
+	fun disableTest(name: String) {
+		preferenceManager.disableTest(name)
+	}
+
+	fun enableTest(name: String) {
+		preferenceManager.enableTest(name)
+	}
+
+	companion object {
+		const val SELECT_ALL = "SELECT_ALL"
+		const val SELECT_SOME = "SELECT_SOME"
+		const val SELECT_NONE = "SELECT_NONE"
+	}
+}