acpt: migrate scenario for Document.styles

Steve Canny · Steve Canny · commit 13b1e74c4c6d · 2015-02-15T03:36:58.000-08:00
diff --git a/features/doc-access-collections.feature b/features/doc-access-collections.feature
@@ -17,3 +17,8 @@ Feature: Access document collections
   Scenario: Access the section collection of a document
      Given a document having sections
       Then document.sections is a Sections object
+
+
+  Scenario: Access the styles collection of a document
+    Given a document having styles
+     Then document.styles is a Styles object
diff --git a/features/doc-styles.feature b/features/doc-styles.feature
@@ -1,22 +1,19 @@
-Feature: Access document styles
-  In order to discover and manipulate document styles
+Feature: Access a document style
+  In order to operate on a particular document style
   As a developer using python-docx
-  I need a way to access document styles
+  I access to each style in the document style collection
 
 
-  Scenario Outline: Access document styles collection
+  Scenario Outline: Access style in style collection
     Given a document having <styles-state>
-     Then I can access the document styles collection
-      And len(styles) is <style-count>
+     Then len(styles) is <style-count>
+      And I can iterate over its styles
+      And I can access a style by style id
+      And I can access a style by its UI name
 
     Examples: having styles or not
       | styles-state   | style-count |
       | a styles part  |      6      |
       | no styles part |      4      |
 
 
-  Scenario: Access style in style collection
-    Given a document having a styles part
-     Then I can iterate over its styles
-      And I can access a style by style id
-      And I can access a style by its UI name
diff --git a/features/steps/document.py b/features/steps/document.py
@@ -13,6 +13,7 @@
 from docx.parts.document import InlineShapes
 from docx.shared import Inches
 from docx.section import Sections
+from docx.styles.styles import Styles
 from docx.table import Table
 
 from helpers import test_docx, test_file
@@ -35,6 +36,11 @@ def given_a_document_having_sections(context):
     context.document = Document(test_docx('doc-access-sections'))
 
 
+@given('a document having styles')
+def given_a_document_having_styles(context):
+    context.document = Document(test_docx('sty-having-styles-part'))
+
+
 @given('a single-section document having portrait layout')
 def given_a_single_section_document_having_portrait_layout(context):
     context.document = Document(test_docx('doc-add-section'))
@@ -152,19 +158,25 @@ def then_document_inline_shapes_is_an_InlineShapes_object(context):
     assert isinstance(inline_shapes, InlineShapes)
 
 
+@then('document.paragraphs is a list containing three paragraphs')
+def then_document_paragraphs_is_a_list_containing_three_paragraphs(context):
+    document = context.document
+    paragraphs = document.paragraphs
+    assert isinstance(paragraphs, list)
+    assert len(paragraphs) == 3
+
+
 @then('document.sections is a Sections object')
 def then_document_sections_is_a_Sections_object(context):
     sections = context.document.sections
     msg = 'document.sections not instance of Sections'
     assert isinstance(sections, Sections), msg
 
 
-@then('document.paragraphs is a list containing three paragraphs')
-def then_document_paragraphs_is_a_list_containing_three_paragraphs(context):
-    document = context.document
-    paragraphs = document.paragraphs
-    assert isinstance(paragraphs, list)
-    assert len(paragraphs) == 3
+@then('document.styles is a Styles object')
+def then_document_styles_is_a_Styles_object(context):
+    styles = context.document.styles
+    assert isinstance(styles, Styles)
 
 
 @then('the document contains a 2 x 2 table')
diff --git a/features/steps/styles.py b/features/steps/styles.py
@@ -9,7 +9,6 @@
 from docx import Document
 from docx.enum.style import WD_STYLE_TYPE
 from docx.styles.latent import _LatentStyle, LatentStyles
-from docx.styles.styles import Styles
 from docx.styles.style import BaseStyle
 from docx.text.paragraph import ParagraphFormat
 from docx.text.run import Font
@@ -331,13 +330,6 @@ def then_I_can_access_a_style_by_style_id(context):
     assert isinstance(style, BaseStyle)
 
 
-@then('I can access the document styles collection')
-def then_I_can_access_the_document_styles_collection(context):
-    document = context.document
-    styles = document.styles
-    assert isinstance(styles, Styles)
-
-
 @then('I can iterate over its styles')
 def then_I_can_iterate_over_its_styles(context):
     styles = [s for s in context.document.styles]
