RubyGems - unpoly-rails - Versions diffs - 0.37.0 → 0.50.0 - Mend

unpoly-rails 0.37.0 → 0.50.0

Potentially problematic release.

This version of unpoly-rails might be problematic. Click here for more details.

Files changed (88) hide show

checksums.yaml +4 -4
data/CHANGELOG.md +127 -25
data/LICENSE +1 -1
data/README_RAILS.md +4 -2
data/Rakefile +6 -1
data/dist/unpoly.js +3192 -2198
data/dist/unpoly.min.js +4 -3
data/lib/assets/javascripts/unpoly/browser.coffee +51 -63
data/lib/assets/javascripts/unpoly/bus.coffee +58 -33
data/lib/assets/javascripts/unpoly/classes/cache.coffee +117 -0
data/lib/assets/javascripts/unpoly/{dom → classes}/extract_cascade.coffee +3 -3
data/lib/assets/javascripts/unpoly/{dom → classes}/extract_plan.coffee +1 -1
data/lib/assets/javascripts/unpoly/classes/field_observer.coffee +57 -0
data/lib/assets/javascripts/unpoly/classes/follow_variant.coffee +52 -0
data/lib/assets/javascripts/unpoly/classes/motion_tracker.coffee +95 -0
data/lib/assets/javascripts/unpoly/classes/record.coffee +16 -0
data/lib/assets/javascripts/unpoly/classes/request.coffee +228 -0
data/lib/assets/javascripts/unpoly/classes/response.coffee +138 -0
data/lib/assets/javascripts/unpoly/dom.coffee +151 -142
data/lib/assets/javascripts/unpoly/feedback.coffee +67 -38
data/lib/assets/javascripts/unpoly/form.coffee +156 -139
data/lib/assets/javascripts/unpoly/history.coffee +22 -19
data/lib/assets/javascripts/unpoly/layout.coffee +108 -90
data/lib/assets/javascripts/unpoly/link.coffee +159 -158
data/lib/assets/javascripts/unpoly/log.coffee +5 -5
data/lib/assets/javascripts/unpoly/modal.coffee +93 -81
data/lib/assets/javascripts/unpoly/motion.coffee +291 -250
data/lib/assets/javascripts/unpoly/popup.coffee +67 -53
data/lib/assets/javascripts/unpoly/protocol.coffee +67 -16
data/lib/assets/javascripts/unpoly/proxy.coffee +282 -211
data/lib/assets/javascripts/unpoly/rails.coffee +3 -14
data/lib/assets/javascripts/unpoly/syntax.coffee +54 -49
data/lib/assets/javascripts/unpoly/tooltip.coffee +18 -25
data/lib/assets/javascripts/unpoly/util.coffee +236 -477
data/lib/assets/javascripts/unpoly.coffee +1 -1
data/lib/unpoly/rails/inspector.rb +67 -22
data/lib/unpoly/rails/version.rb +1 -1
data/package.json +1 -1
data/spec_app/Gemfile.lock +13 -13
data/spec_app/app/assets/javascripts/integration_test.coffee +1 -0
data/spec_app/app/assets/javascripts/jasmine_specs.coffee +1 -1
data/spec_app/app/assets/stylesheets/jasmine_specs.sass +10 -0
data/spec_app/app/controllers/binding_test_controller.rb +19 -2
data/spec_app/app/controllers/method_test_controller.rb +16 -0
data/spec_app/app/views/layouts/jasmine_rails/spec_runner.html.erb +20 -0
data/spec_app/app/views/method_test/form_target.erb +17 -0
data/spec_app/app/views/method_test/page1.erb +11 -0
data/spec_app/app/views/method_test/page2.erb +6 -0
data/spec_app/app/views/pages/start.erb +33 -19
data/spec_app/config/initializers/assets.rb +5 -0
data/spec_app/config/routes.rb +3 -0
data/spec_app/spec/controllers/binding_test_controller_spec.rb +82 -27
data/spec_app/spec/javascripts/helpers/agent_detector.coffee +17 -0
data/spec_app/spec/javascripts/helpers/async_sequence.js.coffee +102 -0
data/spec_app/spec/javascripts/helpers/last_request.js.coffee +1 -1
data/spec_app/spec/javascripts/helpers/mock_ajax.js.coffee +5 -2
data/spec_app/spec/javascripts/helpers/promise_state.js +18 -0
data/spec_app/spec/javascripts/helpers/protect_jasmine_runner.coffee +9 -0
data/spec_app/spec/javascripts/helpers/reset_history.js.coffee +22 -0
data/spec_app/spec/javascripts/helpers/reset_up.js.coffee +11 -3
data/spec_app/spec/javascripts/helpers/show_lib_versions.coffee +10 -0
data/spec_app/spec/javascripts/helpers/to_be_error.coffee +5 -0
data/spec_app/spec/javascripts/helpers/to_match_url.coffee +13 -0
data/spec_app/spec/javascripts/helpers/trigger.js.coffee +13 -6
data/spec_app/spec/javascripts/up/browser_spec.js.coffee +92 -33
data/spec_app/spec/javascripts/up/bus_spec.js.coffee +64 -15
data/spec_app/spec/javascripts/up/classes/.keep +0 -0
data/spec_app/spec/javascripts/up/classes/cache_spec.js.coffee +1 -0
data/spec_app/spec/javascripts/up/dom_spec.js.coffee +759 -551
data/spec_app/spec/javascripts/up/feedback_spec.js.coffee +155 -82
data/spec_app/spec/javascripts/up/form_spec.js.coffee +490 -349
data/spec_app/spec/javascripts/up/history_spec.js.coffee +226 -179
data/spec_app/spec/javascripts/up/layout_spec.js.coffee +253 -185
data/spec_app/spec/javascripts/up/link_spec.js.coffee +416 -270
data/spec_app/spec/javascripts/up/modal_spec.js.coffee +459 -330
data/spec_app/spec/javascripts/up/motion_spec.js.coffee +198 -153
data/spec_app/spec/javascripts/up/namespace_spec.js.coffee +9 -0
data/spec_app/spec/javascripts/up/popup_spec.js.coffee +240 -175
data/spec_app/spec/javascripts/up/protocol_spec.js.coffee +38 -0
data/spec_app/spec/javascripts/up/proxy_spec.js.coffee +777 -303
data/spec_app/spec/javascripts/up/rails_spec.js.coffee +24 -8
data/spec_app/spec/javascripts/up/syntax_spec.js.coffee +40 -23
data/spec_app/spec/javascripts/up/tooltip_spec.js.coffee +80 -66
data/spec_app/spec/javascripts/up/util_spec.js.coffee +227 -201
data/spec_app/vendor/asset-libs/es6-promise-4.1.6/es6-promise.auto.js +1159 -0
metadata +30 -7
data/spec_app/spec/javascripts/helpers/reset_path.js.coffee +0 -7
data/spec_app/spec/javascripts/helpers/to_equal_url.coffee +0 -11

data/spec_app/spec/javascripts/up/history_spec.js.coffee CHANGED Viewed

@@ -1,6 +1,9 @@
 describe 'up.history', ->
   u = up.util
+  beforeEach ->
+    up.history.config.enabled = true
   describe 'JavaScript functions', ->
@@ -10,37 +13,41 @@ describe 'up.history', ->
     describe 'up.history.url', ->
-      it 'does not strip a trailing slash from the current URL', ->
-        history.replaceState?({}, 'title', '/host/path/')
-        expect(up.history.url()).toEqualUrl('/host/path/')
+      describeCapability 'canPushState', ->
+        it 'does not strip a trailing slash from the current URL', ->
+          history.replaceState?({}, 'title', '/host/path/')
+          expect(up.history.url()).toMatchUrl('/host/path/')
     describe 'up.history.isUrl', ->
-      it 'returns true if the given path is the current URL', ->
-        history.replaceState?({}, 'title', '/host/path/')
-        expect(up.history.isUrl('/host/path/')).toBe(true)
+      describeCapability 'canPushState', ->
-      it 'returns false if the given path is not the current URL', ->
-        history.replaceState?({}, 'title', '/host/path/')
-        expect(up.history.isUrl('/host/other-path/')).toBe(false)
+        it 'returns true if the given path is the current URL', ->
+          history.replaceState?({}, 'title', '/host/path/')
+          expect(up.history.isUrl('/host/path/')).toBe(true)
-      it 'returns true if the given full URL is the current URL', ->
-        history.replaceState?({}, 'title', '/host/path/')
-        expect(up.history.isUrl("http://#{location.host}/host/path/")).toBe(true)
+        it 'returns false if the given path is not the current URL', ->
+          history.replaceState?({}, 'title', '/host/path/')
+          expect(up.history.isUrl('/host/other-path/')).toBe(false)
-      it 'returns true if the given path is the current URL, but without a trailing slash', ->
-        history.replaceState?({}, 'title', '/host/path/')
-        expect(up.history.isUrl('/host/path')).toBe(true)
+        it 'returns true if the given full URL is the current URL', ->
+          history.replaceState?({}, 'title', '/host/path/')
+          expect(up.history.isUrl("http://#{location.host}/host/path/")).toBe(true)
-      it 'returns true if the given path is the current URL, but with a trailing slash', ->
-        history.replaceState?({}, 'title', '/host/path')
-        expect(up.history.isUrl('/host/path/')).toBe(true)
+        it 'returns true if the given path is the current URL, but without a trailing slash', ->
+          history.replaceState?({}, 'title', '/host/path/')
+          expect(up.history.isUrl('/host/path')).toBe(true)
+        it 'returns true if the given path is the current URL, but with a trailing slash', ->
+          history.replaceState?({}, 'title', '/host/path')
+          expect(up.history.isUrl('/host/path/')).toBe(true)
   describe 'unobtrusive behavior', ->
     describe 'back button', ->
-      it 'calls destructor functions when destroying compiled elements (bugfix)', (done) ->
+      it 'calls destructor functions when destroying compiled elements (bugfix)', asyncSpec (next) ->
         waitForBrowser = 70
         # By default, up.history will replace the <body> tag when
@@ -65,227 +72,267 @@ describe 'up.history', ->
         expect(constructorSpy).toHaveBeenCalled()
         history.back()
-        u.setTimer waitForBrowser, =>
-          expect(location.pathname).toEqual('/one')
+        next.after waitForBrowser, =>
+          expect(location.pathname).toEqual('/one')
           @respondWith "<div class='container'>restored container text</div>"
-          u.nextFrame ->
-            expect(destructorSpy).toHaveBeenCalled()
-            done()
+        next =>
+          expect(destructorSpy).toHaveBeenCalled()
     describe '[up-back]', ->
-      it 'sets an [up-href] attribute to the previous URL and sets the up-restore-scroll attribute to "true"', ->
-        up.history.push('/one')
-        up.history.push('/two')
-        $element = up.hello(affix('a[href="/three"][up-back]').text('text'))
-        expect($element.attr('href')).toEqualUrl('/three')
-        expect($element.attr('up-href')).toEqualUrl('/one')
-        expect($element.attr('up-restore-scroll')).toBe('')
-        expect($element.attr('up-follow')).toBe('')
+      describeCapability 'canPushState', ->
+        it 'sets an [up-href] attribute to the previous URL and sets the up-restore-scroll attribute to "true"', ->
+          up.history.push('/path1')
+          up.history.push('/path2')
+          $element = up.hello(affix('a[href="/path3"][up-back]').text('text'))
+          expect($element.attr('href')).toMatchUrl('/path3')
+          expect($element.attr('up-href')).toMatchUrl('/path1')
+          expect($element.attr('up-restore-scroll')).toBe('')
+          expect($element.attr('up-follow')).toBe('')
       it 'does not overwrite an existing up-href or up-restore-scroll attribute'
       it 'does not set an up-href attribute if there is no previous URL'
+      describeFallback 'canPushState', ->
+        it 'does not change the element', ->
+          $element = up.hello(affix('a[href="/three"][up-back]').text('text'))
+          expect($element.attr('up-href')).toBeUndefined()
     describe 'scroll restoration', ->
-      afterEach ->
-        $('.viewport').remove()
+      describeCapability 'canPushState', ->
-      it 'restores the scroll position of viewports when the user hits the back button', (done) ->
+        afterEach ->
+          $('.viewport').remove()
-        longContentHtml = """
-          <div class="viewport" style="width: 100px; height: 100px; overflow-y: scroll">
-            <div class="content" style="height: 1000px"></div>
-          </div>
-        """
+        it 'restores the scroll position of viewports when the user hits the back button', asyncSpec (next) ->
+          longContentHtml = """
+            <div class="viewport" style="width: 100px; height: 100px; overflow-y: scroll">
+              <div class="content" style="height: 1000px"></div>
+            </div>
+          """
-        respond = => @respondWith(longContentHtml)
+          respond = => @respondWith(longContentHtml)
-        $viewport = $(longContentHtml).appendTo(document.body)
+          $viewport = $(longContentHtml).appendTo(document.body)
-        up.layout.config.viewports = ['.viewport']
-        up.history.config.popTargets = ['.viewport']
+          up.layout.config.viewports = ['.viewport']
+          up.history.config.popTargets = ['.viewport']
-        up.replace('.content', '/one')
-        respond()
+          up.replace('.content', '/one')
-        $viewport.scrollTop(50)
+          next =>
+            respond()
-        up.replace('.content', '/two')
-        respond()
+          next =>
+            $viewport.scrollTop(50)
+            up.replace('.content', '/two')
-        $('.viewport').scrollTop(150)
+          next =>
+            respond()
-        up.replace('.content', '/three')
-        respond()
-        $('.viewport').scrollTop(250)
+          next =>
+            $('.viewport').scrollTop(150)
+            up.replace('.content', '/three')
-        history.back()
-        u.setTimer 50, ->
-          respond() # we need to respond since we've never requested /two with the popTarget
-          expect($('.viewport').scrollTop()).toBe(150)
+          next =>
+            respond()
-          history.back()
-          u.setTimer 50, ->
+          next =>
+            $('.viewport').scrollTop(250)
+            history.back()
+          next.after 50, =>
+            respond() # we need to respond since we've never requested /two with the popTarget
+          next =>
+            expect($('.viewport').scrollTop()).toBe(150)
+            history.back()
+          next.after 50, =>
             respond() # we need to respond since we've never requested /one with the popTarget
+          next =>
             expect($('.viewport').scrollTop()).toBe(50)
+            history.forward()
+          next.after 50, =>
+            # No need to respond since we requested /two with the popTarget
+            # when we went backwards
+            expect($('.viewport').scrollTop()).toBe(150)
             history.forward()
-            u.setTimer 50, ->
-              # No need to respond since we requested /two with the popTarget
-              # when we went backwards
-              expect($('.viewport').scrollTop()).toBe(150)
-              history.forward()
-              u.setTimer 50, ->
-                respond() # we need to respond since we've never requested /three with the popTarget
-                expect($('.viewport').scrollTop()).toBe(250)
-                done()
-      it 'restores the scroll position of two viewports marked with [up-viewport], but not configured in up.layout.config (bugfix)', (done) ->
-        up.history.config.popTargets = ['.container']
-        html = """
-          <div class="container">
-            <div class="viewport1" up-viewport style="width: 100px; height: 100px; overflow-y: scroll">
-              <div class="content1" style="height: 5000px">content1</div>
-            </div>
-            <div class="viewport2" up-viewport style="width: 100px; height: 100px; overflow-y: scroll">
-              <div class="content2" style="height: 5000px">content2</div>
-            </div>
-          </div>
-        """
+          next.after 50, =>
+            respond() # we need to respond since we've never requested /three with the popTarget
-        respond = => @respondWith(html)
+          next =>
+            expect($('.viewport').scrollTop()).toBe(250)
-        $screen = affix('.screen')
-        $screen.html(html)
+        it 'restores the scroll position of two viewports marked with [up-viewport], but not configured in up.layout.config (bugfix)', asyncSpec (next) ->
+          up.history.config.popTargets = ['.container']
-        up.replace('.content1, .content2', '/one', reveal: false)
-        respond()
+          html = """
+            <div class="container">
+              <div class="viewport1" up-viewport style="width: 100px; height: 100px; overflow-y: scroll">
+                <div class="content1" style="height: 5000px">content1</div>
+              </div>
+              <div class="viewport2" up-viewport style="width: 100px; height: 100px; overflow-y: scroll">
+                <div class="content2" style="height: 5000px">content2</div>
+              </div>
+            </div>
+          """
+          respond = => @respondWith(html)
-        $('.viewport1').scrollTop(3000)
-        $('.viewport2').scrollTop(3050)
+          $screen = affix('.screen')
+          $screen.html(html)
-        expect('.viewport1').toBeScrolledTo(3000)
-        expect('.viewport2').toBeScrolledTo(3050)
+          up.replace('.content1, .content2', '/one', reveal: false)
-        up.replace('.content1, .content2', '/two', reveal: false)
-        respond()
+          next =>
+            respond()
-        u.setTimer 50, ->
+          next =>
+            $('.viewport1').scrollTop(3000)
+            $('.viewport2').scrollTop(3050)
+            expect('.viewport1').toBeScrolledTo(3000)
+            expect('.viewport2').toBeScrolledTo(3050)
-          expect(location.href).toEqualUrl('/two')
+            up.replace('.content1, .content2', '/two', reveal: false)
-          history.back()
+          next =>
+            respond()
-          u.setTimer 50, ->
+          next.after 50, =>
+            expect(location.href).toMatchUrl('/two')
+            history.back()
+          next.after 50, =>
             # we need to respond since we've never requested the original URL with the popTarget
             respond()
-            u.nextFrame ->
-              expect('.viewport1').toBeScrolledTo(3000)
-              expect('.viewport2').toBeScrolledTo(3050)
-              done()
+          next =>
+            expect('.viewport1').toBeScrolledTo(3000)
+            expect('.viewport2').toBeScrolledTo(3050)
     describe 'events', ->
-      it 'emits up:history:* events as the user goes forwards and backwards through history', (done) ->
-        up.proxy.config.cacheSize = 0
-        up.history.config.popTargets = ['.viewport']
+      describeCapability 'canPushState', ->
-        affix('.viewport .content')
-        respond = =>
-          @respondWith """
-            <div class="viewport">
-              <div class="content">content</div>
-            </div>
-            """
+        it 'emits up:history:* events as the user goes forwards and backwards through history', asyncSpec (next) ->
+          up.proxy.config.cacheSize = 0
+          up.history.config.popTargets = ['.viewport']
+          affix('.viewport .content')
+          respond = =>
+            @respondWith """
+              <div class="viewport">
+                <div class="content">content</div>
+              </div>
+              """
-        events = []
-        u.each ['up:history:pushed', 'up:history:restored'], (eventName) ->
-          up.on eventName, (event) ->
-            events.push [eventName, event.url]
+          events = []
+          u.each ['up:history:pushed', 'up:history:restored'], (eventName) ->
+            up.on eventName, (event) ->
+              events.push [eventName, event.url]
-        normalize = up.history.normalizeUrl
+          normalize = up.history.normalizeUrl
-        up.replace('.content', '/one')
-        respond()
+          up.replace('.content', '/foo')
-        expect(events).toEqual [
-          ['up:history:pushed', normalize('/one')]
-        ]
+          next =>
+            respond()
+          next =>
+            expect(events).toEqual [
+              ['up:history:pushed', normalize('/foo')]
+            ]
-        up.replace('.content', '/two')
-        respond()
+            up.replace('.content', '/bar')
-        expect(events).toEqual [
-          ['up:history:pushed', normalize('/one')]
-          ['up:history:pushed', normalize('/two')]
-        ]
+          next =>
+            respond()
-        up.replace('.content', '/three')
-        respond()
+          next =>
+            expect(events).toEqual [
+              ['up:history:pushed', normalize('/foo')]
+              ['up:history:pushed', normalize('/bar')]
+            ]
-        expect(events).toEqual [
-          ['up:history:pushed', normalize('/one')]
-          ['up:history:pushed', normalize('/two')]
-          ['up:history:pushed', normalize('/three')]
-        ]
+            up.replace('.content', '/baz')
-        history.back()
-        u.setTimer 50, ->
-          respond()
-          expect(events).toEqual [
-            ['up:history:pushed', normalize('/one')]
-            ['up:history:pushed', normalize('/two')]
-            ['up:history:pushed', normalize('/three')]
-            ['up:history:restored', normalize('/two')]
-          ]
-          history.back()
-          u.setTimer 50, ->
+          next =>
             respond()
+          next =>
             expect(events).toEqual [
-              ['up:history:pushed', normalize('/one')]
-              ['up:history:pushed', normalize('/two')]
-              ['up:history:pushed', normalize('/three')]
-              ['up:history:restored', normalize('/two')]
-              ['up:history:restored', normalize('/one')]
+              ['up:history:pushed', normalize('/foo')]
+              ['up:history:pushed', normalize('/bar')]
+              ['up:history:pushed', normalize('/baz')]
+            ]
+            history.back()
+          next.after 50, =>
+            respond()
+          next =>
+            expect(events).toEqual [
+              ['up:history:pushed', normalize('/foo')]
+              ['up:history:pushed', normalize('/bar')]
+              ['up:history:pushed', normalize('/baz')]
+              ['up:history:restored', normalize('/bar')]
+            ]
+            history.back()
+          next.after 50, =>
+            respond()
+          next =>
+            expect(events).toEqual [
+              ['up:history:pushed', normalize('/foo')]
+              ['up:history:pushed', normalize('/bar')]
+              ['up:history:pushed', normalize('/baz')]
+              ['up:history:restored', normalize('/bar')]
+              ['up:history:restored', normalize('/foo')]
             ]
             history.forward()
-            u.setTimer 50, ->
-              respond()
-              expect(events).toEqual [
-                ['up:history:pushed', normalize('/one')]
-                ['up:history:pushed', normalize('/two')]
-                ['up:history:pushed', normalize('/three')]
-                ['up:history:restored', normalize('/two')]
-                ['up:history:restored', normalize('/one')]
-                ['up:history:restored', normalize('/two')]
-              ]
-              history.forward()
-              u.setTimer 50, ->
-                respond() # we need to respond since we've never requested /three with the popTarget
-                expect(events).toEqual [
-                  ['up:history:pushed', normalize('/one')]
-                  ['up:history:pushed', normalize('/two')]
-                  ['up:history:pushed', normalize('/three')]
-                  ['up:history:restored', normalize('/two')]
-                  ['up:history:restored', normalize('/one')]
-                  ['up:history:restored', normalize('/two')]
-                  ['up:history:restored', normalize('/three')]
-                ]
-                done()
+          next.after 50, =>
+            respond()
+          next =>
+            expect(events).toEqual [
+              ['up:history:pushed', normalize('/foo')]
+              ['up:history:pushed', normalize('/bar')]
+              ['up:history:pushed', normalize('/baz')]
+              ['up:history:restored', normalize('/bar')]
+              ['up:history:restored', normalize('/foo')]
+              ['up:history:restored', normalize('/bar')]
+            ]
+            history.forward()
+          next.after 50, =>
+            respond() # we need to respond since we've never requested /baz with the popTarget
+          next =>
+            expect(events).toEqual [
+              ['up:history:pushed', normalize('/foo')]
+              ['up:history:pushed', normalize('/bar')]
+              ['up:history:pushed', normalize('/baz')]
+              ['up:history:restored', normalize('/bar')]
+              ['up:history:restored', normalize('/foo')]
+              ['up:history:restored', normalize('/bar')]
+              ['up:history:restored', normalize('/baz')]
+            ]