tbl: add _Cell.vertical_alignment setter

Author: scanny

docx/oxml/table.py

@@ -794,6 +794,13 @@ def vAlign_val(self):
             return None
         return vAlign.val
 
+    @vAlign_val.setter
+    def vAlign_val(self, value):
+        if value is None:
+            self._remove_vAlign()
+            return
+        self.get_or_add_vAlign().val = value
+
     @property
     def vMerge_val(self):
         """

docx/table.py

@@ -282,6 +282,11 @@ def vertical_alignment(self):
             return None
         return tcPr.vAlign_val
 
+    @vertical_alignment.setter
+    def vertical_alignment(self, value):
+        tcPr = self._element.get_or_add_tcPr()
+        tcPr.vAlign_val = value
+
     @property
     def width(self):
         """

features/tbl-cell-props.feature

@@ -15,7 +15,6 @@ Feature: Get and set table cell properties
       | center    | WD_ALIGN_VERTICAL.CENTER |
 
 
-  @wip
   Scenario Outline: Set Cell.vertical_alignment
     Given a _Cell object with <state> vertical alignment as cell
      When I assign <value> to cell.vertical_alignment

tests/test_table.py

@@ -1,8 +1,6 @@
 # encoding: utf-8
 
-"""
-Test suite for the docx.table module
-"""
+"""Test suite for the docx.table module"""
 
 from __future__ import absolute_import, print_function, unicode_literals
 
@@ -344,6 +342,11 @@ def it_knows_its_vertical_alignment(self, alignment_get_fixture):
         vertical_alignment = cell.vertical_alignment
         assert vertical_alignment == expected_value
 
+    def it_can_change_its_vertical_alignment(self, alignment_set_fixture):
+        cell, new_value, expected_xml = alignment_set_fixture
+        cell.vertical_alignment = new_value
+        assert cell._element.xml == expected_xml
+
     def it_knows_its_width_in_EMU(self, width_get_fixture):
         cell, expected_width = width_get_fixture
         assert cell.width == expected_width
@@ -428,6 +431,26 @@ def alignment_get_fixture(self, request):
         cell = _Cell(element(tc_cxml), None)
         return cell, expected_value
 
+    @pytest.fixture(params=[
+        ('w:tc', WD_ALIGN_VERTICAL.TOP,
+         'w:tc/w:tcPr/w:vAlign{w:val=top}'),
+        ('w:tc/w:tcPr', WD_ALIGN_VERTICAL.CENTER,
+         'w:tc/w:tcPr/w:vAlign{w:val=center}'),
+        ('w:tc/w:tcPr/w:vAlign{w:val=center}', WD_ALIGN_VERTICAL.BOTTOM,
+         'w:tc/w:tcPr/w:vAlign{w:val=bottom}'),
+        ('w:tc/w:tcPr/w:vAlign{w:val=center}', None,
+         'w:tc/w:tcPr'),
+        ('w:tc', None,
+         'w:tc/w:tcPr'),
+        ('w:tc/w:tcPr', None,
+         'w:tc/w:tcPr'),
+    ])
+    def alignment_set_fixture(self, request):
+        cxml, new_value, expected_cxml = request.param
+        cell = _Cell(element(cxml), None)
+        expected_xml = xml(expected_cxml)
+        return cell, new_value, expected_xml
+
     @pytest.fixture
     def merge_fixture(self, tc_, tc_2_, parent_, merged_tc_):
         cell, other_cell = _Cell(tc_, parent_), _Cell(tc_2_, parent_)