Enable serverextension in CI! Add publish integration tests.

- Update README
- also, ignore node_modules

Right now we're not automatically configuring the jupyter server. This 
manifests as a situation where requests are sent to the endpoint 
expecting a json payload and it gets back the 404 html page (which it 
doesn't know how to process). The real solution to this is to 
autoinstall the extension, but that won't help in the development case 
so it needs to be documented.

Author: mpacer

.circleci/config.yml

@@ -71,6 +71,7 @@ jobs:
           command: |
             . venv_packaged_integration/bin/activate
             # Install the dependencies for our integration tester
+            jupyter serverextension enable --py bookstore
             npm i
             node ci/integration.js
 

.gitignore

@@ -114,3 +114,5 @@ binder/*run*.ipynb
 jupyter_notebook_config.py
 jupyter_config.py
 
+# node modules to ignore
+node_modules/

README.md

@@ -42,6 +42,16 @@ Since most people are on a regular filesystem, we'll start with writing to the `
 
 ---
 
+## Publishing
+
+The bookstore publishing endpoint is a serverextension to the classic Jupyter server. 
+
+This means if you are developing this you will need to explicitly enable it to use the endpoint. 
+
+To do so you run: `jupyter serverextension enable --py bookstore`. 
+
+If you wish to enable it only for your current environment, run: `jupyter serverextension enable --py bookstore --sys-prefix`.
+
 ## Installation
 
 **bookstore** requires Python 3.6 or higher.

bookstore/handlers.py

@@ -68,8 +68,6 @@ async def _publish(self, model, path):
             )
             self.log.info("Done with published write of %s", path)
 
-        self.log.info(obj)
-
         self.set_status(201)
 
         resp_content = {"s3path": full_s3_path}

ci/integration.js

@@ -1,3 +1,4 @@
+const child_process = require("child_process");
 const path = require("path");
 const fs = require("fs");
 
@@ -167,6 +168,13 @@ const main = async () => {
     originalNotebook
   );
 
+  await jupyterServer.publishNotebook(
+    "ci-published.ipynb",
+    originalNotebook
+  )
+
+  await comparePublishedNotebooks("ci-published.ipynb", originalNotebook);
+
   const basicNotebook = {
     cells: [],
     nbformat: 4,