api: remove old Document.save()

Steve Canny · Steve Canny · commit e5b84e728b8c · 2015-02-15T02:26:29.000-08:00
diff --git a/docx/api.py b/docx/api.py
@@ -85,11 +85,7 @@ def part(self):
         return self._document.part
 
     def save(self, path_or_stream):
-        """
-        Save this document to *path_or_stream*, which can be either a path to
-        a filesystem location (a string) or a file-like object.
-        """
-        self._package.save(path_or_stream)
+        self._document.save(path_or_stream)
 
     @property
     def sections(self):
diff --git a/tests/test_api.py b/tests/test_api.py
@@ -16,9 +16,7 @@
 from docx.opc.constants import CONTENT_TYPE as CT
 from docx.package import Package
 from docx.parts.document import DocumentPart, InlineShapes
-from docx.section import Section
 from docx.styles.styles import Styles
-from docx.text.run import Run
 
 from .unitutil.mock import (
     function_mock, instance_mock, class_mock, property_mock
@@ -95,28 +93,13 @@ def it_provides_access_to_the_document_tables(self, tables_fixture):
         tables = document.tables
         assert tables is tables_
 
-    def it_can_save_the_package(self, save_fixture):
-        document, package_, file_ = save_fixture
-        document.save(file_)
-        package_.save.assert_called_once_with(file_)
-
     def it_provides_access_to_its_styles(self, styles_fixture):
         document, styles_ = styles_fixture
         styles = document.styles
         assert styles is styles_
 
     # fixtures -------------------------------------------------------
 
-    @pytest.fixture
-    def init_fixture(self, docx_, open_):
-        return docx_, open_
-
-    @pytest.fixture
-    def save_fixture(self, request, open_, package_):
-        file_ = instance_mock(request, str)
-        document = Document()
-        return document, package_, file_
-
     @pytest.fixture
     def styles_fixture(self, document, styles_):
         document._document_part.styles = styles_
@@ -163,26 +146,12 @@ def open_(self, request, document_obj_, document_part_, package_):
             return_value=document_obj_
         )
 
-    @pytest.fixture
-    def Package_(self, request, package_):
-        Package_ = class_mock(request, 'docx.api.Package')
-        Package_.open.return_value = package_
-        return Package_
-
     @pytest.fixture
     def package_(self, request, document_part_):
         package_ = instance_mock(request, Package)
         package_.main_document_part = document_part_
         return package_
 
-    @pytest.fixture
-    def run_(self, request):
-        return instance_mock(request, Run)
-
-    @pytest.fixture
-    def section_(self, request):
-        return instance_mock(request, Section)
-
     @pytest.fixture
     def styles_(self, request):
         return instance_mock(request, Styles)
