Merge pull request #5176 from alexbruy/fix-test

[processing] attempt to fix failing test
qgis · Sep 12, 2017 · 592151f · 592151f
2 parents a45913b + a090d7d
commit 592151f
Show file tree

Hide file tree

Showing 2 changed files with 34 additions and 69 deletions.
diff --git a/python/plugins/processing/tests/ToolsTest.py b/python/plugins/processing/tests/ToolsTest.py
@@ -25,28 +25,17 @@
 
 __revision__ = '$Format:%H$'
 
+import os
+import shutil
+import tempfile
+
 from qgis.testing import start_app, unittest
 from processing.tests.TestData import points2
 from processing.tools import vector
 from qgis.core import (QgsVectorLayer, QgsFeatureRequest)
 from processing.core.ProcessingConfig import ProcessingConfig
 
-import os.path
+testDataPath = os.path.join(os.path.dirname(__file__), 'testdata')
-import errno
-import shutil
-
-dataFolder = os.path.join(os.path.dirname(__file__), '../../../../tests/testdata/')
-tmpBaseFolder = os.path.join(os.sep, 'tmp', 'qgis_test', str(os.getpid()))
-
-
-def mkDirP(path):
-    try:
-        os.makedirs(path)
-    except OSError as exc:
-        if exc.errno == errno.EEXIST and os.path.isdir(path):
-            pass
-        else:
-            raise
 
 start_app()
 
@@ -55,54 +44,50 @@ class VectorTest(unittest.TestCase):
 
     @classmethod
     def setUpClass(cls):
-        mkDirP(tmpBaseFolder)
+        cls.cleanup_paths = []
 
     @classmethod
     def tearDownClass(cls):
-        shutil.rmtree(tmpBaseFolder)
+        for path in cls.cleanup_paths:
-        pass
+            shutil.rmtree(path)
 
-    # See http://hub.qgis.org/issues/15698
+    def testOgrLayerNameExtraction(self):
-    def test_ogrLayerName(self):
+        outdir = tempfile.mkdtemp()
-        tmpdir = os.path.join(tmpBaseFolder, 'ogrLayerName')
+        self.cleanup_paths.append(outdir)
-        os.mkdir(tmpdir)
 
-        def linkTestfile(f, t):
+        def _copyFile(dst):
-            os.link(os.path.join(dataFolder, f), os.path.join(tmpdir, t))
+            shutil.copyfile(os.path.join(testDataPath, 'custom', 'grass7', 'weighted.csv'), dst)
 
-        # URI from OGR provider
+        # OGR provider - single layer
-        linkTestfile('geom_data.csv', 'a.csv')
+        _copyFile(os.path.join(outdir, 'a.csv'))
-        name = vector.ogrLayerName(tmpdir)
+        name = vector.ogrLayerName(outdir)
         self.assertEqual(name, 'a')
 
-        # URI from OGR provider
+        # OGR provider - multiple layers
-        linkTestfile('wkt_data.csv', 'b.csv')
+        _copyFile(os.path.join(outdir, 'b.csv'))
-        name = vector.ogrLayerName(tmpdir + '|layerid=0')
+        name = vector.ogrLayerName(outdir + '|layerid=0')
-        self.assertEqual(name, 'a')
-        name = vector.ogrLayerName(tmpdir + '|layerid=1')
         self.assertEqual(name, 'b')
+        name = vector.ogrLayerName(outdir + '|layerid=1')
+        self.assertEqual(name, 'a')
 
-        # URI from OGR provider
+        name = vector.ogrLayerName(outdir + '|layerid=2')
-        name = vector.ogrLayerName(tmpdir + '|layerid=2')
+        self.assertIsNone(name)
-        self.assertEqual(name, 'invalid-layerid')
 
-        # URI from OGR provider
+        # OGR provider - layername takes precedence
-        name = vector.ogrLayerName(tmpdir + '|layername=f')
+        name = vector.ogrLayerName(outdir + '|layername=f')
-        self.assertEqual(name, 'f') # layername takes precedence
+        self.assertEqual(name, 'f')
 
-        # URI from OGR provider
+        name = vector.ogrLayerName(outdir + '|layerid=0|layername=f')
-        name = vector.ogrLayerName(tmpdir + '|layerid=0|layername=f2')
+        self.assertEqual(name, 'f')
-        self.assertEqual(name, 'f2') # layername takes precedence
 
-        # URI from OGR provider
+        name = vector.ogrLayerName(outdir + '|layername=f|layerid=0')
-        name = vector.ogrLayerName(tmpdir + '|layername=f2|layerid=0')
+        self.assertEqual(name, 'f')
-        self.assertEqual(name, 'f2') # layername takes precedence
 
-        # URI from Sqlite provider
+        # SQLiite provider
         name = vector.ogrLayerName('dbname=\'/tmp/x.sqlite\' table="t" (geometry) sql=')
         self.assertEqual(name, 't')
 
-        # URI from PostgreSQL provider
+        # PostgreSQL provider
         name = vector.ogrLayerName('port=5493 sslmode=disable key=\'edge_id\' srid=0 type=LineString table="city_data"."edge" (geom) sql=')
         self.assertEqual(name, 'city_data.edge')
 

diff --git a/python/plugins/processing/tools/vector.py b/python/plugins/processing/tools/vector.py
@@ -525,26 +525,6 @@ def ogrConnectionString(uri):
     return '"' + ogrstr + '"'
 
 
-#
-# The uri parameter is an URI from any QGIS provider,
-# so could have different formats.
-# Example formats:
-#
-# -- PostgreSQL provider
-# port=5493 sslmode=disable key='edge_id' srid=0 type=LineString table="city_data"."edge" (geom) sql=
-#
-# -- Spatialite provider
-# dbname='/tmp/x.sqlite' table="t" (geometry) sql='
-#
-# -- OGR provider (single-layer directory)
-# /tmp/x.gdb
-#
-# -- OGR provider (multi-layer directory)
-# /tmp/x.gdb|layerid=1
-#
-# -- OGR provider (multi-layer directory)
-# /tmp/x.gdb|layername=thelayer
-#
 def ogrLayerName(uri):
 
     # handle URIs of database providers
@@ -580,10 +560,10 @@ def ogrLayerName(uri):
             # take precedence
     ds = ogr.Open(ogruri)
     if not ds:
-        return "invalid-uri"
+        return None
     ly = ds.GetLayer(layerid)
     if not ly:
-        return "invalid-layerid"
+        return None
     name = ly.GetName()
     return name