Import documents - store unquoted path for later matching with metadata

Author: PrimozGodec

orangecontrib/text/import_documents.py

@@ -185,7 +185,7 @@ class UrlReader(Reader, CoreUrlReader):
 
     def __init__(self, path, *args):
         CoreUrlReader.__init__(self, path)
-        Reader.__init__(self, self.filename, *args)
+        Reader.__init__(self, path, *args)
 
     def read_file(self):
         # unquote prevent double quoting when filename is already quoted

orangecontrib/text/tests/test_import_documents.py

@@ -1,11 +1,11 @@
 import unittest
-from unittest.mock import patch
 
 import numpy as np
 import pandas as pd
 
 from orangecontrib.text.import_documents import ImportDocuments, UrlReader, \
     TxtReader, TextData
+from pkg_resources import get_distribution
 
 
 class TestUrlReader(unittest.TestCase):
@@ -51,6 +51,18 @@ def test_name_text_data(self):
         self.assertEqual(text_data.category, "data")
         self.assertEqual(text_data.content, "text")
 
+    def test_remove_quoting(self):
+        """
+        Since URL quoting is implemented in Orange it can be removed from text
+        addon when minimal version of Orange is increased to 3.29.1. When this
+        test start to fail remove the test itself and lines 191 - 194 in
+        import_documents.py
+        """
+        distribution = get_distribution('orange3-text')
+        orange_spec = next(x for x in distribution.requires() if x.name == "Orange3")
+        orange_min_version = tuple(map(int, orange_spec.specs[0][1].split(".")))
+        self.assertLess(orange_min_version, (3, 29, 1))
+
 
 class TestImportDocuments(unittest.TestCase):
     def test_scan_url(self):