Merge pull request #2671 from blink1073/test-imports

Test module importing on Travis

.travis.yml

@@ -59,4 +59,5 @@ install:
 
 script:  
   - python bootstrap.py --test-travis 30
-
+  - python setup.py install
+  - ./continuous_integration/travis_script.sh

continuous_integration/travis_script.sh

@@ -0,0 +1,51 @@
+#!/usr/bin/env bash
+
+set -ex
+
+for f in spyderlib/*.py; do
+  if [[ $f == "spyderlib/restart_app.py" ]]; then
+    continue
+  fi
+  if [[ $f == "spyderlib/spyder.py" ]]; then
+    continue
+  fi
+  if [[ $f == "spyderlib/tour.py" ]]; then
+    continue
+  fi
+  if [[ $f == "spyderlib/start_app.py" ]]; then
+    continue
+  fi
+  if [[ $f == "spyderlib/pil_patch.py" ]]; then
+    continue
+  fi
+  python "$f"
+  if [ $? -ne 0 ]; then
+    exit 1
+  fi
+done
+
+
+for f in spyderlib/*/*.py; do
+  if [[ $f == spyderlib/plugins/*.py ]]; then
+    continue
+  fi
+  if [[ $f == spyderlib/qt/*.py ]]; then
+    continue
+  fi
+  if [[ $f == spyderlib/utils/environ.py ]]; then
+    continue
+  fi
+  if [[ $f == spyderlib/utils/qthelpers.py ]]; then
+    continue
+  fi
+  if [[ $f == spyderlib/utils/windows.py ]]; then
+    continue
+  fi
+  if [[ $f == spyderlib/widgets/*.py ]]; then
+    continue
+  fi
+  python "$f"
+  if [ $? -ne 0 ]; then
+    exit 1
+  fi
+done

spyderlib/utils/bsdsocket.py

@@ -157,13 +157,11 @@ class PacketNotReceived(object):
     client = socket.socket(socket.AF_INET, socket.SOCK_STREAM)
     client.connect( address )
 
-    client.send("data to be catched")
+    client.send("data to be catched".encode('utf-8'))
     # accepted server socket is the one we can read from
     # note that it is different from server socket
     accsock, addr = server.accept()
     print('..got "%s" from %s' % (accsock.recv(4096), addr))
-    client.send("more data for recv")
-    print('..got "%s" from %s' % (accsock.recv(4096), addr))
 
     # accsock.close()
     # client.send("more data for recv")

spyderlib/utils/environ.py

@@ -17,7 +17,6 @@
 from spyderlib.config.base import _
 from spyderlib.widgets.varexp.collectionseditor import CollectionsEditor
 
-
 def envdict2listdict(envdict):
     """Dict --> Dict of lists"""
     sep = os.path.pathsep

spyderlib/utils/misc.py

@@ -228,9 +228,17 @@ def get_common_path(pathlist):
                 return osp.abspath(common)
 
 if __name__ == '__main__':
-    assert get_common_path([
-                            'D:\\Python\\spyder-v21\\spyderlib\\widgets',
-                            'D:\\Python\\spyder\\spyderlib\\utils',
-                            'D:\\Python\\spyder\\spyderlib\\widgets',
-                            'D:\\Python\\spyder-v21\\spyderlib\\utils',
-                            ]) == 'D:\\Python'
+    if os.name == 'nt':
+        assert get_common_path([
+                                'D:\\Python\\spyder-v21\\spyderlib\\widgets',
+                                'D:\\Python\\spyder\\spyderlib\\utils',
+                                'D:\\Python\\spyder\\spyderlib\\widgets',
+                                'D:\\Python\\spyder-v21\\spyderlib\\utils',
+                                ]) == 'D:\\Python'
+    else:
+        assert get_common_path([
+                                '/Python/spyder-v21/spyderlib/widgets',
+                                '/Python/spyder/spyderlib/utils',
+                                '/Python/spyder/spyderlib/widgets',
+                                '/Python/spyder-v21/spyderlib/utils',
+                                ]) == '/Python'

spyderlib/utils/sourcecode.py

@@ -117,8 +117,8 @@ def get_identifiers(source_code):
 if __name__ == '__main__':
     code = 'import functools\nfunctools.partial'
     assert get_primary_at(code, len(code)) == 'functools.partial'
-    assert get_identifiers(code) == ['import', 'functools', 
-                                     'functools.partial']
+    assert set(get_identifiers(code)) == set(['import', 'functools',
+                                              'functools.partial'])
     assert split_source(code) == ['import functools', 'functools.partial']
     code = code.replace('\n', '\r\n')
     assert split_source(code) == ['import functools', 'functools.partial']