Merge pull request #1836 from openedx/hamza/LEARNER-9659

Prevent Deep Link Activation on Orientation Change

.github/workflows/androidTestSuite.yml

@@ -54,6 +54,7 @@ jobs:
           fail_ci_if_error: false
           verbose: true
 
+  # TODO: Re-enable this job when the Instrument Test functionality is restored - Remove 'if: false'
   android_screenshot:
     name: Android Instrumented Tests
     # The macOS VM provided by GitHub Actions has HAXM installed so we are able to create a new AVD
@@ -62,6 +63,7 @@ jobs:
     # Includes Screenshot Tests
     # Ref: https://github.com/marketplace/actions/android-emulator-runner
     runs-on: macos-11
+    if: false
 
     steps:
       # Checkouts the current branch for processing

OpenEdXMobile/src/main/java/org/edx/mobile/view/CourseHomeTabFragment.kt

@@ -273,6 +273,7 @@ class CourseHomeTabFragment : OfflineSupportBaseFragment(), CourseHomeAdapter.On
                 false
             )
             courseUnitDetailLauncher.launch(courseUnitDetailIntent)
+            arguments?.putString(Router.EXTRA_SCREEN_NAME, null)
             screenName = null
         }
     }

OpenEdXMobile/src/main/java/org/edx/mobile/view/CourseTabsDashboardFragment.kt

@@ -130,28 +130,7 @@ class CourseTabsDashboardFragment : BaseFragment() {
     private val fullscreenLoader: FullscreenLoaderDialogFragment?
         get() = getRetainedInstance(childFragmentManager)
 
-    private val courseTabItems: List<FragmentItemModel>
-        get() {
-            val screenName = arguments?.getString(Router.EXTRA_SCREEN_NAME)
-            val items = mutableListOf<FragmentItemModel>()
-
-            items.add(createCourseOutlineItem(screenName))
-            if (environment.config.isCourseVideosEnabled) {
-                items.add(createVideosItem())
-            }
-            if (environment.config.isDiscussionsEnabled &&
-                courseData.course.discussionUrl?.isNotEmpty() == true
-            ) {
-                items.add(createDiscussionsItem())
-            }
-            if (environment.config.isCourseDatesEnabled) {
-                items.add(createDatesItem())
-            }
-            items.add(createHandoutsItem())
-            items.add(createAnnouncementsItem())
-
-            return items
-        }
+    private lateinit var courseTabItems: List<FragmentItemModel>
 
     private val onBackPressedCallback: OnBackPressedCallback =
         object : OnBackPressedCallback(true) {
@@ -191,6 +170,7 @@ class CourseTabsDashboardFragment : BaseFragment() {
         binding = FragmentCourseTabsDashboardBinding.inflate(inflater, container, false)
         courseData =
             arguments?.serializableOrThrow(Router.EXTRA_COURSE_DATA) as EnrolledCoursesResponse
+        courseTabItems = getCourseTabItems()
 
         setHasOptionsMenu(courseData.course.coursewareAccess.hasAccess())
 
@@ -788,6 +768,28 @@ class CourseTabsDashboardFragment : BaseFragment() {
         })
     }
 
+    private fun getCourseTabItems(): List<FragmentItemModel> {
+        val screenName = arguments?.getString(Router.EXTRA_SCREEN_NAME)
+        val items = mutableListOf<FragmentItemModel>()
+
+        items.add(createCourseOutlineItem(screenName))
+        if (environment.config.isCourseVideosEnabled) {
+            items.add(createVideosItem())
+        }
+        if (environment.config.isDiscussionsEnabled &&
+            courseData.course.discussionUrl?.isNotEmpty() == true
+        ) {
+            items.add(createDiscussionsItem())
+        }
+        if (environment.config.isCourseDatesEnabled) {
+            items.add(createDatesItem())
+        }
+        items.add(createHandoutsItem())
+        items.add(createAnnouncementsItem())
+
+        return items
+    }
+
     private fun createCourseOutlineItem(screenName: String?): FragmentItemModel {
         return FragmentItemModel(
             CourseHomeTabFragment::class.java,

codecov.yml

@@ -0,0 +1,7 @@
+# Configuration for GitHub-Codecov Checks
+# Inspiration: https://docs.codecov.com/docs/github-checks#yaml-configuration-for-github-checks-and-codecov
+
+coverage:
+  status:
+    patch: false
+    project: false