navigation_spec.js.coffee in unpoly-rails-0.24.0

- old
+ new
@@ -1,132 +1,168 @@
 describe 'up.navigation', ->
-  
+
+  u = up.util
+
+  beforeEach ->
+    up.modal.config.openAnimation = 'none'
+    up.modal.config.closeAnimation = 'none'
+
   describe 'unobtrusive behavior', ->
 
-    it 'marks a link as .up-current if it links to the current URL', ->
-      spyOn(up.browser, 'url').and.returnValue('/foo')
-      $currentLink = up.hello(affix('a[href="/foo"]'))
-      $otherLink = up.hello(affix('a[href="/bar"]'))
-      expect($currentLink).toHaveClass('up-current')
-      expect($otherLink).not.toHaveClass('up-current')
-      
-    it 'marks any link as .up-current if its up-href attribute matches the current URL', ->
-      spyOn(up.browser, 'url').and.returnValue('/foo')
-      $currentLink = up.hello(affix('span[up-href="/foo"]'))
-      $otherLink = up.hello(affix('span[up-href="/bar"]'))
-      expect($currentLink).toHaveClass('up-current')
-      expect($otherLink).not.toHaveClass('up-current')
+    describe '.up-current', ->
 
-    it 'marks any link as .up-current if any of its space-separated up-alias values matches the current URL', ->
-      spyOn(up.browser, 'url').and.returnValue('/foo')
-      $currentLink = up.hello(affix('span[up-alias="/aaa /foo /bbb"]'))
-      $otherLink = up.hello(affix('span[up-alias="/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', ->
+        spyOn(up.browser, 'url').and.returnValue('/foo')
+        $currentLink = up.hello(affix('a[href="/foo"]'))
+        $otherLink = up.hello(affix('a[href="/bar"]'))
+        expect($currentLink).toHaveClass('up-current')
+        expect($otherLink).not.toHaveClass('up-current')
 
-    it 'does not throw if the current location does not match an up-alias wildcard (bugfix)', ->
-      inserter = -> up.hello(affix('a[up-alias="/qqqq*"]'))
-      expect(inserter).not.toThrow()
+      it 'marks any link as .up-current if its up-href attribute matches the current URL', ->
+        spyOn(up.browser, 'url').and.returnValue('/foo')
+        $currentLink = up.hello(affix('span[up-href="/foo"]'))
+        $otherLink = up.hello(affix('span[up-href="/bar"]'))
+        expect($currentLink).toHaveClass('up-current')
+        expect($otherLink).not.toHaveClass('up-current')
 
-    it 'does not highlight a link to "#" (commonly used for JS-only buttons)', ->
-      $link = up.hello(affix('a[href="#"]'))
-      expect($link).not.toHaveClass('up-current')
+      it 'marks any link as .up-current if any of its space-separated up-alias values matches the current URL', ->
+        spyOn(up.browser, 'url').and.returnValue('/foo')
+        $currentLink = up.hello(affix('a[href="/x"][up-alias="/aaa /foo /bbb"]'))
+        $otherLink = up.hello(affix('a[href="/y"][up-alias="/bar"]'))
+        expect($currentLink).toHaveClass('up-current')
+        expect($otherLink).not.toHaveClass('up-current')
 
-    it 'marks URL prefixes as .up-current if an up-alias value ends in *', ->
-      spyOn(up.browser, 'url').and.returnValue('/foo/123')
-      $currentLink = up.hello(affix('span[up-alias="/aaa /foo/* /bbb"]'))
-      $otherLink = up.hello(affix('span[up-alias="/bar"]'))
-      expect($currentLink).toHaveClass('up-current')
-      expect($otherLink).not.toHaveClass('up-current')
+      it 'does not throw if the current location does not match an up-alias wildcard (bugfix)', ->
+        inserter = -> up.hello(affix('a[up-alias="/qqqq*"]'))
+        expect(inserter).not.toThrow()
 
-    it 'allows to configure a custom "current" class, but always also sets .up-current', ->
-      up.navigation.config.currentClasses = ['highlight']
-      spyOn(up.browser, 'url').and.returnValue('/foo')
-      $currentLink = up.hello(affix('a[href="/foo"]'))
-      expect($currentLink).toHaveClass('highlight up-current')
+      it 'does not highlight a link to "#" (commonly used for JS-only buttons)', ->
+        $link = up.hello(affix('a[href="#"]'))
+        expect($link).not.toHaveClass('up-current')
 
-    describeCapability 'canPushState', ->
-      
-      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')
-        $link.click()
-        @respondWith
-          responseHeaders: { 'X-Up-Location': '/foo/' }
-          responseText: '<div class="main">new-text</div>'
-        expect($link).toHaveClass('up-current')
-      
-      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')
-        $link.click()
-        @respondWith
-          responseHeaders: { 'X-Up-Location': '/foo' }
-          responseText: '<div class="main">new-text</div>'
-        expect($link).toHaveClass('up-current')
+      it 'marks URL prefixes as .up-current if an up-alias value ends in *', ->
+        spyOn(up.browser, 'url').and.returnValue('/foo/123')
+        $currentLink = up.hello(affix('a[href="/x"][up-alias="/aaa /foo/* /bbb"]'))
+        $otherLink = up.hello(affix('a[href="/y"][up-alias="/bar"]'))
+        expect($currentLink).toHaveClass('up-current')
+        expect($otherLink).not.toHaveClass('up-current')
 
-      it 'marks a link as .up-current if it links to an URL currently shown either within or below the modal', (done) ->
-        up.history.replace('/foo')
-        $backgroundLink = affix('a[href="/foo"]')
-        $modalLink = affix('a[href="/bar"][up-modal=".main"]')
-        $unrelatedLink = affix('a[href="/baz]')
+      it 'allows to configure a custom "current" class, but always also sets .up-current', ->
+        up.navigation.config.currentClasses = ['highlight']
+        spyOn(up.browser, 'url').and.returnValue('/foo')
+        $currentLink = up.hello(affix('a[href="/foo"]'))
+        expect($currentLink).toHaveClass('highlight up-current')
 
-        Trigger.click($modalLink)
-        @respondWith('<div class="main">new-text</div>')
-        expect($backgroundLink).toHaveClass('up-current')
-        expect($modalLink).toHaveClass('up-current')
-        expect($unrelatedLink).not.toHaveClass('up-current')
+      describeCapability 'canPushState', ->
 
-        up.modal.close().then ->
+        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')
+          $link.click()
+          @respondWith
+            responseHeaders: { 'X-Up-Location': '/foo/' }
+            responseText: '<div class="main">new-text</div>'
+          expect($link).toHaveClass('up-current')
+
+        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')
+          $link.click()
+          @respondWith
+            responseHeaders: { 'X-Up-Location': '/foo' }
+            responseText: '<div class="main">new-text</div>'
+          expect($link).toHaveClass('up-current')
+
+        it 'marks a link as .up-current if it links to an URL currently shown either within or below the modal', (done) ->
+          up.history.replace('/foo')
+          $backgroundLink = affix('a[href="/foo"]')
+          $modalLink = affix('a[href="/bar"][up-modal=".main"]')
+          $unrelatedLink = affix('a[href="/baz]')
+
+          Trigger.click($modalLink)
+          @respondWith('<div class="main">new-text</div>')
           expect($backgroundLink).toHaveClass('up-current')
-          expect($modalLink).not.toHaveClass('up-current')
+          expect($modalLink).toHaveClass('up-current')
           expect($unrelatedLink).not.toHaveClass('up-current')
-          done()
 
-      it 'marks a link as .up-current if it links to the URL currently either within or below the popup', (done) ->
-        up.history.replace('/foo')
-        $backgroundLink = affix('a[href="/foo"]')
-        $popupLink = affix('a[href="/bar"][up-popup=".main"]')
-        $unrelatedLink = affix('a[href="/baz]')
+          up.modal.close().then ->
+            expect($backgroundLink).toHaveClass('up-current')
+            expect($modalLink).not.toHaveClass('up-current')
+            expect($unrelatedLink).not.toHaveClass('up-current')
+            done()
 
-        $popupLink.click()
-        @respondWith('<div class="main">new-text</div>')
-        expect($backgroundLink).toHaveClass('up-current')
-        expect($popupLink).toHaveClass('up-current')
-        expect($unrelatedLink).not.toHaveClass('up-current')
+        it 'marks a link as .up-current if it links to the URL currently either within or below the popup', (done) ->
+          up.history.replace('/foo')
+          $backgroundLink = affix('a[href="/foo"]')
+          $popupLink = affix('a[href="/bar"][up-popup=".main"]')
+          $unrelatedLink = affix('a[href="/baz]')
 
-        up.popup.close().then ->
+          $popupLink.click()
+          @respondWith('<div class="main">new-text</div>')
           expect($backgroundLink).toHaveClass('up-current')
-          expect($popupLink).not.toHaveClass('up-current')
+          expect($popupLink).toHaveClass('up-current')
           expect($unrelatedLink).not.toHaveClass('up-current')
-          done()
 
-      it 'changes .up-current marks as the URL changes'
-        
-      it 'marks clicked links as .up-active until the request finishes', ->
-        $link = affix('a[href="/foo"][up-target=".main"]')
-        affix('.main')
-        $link.click()
-        expect($link).toHaveClass('up-active')
-        @respondWith('<div class="main">new-text</div>')
-        expect($link).not.toHaveClass('up-active')
-        expect($link).toHaveClass('up-current')
-        
-      it 'marks links with [up-instant] on mousedown as .up-active until the request finishes', ->
-        $link = affix('a[href="/foo"][up-instant][up-target=".main"]')
-        affix('.main')
-        Trigger.mousedown($link)
-        expect($link).toHaveClass('up-active')
-        @respondWith('<div class="main">new-text</div>')
-        expect($link).not.toHaveClass('up-active')
-        expect($link).toHaveClass('up-current')
-    
-      it 'prefers to mark an enclosing [up-expand] click area', ->
-        $area = affix('div[up-expand] a[href="/foo"][up-target=".main"]')
-        up.hello($area)
-        $link = $area.find('a')
-        affix('.main')
-        $link.click()
-        expect($area).toHaveClass('up-active')
-        @respondWith('<div class="main">new-text</div>')
-        expect($area).toHaveClass('up-current')
+          up.popup.close().then ->
+            expect($backgroundLink).toHaveClass('up-current')
+            expect($popupLink).not.toHaveClass('up-current')
+            expect($unrelatedLink).not.toHaveClass('up-current')
+            done()
+
+        it 'changes .up-current marks as the URL changes'
+
+    describe '.up-active', ->
+
+      describeCapability 'canPushState', ->
+
+        it 'marks clicked links as .up-active until the request finishes', ->
+          $link = affix('a[href="/foo"][up-target=".main"]')
+          affix('.main')
+          $link.click()
+          expect($link).toHaveClass('up-active')
+          @respondWith('<div class="main">new-text</div>')
+          expect($link).not.toHaveClass('up-active')
+
+        it 'marks links with [up-instant] on mousedown as .up-active until the request finishes', ->
+          $link = affix('a[href="/foo"][up-instant][up-target=".main"]')
+          affix('.main')
+          Trigger.mousedown($link)
+          expect($link).toHaveClass('up-active')
+          @respondWith('<div class="main">new-text</div>')
+          expect($link).not.toHaveClass('up-active')
+
+        it 'prefers to mark an enclosing [up-expand] click area', ->
+          $area = affix('div[up-expand] a[href="/foo"][up-target=".main"]')
+          up.hello($area)
+          $link = $area.find('a')
+          affix('.main')
+          $link.click()
+          expect($link).not.toHaveClass('up-active')
+          expect($area).toHaveClass('up-active')
+          @respondWith('<div class="main">new-text</div>')
+          expect($area).not.toHaveClass('up-active')
+
+        it 'marks clicked modal openers as .up-active while the modal is loading', ->
+          $link = affix('a[href="/foo"][up-modal=".main"]')
+          affix('.main')
+          $link.click()
+          expect($link).toHaveClass('up-active')
+          @respondWith('<div class="main">new-text</div>')
+          expect($link).not.toHaveClass('up-active')
+
+        it 'removes .up-active from a clicked modal opener if the target is already preloaded (bugfix)', ->
+          $link = affix('a[href="/foo"][up-modal=".main"]')
+          up.proxy.preload($link)
+          @respondWith('<div class="main">new-text</div>')
+          $link.click()
+          expect('.up-modal .main').toHaveText('new-text')
+          expect($link).not.toHaveClass('up-active')
+
+        it 'removes .up-active from a clicked link if the target is already preloaded (bugfix)', ->
+          $link = affix('a[href="/foo"][up-target=".main"]')
+          affix('.main')
+          up.proxy.preload($link)
+          @respondWith('<div class="main">new-text</div>')
+          $link.click()
+          expect('.main').toHaveText('new-text')
+          expect($link).not.toHaveClass('up-active')
 