copying cells copies attachments

add test to verify

notebook/static/notebook/js/textcell.js

@@ -235,12 +235,12 @@ define([
         // We deepcopy the attachments so copied cells don't share the same
         // objects
         if (Object.keys(this.attachments).length > 0) {
-            data.attachments = {};
             if (gc_attachments) {
                 // Garbage collect unused attachments : The general idea is to
                 // render the text, and find used attachments like when we
                 // substitute them in render()
                 var that = this;
+                data.attachments = {};
                 // To find attachments, rendering to HTML is easier than
                 // searching in the markdown source for the multiple ways you
                 // can reference an image in markdown (using []() or a
@@ -262,10 +262,12 @@ define([
                         h.attr('src', '');
                     });
                 });
-            }
-            if (data.attachments.length === 0) {
-                // omit attachments dict if no attachments
-                delete data.attachments;
+                if (data.attachments.length === 0) {
+                    // omit attachments dict if no attachments
+                    delete data.attachments;
+                }
+            } else {
+                data.attachments = JSON.parse(JSON.stringify(this.attachments));
             }
         }
         return data;

notebook/tests/notebook/attachments.js

@@ -13,8 +13,8 @@ casper.notebook_test(function () {
     var selector = '#insert_image > a';
     this.waitForSelector(selector);
     this.thenEvaluate(function(sel) {
-        IPython.notebook.to_markdown();
-        var cell = IPython.notebook.get_selected_cell();
+        Jupyter.notebook.to_markdown();
+        var cell = Jupyter.notebook.get_selected_cell();
         cell.set_text("");
         cell.unrender();
 
@@ -43,14 +43,14 @@ casper.notebook_test(function () {
     // Validate and render the markdown cell
     this.thenClick('#btn_ok');
     this.thenEvaluate(function() {
-        IPython.notebook.get_cell(0).render();
+        Jupyter.notebook.get_cell(0).render();
     });
     this.wait(300);
     // Check that an <img> tag has been inserted and that it contains the
     // image
     this.then(function() {
         var img = this.evaluate(function() {
-            var cell = IPython.notebook.get_cell(0);
+            var cell = Jupyter.notebook.get_cell(0);
             var img = $("div.text_cell_render").find("img");
             return {
                 src: img.attr("src"),
@@ -78,7 +78,7 @@ casper.notebook_test(function () {
     // append a new cell
     this.append_cell('', 'markdown');
     this.thenEvaluate(function() {
-        IPython.notebook.select_next();
+        Jupyter.notebook.select_next();
     });
 
     // and paste the attachments into it
@@ -89,39 +89,60 @@ casper.notebook_test(function () {
     // check that the new cell has attachments
     this.then(function() {
         var cell_attachments = this.evaluate(function() {
-            return IPython.notebook.get_selected_cell().attachments;
+            return Jupyter.notebook.get_selected_cell().attachments;
         });
         var orig_cell_attachments = this.evaluate(function() {
-            return IPython.notebook.get_cell(0).attachments;
+            return Jupyter.notebook.get_cell(0).attachments;
         });
-        var clip = this.evaluate(function() { return IPython.notebook.clipboard_attachments; });
         // Check that the two cells have the same attachments
         this.test.assertEquals(cell_attachments, orig_cell_attachments,
-                               "both cells have the attachments");
+                               "pasted attachments ok");
+    });
+
+    // copy/paste cell includes attachments
+    selector = '#copy_cell > a';
+    this.waitForSelector(selector);
+    this.thenClick(selector);
+
+    selector = '#paste_cell_below > a';
+    this.waitForSelector(selector);
+    this.thenClick(selector);
+
+    // check that the new cell has attachments
+    this.then(function() {
+        var cell_attachments = this.evaluate(function() {
+            return Jupyter.notebook.get_selected_cell().attachments;
+        });
+        var orig_cell_attachments = this.evaluate(function() {
+            return Jupyter.notebook.get_cell(0).attachments;
+        });
+        // Check that the two cells have the same attachments
+        this.test.assertEquals(cell_attachments, orig_cell_attachments,
+                               "pasted cell has attachments");
     });
 
     var nbname = 'attachments_test.ipynb';
     this.thenEvaluate(function(nbname) {
-        IPython.notebook.set_notebook_name(nbname);
+        Jupyter.notebook.set_notebook_name(nbname);
     }, {nbname:nbname});
 
     // -- Save the notebook. This should cause garbage collection for the
     // second cell (since we just pasted the attachments but there is no
     // markdown referencing them)
     this.thenEvaluate(function(nbname) {
-        IPython._checkpoint_created = false;
+        Jupyter._checkpoint_created = false;
         require(['base/js/events'], function (events) {
             events.on('checkpoint_created.Notebook', function (evt, data) {
-                IPython._checkpoint_created = true;
+                Jupyter._checkpoint_created = true;
             });
         });
 
-        IPython.notebook.save_checkpoint();
+        Jupyter.notebook.save_checkpoint();
     }, {nbname:nbname});
 
     this.waitFor(function () {
         return this.evaluate(function(){
-            return IPython._checkpoint_created;
+            return Jupyter._checkpoint_created;
         });
     });
 
@@ -151,16 +172,16 @@ casper.notebook_test(function () {
     this.waitFor(this.kernel_running);
     this.waitFor(function() {
         return this.evaluate(function () {
-            return IPython && IPython.notebook && true;
+            return Jupyter && Jupyter.notebook && true;
         });
     });
 
     this.then(function() {
         var cell0 = this.evaluate(function() {
-            return IPython.notebook.get_cell(0);
+            return Jupyter.notebook.get_cell(0);
         });
         var cell1 = this.evaluate(function() {
-            return IPython.notebook.get_cell(1);
+            return Jupyter.notebook.get_cell(1);
         });
         this.test.assert('black_square_22.png' in cell0.attachments,
                          'cell0 has kept its attachments');