From 507d4fb9ad0de4b860947a8558e7fa9d3cc7c115 2013-10-17 21:09:08
From: Paul Ivanov <pi@berkeley.edu>
Date: 2013-10-17 21:09:08
Subject: [PATCH] adding tests for named_notebook_path

I want to do some refactoring, so I'll put in some tests that will define the
behavior that I want to preserve, so I can verify that my refactoring hasn't
broken anything.

---

diff --git a/IPython/html/services/notebooks/tests/test_nbmanager.py b/IPython/html/services/notebooks/tests/test_nbmanager.py
index 54aa0e9..3a745ca 100644
--- a/IPython/html/services/notebooks/tests/test_nbmanager.py
+++ b/IPython/html/services/notebooks/tests/test_nbmanager.py
@@ -8,8 +8,9 @@ from IPython.utils.tempdir import TemporaryDirectory
 from IPython.utils.traitlets import TraitError
 
 from ..filenbmanager import FileNotebookManager
+from ..nbmanager import NotebookManager
 
-class TestNotebookManager(TestCase):
+class TestFileNotebookManager(TestCase):
 
     def test_nb_dir(self):
         with TemporaryDirectory() as td:
@@ -31,4 +32,21 @@ class TestNotebookManager(TestCase):
         with NamedTemporaryFile() as tf:
             self.assertRaises(TraitError, FileNotebookManager, notebook_dir=tf.name)
 
+class TestNotebookManager(TestCase):
+    def test_named_notebook_path(self):
+        nm = NotebookManager()
+        
+        # doesn't end with ipynb, should just be path
+        name, path = nm.named_notebook_path('hello')
+        self.assertEqual(name, None)
+        self.assertEqual(path, 'hello/')
+
+        name, path = nm.named_notebook_path('hello.ipynb')
+        self.assertEqual(name, 'hello.ipynb')
+        self.assertEqual(path, None)
+        
+        name, path = nm.named_notebook_path('/this/is/a/path/hello.ipynb')
+        self.assertEqual(name, 'hello.ipynb')
+        self.assertEqual(path, '/this/is/a/path/')
+