Finish tests

src/imagewidget/widget.ts

@@ -65,7 +65,6 @@ class ImageWidget extends Widget {
     let transform: string;
     transform = `scale(${value})`;
     scaleNode.style.transform = transform;
-    this.update();
   }
 
   /**
@@ -87,7 +86,7 @@ class ImageWidget extends Widget {
     if (this.isDisposed || !context.isReady) {
       return;
     }
-    let cm = this._context.contentsModel;
+    let cm = context.contentsModel;
     let content = context.model.toString();
     let src = `data:${cm.mimetype};${cm.format},${content}`;
     this.node.querySelector('img').setAttribute('src', src);

test/src/imagewidget/widget.spec.ts

@@ -12,7 +12,7 @@ import {
 } from 'phosphor/lib/core/messaging';
 
 import {
-  WidgetMessage
+  Widget, WidgetMessage
 } from 'phosphor/lib/ui/widget';
 
 import {
@@ -23,6 +23,10 @@ import {
   ImageWidget, ImageWidgetFactory
 } from '../../../lib/imagewidget';
 
+import {
+  createFileContext
+} from '../utils';
+
 
 class LogImage extends ImageWidget {
 
@@ -134,18 +138,53 @@ describe('ImageWidget', () => {
 
   describe('#scale', () => {
 
+    it('should default to 1', () => {
+      expect(widget.scale).to.be(1);
+    });
+
+    it('should be settable', () => {
+      widget.scale = 0.5;
+      expect(widget.scale).to.be(0.5);
+    });
+
   });
 
   describe('#dispose()', () => {
 
+    it('should dispose of the resources used by the widget', () => {
+      expect(widget.isDisposed).to.be(false);
+      widget.dispose();
+      expect(widget.isDisposed).to.be(true);
+      widget.dispose();
+      expect(widget.isDisposed).to.be(true);
+    });
+
   });
 
   describe('#onUpdateRequest()', () => {
 
+    it('should add the image', (done) => {
+      let img: HTMLImageElement = widget.node.querySelector('img');
+      expect(img.src).to.be('');
+      context.ready.then(() => {
+        sendMessage(widget, WidgetMessage.UpdateRequest);
+        expect(widget.methods).to.contain('onUpdateRequest');
+        expect(img.src).to.contain(IMAGE.content);
+        done();
+      }).catch(done);
+    });
+
   });
 
   describe('#onActivateRequest()', () => {
 
+    it('should focus the widget', () => {
+      Widget.attach(widget, document.body);
+      sendMessage(widget, WidgetMessage.ActivateRequest);
+      expect(widget.methods).to.contain('onActivateRequest');
+      expect(widget.node.contains(document.activeElement)).to.be(true);
+    });
+
   });
 
 });
@@ -155,6 +194,17 @@ describe('ImageWidgetFactory', () => {
 
   describe('#createNewWidget', () => {
 
+    it('should create an image widget', () => {
+      let factory = new ImageWidgetFactory({
+        name: 'Image',
+        modelName: 'base64',
+        fileExtensions: ['.png'],
+        defaultFor: ['.png']
+      });
+      let context = createFileContext(IMAGE.path);
+      expect(factory.createNew(context)).to.be.an(ImageWidget);
+    });
+
   });
 
 });