Skip to content

Fix an error in warm-jest-cache by ignoring all filtered_search files

Nick Malcolm requested to merge fix-warm-jest-cache into master

What does this MR do and why?

The warm-jest-cache job is failing in at least two team member MRs for reasons unrelated to the changes they're making. The recommendation in Slack was to ignore the whole filtered_search folder.

If this MR is merged first, they can rebase and having passing pipelines.

Explain quick actions inline in RTE (!158367 - merged) already has this included in their MR, but I want the change too so that my pipeline will pass, and I'm not sure when their MR will get merged 😅

See also: https://gitlab.slack.com/archives/C010NAWPRV4/p1720113685680219 where this fix was suggested

The error

 FAIL  spec/frontend/_warm_cache/036_spec.js
  ● Test suite failed to run
    TypeError: Class extends value undefined is not a constructor or null
       6 | import FilteredSearchDropdown from './filtered_search_dropdown';
       7 |
    >  8 | export default class DropdownEmoji extends FilteredSearchDropdown {
         |                                            ^
       9 |   constructor(options = {}) {
      10 |     super(options);
      11 |     this.config = {
      at Object.FilteredSearchDropdown (app/assets/javascripts/filtered_search/dropdown_emoji.js:8:44)
      at Object.require (app/assets/javascripts/filtered_search/available_dropdown_mappings.js:17:1)
      at Object.require (ee/app/assets/javascripts/filtered_search/available_dropdown_mappings.js:1:1)
      at Object.require (app/assets/javascripts/filtered_search/filtered_search_dropdown_manager.js:2:1)
      at Object.require (app/assets/javascripts/filtered_search/dropdown_utils.js:4:1)
      at Object.require (app/assets/javascripts/filtered_search/filtered_search_dropdown.js:3:1)
      at Object.require (spec/frontend/_warm_cache/036_spec.js:1:1)

      ...
Jest cache warming failed!
If the failure is due to an import error, add the problematic file(s) to the ignore list in scripts/frontend/warm_jest_cache.mjs.

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Screenshots or screen recordings

Screenshots are required for UI changes, and strongly recommended for all other merge requests.

Before After

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

Merge request reports