Validates the service names on user entry

test/test_ghost_inspector.js

 /*
 This file is part of the Juju GUI, which lets users view and manage Juju
 environments within a graphical interface (https://launchpad.net/juju-gui).
 Copyright (C) 2013 Canonical Ltd.
 
 This program is free software: you can redistribute it and/or modify it under
 the terms of the GNU Affero General Public License version 3, as published by
 the Free Software Foundation.
 
 This program is distributed in the hope that it will be useful, but WITHOUT
@@ skipping 40 matching lines @@
   });
 
   afterEach(function(done) {
     cleanIconHelper();
     if (view) {
       if (inspector) {
         view.setInspector(inspector, true);
       }
       view.destroy();
     }
+    db.destroy();
     env.after('destroy', function() { done(); });
     env.destroy();
     container.remove(true);
     window.flags = {};
   });
 
   var setUpInspector = function(data) {
     if (!data) {
       data = charmData;
     }
@@ skipping 60 matching lines @@
       db.services.add({id: 'mediawiki', charm: 'cs:precise/mediawiki'});
       inspector = setUpInspector();
       var serviceNameInput = Y.one('input[name=service-name]');
       // This is usually fired by an event.  The event simulation is broken as
       // of this writing, and we can do more of a unit test this way.
       inspector.updateGhostName(
           {newVal: 'mediawiki42', currentTarget: serviceNameInput});
       assert.isFalse(serviceNameInput.hasClass('invalid'));
       assert.isTrue(serviceNameInput.hasClass('valid'));
     });
+
+    it('won\'t let you deploy a service with an invalid name', function() {
+      db.services.add({id: 'mediawiki', charm: 'cs:precise/mediawiki'});
+      inspector = setUpInspector();
+      var newName = 'foo-2',
+          serviceNameInput = Y.one('input[name=service-name]');

[rharding, 2013/12/03 17:06:27]

do we have to do a Y.one? Isn't there a container we can get at?

[jeff.pihach, 2013/12/03 17:16:32]

copied this from the previous tests.

+      // This is usually fired by an event.  The event simulation is broken as
+      // of this writing, and we can do more of a unit test this way.

[rharding, 2013/12/03 17:06:27]

:(

+      serviceNameInput.set('value', newName);
+      inspector.updateGhostName(
+          {newVal: newName, currentTarget: serviceNameInput});
+      assert.isTrue(serviceNameInput.hasClass('invalid'));
+      assert.isFalse(serviceNameInput.hasClass('valid'));
+      inspector.options.env.deploy = function() {
+        assert.fail('The method should exit before this function is called.');
+      };
+      assert.isFalse(inspector.deployCharm());
+    });
   });
 
   it('updates the service name in the topology when changed in the inspector',
       function(done) {
         // XXX (Jeff) YUI's simulate can't properly simulate focus or blur in
         // IE10 as of 3.9.1, 3.11 https://github.com/yui/yui3/issues/489
         if (Y.UA.ie === 10) {
           done();
         }
         inspector = setUpInspector();
@@ skipping 388 matching lines @@
       inspector = setUpInspector();
       var events = inspector.viewletManager.events;
       assert.equal(
           typeof events['.destroy-service-trigger span'].click, 'function');
       assert.equal(typeof events['.initiate-destroy'].click, 'function');
       assert.equal(typeof events['.cancel-destroy'].click, 'function');
     });
   });
 
 });