Moved loadAllThemes out of test util

- this was skipped on boot, but then called in the test utilities, but only on restart 🙈
- this means that yarn test:acceptance (i.e. running all tests) works, but if you try to run just test/api-acceptance/themes_spec.js it would fail because that uses a fresh boot not a restart/reload
- I've changed this as keeping the test using the real boot, rather than the made-up acceptance-utils tasks as much as possible is way better

core/boot.js

@@ -219,6 +219,10 @@ async function initServices({config}) {
 async function initBackgroundServices({config}) {
     debug('Begin: initBackgroundServices');
 
+    // Load all inactive themes
+    const themeService = require('./server/services/themes');
+    themeService.loadInactiveThemes();
+
     // we don't want to kick off background services that will interfere with tests
     if (process.env.NODE_ENV.match(/^testing/)) {
         return;
@@ -230,10 +234,6 @@ async function initBackgroundServices({config}) {
         await emailAnalyticsJobs.scheduleRecurringJobs();
     }
 
-    // Load all inactive themes
-    const themeService = require('./server/services/themes');
-    themeService.loadInactiveThemes();
-
     const updateCheck = require('./server/update-check');
     updateCheck.scheduleRecurringJobs();
 

test/utils/acceptance-utils.js

@@ -118,9 +118,6 @@ const restartModeGhostStart = async () => {
     // @TODO: why does this happen _after_ URL service
     web.shared.middlewares.customRedirects.reload();
 
-    // Trigger themes to load again
-    themeService.loadInactiveThemes();
-
     // Reload limits service
     limits.init();
 };