api: remove old Document.add_paragraph()

Steve Canny · Steve Canny · commit 11394a0d3743 · 2015-02-14T17:30:36.000-08:00
diff --git a/docx/api.py b/docx/api.py
@@ -49,7 +49,7 @@ class Document(object):
     is loaded.
     """
     def __init__(self, docx=None):
-        document = DocumentNew(docx)
+        self._document = document = DocumentNew(docx)
         self._document_part = document._part
         self._package = document._part.package
 
@@ -78,15 +78,7 @@ def add_page_break(self):
         return p
 
     def add_paragraph(self, text='', style=None):
-        """
-        Return a paragraph newly added to the end of the document, populated
-        with *text* and having paragraph style *style*. *text* can contain
-        tab (``\\t``) characters, which are converted to the appropriate XML
-        form for a tab. *text* can also include newline (``\\n``) or carriage
-        return (``\\r``) characters, each of which is converted to a line
-        break.
-        """
-        return self._document_part.add_paragraph(text, style)
+        return self._document.add_paragraph(text, style)
 
     def add_picture(self, image_path_or_stream, width=None, height=None):
         """
diff --git a/tests/test_api.py b/tests/test_api.py
@@ -104,14 +104,6 @@ def it_should_raise_on_heading_level_out_of_range(self, document):
         with pytest.raises(ValueError):
             document.add_heading(level=10)
 
-    def it_can_add_a_paragraph(self, add_paragraph_fixture):
-        document, text, style, paragraph_ = add_paragraph_fixture
-        paragraph = document.add_paragraph(text, style)
-        document._document_part.add_paragraph.assert_called_once_with(
-            text, style
-        )
-        assert paragraph is paragraph_
-
     def it_can_add_a_page_break(self, add_page_break_fixture):
         document, document_part_, paragraph_, run_ = add_page_break_fixture
         paragraph = document.add_page_break()
@@ -197,16 +189,6 @@ def it_creates_numbering_part_on_first_access_if_not_present(
 
     # fixtures -------------------------------------------------------
 
-    @pytest.fixture(params=[
-        ('',         None),
-        ('',         'Heading 1'),
-        ('foo\rbar', 'Body Text'),
-    ])
-    def add_paragraph_fixture(self, request, document, document_part_,
-                              paragraph_):
-        text, style = request.param
-        return document, text, style, paragraph_
-
     @pytest.fixture(params=[
         (0, 'Title'),
         (1, 'Heading 1'),
