capybara-lockstep 2.1.1 → 2.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 92432e49d223c6a01c09cbe199fec3d3ad0fdc80c5eb5def77883e25f5bace80
-  data.tar.gz: 43fcb6f560accb02c5a6081506a6d1b65c571ab149b0d470efeddf49adb853b9
+  metadata.gz: fa4a510d3f4745ef60812fdf3ab03f6c3e8ede0c6ea4046cb87364d6664e2a7d
+  data.tar.gz: 78be38bdb5aa4fcb9147394ac3adb11f3112c18539ba53a096c214931be753cb
 SHA512:
-  metadata.gz: ef3cb7e6d8789b1108602ad7d18ba7e00bf52e93135d8ebec84d9307c556f800f05218d4b23a750d66888e99f9f2939c37f76d3fd3359cdf765b5799b2a3e182
-  data.tar.gz: d2017577968548a815e9e60c9d1fa5838ab3c16fcbb1122acdaf9185e2c44c36766cca61a53de6c016ed691b4d558ae4fab282ed34cd18aa5348a8db64f527ad
+  metadata.gz: 356df47acf01a2642769be49f54c48736ea030c803a92b3c059c587e5e82bb927fa0b4e5dcf7953060162a46491cf0122c6a2d0debd1f607cbd1b09cc8608ff2
+  data.tar.gz: c5d1189ee309d32197f19765dad9014cc59f13a8d9f6a17287537045cbf02664b480ce9d087e933b3fad1a2c63b4460b5dc5e61e20d3e0b98d76cde9d90eaff3

data/.github/workflows/test.yml

@@ -3,13 +3,13 @@ name: Tests
 on:
   push:
     branches:
-    - master
+    - main
   pull_request:
     branches:
-    - master
+    - main
   workflow_dispatch:
     branches:
-    - master
+    - main
 jobs:
   test:
     runs-on: ubuntu-20.04

data/CHANGELOG.md

@@ -3,6 +3,13 @@ All notable changes to this project will be documented in this file.
 This project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html).
 
 
+# 2.2.0
+
+- We now wait for `<video>` and `<audio>` elements to load their metadata. This addresses a race condition where a media element is inserted into the DOM, but another user action deletes or renames the source before the browser could load the initial metadata frames. 
+- We now wait for `<script type="module">`.
+- We no longer wait for `<img loading="lazy">` or `<iframe loading="lazy">`. This prevents a deadlock where we would wait forever for an element that defers loading until it is scrolled into the viewport.
+
+
 # 2.1.0
 
 - We now synchronize for an additional [JavaScript task](https://jakearchibald.com/2015/tasks-microtasks-queues-and-schedules/) after `history.pushState()`, `history.replaceState()`, `history.forward()`, `history.back()` and `history.go()`.

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    capybara-lockstep (2.1.1)
+    capybara-lockstep (2.2.0)
       activesupport (>= 4.2)
       capybara (>= 3.0)
       ruby2_keywords
@@ -113,4 +113,4 @@ DEPENDENCIES
   thin
 
 BUNDLED WITH
-  4.0.15
+   2.2.32

data/README.md

@@ -71,7 +71,8 @@ When capybara-lockstep synchronizes it will:
 - wait for client-side JavaScript to render or hydrate DOM elements.
 - wait for any pending AJAX requests to finish and their callbacks to be called.
 - wait for dynamically inserted `<script>`s to load (e.g. from [dynamic imports](https://webpack.js.org/guides/code-splitting/#dynamic-imports) or Analytics snippets).
-- waits for dynamically inserted `<img>` or `<iframe>` elements to load.
+- waits for dynamically inserted `<img>` or `<iframe>` elements to load (ignoring [lazy-loaded](https://developer.mozilla.org/en-US/docs/Web/HTML/Element/img#lazy) elements).
+- waits for dynamically inserted `<audio>` and `<video>` elements to load their [metadata](https://developer.mozilla.org/en-US/docs/Web/API/HTMLMediaElement/loadedmetadata_event)
 
 In summary Capybara can no longer observe or interact with the page while HTTP requests are in flight.
 This covers most async work that causes flaky tests.

data/capybara-lockstep.gemspec

@@ -15,7 +15,7 @@ Gem::Specification.new do |spec|
   spec.metadata["source_code_uri"] = spec.homepage
 
   spec.metadata["bug_tracker_uri"] = "https://github.com/makandra/capybara-lockstep/issues"
-  spec.metadata["changelog_uri"] = "https://github.com/makandra/capybara-lockstep/blob/master/CHANGELOG.md"
+  spec.metadata["changelog_uri"] = "https://github.com/makandra/capybara-lockstep/blob/main/CHANGELOG.md"
   spec.metadata["rubygems_mfa_required"] = 'true'
 
   # Specify which files should be added to the gem when it is released.

data/lib/capybara-lockstep/helper.js

@@ -177,39 +177,47 @@ window.CapybaraLockstep = (function() {
   }
 
   function isRemoteScript(element) {
-    if (element.tagName === 'SCRIPT') {
-      let src = element.getAttribute('src')
-      let type = element.getAttribute('type')
+    return element.matches('script[src]') && !hasDataSource(element)
+  }
 
-      return src && (!type || /javascript/i.test(type))
-    }
+  function isTrackableImage(element) {
+    return element.matches('img') &&
+      !element.complete &&
+      !hasDataSource(element) &&
+      element.getAttribute('loading') !== 'lazy'
   }
 
-  function isRemoteImage(element) {
-    if (element.tagName === 'IMG' && !element.complete) {
-      let src = element.getAttribute('src')
-      let srcSet = element.getAttribute('srcset')
+  function isTrackableIFrame(element) {
+    return element.matches('iframe') &&
+      !hasDataSource(element) &&
+      element.getAttribute('loading') !== 'lazy'
+  }
+
+  function hasDataSource(element) {
+    // <img> can have <img src> and <img srcset>
+    // <video> can have <video src> or <video><source src>
+    // <audio> can have <audio src> or <audio><source src>
+    return element.matches('[src*="data:"], [srcset*="data:"]') ||
+      !!element.querySelector('source [src*="data:"], source [srcset*="data:"]')
+  }
 
-      let localSrcPattern = /^data:/
-      let localSrcSetPattern = /(^|\s)data:/
+  function isTrackableMediaElement(element) {
+    return element.matches('audio, video') &&
+      element.readyState === 0 && // no metadata known
+      !hasDataSource(element) &&
+      element.getAttribute('preload') !== 'none'
+  }
 
-      let hasLocalSrc = src && localSrcPattern.test(src)
-      let hasLocalSrcSet = srcSet && localSrcSetPattern.test(srcSet)
+  function trackRemoteElement(element, condition, workTag) {
+    trackLoadingElement(element, condition, workTag, 'load', 'error')
 
-      return (src && !hasLocalSrc) || (srcSet && !hasLocalSrcSet)
-    }
   }
 
-  function isRemoteInlineFrame(element) {
-    if (element.tagName === 'IFRAME') {
-      let src = element.getAttribute('src')
-      let localSrcPattern = /^data:/
-      let hasLocalSrc = src && localSrcPattern.test(src)
-      return (src && !hasLocalSrc)
-    }
+  function trackMediaElement(element, condition, workTag) {
+    trackLoadingElement(element, condition, workTag, 'loadedmetadata', 'error')
   }
 
-  function trackRemoteElement(element, condition, workTag) {
+  function trackLoadingElement(element, condition, workTag, loadEvent, errorEvent) {
     if (!condition(element)) {
       return
     }
@@ -220,8 +228,8 @@ window.CapybaraLockstep = (function() {
 
     let doStop = function() {
       stopped = true
-      element.removeEventListener('load', doStop)
-      element.removeEventListener('error', doStop)
+      element.removeEventListener(loadEvent, doStop)
+      element.removeEventListener(errorEvent, doStop)
       stopWork(workTag)
     }
 
@@ -240,11 +248,11 @@ window.CapybaraLockstep = (function() {
     }
 
     let scheduleCheckCondition = function() {
-      setTimeout(checkCondition, 200)
+      setTimeout(checkCondition, 150)
     }
 
-    element.addEventListener('load', doStop)
-    element.addEventListener('error', doStop)
+    element.addEventListener(loadEvent, doStop)
+    element.addEventListener(errorEvent, doStop)
 
     // We periodically check whether we still think the element will
     // produce a `load` or `error` event.
@@ -256,8 +264,9 @@ window.CapybaraLockstep = (function() {
       change.addedNodes.forEach(function(addedNode) {
         if (addedNode.nodeType === Node.ELEMENT_NODE) {
           trackRemoteElement(addedNode, isRemoteScript, 'Script')
-          trackRemoteElement(addedNode, isRemoteImage, 'Image')
-          trackRemoteElement(addedNode, isRemoteInlineFrame, 'Inline frame')
+          trackRemoteElement(addedNode, isTrackableImage, 'Image')
+          trackRemoteElement(addedNode, isTrackableIFrame, 'Inline frame')
+          trackMediaElement(addedNode, isTrackableMediaElement, 'Media element')
         }
       })
     })

data/lib/capybara-lockstep/version.rb

@@ -1,5 +1,5 @@
 module Capybara
   module Lockstep
-    VERSION = "2.1.1"
+    VERSION = "2.2.0"
   end
 end

metadata

@@ -1,13 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: capybara-lockstep
 version: !ruby/object:Gem::Version
-  version: 2.1.1
+  version: 2.2.0
 platform: ruby
 authors:
 - Henning Koch
+autorequire: 
 bindir: exe
 cert_chain: []
-date: 1980-01-02 00:00:00.000000000 Z
+date: 2024-02-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: capybara
@@ -65,6 +66,7 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+description: 
 email:
 - henning.koch@makandra.de
 executables: []
@@ -105,8 +107,9 @@ metadata:
   homepage_uri: https://github.com/makandra/capybara-lockstep
   source_code_uri: https://github.com/makandra/capybara-lockstep
   bug_tracker_uri: https://github.com/makandra/capybara-lockstep/issues
-  changelog_uri: https://github.com/makandra/capybara-lockstep/blob/master/CHANGELOG.md
+  changelog_uri: https://github.com/makandra/capybara-lockstep/blob/main/CHANGELOG.md
   rubygems_mfa_required: 'true'
+post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
@@ -121,7 +124,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 4.0.15
+rubygems_version: 3.4.3
+signing_key: 
 specification_version: 4
 summary: Synchronize Capybara commands with client-side JavaScript and AJAX requests
 test_files: []