Merge pull request #278 from hybridgroup/add/respond
Add Basestar#respond method to work with events/callbacks
This commit is contained in:
commit
af5300f6cd
|
@ -33,6 +33,23 @@ Utils.subclass(Basestar, EventEmitter);
|
|||
// Returns the klass where the methods have been proxied
|
||||
Basestar.prototype.proxyMethods = Utils.proxyFunctionsToObject;
|
||||
|
||||
// Public: Triggers a callback and emits an event with provided data
|
||||
//
|
||||
// event - name of event to be triggered
|
||||
// callback - callback function to be triggered
|
||||
// ...data - additional arguments to be passed to both event/callback
|
||||
//
|
||||
// Returns nothing
|
||||
Basestar.prototype.respond = function(event, callback) {
|
||||
var args = Array.prototype.slice.call(arguments, 2);
|
||||
|
||||
this.emit.apply(this, [event].concat(args));
|
||||
|
||||
if (typeof callback === "function") {
|
||||
callback.apply(this, args);
|
||||
}
|
||||
};
|
||||
|
||||
// Public: Defines an event handler that proxies events from a source object
|
||||
// to a target object
|
||||
//
|
||||
|
|
|
@ -4,7 +4,11 @@
|
|||
var Adaptor = source("adaptor");
|
||||
|
||||
describe("Adaptor", function() {
|
||||
var adaptor = new Adaptor({ name: "adaptor" });
|
||||
var adaptor;
|
||||
|
||||
beforeEach(function() {
|
||||
adaptor = new Adaptor({ name: "adaptor" });
|
||||
});
|
||||
|
||||
describe("#constructor", function() {
|
||||
it("sets @name to the provided name", function() {
|
||||
|
|
|
@ -50,8 +50,34 @@ describe("Basestar", function() {
|
|||
});
|
||||
});
|
||||
|
||||
describe("#defineEvent", function() {
|
||||
describe("#respond", function() {
|
||||
var listener, callback, child;
|
||||
|
||||
var Child = function Child() {};
|
||||
|
||||
Utils.subclass(Child, Basestar);
|
||||
|
||||
beforeEach(function() {
|
||||
child = new Child();
|
||||
|
||||
listener = spy();
|
||||
callback = spy();
|
||||
|
||||
child.on("event", listener);
|
||||
|
||||
child.respond("event", callback, "arg1", 2, { three: true });
|
||||
});
|
||||
|
||||
it("triggers the callback with all additional arguments", function() {
|
||||
expect(callback).to.be.calledWith("arg1", 2, { three: true });
|
||||
});
|
||||
|
||||
it("emits an event with all additional arguments", function() {
|
||||
expect(listener).to.be.calledWith("arg1", 2, { three: true });
|
||||
});
|
||||
});
|
||||
|
||||
describe("#defineEvent", function() {
|
||||
var ProxyClass = function ProxyClass() {};
|
||||
|
||||
var EmitterClass = function EmitterClass(update) {
|
||||
|
|
Loading…
Reference in New Issue