Merge pull request sigmavirus24#394 from rgbkrk/nb_tests

sigmavirus24 · sigmavirus24 · commit 270f6d9c0978 · 2015-06-17T17:58:50.000-05:00
Initial go at running tests on example notebooks.
diff --git a/.travis.yml b/.travis.yml
@@ -28,6 +28,7 @@ env:
   - TOX_ENV=py27-flake8
   - TOX_ENV=py34-flake8
   - TOX_ENV=docstrings
+  - TOX_ENV=notebooks
 
 matrix:
   fast_finish: true
diff --git a/example-notebooks/iterating_over_public_gists.ipynb b/example-notebooks/iterating_over_public_gists.ipynb
@@ -100,7 +100,7 @@
      "cell_type": "markdown",
      "metadata": {},
      "source": [
-      "If you omit the number argument, github3.py will attempt to iterate over all public gists on GitHub. Unfortunately with a ratelimit of 60 unauthenticated requests per hour you will not be able to list them quickly.  "To increase your ratelimit you would have to do the following (or something similar)."
+      "If you omit the number argument, github3.py will attempt to iterate over all public gists on GitHub. Unfortunately with a ratelimit of 60 unauthenticated requests per hour you will not be able to list them quickly. To increase your ratelimit you would have to do the following (or something similar)."
      ]
     },
     {
diff --git a/example-notebooks/two_factor_authentication.ipynb b/example-notebooks/two_factor_authentication.ipynb
@@ -100,12 +100,11 @@
      "cell_type": "markdown",
      "metadata": {},
      "source": [
-      "The call to `authorize` will create an `Authorization` object which has a token. With the
-      object you can pass the token to `login` and not have to worry about having to supply tokens generated by a mobile application or sent to your phone."
+      "The call to `authorize` will create an `Authorization` object which has a token. With the object you can pass the token to `login` and not have to worry about having to supply tokens generated by a mobile application or sent to your phone."
      ]
     }
    ],
    "metadata": {}
   }
  ]
-}
+}
diff --git a/tests/nbtest.py b/tests/nbtest.py
@@ -0,0 +1,17 @@
+#!/usr/bin/env python
+# -*- coding: utf-8 -*-
+
+import glob
+
+from IPython.config import Config
+from IPython.nbconvert import HTMLExporter
+
+c = Config({
+  'ExecutePreprocessor': {'enabled': True}
+})
+
+exporter = HTMLExporter(config=c)
+
+for filename in glob.glob("example-notebooks/*.ipynb"):
+  print(filename)
+  exporter.from_filename(filename)
diff --git a/tox.ini b/tox.ini
@@ -20,6 +20,12 @@ deps =
     flake8
 commands = flake8 {posargs} github3/
 
+[testenv:notebooks]
+basepython = python3.4
+deps =
+    ipython[notebook]
+commands = python tests/nbtest.py
+
 [testenv:docstrings]
 deps =
     flake8

Original file line number	Diff line number	Diff line change
`@@ -100,7 +100,7 @@`
`100`	`100`	`"cell_type": "markdown",`
`101`	`101`	`"metadata": {},`
`102`	`102`	`"source": [`
`103`		`- "If you omit the number argument, github3.py will attempt to iterate over all public gists on GitHub. Unfortunately with a ratelimit of 60 unauthenticated requests per hour you will not be able to list them quickly. "To increase your ratelimit you would have to do the following (or something similar)."`
	`103`	`+ "If you omit the number argument, github3.py will attempt to iterate over all public gists on GitHub. Unfortunately with a ratelimit of 60 unauthenticated requests per hour you will not be able to list them quickly. To increase your ratelimit you would have to do the following (or something similar)."`
`104`	`104`	`]`
`105`	`105`	`},`
`106`	`106`	`{`
Original file line number	Diff line number	Diff line change
`@@ -100,12 +100,11 @@`
`100`	`100`	`"cell_type": "markdown",`
`101`	`101`	`"metadata": {},`
`102`	`102`	`"source": [`
`103`		- "The call to `authorize` will create an `Authorization` object which has a token. With the
`104`		- object you can pass the token to `login` and not have to worry about having to supply tokens generated by a mobile application or sent to your phone."
	`103`	+ "The call to `authorize` will create an `Authorization` object which has a token. With the object you can pass the token to `login` and not have to worry about having to supply tokens generated by a mobile application or sent to your phone."
`105`	`104`	`]`
`106`	`105`	`}`
`107`	`106`	`],`
`108`	`107`	`"metadata": {}`
`109`	`108`	`}`
`110`	`109`	`]`
`111`		`-}`
	`110`	`+}`