From f740c1a9520bb168afc14c9b80421a44893a6097 Mon Sep 17 00:00:00 2001 From: Jarek Radosz Date: Fri, 30 Jul 2021 18:05:04 +0200 Subject: [PATCH] DEV: Clear all `navItem` information between tests (#13903) Expands the original `clearCustomNavItemHref` from #13025. Fixes issues with discourse-assign tests. --- app/assets/javascripts/discourse/app/models/nav-item.js | 4 +++- .../javascripts/discourse/tests/helpers/qunit-helpers.js | 4 ++-- 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/app/assets/javascripts/discourse/app/models/nav-item.js b/app/assets/javascripts/discourse/app/models/nav-item.js index 0293fbddf8bbb..9090cac154805 100644 --- a/app/assets/javascripts/discourse/app/models/nav-item.js +++ b/app/assets/javascripts/discourse/app/models/nav-item.js @@ -301,8 +301,10 @@ export function customNavItemHref(cb) { NavItem.customNavItemHrefs.push(cb); } -export function clearCustomNavItemHref() { +export function clearNavItems() { NavItem.customNavItemHrefs.clear(); + NavItem.extraArgsCallbacks.clear(); + NavItem.extraNavItemDescriptors.clear(); } export function addNavItem(item) { diff --git a/app/assets/javascripts/discourse/tests/helpers/qunit-helpers.js b/app/assets/javascripts/discourse/tests/helpers/qunit-helpers.js index b6b441043090d..2d5a3fbf5ff62 100644 --- a/app/assets/javascripts/discourse/tests/helpers/qunit-helpers.js +++ b/app/assets/javascripts/discourse/tests/helpers/qunit-helpers.js @@ -40,7 +40,7 @@ import { setTopicList } from "discourse/lib/topic-list-tracker"; import sinon from "sinon"; import siteFixtures from "discourse/tests/fixtures/site-fixtures"; import { clearResolverOptions } from "discourse-common/resolver"; -import { clearCustomNavItemHref } from "discourse/models/nav-item"; +import { clearNavItems } from "discourse/models/nav-item"; import { cleanUpComposerUploadHandler, cleanUpComposerUploadMarkdownResolver, @@ -274,7 +274,7 @@ export function acceptance(name, optionsOrCallback) { resetOneboxCache(); resetCustomPostMessageCallbacks(); resetUserSearchCache(); - clearCustomNavItemHref(); + clearNavItems(); setTopicList(null); _clearSnapshots(); setURLContainer(null);