style: add _LatentStyle on/off prop setters

Steve Canny · Steve Canny · commit 8562eb7ff6d3 · 2015-02-08T00:01:48.000-08:00
diff --git a/docx/oxml/styles.py b/docx/oxml/styles.py
@@ -92,6 +92,12 @@ def on_off_prop(self, attr_name):
         """
         return getattr(self, attr_name)
 
+    def set_on_off_prop(self, attr_name, value):
+        """
+        Set the on/off attribute having *attr_name* to *value*.
+        """
+        setattr(self, attr_name, value)
+
 
 class CT_Style(BaseOxmlElement):
     """
diff --git a/docx/styles/latent.py b/docx/styles/latent.py
@@ -135,6 +135,10 @@ def hidden(self):
         """
         return self._element.on_off_prop('semiHidden')
 
+    @hidden.setter
+    def hidden(self, value):
+        self._element.set_on_off_prop('semiHidden', value)
+
     @property
     def locked(self):
         """
@@ -146,6 +150,10 @@ def locked(self):
         """
         return self._element.on_off_prop('locked')
 
+    @locked.setter
+    def locked(self, value):
+        self._element.set_on_off_prop('locked', value)
+
     @property
     def name(self):
         """
@@ -174,6 +182,10 @@ def quick_style(self):
         """
         return self._element.on_off_prop('qFormat')
 
+    @quick_style.setter
+    def quick_style(self, value):
+        self._element.set_on_off_prop('qFormat', value)
+
     @property
     def unhide_when_used(self):
         """
@@ -184,3 +196,7 @@ def unhide_when_used(self):
         object.
         """
         return self._element.on_off_prop('unhideWhenUsed')
+
+    @unhide_when_used.setter
+    def unhide_when_used(self, value):
+        self._element.set_on_off_prop('unhideWhenUsed', value)
diff --git a/features/sty-latent-props.feature b/features/sty-latent-props.feature
@@ -80,7 +80,6 @@ Feature: Get and set latent style properties
       | unhide_when_used | no setting | None  |
 
 
-  @wip
   Scenario Outline: Set on/off latent style properties
     Given a latent style having <prop-name> set <setting>
      When I assign <new-value> to latent_style.<prop-name>
diff --git a/tests/styles/test_latent.py b/tests/styles/test_latent.py
@@ -35,6 +35,11 @@ def it_knows_its_on_off_properties(self, on_off_get_fixture):
         actual_value = getattr(latent_style, prop_name)
         assert actual_value == expected_value
 
+    def it_can_change_its_on_off_properties(self, on_off_set_fixture):
+        latent_style, prop_name, value, expected_xml = on_off_set_fixture
+        setattr(latent_style, prop_name, value)
+        assert latent_style.element.xml == expected_xml
+
     # fixtures -------------------------------------------------------
 
     @pytest.fixture(params=[
@@ -64,6 +69,28 @@ def on_off_get_fixture(self, request):
         latent_style = _LatentStyle(element(lsdException_cxml))
         return latent_style, prop_name, expected_value
 
+    @pytest.fixture(params=[
+        ('w:lsdException',                 'hidden',           True,
+         'w:lsdException{w:semiHidden=1}'),
+        ('w:lsdException{w:semiHidden=1}', 'hidden',           False,
+         'w:lsdException{w:semiHidden=0}'),
+        ('w:lsdException{w:semiHidden=0}', 'hidden',           None,
+         'w:lsdException'),
+        ('w:lsdException',                 'locked',           True,
+         'w:lsdException{w:locked=1}'),
+        ('w:lsdException',                 'quick_style',      False,
+         'w:lsdException{w:qFormat=0}'),
+        ('w:lsdException',                 'unhide_when_used', True,
+         'w:lsdException{w:unhideWhenUsed=1}'),
+        ('w:lsdException{w:locked=1}',     'locked',           None,
+         'w:lsdException'),
+    ])
+    def on_off_set_fixture(self, request):
+        lsdException_cxml, prop_name, value, expected_cxml = request.param
+        latent_styles = _LatentStyle(element(lsdException_cxml))
+        expected_xml = xml(expected_cxml)
+        return latent_styles, prop_name, value, expected_xml
+
     @pytest.fixture(params=[
         ('w:lsdException',                  None),
         ('w:lsdException{w:uiPriority=42}', 42),
