api: remove old Document.paragraphs

Steve Canny · Steve Canny · commit 8e2f61ad1559 · 2015-02-15T01:47:56.000-08:00
diff --git a/docx/api.py b/docx/api.py
@@ -78,12 +78,7 @@ def inline_shapes(self):
 
     @property
     def paragraphs(self):
-        """
-        A list of |Paragraph| instances corresponding to the paragraphs in
-        the document, in document order. Note that paragraphs within revision
-        marks such as ``<w:ins>`` or ``<w:del>`` do not appear in this list.
-        """
-        return self._document_part.paragraphs
+        return self._document.paragraphs
 
     @property
     def part(self):
diff --git a/docx/parts/document.py b/docx/parts/document.py
@@ -113,15 +113,6 @@ def numbering_part(self):
             self.relate_to(numbering_part, RT.NUMBERING)
             return numbering_part
 
-    @property
-    def paragraphs(self):
-        """
-        A list of |Paragraph| instances corresponding to the paragraphs in
-        the document, in document order. Note that paragraphs within revision
-        marks such as inserted or deleted do not appear in this list.
-        """
-        return self.body.paragraphs
-
     @lazyproperty
     def sections(self):
         """
diff --git a/tests/parts/test_document.py b/tests/parts/test_document.py
@@ -41,12 +41,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_paragraphs(
-            self, paragraphs_fixture):
-        document_part, paragraphs_ = paragraphs_fixture
-        paragraphs = document_part.paragraphs
-        assert paragraphs is paragraphs_
-
     def it_provides_access_to_the_document_sections(self, sections_fixture):
         document, document_elm, Sections_ = sections_fixture
         sections = document.sections
@@ -212,12 +206,6 @@ def nmprt_get_fixture(self, part_related_by_, numbering_part_):
         part_related_by_.return_value = numbering_part_
         return document_part, numbering_part_
 
-    @pytest.fixture
-    def paragraphs_fixture(self, document_part_body_, body_, paragraphs_):
-        document_part = DocumentPart(None, None, None, None)
-        body_.paragraphs = paragraphs_
-        return document_part, paragraphs_
-
     @pytest.fixture
     def sections_fixture(self, Sections_):
         document_elm = a_document().with_nsdecls().element
@@ -317,10 +305,6 @@ def p_(self, request):
     def package_(self, request):
         return instance_mock(request, Package)
 
-    @pytest.fixture
-    def paragraphs_(self, request):
-        return instance_mock(request, list)
-
     @pytest.fixture
     def part_related_by_(self, request):
         return method_mock(request, DocumentPart, 'part_related_by')
diff --git a/tests/test_api.py b/tests/test_api.py
@@ -86,12 +86,6 @@ def Package_(self, request):
 
 class DescribeDocumentOld(object):
 
-    def it_provides_access_to_the_document_paragraphs(
-            self, paragraphs_fixture):
-        document, paragraphs_ = paragraphs_fixture
-        paragraphs = document.paragraphs
-        assert paragraphs is paragraphs_
-
     def it_provides_access_to_the_document_sections(self, document):
         body = document.sections
         assert body is document._document_part.sections
@@ -117,10 +111,6 @@ def it_provides_access_to_its_styles(self, styles_fixture):
     def init_fixture(self, docx_, open_):
         return docx_, open_
 
-    @pytest.fixture
-    def paragraphs_fixture(self, document, paragraphs_):
-        return document, paragraphs_
-
     @pytest.fixture
     def save_fixture(self, request, open_, package_):
         file_ = instance_mock(request, str)
@@ -153,11 +143,10 @@ def document_obj_(self, request):
         return instance_mock(request, docx.document.Document)
 
     @pytest.fixture
-    def document_part_(self, request, paragraphs_, tables_):
+    def document_part_(self, request, tables_):
         document_part_ = instance_mock(
             request, DocumentPart, content_type=CT.WML_DOCUMENT_MAIN
         )
-        document_part_.paragraphs = paragraphs_
         document_part_.tables = tables_
         return document_part_
 
@@ -186,10 +175,6 @@ def package_(self, request, document_part_):
         package_.main_document_part = document_part_
         return package_
 
-    @pytest.fixture
-    def paragraphs_(self, request):
-        return instance_mock(request, list)
-
     @pytest.fixture
     def run_(self, request):
         return instance_mock(request, Run)
