Skip to content

Commit

Permalink
PMM-13322 fix cli tests
Browse files Browse the repository at this point in the history
  • Loading branch information
yurkovychv committed Aug 22, 2024
1 parent c0d0a89 commit ad23516
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 3 deletions.
2 changes: 1 addition & 1 deletion cli/tests/pmm-server-only.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ test.describe('PMM Server CLI tests for Docker Environment Variables', async ()
let out: ExecReturn;

await expect(async () => {
out = await cli.exec('docker logs PMM-T225 2>&1 | grep \'Configuration warning: unknown environment variable\'');
out = await cli.exec('docker logs PMM-T225 2>&1 | grep "Configuration warning: unknown environment variable"');
await out.exitCodeEquals(0);
}).toPass({
// Probe, wait 1s, probe, wait 2s, probe, wait 2s, probe, wait 2s, probe, ....
Expand Down
3 changes: 1 addition & 2 deletions cli/tests/remove.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -36,8 +36,7 @@ test.describe('PMM Server CLI tests for Docker Environment Variables', async ()
const output = await cli.exec(`docker exec pmm-client-remove pmm-admin remove ${services[i]}`);
await output.exitCodeEquals(1);
await output.outContains(
`We could not find a service associated with the local node. Please provide "Service ID" or "Service name".
${output.stdout} \n ${JSON.stringify(output.stderr)} \n ${output.code}`,
'We could not find a service associated with the local node. Please provide "Service ID" or "Service name".',
);
}

Expand Down

0 comments on commit ad23516

Please sign in to comment.