From 394bf1f497f5cf575bb5efc3746ec18559617015 Mon Sep 17 00:00:00 2001 From: Kristiyan Kostadinov Date: Sat, 7 Jan 2023 12:15:46 +0100 Subject: [PATCH] build: re-enable material e2e tests The Material tests had to be disabled temporarily during the TS 4.9 update, because the peer dependency range didn't allow for `next` versions. Now that the range was expanded in https://github.com/angular/components/pull/26308, the tests should work again. --- tests/legacy-cli/e2e/tests/build/material.ts | 3 --- tests/legacy-cli/e2e/tests/commands/add/add-material.ts | 3 --- .../e2e/tests/misc/invalid-schematic-dependencies.ts | 3 --- 3 files changed, 9 deletions(-) diff --git a/tests/legacy-cli/e2e/tests/build/material.ts b/tests/legacy-cli/e2e/tests/build/material.ts index 9c9e36417659..a010ea834791 100644 --- a/tests/legacy-cli/e2e/tests/build/material.ts +++ b/tests/legacy-cli/e2e/tests/build/material.ts @@ -7,9 +7,6 @@ import { isPrereleaseCli, updateJsonFile } from '../../utils/project'; const snapshots = require('../../ng-snapshot/package.json'); export default async function () { - // TODO(crisbeto): temporarily disabled until Material is updated - return; - let tag = (await isPrereleaseCli()) ? '@next' : ''; await ng('add', `@angular/material${tag}`, '--skip-confirmation'); diff --git a/tests/legacy-cli/e2e/tests/commands/add/add-material.ts b/tests/legacy-cli/e2e/tests/commands/add/add-material.ts index 8fd27b768e57..bca90ca6c032 100644 --- a/tests/legacy-cli/e2e/tests/commands/add/add-material.ts +++ b/tests/legacy-cli/e2e/tests/commands/add/add-material.ts @@ -5,9 +5,6 @@ import { ng } from '../../../utils/process'; import { isPrereleaseCli } from '../../../utils/project'; export default async function () { - // TODO(crisbeto): temporarily disabled until Material is updated - return; - // forcibly remove in case another test doesn't clean itself up await rimraf('node_modules/@angular/material'); diff --git a/tests/legacy-cli/e2e/tests/misc/invalid-schematic-dependencies.ts b/tests/legacy-cli/e2e/tests/misc/invalid-schematic-dependencies.ts index fbcd84890a1f..432f21167cdb 100644 --- a/tests/legacy-cli/e2e/tests/misc/invalid-schematic-dependencies.ts +++ b/tests/legacy-cli/e2e/tests/misc/invalid-schematic-dependencies.ts @@ -4,9 +4,6 @@ import { installPackage, uninstallPackage } from '../../utils/packages'; import { isPrereleaseCli } from '../../utils/project'; export default async function () { - // TODO(crisbeto): temporarily disabled until Material is updated - return; - // Must publish old version to local registry to allow install. This is especially important // for release commits as npm will try to request tooling packages that are not on the npm registry yet await publishOutdated('@schematics/angular@7');