Commit 885047a

committed

test: workaround npm cleanup for ng add tailwind E2E

Add an explicit removal of previous `tailwindcss` at the start of the `commands/add/add-tailwindcss.ts` E2E test to workaround issues with npm not fully uninstalling the package from previous tests.

1 parent 301b50d commit 885047aCopy full SHA for 885047a

File tree

1 file changed

-2

lines changed

tests/legacy-cli/e2e/tests/commands/add
- add-tailwindcss.ts

1 file changed

-2

lines changed

`‎tests/legacy-cli/e2e/tests/commands/add/add-tailwindcss.ts`

Lines changed: 9 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -1,8 +1,15 @@`
`1`		`-import { expectFileToExist, expectFileToMatch } from '../../../utils/fs';`
`2`		`-import { uninstallPackage } from '../../../utils/packages';`
	`1`	`+import { expectFileToExist, expectFileToMatch,rimraf } from '../../../utils/fs';`
	`2`	`+import { getActivePackageManager,uninstallPackage } from '../../../utils/packages';`
`3`	`3`	`import { ng } from '../../../utils/process';`
`4`	`4`
`5`	`5`	`export default async function () {`
	`6`	`+ // In case a previous test installed tailwindcss, clear it.`
	`7`	`+ // (we don't clear node module directories between tests)`
	`8`	`+ // npm does not appear to fully uninstall sometimes`
	`9`	`+ if (getActivePackageManager() === 'npm') {`
	`10`	`+ await rimraf('node_modules/tailwindcss');`
	`11`	`+ }`
	`12`	`+`
`6`	`13`	`try {`
`7`	`14`	`await ng('add', 'tailwindcss', '--skip-confirmation');`
`8`	`15`	`await expectFileToExist('.postcssrc.json');`

0 commit comments

Comments

(0)

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commit 885047a

File tree

1 file changed

1 file changed

`‎tests/legacy-cli/e2e/tests/commands/add/add-tailwindcss.ts`

0 commit comments