fixing up entirely-skipped tests

tests/scenarios/engines-test.ts

@@ -81,11 +81,7 @@ let engineScenarios = appScenarios.map('engines', project => {
 });
 
 engineScenarios
-  .skip('lts_3_28-engines') // this skip should be removed before https://github.com/embroider-build/embroider/pull/1435 is merged
-  .skip('lts_4_4-engines') // this skip should be removed before https://github.com/embroider-build/embroider/pull/1435 is merged
-  .skip('release-engines') // this skip should be removed before https://github.com/embroider-build/embroider/pull/1435 is merged
-  .skip('lts_5_8-engines') // this shouldn't be run
-  .skip('canary-engines') // this shouldn't be run
+  .skip()
   .map('without-fastboot', () => {})
   .forEachScenario(scenario => {
     Qmodule(scenario.name, function (hooks) {

tests/scenarios/fastboot-app-test.ts

@@ -83,8 +83,7 @@ appScenarios
     merge(project.files, loadFromFixtureData('fastboot-app'));
   })
   // TODO remove once https://github.com/ember-fastboot/ember-cli-fastboot/issues/925 is fixed
-  .skip('lts_5_8-fastboot-app-test')
-  .skip('canary-fastboot-app-test')
+  .skip()
   .forEachScenario(scenario => {
     Qmodule(scenario.name, function (hooks) {
       let app: PreparedApp;

tests/scenarios/watch-mode-test.ts

@@ -9,15 +9,12 @@ import CommandWatcher, { DEFAULT_TIMEOUT } from './helpers/command-watcher';
 
 const { module: Qmodule, test } = QUnit;
 
-let app = appScenarios
-  .skip('canary')
-  .skip('lts_5_8')
-  .map('watch-mode', () => {
-    /**
-     * We will create files as a part of the watch-mode tests,
-     * because creating files should cause appropriate watch/update behavior
-     */
-  });
+let app = appScenarios.skip().map('watch-mode', () => {
+  /**
+   * We will create files as a part of the watch-mode tests,
+   * because creating files should cause appropriate watch/update behavior
+   */
+});
 
 class File {
   constructor(readonly label: string, readonly fullPath: string) {}