remove PDF post processor

IPython/nbconvert/exporters/tests/test_pdf.py

@@ -0,0 +1,36 b''
+"""Tests for PDF export"""
+
+# Copyright (c) IPython Development Team.
+# Distributed under the terms of the Modified BSD License.
+
+import logging
+import os
+
+from IPython.testing import decorators as dec
+
+from .base import ExportersTestsBase
+from ..pdf import PDFExporter
+
+
+#-----------------------------------------------------------------------------
+# Class
+#-----------------------------------------------------------------------------
+
+class TestPDF(ExportersTestsBase):
+    """Test PDF export"""
+
+    exporter_class = PDFExporter
+
+    def test_constructor(self):
+        """Can a PDFExporter be constructed?"""
+        self.exporter_class()
+
+
+    @dec.onlyif_cmds_exist('pdflatex')
+    @dec.onlyif_cmds_exist('pandoc')
+    def test_export(self):
+        """Smoke test PDFExporter"""
+        (output, resources) = self.exporter_class(latex_count=1).from_filename(self._get_notebook())
+        self.assertIsInstance(output, bytes)
+        assert len(output) > 0
+

IPython/nbconvert/nbconvertapp.py

@@ -164,8 +164,7 b' class NbConvertApp(BaseIPythonApplication):'
     postprocessor_class = DottedOrNone(config=True, 
                                     help="""PostProcessor class used to write the 
                                     results of the conversion""")
-    postprocessor_aliases = {'pdf': 'IPython.nbconvert.postprocessors.pdf.PDFPostProcessor',
-                              'serve': 'IPython.nbconvert.postprocessors.serve.ServePostProcessor'}
+    postprocessor_aliases = {'serve': 'IPython.nbconvert.postprocessors.serve.ServePostProcessor'}
     postprocessor_factory = Type()
 
     def _postprocessor_class_changed(self, name, old, new):

IPython/nbconvert/postprocessors/__init__.py

@@ -1,5 +1,4 b''
 from .base import PostProcessorBase
-from .pdf import PDFPostProcessor
 
 # protect against unavailable tornado
 try:

IPython/nbconvert/tests/test_nbconvertapp.py

@@ -1,22 +1,15 b''
 # -*- coding: utf-8 -*- 
 """Test NbConvertApp"""
 
-#-----------------------------------------------------------------------------
-#  Copyright (C) 2013 The IPython Development Team
-#
-#  Distributed under the terms of the BSD License.  The full license is in
-#  the file COPYING, distributed as part of this software.
-#-----------------------------------------------------------------------------
-
-#-----------------------------------------------------------------------------
-# Imports
-#-----------------------------------------------------------------------------
+# Copyright (c) IPython Development Team.
+# Distributed under the terms of the Modified BSD License.
 
 import os
 import glob
 import sys
 
 from .base import TestsBase
+from ..postprocessors import PostProcessorBase
 
 import IPython.testing.tools as tt
 from IPython.testing import decorators as dec
@@ -25,6 +18,10 b' from IPython.testing import decorators as dec'
 # Classes and functions
 #-----------------------------------------------------------------------------
 
+class DummyPost(PostProcessorBase):
+    def postprocess(self, filename):
+        print("Dummy:%s" % filename)
+
 class TestNbConvertApp(TestsBase):
     """Collection of NbConvertApp tests"""
 
@@ -79,24 +76,19 b' class TestNbConvertApp(TestsBase):'
         """
         with self.create_temp_cwd(['notebook2.ipynb']):
             os.rename('notebook2.ipynb', 'notebook with spaces.ipynb')
-            self.call('nbconvert --log-level 0 --to latex '
-                            '"notebook with spaces" --post PDF '
-                            '--PDFPostProcessor.verbose=True')
-            assert os.path.isfile('notebook with spaces.tex')
-            assert os.path.isdir('notebook with spaces_files')
+            self.call('nbconvert --log-level 0 --to pdf'
+                    ' "notebook with spaces"'
+                    ' --PDFExporter.latex_count=1'
+                    ' --PDFExporter.verbose=True'
+            )
             assert os.path.isfile('notebook with spaces.pdf')
 
-    @dec.onlyif_cmds_exist('pdflatex')
-    @dec.onlyif_cmds_exist('pandoc')
     def test_post_processor(self):
-        """
-        Do post processors work?
-        """
+        """Do post processors work?"""
         with self.create_temp_cwd(['notebook1.ipynb']):
-            self.call('nbconvert --log-level 0 --to latex notebook1 '
-                      '--post PDF --PDFPostProcessor.verbose=True')
-            assert os.path.isfile('notebook1.tex')
-            assert os.path.isfile('notebook1.pdf')
+            out, err = self.call('nbconvert --log-level 0 --to python notebook1 '
+                      '--post IPython.nbconvert.tests.test_nbconvertapp.DummyPost')
+            self.assertIn('Dummy:notebook1.py', out)
 
     @dec.onlyif_cmds_exist('pandoc')
     def test_spurious_cr(self):
@@ -195,8 +187,7 b' class TestNbConvertApp(TestsBase):'
         """
         with self.create_temp_cwd():
             self.create_empty_notebook(u'nb1_análisis.ipynb')
-            self.call('nbconvert --log-level 0 --to latex '
-                            '"nb1_*" --post PDF '
-                            '--PDFPostProcessor.verbose=True')
-            assert os.path.isfile(u'nb1_análisis.tex')
+            self.call('nbconvert --log-level 0 --to pdf "nb1_*"'
+                    ' --PDFExporter.latex_count=1'
+                    ' --PDFExporter.verbose=True')
             assert os.path.isfile(u'nb1_análisis.pdf')