upjs-rails 0.2.1 → 0.2.2

data/spec_app/spec/javascripts/up/navigation_spec.js.coffee

@@ -1,27 +1,63 @@
 describe 'up.navigation', ->
+  
+  describe 'unobtrusive behavior', ->
 
-  it 'marks links as .up-current iff they link to the current URL', ->
-    spyOn(up.browser, 'url').and.returnValue('/foo')
-    $currentLink = up.ready(affix('a[href="/foo"]'))
-    $otherLink = up.ready(affix('a[href="/bar"]'))
-    expect($currentLink).toHaveClass('up-current')
-    expect($otherLink).not.toHaveClass('up-current')
+    it 'marks a link as .up-current iff it links to the current URL', ->
+      spyOn(up.browser, 'url').and.returnValue('/foo')
+      $currentLink = up.ready(affix('a[href="/foo"]'))
+      $otherLink = up.ready(affix('a[href="/bar"]'))
+      expect($currentLink).toHaveClass('up-current')
+      expect($otherLink).not.toHaveClass('up-current')
+      
+    it 'marks a link as .up-current if it links to the current URL, but is missing a trailing slash', ->
+      $link = affix('a[href="/foo"][up-target=".main"]')
+      affix('.main')
+      jasmine.Ajax.install()
+      $link.click()
+      jasmine.Ajax.requests.mostRecent().respondWith
+        status: 200
+        contentType: 'text/html'
+        responseHeaders: { 'X-Up-Current-Location': '/foo/' }
+        responseText: '<div class="main">new-text</div>'
+      expect($link).toHaveClass('up-current')
     
-  it 'changes .up-current marks as the URL changes'
-    
-  it 'marks clicked linked as .up-active until the request finishes', ->
-    $link = affix('a[href="/foo"][up-target=".main"]')
-    affix('.main')
-    jasmine.Ajax.install()
-    $link.click()
-    expect($link).toHaveClass('up-active')
-    jasmine.Ajax.requests.mostRecent().respondWith
-      status: 200
-      contentType: '/text/html'
-      responseText: '<div class="main">new-text</div>'
-    expect($link).not.toHaveClass('up-active')
-    expect($link).toHaveClass('up-current')
-    
-
-  it 'prefers to mark an enclosing [up-follow] click area'
- 
+    it 'marks a link as .up-current if it links to the current URL, but has an extra trailing slash', ->
+      $link = affix('a[href="/foo/"][up-target=".main"]')
+      affix('.main')
+      jasmine.Ajax.install()
+      $link.click()
+      jasmine.Ajax.requests.mostRecent().respondWith
+        status: 200
+        contentType: 'text/html'
+        responseHeaders: { 'X-Up-Current-Location': '/foo' }
+        responseText: '<div class="main">new-text</div>'
+      expect($link).toHaveClass('up-current')
+      
+    it 'changes .up-current marks as the URL changes'
+      
+    it 'marks clicked linked as .up-active until the request finishes', ->
+      $link = affix('a[href="/foo"][up-target=".main"]')
+      affix('.main')
+      jasmine.Ajax.install()
+      $link.click()
+      expect($link).toHaveClass('up-active')
+      jasmine.Ajax.requests.mostRecent().respondWith
+        status: 200
+        contentType: 'text/html'
+        responseText: '<div class="main">new-text</div>'
+      expect($link).not.toHaveClass('up-active')
+      expect($link).toHaveClass('up-current')
+  
+    it 'prefers to mark an enclosing [up-follow] click area', ->
+      $area = affix('div[up-follow] a[href="/foo"][up-target=".main"]')
+      $link = $area.find('a')
+      affix('.main')
+      jasmine.Ajax.install()
+      $link.click()
+      expect($area).toHaveClass('up-active')
+      jasmine.Ajax.requests.mostRecent().respondWith
+        status: 200
+        contentType: 'text/html'
+        responseText: '<div class="main">new-text</div>'
+      expect($area).toHaveClass('up-current')
+      

data/spec_app/spec/javascripts/up/popup_spec.js.coffee

@@ -0,0 +1,30 @@
+describe 'up.popup', ->
+
+  describe 'Javascript functions', ->
+
+    describe 'up.popup.defaults', ->
+
+      it 'should have tests'
+
+    describe 'up.popup.open', ->
+
+      it 'should have tests'
+
+    describe 'up.popup.close', ->
+
+      it 'should have tests'
+
+    describe 'up.popup.source', ->
+
+      it 'should have tests'
+
+  describe 'unobtrusive behavior', ->
+
+    describe 'a[up-popup]', ->
+
+      it 'should have tests'
+
+    describe '[up-close]', ->
+
+      it 'should have tests'
+      

data/spec_app/spec/javascripts/up/tooltip_spec.js.coffee

@@ -0,0 +1,46 @@
+describe 'up.tooltip', ->
+  
+  describe 'Javascript functions', ->
+    
+    describe 'up.tooltip.open', ->
+      
+      it 'opens a tooltop above the center of the given element', ->
+        $link = affix('span').text('button label')
+        $link.css(
+          position: 'absolute'
+          left: '200px'
+          top: '200px'
+          width: '50px'
+          height: '50px'
+        )
+        up.tooltip.open($link, html: 'tooltip text')
+        $tooltip = $('.up-tooltip')
+        expect($tooltip).toBeInDOM()
+        expect($tooltip).toHaveText('tooltip text')
+        tooltipBox = up.util.measure($tooltip, relative: true)
+        linkBox = up.util.measure($link, relative: true)
+        expect(tooltipBox.top).toBeCloseTo(linkBox.top - tooltipBox.height, 15)
+        expect(tooltipBox.left).toBeCloseTo(linkBox.left + 0.5 * (linkBox.width - tooltipBox.width), 15)
+        
+      it 'allows HTML for the tooltip text'
+        
+      it 'closes an existing tooltip'
+      
+      describe 'with origin option', ->
+        
+        it 'anchors the tooltip at a different edge of the element'
+      
+    describe 'up.tooltip.close', ->
+
+      it 'should have tests'
+  
+  describe 'unobtrusive behavior', ->
+    
+    describe '[up-tooltip]', ->
+
+      it 'should have tests'
+      
+    describe 'body', ->
+      
+      it 'closes a tooltip when clicked'
+      

data/spec_app/spec/javascripts/up/util_spec.js.coffee

@@ -1,27 +1,29 @@
 describe 'up.util', ->
+  
+  describe 'Javascript functions', ->
 
-  describe '.isBlank', ->
-
-    it 'returns false for false', ->
-      expect(up.util.isBlank(false)).toBe(false)
-      
-    it 'returns false for true', ->
-      expect(up.util.isBlank(true)).toBe(false)
-
-    it 'returns true for null', ->
-      expect(up.util.isBlank(null)).toBe(true)
-      
-    it 'returns true for undefined', ->
-      expect(up.util.isBlank(undefined)).toBe(true)
-      
-    it 'returns true for an empty String', ->
-      expect(up.util.isBlank('')).toBe(true)
-      
-    it 'returns false for a String with at least one character', ->
-      expect(up.util.isBlank('string')).toBe(false)
-      
-    it 'returns true for an empty array', ->
-      expect(up.util.isBlank([])).toBe(true)
-      
-    it 'returns false for an array with at least one element', ->
-      expect(up.util.isBlank(['element'])).toBe(false)
+    describe '.isBlank', ->
+  
+      it 'returns false for false', ->
+        expect(up.util.isBlank(false)).toBe(false)
+        
+      it 'returns false for true', ->
+        expect(up.util.isBlank(true)).toBe(false)
+  
+      it 'returns true for null', ->
+        expect(up.util.isBlank(null)).toBe(true)
+        
+      it 'returns true for undefined', ->
+        expect(up.util.isBlank(undefined)).toBe(true)
+        
+      it 'returns true for an empty String', ->
+        expect(up.util.isBlank('')).toBe(true)
+        
+      it 'returns false for a String with at least one character', ->
+        expect(up.util.isBlank('string')).toBe(false)
+        
+      it 'returns true for an empty array', ->
+        expect(up.util.isBlank([])).toBe(true)
+        
+      it 'returns false for an array with at least one element', ->
+        expect(up.util.isBlank(['element'])).toBe(false)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: upjs-rails
 version: !ruby/object:Gem::Version
-  version: 0.2.1
+  version: 0.2.2
 platform: ruby
 authors:
 - Henning Koch
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-02-05 00:00:00.000000000 Z
+date: 2015-02-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -140,8 +140,8 @@ files:
 - lib/assets/stylesheets/up/popup.css.sass
 - lib/assets/stylesheets/up/tooltip.css.sass
 - lib/upjs-rails.rb
+- lib/upjs/rails/current_location.rb
 - lib/upjs/rails/engine.rb
-- lib/upjs/rails/redirection.rb
 - lib/upjs/rails/request.rb
 - lib/upjs/rails/version.rb
 - spec_app/.firefox-version
@@ -158,6 +158,7 @@ files:
 - spec_app/app/assets/stylesheets/blocks/controls.css.sass
 - spec_app/app/assets/stylesheets/blocks/menu.css.sass
 - spec_app/app/assets/stylesheets/blocks/panel.css.sass
+- spec_app/app/assets/stylesheets/jasmine_specs.css
 - spec_app/app/controllers/application_controller.rb
 - spec_app/app/controllers/cards_controller.rb
 - spec_app/app/controllers/concerns/.keep
@@ -224,9 +225,18 @@ files:
 - spec_app/spec/javascripts/helpers/reset_path.js.coffee
 - spec_app/spec/javascripts/helpers/reset_up.js.coffee
 - spec_app/spec/javascripts/support/jasmine.yml
+- spec_app/spec/javascripts/up/bus_spec.js.coffee
 - spec_app/spec/javascripts/up/flow_spec.js.coffee
+- spec_app/spec/javascripts/up/form_spec.js.coffee
+- spec_app/spec/javascripts/up/history_spec.js.coffee
+- spec_app/spec/javascripts/up/link_spec.js.coffee
+- spec_app/spec/javascripts/up/magic_spec.js.coffee
 - spec_app/spec/javascripts/up/marker_spec.js.coffee
+- spec_app/spec/javascripts/up/modal_spec.js.coffee
+- spec_app/spec/javascripts/up/motion_spec.js.coffee
 - spec_app/spec/javascripts/up/navigation_spec.js.coffee
+- spec_app/spec/javascripts/up/popup_spec.js.coffee
+- spec_app/spec/javascripts/up/tooltip_spec.js.coffee
 - spec_app/spec/javascripts/up/util_spec.js.coffee
 - spec_app/test/controllers/.keep
 - spec_app/test/fixtures/.keep

data/lib/upjs/rails/redirection.rb

@@ -1,26 +0,0 @@
-module Upjs
-  module Rails
-    module Redirection
-
-      def redirect_to(*args)
-        super
-        flash[:redirected_to] = self.location if request.up?
-      end
-
-      def self.included(base)
-        base.before_filter :extract_redirect_location
-      end
-
-      private
-
-      def extract_redirect_location
-        if location = flash[:redirected_to]
-          headers['X-Up-Previous-Redirect-Location'] = location
-        end
-      end
-
-      ActionController::Base.include(self)
-
-    end
-  end
-end