acpt: add scenarios for Table.table_direction

Steve Canny · Steve Canny · commit d2781975da2e · 2015-02-21T15:20:10.000-08:00
diff --git a/docx/table.py b/docx/table.py
@@ -145,6 +145,15 @@ def table(self):
         """
         return self
 
+    @property
+    def table_direction(self):
+        """
+        A member of :ref:`WdTableDirection` indicating the direction in which
+        the table cells are ordered, e.g. `WD_TABLE_DIRECTION.LTR`. |None|
+        indicates the value is inherited from the style hierarchy.
+        """
+        raise NotImplementedError
+
     @property
     def _cells(self):
         """
diff --git a/features/steps/table.py b/features/steps/table.py
@@ -11,7 +11,7 @@
 from behave import given, then, when
 
 from docx import Document
-from docx.enum.table import WD_TABLE_ALIGNMENT
+from docx.enum.table import WD_TABLE_ALIGNMENT, WD_TABLE_DIRECTION
 from docx.shared import Inches
 from docx.table import _Column, _Columns, _Row, _Rows
 
@@ -112,6 +112,17 @@ def given_a_table_having_style(context, style):
     context.table_ = document.tables[table_idx]
 
 
+@given('a table having table direction set {setting}')
+def given_a_table_having_table_direction_setting(context, setting):
+    table_idx = [
+        'to inherit',
+        'right-to-left',
+        'left-to-right'
+    ].index(setting)
+    document = Document(test_docx('tbl-on-off-props'))
+    context.table_ = document.tables[table_idx]
+
+
 @given('a table having two columns')
 def given_a_table_having_two_columns(context):
     docx_path = test_docx('blk-containing-table')
@@ -165,6 +176,14 @@ def when_apply_value_to_table_style(context, value):
     table.style = new_value
 
 
+@when('I assign {value} to table.table_direction')
+def when_assign_value_to_table_table_direction(context, value):
+    new_value = (
+        None if value == 'None' else getattr(WD_TABLE_DIRECTION, value)
+    )
+    context.table_.table_direction = new_value
+
+
 @when('I merge from cell {origin} to cell {other}')
 def when_I_merge_from_cell_origin_to_cell_other(context, origin, other):
     def cell(table, idx):
@@ -274,6 +293,15 @@ def then_table_style_is_styles_style_name(context, style_name):
     assert table.style == expected_style, "got '%s'" % table.style
 
 
+@then('table.table_direction is {value}')
+def then_table_table_direction_is_value(context, value):
+    expected_value = (
+        None if value == 'None' else getattr(WD_TABLE_DIRECTION, value)
+    )
+    actual_value = context.table_.table_direction
+    assert actual_value == expected_value, "got '%s'" % actual_value
+
+
 @then('the column cells text is {expected_text}')
 def then_the_column_cells_text_is_expected_text(context, expected_text):
     table = context.table_
diff --git a/features/steps/test_files/tbl-on-off-props.docx b/features/steps/test_files/tbl-on-off-props.docx
diff --git a/features/tbl-props.feature b/features/tbl-props.feature
@@ -4,7 +4,7 @@ Feature: Get and set table properties
   I need a way to get and set a table's properties
 
 
-  Scenario Outline: Determine table alignment
+  Scenario Outline: Get table alignment
     Given a table having <alignment> alignment
      Then table.alignment is <value>
 
@@ -53,3 +53,28 @@ Feature: Get and set table properties
       | autofit             | autofit     | autofit          |
       | fixed               | fixed       | fixed            |
       | autofit             | fixed       | fixed            |
+
+
+  @wip
+  Scenario Outline: Get table direction
+    Given a table having table direction set <setting>
+     Then table.table_direction is <value>
+
+    Examples: Table on/off property values
+      | setting       | value |
+      | to inherit    | None  |
+      | right-to-left | RTL   |
+      | left-to-right | LTR   |
+
+
+  @wip
+  Scenario Outline: Set table direction
+    Given a table having table direction set <setting>
+     When I assign <new-value> to table.table_direction
+     Then table.table_direction is <value>
+
+    Examples: Results of assignment to Table.table_direction
+      | setting       | new-value | value |
+      | to inherit    | RTL       | RTL   |
+      | right-to-left | LTR       | LTR   |
+      | left-to-right | None      | None  |
