Fix #5148: Use DialogUtil method in Uploadactivity (#5149)

* Replace showInfoAlert with available DialogUtil method

* Remove test case that is no longer relevant

app/src/main/java/fr/free/nrw/commons/upload/UploadActivity.java

@@ -225,9 +225,13 @@ public class UploadActivity extends BaseActivity implements UploadContract.View,
                 .subscribeOn(Schedulers.io())
                 .observeOn(AndroidSchedulers.mainThread())
                 .filter(result -> result)
-                .subscribe(result -> showInfoAlert(R.string.block_notification_title,
-                        R.string.block_notification, UploadActivity.this::finish)
-                ));
+                .subscribe(result -> DialogUtil.showAlertDialog(
+                    this,
+                    getString(R.string.block_notification_title),
+                    getString(R.string.block_notification),
+                    getString(R.string.ok),
+                    this::finish,
+                    true)));
     }
 
     private void checkStoragePermissions() {
@@ -457,18 +461,6 @@ public class UploadActivity extends BaseActivity implements UploadContract.View,
         finish();
     }
 
-    private void showInfoAlert(int titleStringID, int messageStringId, Runnable positive, String... formatArgs) {
-        new AlertDialog.Builder(this)
-                .setTitle(titleStringID)
-                .setMessage(getString(messageStringId, (Object[]) formatArgs))
-                .setCancelable(true)
-                .setPositiveButton(android.R.string.ok, (dialog, id) -> {
-                    positive.run();
-                    dialog.cancel();
-                })
-                .create()
-                .show();
-    }
 
     @Override
     public void showAlertDialog(int messageResourceId, Runnable onPositiveClick) {

app/src/test/kotlin/fr/free/nrw/commons/upload/UploadActivityUnitTests.kt

@@ -233,26 +233,6 @@ class UploadActivityUnitTests {
         method.invoke(activity)
     }
 
-    @Test
-    @Throws(Exception::class)
-    fun testShowInfoAlert() {
-        val method: Method = UploadActivity::class.java.getDeclaredMethod(
-            "showInfoAlert",
-            Int::class.java,
-            Int::class.java,
-            Runnable::class.java,
-            Array<String>::class.java
-        )
-        method.isAccessible = true
-        method.invoke(
-            activity,
-            R.string.block_notification_title,
-            R.string.block_notification,
-            mock(Runnable::class.java),
-            arrayOf("")
-        )
-    }
-
     @Test
     @Throws(Exception::class)
     fun testOnNextButtonClicked() {