From b8d05ef5c0ddb400c98439e90cd798989d115156 Mon Sep 17 00:00:00 2001 From: Robin Malfait Date: Fri, 10 Jun 2022 11:12:41 +0200 Subject: [PATCH] restructure test a little bit Instead of relying on a arbitrary setTimout value, let's wait for the file to be created instead. --- integrations/tailwindcss-cli/tests/cli.test.js | 17 +++++++++++++---- 1 file changed, 13 insertions(+), 4 deletions(-) diff --git a/integrations/tailwindcss-cli/tests/cli.test.js b/integrations/tailwindcss-cli/tests/cli.test.js index 7b421bc60d2d..9ebcd3012b91 100644 --- a/integrations/tailwindcss-cli/tests/cli.test.js +++ b/integrations/tailwindcss-cli/tests/cli.test.js @@ -5,7 +5,14 @@ let resolveToolRoot = require('../../resolve-tool-root') let version = require('../../../package.json').version -let { readOutputFile, writeInputFile, cleanupFile, fileExists, removeFile } = require('../../io')({ +let { + cleanupFile, + fileExists, + readOutputFile, + removeFile, + waitForOutputFileCreation, + writeInputFile, +} = require('../../io')({ output: 'dist', input: 'src', }) @@ -388,11 +395,11 @@ describe('Build command', () => { ` ) - let proc = $( + let runningProcess = $( `${EXECUTABLE} --watch --input ./src/test.css --content ./src/index.html --output ./dist/main.css` ) - await new Promise((r) => setTimeout(r, 500)) - await proc.stop() + + await waitForOutputFileCreation('main.css') expect(await readOutputFile('main.css')).toIncludeCss( css` @@ -403,6 +410,8 @@ describe('Build command', () => { } ` ) + + return runningProcess.stop() }) test('postcss-import is included when using a custom postcss configuration', async () => {