Add automated tests for password-protected ODT files

.github/workflows/build_test.yml

@@ -165,4 +165,43 @@ jobs:
         force-avd-creation: false
         emulator-options: -no-snapshot-save -no-window -gpu swiftshader_indirect -noaudio -no-boot-anim -camera-back none
         disable-animations: true
-        script: ./gradlew connectedCheck
+        script: |
+          # Clear logcat before tests
+          adb logcat -c
+
+          # Run tests
+          ./gradlew connectedCheck
+
+          # Dump logcat after tests
+          adb logcat -d > logcat.txt
+
+    - name: Upload test results
+      if: always()
+      uses: actions/upload-artifact@v4
+      with:
+        name: test-results-${{ matrix.api-level }}
+        path: |
+          app/build/reports/androidTests/
+          app/build/outputs/androidTest-results/
+        if-no-files-found: warn
+
+    - name: Upload test logs
+      if: always()
+      uses: actions/upload-artifact@v4
+      with:
+        name: test-logs-${{ matrix.api-level }}
+        path: |
+          app/build/outputs/logs/
+          app/build/test-results/
+        if-no-files-found: warn
+
+    - name: Upload emulator logs
+      if: failure()
+      uses: actions/upload-artifact@v4
+      with:
+        name: emulator-logs-${{ matrix.api-level }}
+        path: |
+          ~/.android/avd/*.avd/config.ini
+          ~/.android/avd/*.avd/*.log
+          logcat.txt
+        if-no-files-found: warn

app/src/androidTest/java/at/tomtasche/reader/test/CoreTest.java

@@ -25,6 +25,7 @@
 @RunWith(AndroidJUnit4.class)
 public class CoreTest {
     private File m_testFile;
+    private File m_passwordTestFile;
 
     @Before
     public void initializeCore() {
@@ -36,19 +37,26 @@ public void initializeCore() {
     public void extractTestFile() throws IOException {
         Context appCtx = InstrumentationRegistry.getInstrumentation().getTargetContext();
         m_testFile = new File(appCtx.getCacheDir(), "test.odt");
+        m_passwordTestFile = new File(appCtx.getCacheDir(), "password-test.odt");
 
         Context testCtx = InstrumentationRegistry.getInstrumentation().getContext();
         AssetManager assetManager = testCtx.getAssets();
         try (InputStream inputStream = assetManager.open("test.odt")) {
             copy(inputStream, m_testFile);
         }
+        try (InputStream inputStream = assetManager.open("password-test.odt")) {
+            copy(inputStream, m_passwordTestFile);
+        }
     }
 
     @After
     public void cleanupTestFile() {
         if (null != m_testFile) {
             m_testFile.delete();
         }
+        if (null != m_passwordTestFile) {
+            m_passwordTestFile.delete();
+        }
     }
 
     private static void copy(InputStream src, File dst) throws IOException {
@@ -84,4 +92,48 @@ public void test() {
         CoreWrapper.CoreResult result = CoreWrapper.backtranslate(coreOptions, htmlDiff);
         Assert.assertEquals(0, coreResult.errorCode);
     }
+
+    @Test
+    public void testPasswordProtectedDocumentWithoutPassword() {
+        File cacheDir = InstrumentationRegistry.getInstrumentation().getTargetContext().getCacheDir();
+        File outputDir = new File(cacheDir, "output_password_test");
+
+        CoreWrapper.CoreOptions coreOptions = new CoreWrapper.CoreOptions();
+        coreOptions.inputPath = m_passwordTestFile.getAbsolutePath();
+        coreOptions.outputPath = outputDir.getPath();
+        coreOptions.editable = false;
+
+        CoreWrapper.CoreResult coreResult = CoreWrapper.parse(coreOptions);
+        Assert.assertEquals(-2, coreResult.errorCode);
+    }
+
+    @Test
+    public void testPasswordProtectedDocumentWithWrongPassword() {
+        File cacheDir = InstrumentationRegistry.getInstrumentation().getTargetContext().getCacheDir();
+        File outputDir = new File(cacheDir, "output_password_test");
+
+        CoreWrapper.CoreOptions coreOptions = new CoreWrapper.CoreOptions();
+        coreOptions.inputPath = m_passwordTestFile.getAbsolutePath();
+        coreOptions.outputPath = outputDir.getPath();
+        coreOptions.password = "wrongpassword";
+        coreOptions.editable = false;
+
+        CoreWrapper.CoreResult coreResult = CoreWrapper.parse(coreOptions);
+        Assert.assertEquals(-2, coreResult.errorCode);
+    }
+
+    @Test
+    public void testPasswordProtectedDocumentWithCorrectPassword() {
+        File cacheDir = InstrumentationRegistry.getInstrumentation().getTargetContext().getCacheDir();
+        File outputDir = new File(cacheDir, "output_password_test");
+
+        CoreWrapper.CoreOptions coreOptions = new CoreWrapper.CoreOptions();
+        coreOptions.inputPath = m_passwordTestFile.getAbsolutePath();
+        coreOptions.outputPath = outputDir.getPath();
+        coreOptions.password = "passwort";
+        coreOptions.editable = false;
+
+        CoreWrapper.CoreResult coreResult = CoreWrapper.parse(coreOptions);
+        Assert.assertEquals(0, coreResult.errorCode);
+    }
 }

app/src/androidTest/java/at/tomtasche/reader/test/MainActivityTests.java

@@ -1,15 +1,20 @@
 package at.tomtasche.reader.test;
 
 import static androidx.test.espresso.Espresso.onView;
+import static androidx.test.espresso.action.ViewActions.clearText;
 import static androidx.test.espresso.action.ViewActions.click;
+import static androidx.test.espresso.action.ViewActions.typeText;
+import static androidx.test.espresso.assertion.ViewAssertions.matches;
 import static androidx.test.espresso.intent.matcher.IntentMatchers.hasAction;
 import static androidx.test.espresso.matcher.ViewMatchers.isDisplayed;
 import static androidx.test.espresso.matcher.ViewMatchers.isEnabled;
+import static androidx.test.espresso.matcher.ViewMatchers.withClassName;
 import static androidx.test.espresso.matcher.ViewMatchers.withContentDescription;
 import static androidx.test.espresso.matcher.ViewMatchers.withId;
 import static androidx.test.espresso.matcher.ViewMatchers.withText;
 import static org.hamcrest.Matchers.allOf;
 import static org.hamcrest.Matchers.anyOf;
+import static org.hamcrest.Matchers.equalTo;
 
 import android.app.Activity;
 import android.app.Instrumentation;
@@ -18,6 +23,7 @@
 import android.content.res.AssetManager;
 import android.net.Uri;
 import android.util.ArrayMap;
+import android.util.Log;
 
 import androidx.core.content.FileProvider;
 import androidx.test.espresso.IdlingRegistry;
@@ -70,15 +76,23 @@ public void setUp() {
         mainActivity.sendBroadcast(new Intent(Intent.ACTION_CLOSE_SYSTEM_DIALOGS));
 
         Intents.init();
+
+        // Log test setup for debugging
+        Log.d("MainActivityTests", "setUp() called for test: " + getClass().getName());
     }
 
     @After
     public void tearDown() {
+        Log.d("MainActivityTests", "tearDown() called");
+
         Intents.release();
 
         if (null != m_idlingResource) {
             IdlingRegistry.getInstance().unregister(m_idlingResource);
         }
+
+        // Ensure activity is finished to reset state between tests
+        mainActivityActivityTestRule.getActivity().finish();
     }
 
     private static void copy(InputStream src, File dst) throws IOException {
@@ -104,7 +118,7 @@ public static void extractTestFiles() throws IOException {
 
         AssetManager testAssetManager = instrumentation.getContext().getAssets();
 
-        for (String filename: new String[] {"test.odt", "dummy.pdf"}) {
+        for (String filename: new String[] {"test.odt", "dummy.pdf", "password-test.odt"}) {
             File targetFile = new File(testDocumentsDir, filename);
             try (InputStream inputStream = testAssetManager.open(filename)) {
                 copy(inputStream, targetFile);
@@ -184,4 +198,69 @@ public void testPDF() {
                         .perform(click());
             });
     }
+
+    @Test
+    public void testPasswordProtectedODT() {
+        File testFile = s_testFiles.get("password-test.odt");
+        Assert.assertNotNull(testFile);
+
+        // Check if the file exists and is readable
+        Assert.assertTrue("Password test file does not exist: " + testFile.getAbsolutePath(), testFile.exists());
+        Assert.assertTrue("Password test file is not readable: " + testFile.getAbsolutePath(), testFile.canRead());
+
+        // Log file info for debugging CI issues
+        Log.d("MainActivityTests", "Password test file path: " + testFile.getAbsolutePath());
+        Log.d("MainActivityTests", "Password test file size: " + testFile.length());
+        Log.d("MainActivityTests", "All test files: " + s_testFiles.keySet());
+
+        // Double-check we're using the right file
+        Assert.assertEquals("password-test.odt file size mismatch", 12671L, testFile.length());
+
+        Context appCtx = InstrumentationRegistry.getInstrumentation().getTargetContext();
+        Uri testFileUri = FileProvider.getUriForFile(appCtx, appCtx.getPackageName() + ".provider", testFile);
+        Intents.intending(hasAction(Intent.ACTION_OPEN_DOCUMENT)).respondWith(
+                new Instrumentation.ActivityResult(Activity.RESULT_OK,
+                        new Intent()
+                                .setData(testFileUri)
+                                .addFlags(Intent.FLAG_GRANT_READ_URI_PERMISSION)
+                )
+        );
+
+        onView(allOf(withId(R.id.menu_open), withContentDescription("Open document"), isDisplayed()))
+            .perform(click());
+
+        onView(allOf(withId(android.R.id.text1), anyOf(withText("Documents"), withText("Files")), isDisplayed()))
+                .perform(click());
+
+        // Wait for the password dialog to appear
+        onView(withText("This document is password-protected"))
+                .check(matches(isDisplayed()));
+
+        // Enter wrong password first
+        onView(withClassName(equalTo("android.widget.EditText")))
+                .perform(typeText("wrongpassword"));
+
+        onView(withId(android.R.id.button1))
+                .perform(click());
+
+        // Should show password dialog again for wrong password
+        onView(withText("This document is password-protected"))
+                .check(matches(isDisplayed()));
+
+        // Clear the text field and enter correct password
+        onView(withClassName(equalTo("android.widget.EditText")))
+                .perform(clearText(), typeText("passwort"));
+
+        onView(withId(android.R.id.button1))
+                .perform(click());
+
+        // Check if edit button becomes available (indicating successful load)
+        onView(allOf(withId(R.id.menu_edit), withContentDescription("Edit document"), isEnabled()))
+            .withFailureHandler((error, viewMatcher) -> {
+                onView(allOf(withContentDescription("More options"), isDisplayed())).perform(click());
+
+                onView(allOf(withId(R.id.menu_edit), withContentDescription("Edit document"), isDisplayed()))
+                        .perform(click());
+            });
+    }
 }