font: add Font.size setter

Steve Canny · Steve Canny · commit a8237a0d3cc5 · 2015-02-08T00:01:41.000-08:00
diff --git a/docx/oxml/text/run.py b/docx/oxml/text/run.py
@@ -251,6 +251,14 @@ def sz_val(self):
             return None
         return sz.val
 
+    @sz_val.setter
+    def sz_val(self, value):
+        if value is None:
+            self._remove_sz()
+            return
+        sz = self.get_or_add_sz()
+        sz.val = value
+
     @property
     def underline(self):
         """
diff --git a/docx/text/run.py b/docx/text/run.py
@@ -421,6 +421,11 @@ def size(self):
             return None
         return rPr.sz_val
 
+    @size.setter
+    def size(self, emu):
+        rPr = self._element.get_or_add_rPr()
+        rPr.sz_val = emu
+
 
 class _Text(object):
     """
diff --git a/features/txt-font-props.feature b/features/txt-font-props.feature
@@ -36,7 +36,6 @@ Feature: Get or set font properties
       | 14 pt       | 177800 |
 
 
-  @wip
   Scenario Outline: Set font size
     Given a font of size <size>
      When I assign <value> to font.size
diff --git a/tests/text/test_run.py b/tests/text/test_run.py
@@ -385,6 +385,11 @@ def it_knows_its_size(self, size_get_fixture):
         font, expected_value = size_get_fixture
         assert font.size == expected_value
 
+    def it_can_change_its_size(self, size_set_fixture):
+        font, value, expected_xml = size_set_fixture
+        font.size = value
+        assert font._element.xml == expected_xml
+
     # fixtures -------------------------------------------------------
 
     @pytest.fixture(params=[
@@ -423,3 +428,15 @@ def size_get_fixture(self, request):
         r_cxml, expected_value = request.param
         font = Font(element(r_cxml))
         return font, expected_value
+
+    @pytest.fixture(params=[
+        ('w:r',                      Pt(12), 'w:r/w:rPr/w:sz{w:val=24}'),
+        ('w:r/w:rPr',                Pt(12), 'w:r/w:rPr/w:sz{w:val=24}'),
+        ('w:r/w:rPr/w:sz{w:val=24}', Pt(18), 'w:r/w:rPr/w:sz{w:val=36}'),
+        ('w:r/w:rPr/w:sz{w:val=36}', None,   'w:r/w:rPr'),
+    ])
+    def size_set_fixture(self, request):
+        r_cxml, value, expected_r_cxml = request.param
+        font = Font(element(r_cxml))
+        expected_xml = xml(expected_r_cxml)
+        return font, value, expected_xml
