tabs: add TabStop.alignment setter

DKWoods · Steve Canny · commit 7f76a129117e · 2016-05-27T12:48:11.000-07:00
diff --git a/docx/text/tabstops.py b/docx/text/tabstops.py
@@ -72,6 +72,10 @@ def alignment(self):
         """
         return self._tab.val
 
+    @alignment.setter
+    def alignment(self, value):
+        self._tab.val = value
+
     @property
     def leader(self):
         """
diff --git a/features/tab-tabstop-props.feature b/features/tab-tabstop-props.feature
@@ -36,7 +36,6 @@ Feature: Tab stop properties
       | RIGHT     |
 
 
-  @wip
   Scenario Outline: Set TabStop.alignment
    Given a tab stop having <alignment> alignment
     When I assign <member> to tab_stop.alignment
diff --git a/tests/text/test_tabstops.py b/tests/text/test_tabstops.py
@@ -35,6 +35,11 @@ def it_knows_its_alignment(self, alignment_get_fixture):
         tab_stop, expected_value = alignment_get_fixture
         assert tab_stop.alignment == expected_value
 
+    def it_can_change_its_alignment(self, alignment_set_fixture):
+        tab_stop, value, expected_xml = alignment_set_fixture
+        tab_stop.alignment = value
+        assert tab_stop._element.xml == expected_xml
+
     def it_knows_its_leader(self, leader_get_fixture):
         tab_stop, expected_value = leader_get_fixture
         assert tab_stop.leader == expected_value
@@ -51,6 +56,17 @@ def alignment_get_fixture(self, request):
         expected_value = getattr(WD_TAB_ALIGNMENT, member)
         return tab_stop, expected_value
 
+    @pytest.fixture(params=[
+        ('w:tab{w:val=left}',  'RIGHT', 'w:tab{w:val=right}'),
+        ('w:tab{w:val=right}', 'LEFT',  'w:tab{w:val=left}'),
+    ])
+    def alignment_set_fixture(self, request):
+        tab_stop_cxml, member, expected_cxml = request.param
+        tab_stop = TabStop(element(tab_stop_cxml))
+        expected_xml = xml(expected_cxml)
+        value = getattr(WD_TAB_ALIGNMENT, member)
+        return tab_stop, value, expected_xml
+
     @pytest.fixture(params=[
         ('w:tab',                'SPACES'),
         ('w:tab{w:leader=none}', 'SPACES'),
