Correct function naming, actually get a test passing.

This commit is contained in:
Ian Gulliver
2016-01-08 15:27:34 -08:00
parent c9cb04d420
commit 3a4614f977
2 changed files with 10 additions and 26 deletions

View File

@@ -59,9 +59,9 @@ hogfather.PublicChat = function(cosmo, id) {
* @param {Cosmopolite} cosmo
* @return {Promise}
*/
hogfather.PublicChat.Create = function(cosmo) {
hogfather.PublicChat.create = function(cosmo) {
var id = cosmo.uuid();
return hogfather.PublicChat.Join(cosmo, id);
return hogfather.PublicChat.join(cosmo, id);
};
@@ -70,10 +70,10 @@ hogfather.PublicChat.Create = function(cosmo) {
* @param {string} id
* @return {Promise}
*/
hogfather.PublicChat.Join = function(cosmo, id) {
hogfather.PublicChat.join = function(cosmo, id) {
return new Promise(function(resolve, reject) {
var chat = new hogfather.PublicChat(cosmo, id);
chat.Start_().then(function() {
chat.start_().then(function() {
resolve(chat);
}).catch(function(err) {
reject(err);
@@ -86,7 +86,7 @@ hogfather.PublicChat.Join = function(cosmo, id) {
* @return {Promise}
* @private
*/
hogfather.PublicChat.prototype.Start_ = function() {
hogfather.PublicChat.prototype.start_ = function() {
return new Promise(function(resolve, reject) {
this.cosmo_.subscribe(this.subject_, -1).then(function() {
console.log(this.loggingPrefix_(), 'ready');
@@ -101,7 +101,7 @@ hogfather.PublicChat.prototype.Start_ = function() {
/**
* @return {Promise}
*/
hogfather.PublicChat.prototype.Shutdown = function() {
hogfather.PublicChat.prototype.shutdown = function() {
console.log(this.loggingPrefix_(), 'shutdown start');
this.cosmo_.removeEventListener('message', this.boundOnMessage_);
return this.cosmo_.unsubscribe(this.subject_);

View File

@@ -947,34 +947,18 @@ QUnit.asyncTest('sendMessage admin ACL', function(assert) {
QUnit.module('Hogfather');
QUnit.test('Construct/shutdown', function(assert) {
assert.expect(4);
var cosmo = new Cosmopolite(null, randstring());
assert.ok(true, 'new Cosmopolite() succeeds');
var hogfather = new Hogfather(cosmo, randstring());
assert.ok(true, 'new Hogfather()) succeeds');
hogfather.shutdown();
assert.ok(true, 'Hogfather.shutdown() succeeds');
cosmo.shutdown();
assert.ok(true, 'Cosmopolite.shutdown() succeeds');
});
QUnit.asyncTest('Create', function(assert) {
QUnit.asyncTest('PublicChat.create', function(assert) {
assert.expect(5);
var cosmo = new Cosmopolite(null, randstring());
assert.ok(true, 'new Cosmopolite() succeeds');
Hogfather.Create(cosmo).then(function(hogfather) {
hogfather.PublicChat.create(cosmo).then(function(chat) {
assert.ok(true, 'Hogfather.Create() succeeds');
assert.ok(hogfather, 'Hogfather.Create() returns something');
assert.ok(chat, 'Hogfather.Create() returns something');
hogfather.shutdown();
chat.shutdown();
assert.ok(true, 'Hogfather.shutdown() succeeds');
cosmo.shutdown();