mercury-rails 0.2.3 → 0.3.1

data/spec/javascripts/mercury/mercury_spec.js.coffee

@@ -1,7 +1,8 @@
-require '/assets/mercury.js'
-
 describe "Mercury", ->
 
+  afterEach: ->
+    Mercury.config.localization.enabled = false
+
   describe "supported:", ->
 
     # this is here for documentation -- unable to test, because this is evaluated on script load
@@ -10,27 +11,60 @@ describe "Mercury", ->
     it "disallows konqueror and msie", ->
 
 
-  describe "#bind", ->
+  describe ".on", ->
 
     it "binds an event prefixed with 'mercury:' to the top window", ->
       callCount = 0
-      Mercury.bind('test', -> callCount += 1)
+      Mercury.on('test', -> callCount += 1)
       $(top).trigger("mercury:test")
       expect(callCount).toEqual(1)
 
 
-  describe "#trigger", ->
+  describe ".trigger", ->
 
     it "triggers an event prefixed with 'mercury:' on the top window", ->
       argsForCall = []
       callCount = 0
-      Mercury.bind('test', -> argsForCall[callCount] = arguments; callCount += 1)
+      Mercury.on('test', -> argsForCall[callCount] = arguments; callCount += 1)
       Mercury.trigger("test", {foo: 'bar'})
       expect(callCount).toEqual(1)
       expect(argsForCall[0][1]).toEqual({foo: 'bar'})
 
 
-  describe "#log", ->
+  describe ".notify", ->
+
+    beforeEach ->
+      @alertSpy = spyOn(window, 'alert').andCallFake(=>)
+      @i18nSpy = spyOn(Mercury, 'I18n').andCallFake(=>)
+      Mercury.notify('hello world!')
+
+    it "translates the text first by calling Mercury.I18n", ->
+      expect(@i18nSpy.callCount).toEqual(1)
+      expect(@i18nSpy.argsForCall[0]).toEqual(['hello world!'])
+
+    it "alerts the message", ->
+      expect(@alertSpy.callCount).toEqual(1)
+      expect(@i18nSpy.argsForCall[0]).toEqual(['hello world!'])
+
+
+  describe ".warn", ->
+
+    beforeEach ->
+      window.console = {warn: (-> ''), trace: (-> '')}
+      @warnSpy = spyOn(window.console, 'warn').andCallFake(=>)
+      @notifySpy = spyOn(Mercury, 'notify').andCallFake(=>)
+
+    it "calls console.warn", ->
+      Mercury.warn('message', 2)
+      expect(@warnSpy.callCount).toEqual(1)
+
+    it "calls Mercury.notify if there's no console", ->
+      window.console = null
+      Mercury.warn('message', 2)
+      expect(@notifySpy.callCount).toEqual(1)
+
+
+  describe ".log", ->
 
     beforeEach ->
       window.console = {debug: -> ''}
@@ -50,3 +84,49 @@ describe "Mercury", ->
       window.console = null
       Mercury.log(1, 2)
       expect(@debugSpy.callCount).toEqual(0)
+
+
+  describe ".locale", ->
+
+    beforeEach ->
+      @translationSource = Mercury.I18n['en'] = {
+        'original-top': 'translated-top'
+        _US_: {'original-sub': 'translated-sub'}
+      }
+
+    it "memoizes array for what the browsers language is set to (breaks with a different language set)", ->
+      Mercury.config.localization.enabled = true
+      expect(Mercury.determinedLocale).toEqual(undefined)
+      expect(Mercury.locale()).toEqual({top: @translationSource, sub: @translationSource['_US_']})
+      expect(Mercury.determinedLocale).toEqual({top: @translationSource, sub: @translationSource['_US_']})
+
+
+  describe ".I18n", ->
+
+    beforeEach ->
+      Mercury.I18n['foo'] =
+        'originaL': 'translated -- top level'
+        'uniquE': 'translated unique'
+        'complex %s with %d, and %f': 'translated %s with %d, and %f'
+        BAR:
+          'originaL': 'translated -- sub level'
+          'unique': 'undesired unique'
+      Mercury.determinedLocale = {top: Mercury.I18n['foo'], sub: Mercury.I18n['foo']['BAR']}
+
+
+    it "translates from a top level locale", ->
+      Mercury.determinedLocale.sub = {}
+      expect(Mercury.I18n('originaL')).toEqual('translated -- top level')
+
+    it "translates from a sub level locale", ->
+      expect(Mercury.I18n('originaL')).toEqual('translated -- sub level')
+
+    it "falls back from a sub level locale", ->
+      expect(Mercury.I18n('uniquE')).toEqual('translated unique')
+
+    it "falls back to no translation", ->
+      Mercury.determinedLocale = {top: {}, sub: {}}
+      expect(Mercury.I18n('original')).toEqual('original')
+
+    it "uses printf to get any number of variables into the translation", ->
+      expect(Mercury.I18n('complex %s with %d, and %f', 'string', 1, 2.4)).toEqual('translated string with 1, and 2.4')

data/spec/javascripts/mercury/modal_spec.js.coffee

@@ -1,5 +1,3 @@
-require '/assets/mercury.js'
-
 describe "Mercury.modal", ->
 
   template 'mercury/modal.html'
@@ -7,8 +5,12 @@ describe "Mercury.modal", ->
   beforeEach ->
     $.fx.off = true
     Mercury.displayRect = {fullHeight: 200}
+    Mercury.determinedLocale =
+      top: {'hello world!': 'bork! bork!'}
+      sub: {'foo': 'Bork!'}
 
   afterEach ->
+    Mercury.config.localization.enabled = false
     Mercury.modal.initialized = false
     Mercury.modal.visible = false
     $(window).unbind('mercury:refresh')
@@ -119,11 +121,6 @@ describe "Mercury.modal", ->
       expect($('#modal_container .mercury-modal').length).toEqual(1)
       expect($('#modal_container .mercury-modal-overlay').length).toEqual(1)
 
-    it "updates the title to reflect what was passed in the options", ->
-      Mercury.modal.options.title = 'title'
-      Mercury.modal.build()
-      expect($('#test .mercury-modal-title span').html()).toEqual('title')
-
 
   describe "observed events", ->
 
@@ -146,13 +143,21 @@ describe "Mercury.modal", ->
         Mercury.trigger('resize')
         expect(spy.callCount).toEqual(1)
 
-    describe "clicking on the overlay", ->
+    describe "clicking on the overlay (options.allowHideUsingOverlay = true)", ->
 
       it "calls hide", ->
+        Mercury.modal.options.allowHideUsingOverlay = true
         spy = spyOn(Mercury.modal, 'hide').andCallFake(=>)
         jasmine.simulate.click($('.mercury-modal-overlay').get(0))
         expect(spy.callCount).toEqual(1)
 
+    describe "clicking on the overlay (options.allowHideUsingOverlay = false)", ->
+
+      it "doesn't call hide", ->
+        spy = spyOn(Mercury.modal, 'hide').andCallFake(=>)
+        jasmine.simulate.click($('.mercury-modal-overlay').get(0))
+        expect(spy.callCount).toEqual(0)
+
     describe "clicking on the close button", ->
 
       it "calls hide", ->
@@ -170,6 +175,17 @@ describe "Mercury.modal", ->
         jasmine.simulate.keydown(document, {keyCode: 27})
         expect(spy.callCount).toEqual(1)
 
+    describe "ajax:beforeSend", ->
+
+      it "sets a success that will load the contents of the response", ->
+        options = {}
+        spy = spyOn(Mercury.modal, 'loadContent').andCallFake(=>)
+        Mercury.modal.element.trigger('ajax:beforeSend', [null, options])
+        expect(options.success).toBeDefined()
+        options.success('new content')
+        expect(spy.callCount).toEqual(1)
+        expect(spy.argsForCall[0]).toEqual(['new content'])
+
 
   describe "#appear", ->
 
@@ -178,7 +194,7 @@ describe "Mercury.modal", ->
       spyOn(Mercury.modal, 'update').andCallFake(=>)
       @loadSpy = spyOn(Mercury.modal, 'load').andCallFake(=>)
       @positionSpy = spyOn(Mercury.modal, 'position').andCallFake(=>)
-      Mercury.modal('/evergreen/responses/blank.html', {appendTo: $('#test')})
+      Mercury.modal('/blank.html', {appendTo: $('#test')})
 
     it "calls position", ->
       Mercury.modal.appear()
@@ -223,7 +239,7 @@ describe "Mercury.modal", ->
 
     beforeEach ->
       spyOn(Mercury.modal, 'appear').andCallFake(=>)
-      Mercury.modal('/evergreen/responses/blank.html', {appendTo: $('#test')})
+      Mercury.modal('/blank.html', {appendTo: $('#test')})
       Mercury.modal.contentPane = $()
 
     it "will keep the content element visible if asked to do so", ->
@@ -277,7 +293,7 @@ describe "Mercury.modal", ->
     beforeEach ->
       spyOn(Mercury.modal, 'appear').andCallFake(=>)
       @ajaxSpy = spyOn($, 'ajax')
-      Mercury.modal('/evergreen/responses/blank.html', {appendTo: $('#test')})
+      Mercury.modal('/blank.html', {appendTo: $('#test')})
 
     it "does nothing if there's no url", ->
       Mercury.modal.url = null
@@ -297,8 +313,8 @@ describe "Mercury.modal", ->
     describe "on a preloaded view", ->
 
       beforeEach ->
-        @setTimeoutSpy = spyOn(window, 'setTimeout').andCallFake((callback) => callback())
-        Mercury.preloadedViews = {'/evergreen/responses/blank.html': 'this is the preloaded content'}
+        @setTimeoutSpy = spyOn(window, 'setTimeout').andCallFake((timeout, callback) => callback())
+        Mercury.preloadedViews = {'/blank.html': 'this is the preloaded content'}
 
       afterEach ->
         Mercury.preloadedViews = {}
@@ -313,8 +329,10 @@ describe "Mercury.modal", ->
 
       it "makes an ajax request", ->
         @ajaxSpy.andCallFake(=>)
+        spyOn(Mercury, 'ajaxHeaders').andCallFake(=> {'X-CSRFToken': 'f00'})
         Mercury.modal.load()
         expect(@ajaxSpy.callCount).toEqual(1)
+        expect(@ajaxSpy.argsForCall[0][1]['headers']).toEqual({'X-CSRFToken': 'f00'})
 
       describe "on success", ->
 
@@ -343,7 +361,7 @@ describe "Mercury.modal", ->
           spy = spyOn(window, 'alert').andCallFake(=>)
           Mercury.modal.load()
           expect(spy.callCount).toEqual(1)
-          expect(spy.argsForCall[0]).toEqual(['Mercury was unable to load /evergreen/responses/blank.html for the modal.'])
+          expect(spy.argsForCall[0]).toEqual(['Mercury was unable to load /blank.html for the modal.'])
 
 
   describe "#loadContent", ->
@@ -351,7 +369,7 @@ describe "Mercury.modal", ->
     beforeEach ->
       spyOn(Mercury.modal, 'appear').andCallFake(=>)
       @resizeSpy = spyOn(Mercury.modal, 'resize').andCallFake(=>)
-      Mercury.modal('/evergreen/responses/blank.html', {appendTo: $('#test'), title: 'title'})
+      Mercury.modal('/blank.html', {appendTo: $('#test'), title: 'title'})
 
     it "accepts options and sets them to the instance options", ->
       Mercury.modal.loadContent('content', {title: 'title'})
@@ -378,8 +396,8 @@ describe "Mercury.modal", ->
       expect($('.mercury-modal').hasClass('loading')).toEqual(false)
 
     it "sets the content elements html to whatever was passed", ->
-      Mercury.modal.loadContent('content')
-      expect($('.mercury-modal-content').html()).toEqual('content')
+      Mercury.modal.loadContent('<span>content</span>')
+      expect($('.mercury-modal-content').html()).toEqual('<span>content</span>')
 
     it "hides the contentElement", ->
       $('.mercury-modal-content').css('display', 'block')
@@ -388,9 +406,9 @@ describe "Mercury.modal", ->
       expect($('.mercury-modal-content').css('visibility')).toEqual('hidden')
 
     it "finds the content panes and control elements in case they were added with the content", ->
-      Mercury.modal.loadContent('<div class="mercury-modal-pane-container"></div><div class="mercury-modal-controls"></div>')
-      expect(Mercury.modal.contentPane.get(0)).toEqual($('#test .mercury-modal-pane-container').get(0))
-      expect(Mercury.modal.contentControl.get(0)).toEqual($('#test .mercury-modal-controls').get(0))
+      Mercury.modal.loadContent('<div class="mercury-display-pane-container"></div><div class="mercury-display-controls"></div>')
+      expect(Mercury.modal.contentPane.get(0)).toEqual($('#test .mercury-display-pane-container').get(0))
+      expect(Mercury.modal.contentControl.get(0)).toEqual($('#test .mercury-display-controls').get(0))
 
     it "calls an afterLoad callback (if provided in options)", ->
       callCount = 0
@@ -403,6 +421,11 @@ describe "Mercury.modal", ->
       Mercury.modal.loadContent('content', {handler: 'foo'})
       expect(callCount).toEqual(1)
 
+    it "translates the content if configured", ->
+      Mercury.config.localization.enabled = true
+      Mercury.modal.loadContent('<span>foo</span>')
+      expect($('.mercury-modal-content').html()).toEqual('<span>Bork!</span>')
+
     it "calls resize", ->
       Mercury.modal.loadContent('content')
       expect(@resizeSpy.callCount).toEqual(1)
@@ -412,19 +435,26 @@ describe "Mercury.modal", ->
 
     beforeEach ->
       spyOn(Mercury.modal, 'appear').andCallFake(=>)
-      Mercury.modal('/evergreen/responses/blank.html', {appendTo: $('#test'), title: 'title'})
+      Mercury.modal('/blank.html', {appendTo: $('#test'), title: 'title'})
 
     it "sets the the title contents to what was provided in the options", ->
       Mercury.modal.options = {title: 'new title'}
       Mercury.modal.setTitle()
       expect($('.mercury-modal-title span').html()).toEqual('new title')
+      expect($('.mercury-modal-title a').css('display')).toEqual('inline')
+
+    it "hides the close button if the options.closeButton is false", ->
+      Mercury.modal.options = {title: 'new title', closeButton: false}
+      Mercury.modal.setTitle()
+      expect($('.mercury-modal-title a').css('display')).toEqual('none')
+
 
 
   describe "#reset", ->
 
     beforeEach ->
       spyOn(Mercury.modal, 'appear').andCallFake(=>)
-      Mercury.modal('/evergreen/responses/blank.html', {appendTo: $('#test'), title: 'title'})
+      Mercury.modal('/blank.html', {appendTo: $('#test'), title: 'title'})
 
     it "clears the title and content elements", ->
       $('.mercury-modal-content').html('content')
@@ -437,7 +467,7 @@ describe "Mercury.modal", ->
 
     beforeEach ->
       spyOn(Mercury.modal, 'appear').andCallFake(=>)
-      Mercury.modal('/evergreen/responses/blank.html', {appendTo: $('#test')})
+      Mercury.modal('/blank.html', {appendTo: $('#test')})
 
     it "triggers the focus:frame event", ->
       spy = spyOn(Mercury, 'trigger').andCallFake(=>)

data/spec/javascripts/mercury/modals/htmleditor_spec.js.coffee

@@ -1,5 +1,3 @@
-require '/assets/mercury.js'
-
 describe "Mercury.modalHandlers.htmlEditor", ->
 
   template 'mercury/modals/htmleditor.html'

data/spec/javascripts/mercury/modals/insertcharacter_spec.js.coffee

@@ -1,5 +1,3 @@
-require '/assets/mercury.js'
-
 describe "Mercury.modalHandlers.insertCharacter", ->
 
   template 'mercury/modals/insertcharacter.html'
@@ -7,7 +5,8 @@ describe "Mercury.modalHandlers.insertCharacter", ->
   beforeEach ->
     @modal =
       element: $('#test')
-    @modalHideSpy = spyOn(Mercury.modal, 'hide').andCallFake(=>)
+      hide: ->
+    @modalHideSpy = spyOn(@modal, 'hide').andCallFake(=>)
     Mercury.modalHandlers.insertCharacter.call(@modal)
 
   describe "clicking on a character", ->

data/spec/javascripts/mercury/modals/insertlink_spec.js.coffee

@@ -1,5 +1,3 @@
-require '/assets/mercury.js'
-
 describe "Mercury.modalHandlers.insertLink", ->
 
   template 'mercury/modals/insertlink.html'

data/spec/javascripts/mercury/modals/insertmedia_spec.js.coffee

@@ -1,11 +1,10 @@
-require '/assets/mercury.js'
-
 describe "Mercury.modalHandlers.insertMedia", ->
 
   template 'mercury/modals/insertmedia.html'
 
   beforeEach ->
     Mercury.region = null
+    spyOn(window, 'setTimeout').andCallFake((timeout, callback) => callback())
     @modal =
       element: $('#test')
       hide: ->
@@ -21,6 +20,7 @@ describe "Mercury.modalHandlers.insertMedia", ->
       jasmine.simulate.click($('#checkbox1').get(0))
       expect(spy.callCount).toEqual(1)
 
+
   describe "focusing an input", ->
 
     beforeEach ->

data/spec/javascripts/mercury/modals/insertsnippet_spec.js.coffee

@@ -1,5 +1,3 @@
-require '/assets/mercury.js'
-
 describe "Mercury.modalHandlers.insertSnippet", ->
 
   template 'mercury/modals/insertsnippet.html'

data/spec/javascripts/mercury/modals/inserttable_spec.js.coffee

@@ -1,5 +1,3 @@
-require '/assets/mercury.js'
-
 describe "Mercury.modalHandlers.insertTable", ->
 
   template 'mercury/modals/inserttable.html'
@@ -148,11 +146,11 @@ describe "Mercury.modalHandlers.insertTable", ->
       jasmine.simulate.click($('#submit').get(0))
       expect(spy.callCount).toEqual(1)
       expect(spy.argsForCall[0][0]).toEqual('action')
-      expect(spy.argsForCall[0][1]['action']).toEqual('insertHTML')
+      expect(spy.argsForCall[0][1]['action']).toEqual('insertTable')
       value = spy.argsForCall[0][1]['value']
       expect(value).toContain('border="1"')
       expect(value).toContain('cellspacing="0"')
-      expect(value).toContain('<td id="cell2">&nbsp;</td>')
+      expect(value).toContain('<td id="cell2"><br></td>')
 
     it "hides the modal", ->
       spy = spyOn(@modal, 'hide').andCallFake(=>)

data/spec/javascripts/mercury/native_extensions_spec.js.coffee

@@ -1,5 +1,3 @@
-require '/assets/mercury.js'
-
 describe "String", ->
 
   describe "#titleize", ->
@@ -15,6 +13,26 @@ describe "String", ->
       expect('rgb(255, 255, 0)'.toHex()).toEqual('#FFFF00')
 
 
+  describe "#regExpEscape", ->
+
+    it "escapes characters used in regular expressions", ->
+      expect('/.*+?|()[]{}\\'.regExpEscape()).toEqual('\\/\\.\\*\\+\\?\\|\\(\\)\\[\\]\\{\\}\\\\')
+
+
+  describe "#printf", ->
+
+    it "works something like a basic implementation of the standard sprintf", ->
+      expect('int %d'.printf(2.1)).toEqual('int 2')
+      expect('int%d'.printf(2.1)).toEqual('int2')
+      expect('%d-int'.printf(2.1)).toEqual('2-int')
+      expect('%f float'.printf(2.1)).toEqual('2.1 float')
+      expect('%s string'.printf(2.1)).toEqual('2.1 string')
+      expect('%% a'.printf(2.1)).toEqual('% a')
+      expect('a %% b'.printf()).toEqual('a % b')
+      expect('a %% %d'.printf(2.1)).toEqual('a % 2')
+      expect('%d\n%s'.printf(2.1, 'string')).toEqual('2\nstring')
+
+
 describe "Number", ->
 
   describe "#toHex", ->

data/spec/javascripts/mercury/page_editor_spec.js.coffee

@@ -1,11 +1,9 @@
-require '/assets/mercury.js'
-
 describe "Mercury.PageEditor", ->
 
   template 'mercury/page_editor.html'
 
   beforeEach ->
-    Mercury.config.regionClass = 'custom-region-class'
+    Mercury.config.regions.className = 'custom-region-class'
 
   afterEach ->
     @pageEditor = null
@@ -15,6 +13,8 @@ describe "Mercury.PageEditor", ->
     $(window).unbind('mercury:focus:frame')
     $(window).unbind('mercury:focus:window')
     $(window).unbind('mercury:toggle:interface')
+    $(window).unbind('mercury:reinitialize')
+    $(window).unbind('mercury:mode')
     $(window).unbind('mercury:action')
     $(window).unbind('resize')
     $(document).unbind('mousedown')
@@ -24,13 +24,6 @@ describe "Mercury.PageEditor", ->
     beforeEach ->
       @initializeInterfaceSpy = spyOn(Mercury.PageEditor.prototype, 'initializeInterface').andCallFake(=>)
 
-    it "throws an error if it's not supported", ->
-      Mercury.supported = false
-      expect(=>
-        new Mercury.PageEditor()
-      ).toThrow('Mercury.PageEditor is unsupported in this client. Supported browsers are chrome 10+, firefix 4+, and safari 5+.')
-      Mercury.supported = true
-
     it "throws an error if it's already instantiated", ->
       window.mercuryInstance = true
       expect(=>
@@ -73,6 +66,7 @@ describe "Mercury.PageEditor", ->
   describe "#initializeInterface", ->
 
     beforeEach ->
+      @resizeSpy = spyOn(Mercury.PageEditor.prototype, 'resize').andCallFake(=>)
       Mercury.Toolbar = -> {toolbar: true}
       Mercury.Statusbar = -> {statusbar: true}
       @iframeSrcSpy = spyOn(Mercury.PageEditor.prototype, 'iframeSrc').andCallFake(=> '/foo')
@@ -98,6 +92,10 @@ describe "Mercury.PageEditor", ->
       @pageEditor = new Mercury.PageEditor('', {appendTo: $('#test')})
       expect(@pageEditor.statusbar).toEqual({statusbar: true})
 
+    it "calls resize", ->
+      @pageEditor = new Mercury.PageEditor('', {appendTo: $('#test')})
+      expect(@resizeSpy.callCount).toEqual(1)
+
 
   describe "#initializeFrame", ->
 
@@ -149,7 +147,7 @@ describe "Mercury.PageEditor", ->
     it "calls resize", ->
       @finalizeInterfaceSpy.andCallFake(=>)
       @pageEditor.initializeFrame()
-      expect(@resizeSpy.callCount).toEqual(1)
+      expect(@resizeSpy.callCount).toEqual(2)
 
     it "calls initializeRegions", ->
       @finalizeInterfaceSpy.andCallFake(=>)
@@ -209,22 +207,30 @@ describe "Mercury.PageEditor", ->
   describe "#initializeRegions", ->
 
     beforeEach ->
+      @resizeSpy = spyOn(Mercury.PageEditor.prototype, 'resize').andCallFake(=>)
       Mercury.PageEditor.prototype.initializeFrame = ->
       @pageEditor = new Mercury.PageEditor('', {appendTo: $('#test')})
       @pageEditor.document = $(document)
-      @buildRegionSpy = spyOn(Mercury.PageEditor.prototype, 'buildRegion').andCallFake(=>)
 
     it "it calls buildRegion for all the regions found in a document", ->
+      spy = spyOn(Mercury.PageEditor.prototype, 'buildRegion').andCallFake(=>)
       @pageEditor.initializeRegions()
-      expect(@buildRegionSpy.callCount).toEqual(3)
+      expect(spy.callCount).toEqual(4)
 
     it "focuses the first region", ->
-      firstFocusCalled = false
-      @pageEditor.regions = [{focus: => firstFocusCalled = true}, {}, {}]
+      regionIndex = 0
+      regionFocusCall = null;
+      spy = spyOn(Mercury.PageEditor.prototype, 'buildRegion').andCallFake ->
+        @regions.push({focus: ->
+          regionIndex += 1
+          regionFocusCall = regionIndex
+        })
       @pageEditor.initializeRegions()
-      expect(firstFocusCalled).toEqual(true)
+      expect(spy.callCount).toEqual(4)
+      expect(regionFocusCall).toEqual(1)
 
     it "doesn't focus the first region if it's not supposed to be visible", ->
+      spy = spyOn(Mercury.PageEditor.prototype, 'buildRegion').andCallFake(=>)
       firstFocusCalled = false
       @pageEditor.regions = [{focus: => firstFocusCalled = true}, {}, {}]
       @pageEditor.options.visible = false
@@ -233,10 +239,19 @@ describe "Mercury.PageEditor", ->
 
 
   describe "#buildRegion", ->
+#    it "throws an error if it's not supported", ->
+#      Mercury.supported = false
+#      expect(=>
+#                new Mercury.PageEditor()
+#      ).toThrow('Mercury.PageEditor is unsupported in this client. Supported browsers are chrome 10+, firefix 4+, and safari 5+.')
+#      Mercury.supported = true
+#
 
     beforeEach ->
+      @resizeSpy = spyOn(Mercury.PageEditor.prototype, 'resize').andCallFake(=>)
       Mercury.PageEditor.prototype.initializeFrame = ->
       Mercury.Regions.Editable = -> {region: true}
+      Mercury.Regions.Editable.supported = true
       @pageEditor = new Mercury.PageEditor('', {appendTo: $('#test')})
 
     it "instantiates the region and pushes it into the regions array", ->
@@ -244,18 +259,41 @@ describe "Mercury.PageEditor", ->
       expect(@pageEditor.regions.length).toEqual(1)
       expect(@pageEditor.regions[0]).toEqual({region: true})
 
-    it "alerts on errors", ->
-      Mercury.debug = false
-      Mercury.Regions.Editable = -> throw('error!')
-      spy = spyOn(window, 'alert').andCallFake(=>)
+    it "throws an exception when there's no data-type attribute", ->
+      expect(=> @pageEditor.buildRegion($('#region4'))).toThrow('Region type is malformed, no data-type provided, or "Unknown" is unknown for the "region4" region.')
+      expect(=> @pageEditor.buildRegion($('#region4'))).toThrow('Region type is malformed, no data-type provided, or "Unknown" is unknown for the "region4" region.')
+
+    it "throws an exception when the data-type isn't known", ->
+      expect(=> @pageEditor.buildRegion($('#region4'))).toThrow('Region type is malformed, no data-type provided, or "Unknown" is unknown for the "region4" region.')
+      $('#region4').attr('data-type', 'foo')
+      expect(=> @pageEditor.buildRegion($('#region4'))).toThrow('Region type is malformed, no data-type provided, or "Foo" is unknown for the "region4" region.')
+
+    it "doesn't re-instantiate the region if the element's already initialized", ->
+      $('#region2').data('region', {foo: 'bar'})
       @pageEditor.buildRegion($('#region2'))
-      expect(spy.callCount).toEqual(1)
-      expect(spy.argsForCall[0]).toEqual(['Region type is malformed, no data-type provided, or "Editable" is unknown.'])
+      expect(@pageEditor.regions.length).toEqual(1)
+      expect(@pageEditor.regions[0]).toEqual({foo: 'bar'})
+
+    it "calls togglePreview on the region if in preview mode", ->
+      callCount = 0
+      Mercury.Regions.Editable = -> {region: true, togglePreview: -> callCount += 1 }
+      Mercury.Regions.Editable.supported = true
+      @pageEditor.previewing = true
+      @pageEditor.buildRegion($('#region2'))
+      expect(callCount).toEqual(1)
+
+    it "doesn't call togglePreview if not in preview mode", ->
+      callCount = 0
+      Mercury.Regions.Editable = -> {region: true, togglePreview: -> callCount += 1 }
+      Mercury.Regions.Editable.supported = true
+      @pageEditor.buildRegion($('#region2'))
+      expect(callCount).toEqual(0)
 
 
   describe "#finalizeInterface", ->
 
     beforeEach ->
+      @resizeSpy = spyOn(Mercury.PageEditor.prototype, 'resize').andCallFake(=>)
       Mercury.PageEditor.prototype.initializeFrame = ->
       Mercury.SnippetToolbar = -> {snippetToolbar: true}
       @pageEditor = new Mercury.PageEditor('', {appendTo: $('#test')})
@@ -292,7 +330,7 @@ describe "Mercury.PageEditor", ->
 
       it "calls initializeFrame", ->
         @initializeFrameSpy = spyOn(Mercury.PageEditor.prototype, 'initializeFrame').andCallFake(=>)
-        @setTimeoutSpy = spyOn(window, 'setTimeout').andCallFake((callback) -> callback())
+        @setTimeoutSpy = spyOn(window, 'setTimeout').andCallFake((timeout, callback) -> callback())
         Mercury.trigger('initialize:frame')
         expect(@initializeFrameSpy.callCount).toEqual(1)
         expect(@setTimeoutSpy.callCount).toEqual(1)
@@ -310,7 +348,7 @@ describe "Mercury.PageEditor", ->
       it "calls focus on a focusable element", ->
         callCount = 0
         @pageEditor.focusableElement = {focus: -> callCount += 1}
-        @setTimeoutSpy = spyOn(window, 'setTimeout').andCallFake((callback) -> callback())
+        @setTimeoutSpy = spyOn(window, 'setTimeout').andCallFake((timeout, callback) -> callback())
         Mercury.trigger('focus:window')
         expect(callCount).toEqual(1)
 
@@ -321,6 +359,23 @@ describe "Mercury.PageEditor", ->
         Mercury.trigger('toggle:interface')
         expect(spy.callCount).toEqual(1)
 
+    describe "custom event: mode", ->
+
+      it "toggles preview mode if needed", ->
+        Mercury.trigger('mode', {mode: 'preview'})
+        expect(@pageEditor.previewing).toEqual(true)
+        Mercury.trigger('mode', {mode: 'preview'})
+        expect(@pageEditor.previewing).toEqual(false)
+        Mercury.trigger('mode', {mode: 'foo'})
+        expect(@pageEditor.previewing).toEqual(false)
+
+    describe "custom event: reinitialize", ->
+
+      it "calls initializeRegions", ->
+        spy = spyOn(Mercury.PageEditor.prototype, 'initializeRegions').andCallFake(=>)
+        Mercury.trigger('reinitialize')
+        expect(spy.callCount).toEqual(1)
+
     describe "custom event: action", ->
 
       it "calls save if the action was save", ->
@@ -350,16 +405,16 @@ describe "Mercury.PageEditor", ->
 
     describe "window resize", ->
 
+      # untestable
       it "calls resize", ->
-        # untestable
         #spy = spyOn(Mercury.PageEditor.prototype, 'resize').andCallFake(=>)
         #resizeTo($(window).width() - 1, $(window).height() - 1)
         #expect(spy.callCount).toEqual(1)
 
     describe "onbeforeunload", ->
 
+      # untestable
       it "calls Mercury.beforeUnload", ->
-        # untestable
         #spy = spyOn(Mercury, 'beforeUnload').andCallFake(=>)
         #window.onbeforeunload()
         #expect(spy.callCount).toEqual(1)
@@ -373,8 +428,18 @@ describe "Mercury.PageEditor", ->
       spec.toolbarHideCallCount = 0
       spec.statusbarShowCallCount = 0
       spec.statusbarHideCallCount = 0
-      Mercury.Toolbar = -> {toolbar: true, height: (-> 100), show: (=> spec.toolbarShowCallCount += 1), hide: (=> spec.toolbarHideCallCount += 1)}
-      Mercury.Statusbar = -> {statusbar: true, top: (-> 500), show: (=> spec.statusbarShowCallCount += 1), hide: (=> spec.statusbarHideCallCount += 1)}
+      Mercury.Toolbar = -> {
+        toolbar: true,
+        height: (-> 100),
+        show: (=> spec.toolbarShowCallCount += 1),
+        hide: (=> spec.toolbarHideCallCount += 1)
+      }
+      Mercury.Statusbar = -> {
+        statusbar: true,
+        top: (-> 500),
+        show: (=> spec.statusbarShowCallCount += 1),
+        hide: (=> spec.statusbarHideCallCount += 1)
+      }
       Mercury.PageEditor.prototype.initializeFrame = ->
       @pageEditor = new Mercury.PageEditor('', {appendTo: $('#test')})
 
@@ -395,6 +460,14 @@ describe "Mercury.PageEditor", ->
       beforeEach ->
         @pageEditor.visible = true
 
+      it "triggers an extra preview mode event if currently previewing", ->
+        @pageEditor.previewing = true
+        spy = spyOn(Mercury, 'trigger').andCallFake(=>)
+        @pageEditor.toggleInterface()
+        expect(spy.callCount).toEqual(3)
+        expect(spy.argsForCall[0]).toEqual(['mode', {mode: 'preview'}])
+        expect(spy.argsForCall[1]).toEqual(['mode', {mode: 'preview'}])
+
       it "sets visible to false", ->
         @pageEditor.toggleInterface()
         expect(@pageEditor.visible).toEqual(false)
@@ -457,6 +530,16 @@ describe "Mercury.PageEditor", ->
     it "takes the location and removes the /editor", ->
       expect(@pageEditor.iframeSrc('http://foo.com/editor/path')).toEqual('http://foo.com/path')
 
+    it "uses the configured url regex to remove the editor", ->
+      original = Mercury.config.editorUrlRegEx
+      Mercury.config.editorUrlRegEx = /([http|https]:\/\/[^\/]*)\/(.*)\/edit\/?/i
+      expect(@pageEditor.iframeSrc('http://foo.com/path/edit')).toEqual('http://foo.com/path')
+      Mercury.config.editorUrlRegEx = original
+
+    it "adds query params", ->
+      expect(@pageEditor.iframeSrc('http://foo.com/editor/path', true)).toEqual('http://foo.com/path?mercury_frame=true')
+      expect(@pageEditor.iframeSrc('http://foo.com/editor/path?something=true', true)).toEqual('http://foo.com/path?something=true&mercury_frame=true')
+
 
   describe "#hijackLinksAndForms", ->
 
@@ -474,12 +557,12 @@ describe "Mercury.PageEditor", ->
       @pageEditor.hijackLinksAndForms()
       expect($('#anchor1').attr('target')).toEqual('_top')
       expect($('#anchor2').attr('target')).toEqual('_blank')
-      expect($('#anchor3').attr('target')).toEqual('_top')
-      expect($('#anchor4').attr('target')).toEqual('_top')
+      expect($('#anchor3').attr('target')).toEqual('_parent')
+      expect($('#anchor4').attr('target')).toEqual('_parent')
       expect($('#form1').attr('target')).toEqual('_top')
       expect($('#form2').attr('target')).toEqual('_blank')
-      expect($('#form3').attr('target')).toEqual('_top')
-      expect($('#form4').attr('target')).toEqual('_top')
+      expect($('#form3').attr('target')).toEqual('_parent')
+      expect($('#form4').attr('target')).toEqual('_parent')
 
     it "ignores links in regions", ->
       @pageEditor.hijackLinksAndForms()
@@ -590,10 +673,10 @@ describe "Mercury.PageEditor", ->
         @pageEditor.save()
         expect(@ajaxSpy.argsForCall[0][1]['data']).toEqual({content: {region1: 'region1'}})
 
-      it "sets headers by calling #saveHeaders", ->
+      it "sets headers by calling Mercury.ajaxHeaders", ->
         @ajaxSpy.andCallFake(=>)
         spyOn(Mercury.PageEditor.prototype, 'serialize').andCallFake(=> {region1: 'region1'})
-        spy = spyOn(Mercury.PageEditor.prototype, 'saveHeaders').andCallFake(=> {'X-CSRFToken': 'f00'})
+        spyOn(Mercury, 'ajaxHeaders').andCallFake(=> {'X-CSRFToken': 'f00'})
         @pageEditor.save()
         expect(@ajaxSpy.argsForCall[0][1]['headers']).toEqual({'X-CSRFToken': 'f00'})
 
@@ -613,6 +696,18 @@ describe "Mercury.PageEditor", ->
           @pageEditor.save()
           expect(Mercury.changes).toEqual(false)
 
+        it "calls a callback if one was provided", ->
+          callback = -> callback.callCount += 1
+          callback.callCount = 0;
+          @pageEditor.save(callback)
+          expect(callback.callCount).toEqual(1)
+
+        it "fires an event", ->
+          spy = spyOn(Mercury, 'trigger').andCallFake(=>)
+          @pageEditor.save()
+          expect(spy.callCount).toEqual(1)
+          expect(spy.argsForCall[0]).toEqual(['saved'])
+
       describe "on failed ajax request", ->
 
         beforeEach ->
@@ -652,4 +747,4 @@ describe "Mercury.PageEditor", ->
 
     it "returns an object with the region name, and it's serialized value", ->
       ret = @pageEditor.serialize()
-      expect(ret).toEqual({region1: 'region1', region2: 'region2'})
+      expect(ret).toEqual({region1: 'region1', region2: 'region2'})