Fix bug in RecordingContext

hammerlab · Sep 24, 2015 · c728de0 · c728de0
1 parent 0a606a6
commit c728de0
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/src/main/data-canvas.js b/src/main/data-canvas.js
@@ -69,7 +69,7 @@ function RecordingContext(ctx: CanvasRenderingContext2D) {
         // TODO: record current drawing style
         var args = Array.prototype.slice.call(arguments);
         calls.push([k].concat(args));
-        ctx[k].apply(ctx, arguments);
+        return ctx[k].apply(ctx, arguments);
       };
     })(k);
   }

diff --git a/src/test/data-canvas-test.js b/src/test/data-canvas-test.js
@@ -147,5 +147,13 @@ describe('data-canvas', function() {
 
       // TODO: check drawing styles
     });
+
+    it('should return values from proxied functions', function() {
+      var dtx = new dataCanvas.RecordingContext(ctx);
+      var metrics = dtx.measureText('Hello');
+
+      expect(dtx.calls).to.deep.equal([['measureText', 'Hello']]);
+      expect(metrics.width).to.be.greaterThan(0);
+    });
   });
 });