Set view options before Behaviors are initialized

mxriverlynn · Apr 24, 2014 · 9cfe7a2 · 9cfe7a2
1 parent 6a29e7a
commit 9cfe7a2
Show file tree

Hide file tree

Showing 2 changed files with 42 additions and 8 deletions.
diff --git a/spec/javascripts/behaviors.spec.js b/spec/javascripts/behaviors.spec.js
@@ -134,16 +134,49 @@ describe("Behaviors", function(){
     });
   });
 
-  describe("behavior initialize", function() {
-    var Behavior = Marionette.Behavior.extend({
-      initialize: sinon.spy()
+  describe('behavior initialize', function() {
+    var View, Behavior, Obj;
+    var behaviorOptions, viewOptions;
+
+    beforeEach(function() {
+      Behavior = Marionette.Behavior.extend({
+        initialize: sinon.spy()
+      });
+
+      Obj = {
+        Tooltip: Marionette.Behavior.extend({
+          initialize: function(options, view) {
+            behaviorOptions = options;
+            viewOptions = view.options;
+          }
+        })
+      };
+
+      View = Marionette.ItemView.extend({
+        template: _.template(''),
+        behaviors: {
+          Tooltip: {
+            position: 'left'
+          }
+        }
+      });
+
+      Marionette.Behaviors.behaviorsLookup = Obj;
     });
 
     it("should call initialize when a behavior is created", function() {
       var b = new Behavior({}, {});
-
       expect(b.initialize).toHaveBeenCalled();
     });
+
+    it('should call initialize when a behavior is created', function() {
+      var view = new View({
+        words: 'big'
+      });
+
+      expect(viewOptions).toEqual(view.options);
+      expect(behaviorOptions).toEqual(View.prototype.behaviors.Tooltip);
+    });
   });
 
   describe("behavior events", function() {

diff --git a/src/marionette.view.js b/src/marionette.view.js
@@ -7,10 +7,6 @@ Marionette.View = Backbone.View.extend({
   constructor: function(options){
     _.bindAll(this, "render");
 
-    if (_.isObject(this.behaviors)) {
-      new Marionette.Behaviors(this);
-    }
-
     // this exposes view options to the view initializer
     // this is a backfill since backbone removed the assignment
     // of this.options
@@ -19,6 +15,11 @@ Marionette.View = Backbone.View.extend({
 
     // parses out the @ui DSL for events
     this.events = this.normalizeUIKeys(_.result(this, 'events'));
+
+    if (_.isObject(this.behaviors)) {
+      new Marionette.Behaviors(this);
+    }
+
     Backbone.View.prototype.constructor.apply(this, arguments);
 
     Marionette.MonitorDOMRefresh(this);