You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tika.apache.org by dm...@apache.org on 2021/12/31 12:42:43 UTC

[tika] 01/02: TIKA-3636: Updated testNoConfig to always run

This is an automated email from the ASF dual-hosted git repository.

dmeikle pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/tika.git

commit 133c53a6c882f0490f25c69780664e0e3ea2caef
Author: David Meikle <dm...@apache.org>
AuthorDate: Fri Dec 31 12:41:48 2021 +0000

    TIKA-3636: Updated testNoConfig to always run
---
 .../tika/language/translate/impl/MarianTranslatorTest.java     | 10 +++-------
 1 file changed, 3 insertions(+), 7 deletions(-)

diff --git a/tika-translate/src/test/java/org/apache/tika/language/translate/impl/MarianTranslatorTest.java b/tika-translate/src/test/java/org/apache/tika/language/translate/impl/MarianTranslatorTest.java
index 0fc127a..22afc68 100644
--- a/tika-translate/src/test/java/org/apache/tika/language/translate/impl/MarianTranslatorTest.java
+++ b/tika-translate/src/test/java/org/apache/tika/language/translate/impl/MarianTranslatorTest.java
@@ -60,14 +60,10 @@ public class MarianTranslatorTest {
     @Test
     public void testNoConfig() throws Exception {
         String source = "Apache Tika is a wonderful tool";
-        String expected = "Apache Tika is a wonderful tool";
+        String expected = "Apache Tika is a wonderful tool"; // Pattern from other Translators is to return source
         String translated = translator.translate(source, "en", "zz");
-        if (translator.isAvailable()) {
-            assertTrue("Translate " + source + " to " + expected + " (was " + translated + ")",
-                    expected.equalsIgnoreCase(translated));
-        } else {
-            throw new AssumptionViolatedException("Engine not available");
-        }
+        assertTrue("Translate " + source + " to " + expected + " (was " + translated + ")",
+                   expected.equalsIgnoreCase(translated));
     }
 
 }