govuk_publishing_components 66.6.0 → 66.6.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ab8154e3861817b4b3715d2eea62bb92736ad4bd6fe842774b92aaf677252812
-  data.tar.gz: 2ba507170514ffb2c67ea8fc9bd2241c1943688d004c5ae0ce16ef2790b6b5fc
+  metadata.gz: 15dc9f3d8df89afd884073edc445ee356db19847cadca1102a003d3a63b02ef3
+  data.tar.gz: 729ba4b1052993fa429a1f43cd1ff2f0832a7ad18fddbac4698aff13a04dbaf3
 SHA512:
-  metadata.gz: 3e297e668a6c2cce7746b0408d1652b6293a9c737c755c6db89b42912c872428ab74b8ede5423e8c2efc9ac1fffcbc94b29500e648696fb0ea6aa1df337bb9ca
-  data.tar.gz: 8a5df50c254dcb946ca52b0d1f0efcb7df2ae1b30abcf100f75bac2c0a1d4e3e3dbfef068ec283bcf0e7379d430c9924d6218e6800196f40c746443b6e2ab26d
+  metadata.gz: ce997f38b851195443763ffb14970e5d48f5eb5b46db659959152d00e56cc0e183ce702b33972c9597f82922982e21685b2620bbe1fcbf4aa58dabf555e0dfff
+  data.tar.gz: 755231805c81bea3994908507f771ae0fe37de4d6412c6bd9cc3cba86b98143e9d69d66d5a0e60dd32323be7cc63e4029c86f36522b27905f67ab69f2d351b48

data/app/assets/javascripts/govuk_publishing_components/lib/filter-list.js

@@ -16,10 +16,21 @@
   FilterList.prototype.appendFilterInput = function () {
     const form = document.createElement('div')
     form.classList.add('govuk-form-group')
-    form.innerHTML = `
-      <label for="filterInput" class="gem-c-label govuk-label">${this.labelText}</label>
-      <input class="gem-c-input govuk-input" id="filterInput" name="name" spellcheck="false" type="text">
-    `
+
+    const formLabel = document.createElement('label')
+    formLabel.htmlFor = 'filterInput'
+    formLabel.classList.add('gem-c-label', 'govuk-label')
+    formLabel.textContent = this.labelText
+
+    const formInput = document.createElement('input')
+    formInput.classList.add('gem-c-input', 'govuk-input')
+    formInput.id = 'filterInput'
+    formInput.name = 'name'
+    formInput.spellcheck = false
+    formInput.type = 'text'
+
+    form.append(formLabel, formInput)
+
     this.module.prepend(form)
     const input = form.querySelector('.govuk-input')
     input.addEventListener('submit', function (e) { e.preventDefault() })

data/app/assets/stylesheets/govuk_publishing_components/components/govspeak/_typography.scss

@@ -30,6 +30,13 @@
     }
   }
 
+  // restore styles for cards component inside govspeak
+  .gem-c-cards__list {
+    list-style: none;
+    padding: 0;
+    margin: 0;
+  }
+
   // Block quotes
 
   blockquote {

data/app/models/govuk_publishing_components/audit_applications.rb

@@ -64,6 +64,7 @@ module GovukPublishingComponents
           tests_path: "spec/components/",
           javascript_tests_path: "spec/javascripts/components/",
           helpers_path: "app/helpers/",
+          helpers_tests_path: "spec/helpers/",
         }
         application_components = AuditComponents.new(path, options)
         application_components = application_components.data if application_components

data/app/models/govuk_publishing_components/audit_components.rb

@@ -11,6 +11,7 @@ module GovukPublishingComponents
       @tests_path = options[:tests_path] || "spec/components"
       @javascript_tests_path = options[:javascript_tests_path] || "spec/javascripts/components"
       @helpers_path = options[:helpers_path] || "lib/govuk_publishing_components/presenters"
+      @helpers_tests_path = options[:helpers_tests_path] || "spec/lib/govuk_publishing_components/presenters"
 
       @application_name = options[:application_name] || "govuk_publishing_components"
       @application_dir = options[:application_dir] || "govuk_publishing_components"
@@ -27,6 +28,7 @@ module GovukPublishingComponents
         test: 0,
         javascript_test: 0,
         helper: 0,
+        helper_test: 0,
       }
 
       @data = {
@@ -91,6 +93,10 @@ module GovukPublishingComponents
             type: "helper",
             file: "#{[path, @helpers_path].join('/')}/#{component.gsub(' ', '_')}_helper.rb",
           },
+          {
+            type: "helper_test",
+            file: "#{[path, @helpers_tests_path].join('/')}/#{component.gsub(' ', '_')}_helper_spec.rb",
+          },
         ]
         file_details.each do |detail|
           component_detail = component_detail.merge(get_component_asset_detail(detail, component))
@@ -162,6 +168,7 @@ module GovukPublishingComponents
         .gsub(".js", "")
         .gsub("spec", "")
         .gsub("helper.rb", "")
+        .gsub("helper_spec.rb", "")
         .gsub(".rb", "")
         .strip
     end
@@ -244,6 +251,7 @@ module GovukPublishingComponents
       link = "#{url}/#{repo}/#{blob}/#{@tests_path}/#{component.gsub(' ', '_')}_spec.rb" if a_thing == "test"
       link = "#{url}/#{repo}/#{blob}/#{@javascript_tests_path}/#{component.gsub(' ', '-')}-spec.js" if a_thing == "javascript_test"
       link = "#{url}/#{repo}/#{blob}/#{@helpers_path}/#{component.gsub(' ', '_')}_helper.rb" if a_thing == "helper"
+      link = "#{url}/#{repo}/#{blob}/#{@helpers_tests_path}/#{component.gsub(' ', '_')}_helper_spec.rb" if a_thing == "helper_test"
 
       link
     end

data/app/views/govuk_publishing_components/audit/_component_contents.html.erb

@@ -60,6 +60,12 @@
             <span class="component__count"><%= passed_components[:component_numbers][:helper] %></span>
           <% end %>
         </th>
+        <th scope="col" class="govuk-table__header sticky-table-header" data-component-type="helper_test" title="Component has a helper test file">
+          Helper test
+          <% unless show_application_name %>
+            <span class="component__count"><%= passed_components[:component_numbers][:helper_test] %></span>
+          <% end %>
+        </th>
       </tr>
     </thead>
     <tbody class="govuk-table__body" <% if show_application_name %> data-audit-list<% end %>>
@@ -148,6 +154,14 @@
               </a>
             <% end %>
           </td>
+          <td class="govuk-table__cell" data-component-type="helper_test">
+            <% if component[:helper_test_exists] %>
+              <a href="<%= component[:helper_test_link] %>" class="govuk-link" title="This file has <%= component[:helper_test_lines] %> lines">
+                <%= component[:helper_test_lines] %>
+                <span class="govuk-visually-hidden">lines of code in <%= component[:name] %> helper test</span>
+              </a>
+            <% end %>
+          </td>
         </tr>
       <% end %>
     </tbody>

data/lib/govuk_publishing_components/version.rb

@@ -1,3 +1,3 @@
 module GovukPublishingComponents
-  VERSION = "66.6.0".freeze
+  VERSION = "66.6.1".freeze
 end

data/node_modules/axe-core/axe.d.ts

@@ -352,7 +352,7 @@ declare namespace axe {
     helpUrl: string;
     tags: string[];
     actIds?: string[];
-    enabled: boolean;
+    enabled?: boolean;
   }
   interface SerialDqElement {
     source: string;
@@ -623,7 +623,9 @@ declare namespace axe {
    * @param  {Array}  tags  Optional array of tags
    * @return {Array}  Array of rules
    */
-  function getRules(tags?: string[]): RuleMetadata[];
+  function getRules(
+    tags?: string[]
+  ): (Omit<RuleMetadata, 'enabled'> & { enabled: boolean })[];
 
   /**
    * Restores the default axe configuration

data/node_modules/axe-core/axe.js

@@ -1,4 +1,4 @@
-/*! axe v4.12.0
+/*! axe v4.12.1
  * Copyright (c) 2015 - 2026 Deque Systems, Inc.
  *
  * Your use of this Source Code Form is subject to the terms of the Mozilla Public
@@ -22,7 +22,7 @@
     }, _typeof(o);
   }
   var axe = axe || {};
-  axe.version = '4.12.0';
+  axe.version = '4.12.1';
   if (typeof define === 'function' && define.amd) {
     define('axe-core', [], function() {
       return axe;
@@ -6889,12 +6889,18 @@
     Object.freeze(constants.resultGroupMap);
     Object.freeze(constants);
     var constants_default = constants;
+    var logger;
     function log() {
-      if ((typeof console === 'undefined' ? 'undefined' : _typeof(console)) === 'object' && console.log) {
-        Function.prototype.apply.call(console.log, console, arguments);
+      if (logger) {
+        logger.apply(void 0, arguments);
+      } else if ((typeof console === 'undefined' ? 'undefined' : _typeof(console)) === 'object' && console.log) {
+        var _console;
+        (_console = console).log.apply(_console, arguments);
       }
     }
-    var log_default = log;
+    function setLogger(fn) {
+      logger = fn;
+    }
     var whitespaceRegex = /[\t\r\n\f]/g;
     var AbstractVirtualNode = function() {
       function AbstractVirtualNode() {
@@ -11706,7 +11712,7 @@
         err2 = e;
         setTimeout(function() {
           if (err2 !== void 0 && err2 !== null) {
-            log_default('Uncaught error (of queue)', err2);
+            log('Uncaught error (of queue)', err2);
           }
         }, 1);
       };
@@ -12193,7 +12199,7 @@
       var win = node.contentWindow;
       var pingWaitTime = (_parameters$options$p = (_parameters$options = parameters.options) === null || _parameters$options === void 0 ? void 0 : _parameters$options.pingWaitTime) !== null && _parameters$options$p !== void 0 ? _parameters$options$p : 500;
       if (!win) {
-        log_default('Frame does not have a content window', node);
+        log('Frame does not have a content window', node);
         resolve(null);
         return;
       }
@@ -16876,7 +16882,7 @@
         return accName || step(virtualNode, context);
       }, '');
       if (context.debug) {
-        log_default(valueString || '{empty-value}', actualNode, context);
+        log(valueString || '{empty-value}', actualNode, context);
       }
       return valueString;
     }
@@ -19730,7 +19736,7 @@
           axeStartTime = now();
         },
         _log: function _log(message) {
-          log_default(message);
+          log(message);
         }
       };
     }();
@@ -20784,12 +20790,12 @@
     }
     function _loadElementInternals() {
       _loadElementInternals = _asyncToGenerator(_regenerator().m(function _callee2() {
-        var logger, promiseValue, internalsMap, _i43, _internalsMap$_i, internals, ancestry, node, vNode, _i44, _Object$entries, _Object$entries$_i, _key12, val, type2, value, _args2 = arguments;
+        var logger2, promiseValue, internalsMap, _i43, _internalsMap$_i, internals, ancestry, node, vNode, _i44, _Object$entries, _Object$entries$_i, _key12, val, type2, value, _args2 = arguments;
         return _regenerator().w(function(_context3) {
           while (1) {
             switch (_context3.n) {
              case 0:
-              logger = _args2.length > 0 && _args2[0] !== undefined ? _args2[0] : log_default;
+              logger2 = _args2.length > 0 && _args2[0] !== undefined ? _args2[0] : log;
               if (getElementInternals2) {
                 _context3.n = 1;
                 break;
@@ -20808,7 +20814,7 @@
                 _context3.n = 3;
                 break;
               }
-              logger('externalAPIs.getElementInternals() did not return an array');
+              logger2('externalAPIs.getElementInternals() did not return an array');
               return _context3.a(2);
 
              case 3:
@@ -20823,7 +20829,7 @@
                 _context3.n = 5;
                 break;
               }
-              logger('externalAPIs.getElementInternals()['.concat(_i43, '] is not an object'));
+              logger2('externalAPIs.getElementInternals()['.concat(_i43, '] is not an object'));
               return _context3.a(3, 13);
 
              case 5:
@@ -20833,7 +20839,7 @@
                 _context3.n = 6;
                 break;
               }
-              logger('externalAPIs.getElementInternals()['.concat(_i43, '].internals is not an object'));
+              logger2('externalAPIs.getElementInternals()['.concat(_i43, '].internals is not an object'));
               return _context3.a(3, 13);
 
              case 6:
@@ -20841,7 +20847,7 @@
                 _context3.n = 7;
                 break;
               }
-              logger('externalAPIs.getElementInternals()['.concat(_i43, '].ancestry is not a string or an array of strings'));
+              logger2('externalAPIs.getElementInternals()['.concat(_i43, '].ancestry is not a string or an array of strings'));
               return _context3.a(3, 13);
 
              case 7:
@@ -20851,7 +20857,7 @@
                 _context3.n = 8;
                 break;
               }
-              logger('Unable to locate node using selector '.concat(ancestry, ' from externalAPIs.getElementInternals()[').concat(_i43, ']'));
+              logger2('Unable to locate node using selector '.concat(ancestry, ' from externalAPIs.getElementInternals()[').concat(_i43, ']'));
               return _context3.a(3, 13);
 
              case 8:
@@ -20873,10 +20879,10 @@
              case 10:
               type2 = val.type, value = val.value;
               if (!type2) {
-                logger('externalAPIs.getElementInternals()['.concat(_i43, '].internals.').concat(_key12, ' is an object but has no "type" property'));
+                logger2('externalAPIs.getElementInternals()['.concat(_i43, '].internals.').concat(_key12, ' is an object but has no "type" property'));
               }
               if (!value) {
-                logger('externalAPIs.getElementInternals()['.concat(_i43, '].internals.').concat(_key12, ' is an object but has no "value" property'));
+                logger2('externalAPIs.getElementInternals()['.concat(_i43, '].internals.').concat(_key12, ' is an object but has no "value" property'));
               }
               if (type2 === 'HTMLElement') {
                 setHTMLElement(internals, _key12, value);
@@ -20954,12 +20960,13 @@
     }
     function getRules(tags) {
       tags = tags || [];
-      var matchingRules = !tags.length ? axe._audit.rules : axe._audit.rules.filter(function(item) {
+      var _axe$_audit = axe._audit, rules = _axe$_audit.rules, data = _axe$_audit.data, tagExclude = _axe$_audit.tagExclude;
+      var matchingRules = !tags.length ? rules : rules.filter(function(item) {
         return !!tags.filter(function(tag) {
           return item.tags.indexOf(tag) !== -1;
         }).length;
       });
-      var ruleData = axe._audit.data.rules || {};
+      var ruleData = data.rules || {};
       return matchingRules.map(function(matchingRule) {
         var rd = ruleData[matchingRule.id] || {};
         return {
@@ -20969,7 +20976,9 @@
           helpUrl: rd.helpUrl,
           tags: matchingRule.tags,
           actIds: matchingRule.actIds,
-          enabled: matchingRule.enabled
+          enabled: matchingRule.enabled && !matchingRule.tags.some(function(tag) {
+            return tagExclude.includes(tag);
+          })
         };
       });
     }
@@ -28874,7 +28883,7 @@
       this._markChecksEnd = 'mark_runchecks_end_' + this.id;
     };
     Rule.prototype._logGatherPerformance = function _logGatherPerformance(nodes) {
-      log_default('gather for '.concat(this.id, ' (').concat(nodes.length, ' nodes): ').concat(performance_timer_default.timeElapsed(), 'ms'));
+      log('gather for '.concat(this.id, ' (').concat(nodes.length, ' nodes): ').concat(performance_timer_default.timeElapsed(), 'ms'));
       performance_timer_default.mark(this._markChecksStart);
     };
     Rule.prototype._logRulePerformance = function _logRulePerformance() {
@@ -29613,7 +29622,7 @@
             resolve(results, teardown_default);
           } catch (e) {
             teardown_default();
-            log_default(e);
+            log(e);
           }
         } catch (e) {
           teardown_default();
@@ -29753,7 +29762,7 @@
       });
     }
     function normalizeRunParams(_ref147) {
-      var _ref149, _options$reporter, _axe$_audit;
+      var _ref149, _options$reporter, _axe$_audit2;
       var _ref148 = _slicedToArray(_ref147, 3), context = _ref148[0], options = _ref148[1], callback = _ref148[2];
       var typeErr = new TypeError('axe.run arguments are invalid');
       if (!_isContextSpec(context)) {
@@ -29775,7 +29784,7 @@
         throw typeErr;
       }
       options = clone2(options);
-      options.reporter = (_ref149 = (_options$reporter = options.reporter) !== null && _options$reporter !== void 0 ? _options$reporter : (_axe$_audit = axe._audit) === null || _axe$_audit === void 0 ? void 0 : _axe$_audit.reporter) !== null && _ref149 !== void 0 ? _ref149 : 'v1';
+      options.reporter = (_ref149 = (_options$reporter = options.reporter) !== null && _options$reporter !== void 0 ? _options$reporter : (_axe$_audit2 = axe._audit) === null || _axe$_audit2 === void 0 ? void 0 : _axe$_audit2.reporter) !== null && _ref149 !== void 0 ? _ref149 : 'v1';
       return {
         context: context,
         options: options,
@@ -29924,14 +29933,14 @@
       });
     }
     function finishRun(partialResults) {
-      var _ref152, _options$reporter2, _axe$_audit2;
+      var _ref152, _options$reporter2, _axe$_audit3;
       var options = arguments.length > 1 && arguments[1] !== undefined ? arguments[1] : {};
       options = clone2(options);
       var _ref151 = partialResults.find(function(r) {
         return r.environmentData;
       }) || {}, environmentData = _ref151.environmentData;
       _normalizeRunOptions(options);
-      options.reporter = (_ref152 = (_options$reporter2 = options.reporter) !== null && _options$reporter2 !== void 0 ? _options$reporter2 : (_axe$_audit2 = axe._audit) === null || _axe$_audit2 === void 0 ? void 0 : _axe$_audit2.reporter) !== null && _ref152 !== void 0 ? _ref152 : 'v1';
+      options.reporter = (_ref152 = (_options$reporter2 = options.reporter) !== null && _options$reporter2 !== void 0 ? _options$reporter2 : (_axe$_audit3 = axe._audit) === null || _axe$_audit3 === void 0 ? void 0 : _axe$_audit3.reporter) !== null && _ref152 !== void 0 ? _ref152 : 'v1';
       setFrameSpec(partialResults);
       var results = merge_results_default(partialResults);
       results = axe._audit.after(results, options);
@@ -30125,7 +30134,8 @@
     var exposed_for_testing_default = _thisWillBeDeletedDoNotUse;
     axe._thisWillBeDeletedDoNotUse = exposed_for_testing_default;
     axe.constants = constants_default;
-    axe.log = log_default;
+    axe.log = log;
+    axe._setLogger = setLogger;
     axe.AbstractVirtualNode = abstract_virtual_node_default;
     axe.SerialVirtualNode = serial_virtual_node_default;
     axe.VirtualNode = virtual_node_default;