remove quotes, add first (failing) test

.travis.yml

@@ -0,0 +1,8 @@
+language: python
+python: 
+     - "2.7"
+install:
+    pip install mock
+script:
+    - export PYTHONPATH=$PYTHONPATH:$(pwd)
+    - py.test

mGui/core/__init__.py

@@ -89,8 +89,8 @@ def __new__(mcs, name, parents, kwargs):
         # response for gui.wrap will need to be the first class defined
         # for this to work properly. If we run into future isses we may need to make
         # this an explicit class attribute instead
-        if maya_cmd and not REGISTRY.get(maya_cmd.__name__):
-            REGISTRY[maya_cmd.__name__] = completed_type
+        if maya_cmd and not REGISTRY.get(maya_cmd):
+            REGISTRY[maya_cmd] = completed_type
 
         return completed_type
 

mGui/events.py

@@ -6,6 +6,7 @@
 they don't keep their handlers alive if they are otherwise out of scope.
 
 """
+from __future__ import print_function
 import weakref
 import maya.utils
 from functools import partial, wraps
@@ -165,7 +166,7 @@ def _handler_count(self):
     __isub__ = _remove_handler
 
     def __del__(self):
-        print 'event expired'
+        print ('event expired')
 
 
 class MayaEvent(Event):

tests/mock_maya.py

@@ -0,0 +1,19 @@
+import mock
+import sys
+from types import ModuleType
+
+# this creates and 'imports' a maya and a maya.cmds module
+# as mocks
+
+_maya = ModuleType('maya')
+_cmds = ModuleType('cmds')
+_utils = ModuleType('utils')
+_mel = ModuleType('mel')
+_maya.cmds = mock.MagicMock()
+_maya.utils = mock.MagicMock()
+_maya.mel = mock.MagicMock()
+_maya.cmds.about =mock.MagicMock(return_value = '2018.1')
+sys.modules['maya'] = _maya
+sys.modules['maya.cmds'] = _cmds
+sys.modules['maya.utils'] = _utils
+sys.modules['maya.mel'] = _mel

tests/test_Bindings.py

@@ -3,14 +3,11 @@
 
 @author: Stephen Theodore
 '''
-import maya.standalone
-
-maya.standalone.initialize()
+import mock_maya
 import mGui.bindings as bindings
 from unittest import TestCase
-
 import maya.cmds as cmds
-import pymel.core as pm
+#import pymel.core as pm
 
 
 class Test_Accessors(TestCase):
@@ -207,44 +204,48 @@ def set_val(self, **kwargs):
 
     def test_cmds_accessor_get(self):
         cmds.file(new=True, f=True)
+        cmds.polyCube.side_effect= [('pCube1', 'polyCube1')]
         test_obj, _ = cmds.polyCube()
         cmds.xform(test_obj, rotation=(10, 10, 10))
         ac = bindings.CmdsAccessor(test_obj, 'r')
+        cmds.getAttr.side_effect = [[(10,10,10)]]  # nesting is intentional!
         assert ac.pull() == [(10, 10, 10)]
+        assert cmds.getAttr.called_with('pCube1.r', q=True)
 
     def test_cmds_accessor_set(self):
         cmds.file(new=True, f=True)
         ac = bindings.CmdsAccessor('front', 'tz')
         ac.push(55)
-        assert cmds.getAttr('front.tz') == 55
-
-    def test_py_accessor_get(self):
-        cmds.file(new=True, f=True)
-        test_obj, _ = cmds.polyCube()
-        pynode = pm.PyNode(test_obj)
-        ac = bindings.PyNodeAccessor(pynode, 'rx')
-        assert ac.pull() == 0
-
-    def test_py_accessor_set(self):
-        cmds.file(new=True, f=True)
-        front = pm.PyNode('front')
-        ac = bindings.PyNodeAccessor(front, 'rx')
-        ac.push(55)
-        assert front.attr('rx').get() == 55
-
-    def test_py_attrib_accessor_get(self):
-        cmds.file(new=True, f=True)
-        front = pm.PyNode('front')
-        ac = bindings.PyAttributeAccessor(front.rx, None)
-        ac.push(55)
-        assert front.attr('rx').get() == 55
-
-    def test_py_attrib_accessor_set(self):
-        cmds.file(new=True, f=True)
-        front = pm.PyNode('front')
-        ac = bindings.PyAttributeAccessor(front.rx, None)
-        ac.push(55)
-        assert front.attr('rx').get() == 55
+        assert cmds.setAttr.called_with('front.tz', q=True)
+#        assert cmds.getAttr('front.tz') == 55
+
+    # def test_py_accessor_get(self):
+    #     cmds.file(new=True, f=True)
+    #     test_obj, _ = cmds.polyCube()
+    #     pynode = pm.PyNode(test_obj)
+    #     ac = bindings.PyNodeAccessor(pynode, 'rx')
+    #     assert ac.pull() == 0
+
+    # def test_py_accessor_set(self):
+    #     cmds.file(new=True, f=True)
+    #     front = pm.PyNode('front')
+    #     ac = bindings.PyNodeAccessor(front, 'rx')
+    #     ac.push(55)
+    #     assert front.attr('rx').get() == 55
+
+    # def test_py_attrib_accessor_get(self):
+    #     cmds.file(new=True, f=True)
+    #     front = pm.PyNode('front')
+    #     ac = bindings.PyAttributeAccessor(front.rx, None)
+    #     ac.push(55)
+    #     assert front.attr('rx').get() == 55
+
+    # def test_py_attrib_accessor_set(self):
+    #     cmds.file(new=True, f=True)
+    #     front = pm.PyNode('front')
+    #     ac = bindings.PyAttributeAccessor(front.rx, None)
+    #     ac.push(55)
+    #     assert front.attr('rx').get() == 55
 
 
 class TestAccessorFactory(TestCase):
@@ -295,32 +296,34 @@ def test_cmds_accessor(self):
 
     def test_cmds_accessor_excepts_for_nonexistent_object(self):
         cmds.file(new=True, f=True)
+        cmds.getAttr.side_effect = RuntimeError
         self.assertRaises(bindings.BindingError, lambda: bindings.get_accessor('dont_exist', 'tx'))
 
     def test_cmds_accessor_excepts_for_nonexistent_attrrib(self):
         cmds.file(new=True, f=True)
+        cmds.getAttr.side_effect = RuntimeError
         self.assertRaises(bindings.BindingError, lambda: bindings.get_accessor('persp', 'dontexist'))
 
-    def test_pynode_accessor(self):
-        cmds.file(new=True, f=True)
-        cube, shape = pm.polyCube()
-        ac = bindings.get_accessor(cube, 'rx')
-        assert isinstance(ac, bindings.PyNodeAccessor)
-        ac2 = bindings.get_accessor(shape, 'width')
-        assert isinstance(ac2, bindings.PyNodeAccessor)
+    # def test_pynode_accessor(self):
+    #     cmds.file(new=True, f=True)
+    #     cube, shape = pm.polyCube()
+    #     ac = bindings.get_accessor(cube, 'rx')
+    #     assert isinstance(ac, bindings.PyNodeAccessor)
+    #     ac2 = bindings.get_accessor(shape, 'width')
+    #     assert isinstance(ac2, bindings.PyNodeAccessor)
 
-    def test_pynode_accessor_excepts_for_nonexistent_attrib(self):
-        cmds.file(new=True, f=True)
-        cube, _ = pm.polyCube()
-        self.assertRaises(bindings.BindingError, lambda: bindings.get_accessor(cube, 'xyz'))
+    # def test_pynode_accessor_excepts_for_nonexistent_attrib(self):
+    #     cmds.file(new=True, f=True)
+    #     cube, _ = pm.polyCube()
+    #     self.assertRaises(bindings.BindingError, lambda: bindings.get_accessor(cube, 'xyz'))
 
-    def test_pyattr_accessor(self):
-        cmds.file(new=True, f=True)
-        cube, shape = pm.polyCube()
-        ac = bindings.get_accessor(cube.rx)
-        assert isinstance(ac, bindings.PyAttributeAccessor)
-        ac2 = bindings.get_accessor(shape.width)
-        assert isinstance(ac2, bindings.PyAttributeAccessor)
+    # def test_pyattr_accessor(self):
+    #     cmds.file(new=True, f=True)
+    #     cube, shape = pm.polyCube()
+    #     ac = bindings.get_accessor(cube.rx)
+    #     assert isinstance(ac, bindings.PyAttributeAccessor)
+    #     ac2 = bindings.get_accessor(shape.width)
+    #     assert isinstance(ac2, bindings.PyAttributeAccessor)
 
 
 class TestBindings(TestCase):
@@ -433,21 +436,21 @@ def test_bind_to_cmds_string(self):
         tester2()
         assert cmds.getAttr('pCube1.ty') == 45
 
-    def test_bind_to_pyAttr(self):
-        ex = self.Example('cube', 45)
-        cmds.file(new=True, f=True)
-        cube, shape = pm.polyCube()
-        tester = ex & 'val' > bindings.bind() > cube.tx
-        tester()
-        assert cmds.getAttr('pCube1.tx') == 45
-
-    def test_bind_to_pyNode(self):
-        ex = self.Example('cube', 45)
-        cmds.file(new=True, f=True)
-        cube, shape = pm.polyCube()
-        tester = ex & 'val' > bindings.bind() > (cube, 'tx')
-        tester()
-        assert cmds.getAttr('pCube1.tx') == 45
+    # def test_bind_to_pyAttr(self):
+    #     ex = self.Example('cube', 45)
+    #     cmds.file(new=True, f=True)
+    #     cube, shape = pm.polyCube()
+    #     tester = ex & 'val' > bindings.bind() > cube.tx
+    #     tester()
+    #     assert cmds.getAttr('pCube1.tx') == 45
+
+    # def test_bind_to_pyNode(self):
+    #     ex = self.Example('cube', 45)
+    #     cmds.file(new=True, f=True)
+    #     cube, shape = pm.polyCube()
+    #     tester = ex & 'val' > bindings.bind() > (cube, 'tx')
+    #     tester()
+    #     assert cmds.getAttr('pCube1.tx') == 45
 
 
 class TestBindableObject(TestCase):

tests/test_api.py

@@ -1,10 +1,5 @@
 from unittest import TestCase, main
-import maya.standalone
-
-try:
-    maya.standalone.initialize()
-except:
-    pass
+import mock_maya
 
 from mGui.gui import *
 from mGui.forms import *
@@ -276,9 +271,6 @@ def test_PopupMenu(self):
     def test_ProgressBar(self):
         assert ProgressBar
 
-    def test_REGISTRY(self):
-        assert REGISTRY
-
     def test_RadioButton(self):
         assert RadioButton
 

tests/test_controls.py

@@ -1,40 +1,11 @@
-'''
-Created on Mar 3, 2014
-
-@author: Stephen Theodore
-'''
-
+import mock_maya
 from unittest import TestCase, main
-
-LAST_ARGS = {}
-
-
-def control_mock(*args, **kwargs):
-    LAST_ARGS['args'] = args
-    LAST_ARGS['kwargs'] = kwargs
-
-
-import maya.standalone
-
-maya.standalone.initialize()
-
+import inspect
+import mGui.properties as properties
+import mGui.gui as gui
+import mGui.core.progress as progress
 import maya.cmds as cmds
 
-cmds.control = control_mock
-# ===============================================================================
-# cmds.layout = control_mock
-# cmds.window = control_mock
-# cmds.menu = control_mock
-# cmds.menuItem = control_mock
-# 
-# import mGui.styles as styles
-# class MockStyled(object):
-#    CMD = cmds.control
-# 
-# styles.Styled = MockStyled
-# ===============================================================================
-
-
 CONTROL_CMDS = ['attrColorSliderGrp',
                 'attrControlGrp',
                 'attrFieldGrp',
@@ -129,10 +100,6 @@ def control_mock(*args, **kwargs):
     'tabLayout',
     'toolBar']
 
-import inspect
-import mGui.properties as properties
-import mGui.gui as gui
-import mGui.core.progress as progress
 
 
 class test_CtlProperty(TestCase):
@@ -149,50 +116,33 @@ def __init__(self, *args, **kwargs):
         fred = properties.CtlProperty("fred", CMD)
         barney = properties.CtlProperty("barney", CMD)
 
-    def setUp(self):
-        LAST_ARGS['args'] = (None,)
-        LAST_ARGS['kwargs'] = {}
-
     def test_call_uses_widget(self):
         t = self.Example()
-        get = t.fred
-        assert LAST_ARGS['args'][0] == 'path|to|widget'
-
-    def test_call_uses_q_flag(self):
-        t = self.Example()
-        get = t.fred
-        assert 'q' in LAST_ARGS['kwargs']
-
-    def test_call_uses_q_control_flag(self):
-        t = self.Example()
-        get = t.fred
-        assert 'fred' in LAST_ARGS['kwargs']
+        _ = t.fred
+        cmds.control.assert_called_with(t.widget, fred=True, q=True)
 
     def test_set_uses_widget(self):
         t = self.Example()
         t.fred = 999
-        assert LAST_ARGS['args'][0] == 'path|to|widget'
+        cmds.control.assert_called_with(t.widget, e=True, fred=999)
 
-    def test_set_uses_e_flag(self):
-        t = self.Example()
-        t.fred = 999
-        assert 'e' in LAST_ARGS['kwargs']
 
     def test_each_property_has_own_command(self):
         t = self.Example()
-        get = t.fred
-        assert 'fred' in LAST_ARGS['kwargs']
-        get = t.barney
-        assert 'barney' in LAST_ARGS['kwargs']
+        _ = t.fred
+        cmds.control.assert_called_with(t.widget, q=True, fred=True)
+
+        _ = t.barney
+        cmds.control.assert_called_with(t.widget, q=True, barney=True)
 
     def test_access_via_getattr(self):
         t = self.Example()
-        get = getattr(t, 'fred')
-        assert 'fred' in LAST_ARGS['kwargs']
+        _ = getattr(t, 'fred')
+        cmds.control.assert_called_with(t.widget, fred=True, q=True)
 
     def test_access_via_dict_fails(self):
         t = self.Example()
-        assert not 'fred' in t.__dict__
+        assert 'fred' not in t.__dict__
 
 
 class TestControlsExist(TestCase):
@@ -224,6 +174,5 @@ def test_has_MenuItem(self):
         assert 'MenuItem' in gui_items
 
 
-
 if __name__ == '__main__':
-    main()
+    main()