api: remove old Document.sections

Steve Canny · Steve Canny · commit 804aad19906d · 2015-02-15T03:22:31.000-08:00
diff --git a/docx/api.py b/docx/api.py
@@ -89,10 +89,7 @@ def save(self, path_or_stream):
 
     @property
     def sections(self):
-        """
-        Return a reference to the |Sections| instance for this document.
-        """
-        return self._document_part.sections
+        return self._document.sections
 
     @property
     def styles(self):
diff --git a/docx/parts/document.py b/docx/parts/document.py
@@ -13,7 +13,6 @@
 from .numbering import NumberingPart
 from ..opc.constants import RELATIONSHIP_TYPE as RT
 from ..opc.part import XmlPart
-from ..section import Sections
 from ..shape import InlineShape
 from ..shared import lazyproperty, Parented
 from .styles import StylesPart
@@ -118,13 +117,6 @@ def save(self, path_or_stream):
         """
         self.package.save(path_or_stream)
 
-    @lazyproperty
-    def sections(self):
-        """
-        The |Sections| instance organizing the sections in this document.
-        """
-        return Sections(self._element)
-
     @property
     def styles(self):
         """
diff --git a/features/steps/document.py b/features/steps/document.py
@@ -10,8 +10,9 @@
 
 from docx import Document
 from docx.enum.section import WD_ORIENT, WD_SECTION
-from docx.parts.document import InlineShapes, Sections
+from docx.parts.document import InlineShapes
 from docx.shared import Inches
+from docx.section import Sections
 from docx.table import Table
 
 from helpers import test_docx, test_file
diff --git a/tests/parts/test_document.py b/tests/parts/test_document.py
@@ -45,12 +45,6 @@ def it_has_a_body(self, body_fixture):
         _Body_.assert_called_once_with(body_elm, document_part)
         assert _body is _Body_.return_value
 
-    def it_provides_access_to_the_document_sections(self, sections_fixture):
-        document, document_elm, Sections_ = sections_fixture
-        sections = document.sections
-        Sections_.assert_called_once_with(document_elm)
-        assert sections is Sections_.return_value
-
     def it_provides_access_to_the_document_tables(self, tables_fixture):
         document_part, tables_ = tables_fixture
         tables = document_part.tables
@@ -216,12 +210,6 @@ def save_fixture(self, package_):
         file_ = 'foobar.docx'
         return document_part, file_
 
-    @pytest.fixture
-    def sections_fixture(self, Sections_):
-        document_elm = a_document().with_nsdecls().element
-        document = DocumentPart(None, None, document_elm, None)
-        return document, document_elm, Sections_
-
     @pytest.fixture
     def styles_fixture(self, _styles_part_prop_, styles_part_, styles_):
         document_part = DocumentPart(None, None, None, None)
@@ -329,10 +317,6 @@ def relate_to_(self, request, rId_):
     def rId_(self, request):
         return instance_mock(request, str)
 
-    @pytest.fixture
-    def Sections_(self, request):
-        return class_mock(request, 'docx.parts.document.Sections')
-
     @pytest.fixture
     def style_(self, request):
         return instance_mock(request, BaseStyle)
diff --git a/tests/test_api.py b/tests/test_api.py
@@ -84,10 +84,6 @@ def Package_(self, request):
 
 class DescribeDocumentOld(object):
 
-    def it_provides_access_to_the_document_sections(self, document):
-        body = document.sections
-        assert body is document._document_part.sections
-
     def it_provides_access_to_the_document_tables(self, tables_fixture):
         document, tables_ = tables_fixture
         tables = document.tables
