Skip to content

Commit

Permalink
readme
Browse files Browse the repository at this point in the history
  • Loading branch information
kb-kerem committed Apr 18, 2024
1 parent 2158431 commit 4d56416
Showing 1 changed file with 3 additions and 4 deletions.
7 changes: 3 additions & 4 deletions tests/env-vars.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,7 @@ describe('Env var tests', () => {
'runs the docker image with SAUCE_VISUAL_BUILD_ID in place',
async () => {
const result = await execute(
`docker run -e SAUCE_USERNAME -e SAUCE_ACCESS_KEY \\
`docker run --rm -e SAUCE_USERNAME -e SAUCE_ACCESS_KEY \\
-e SAUCE_VISUAL_BUILD_ID \\
${process.env.CONTAINER_IMAGE_NAME}`,
{
Expand Down Expand Up @@ -127,7 +127,7 @@ describe('Env var tests', () => {
);

it(
'screenshots are linked to the external build',
'screenshots are linked to the external build with customId',
async () => {
const build = await visualApi.buildWithDiffsByCustomId(customId);
expect(build).toBeTruthy();
Expand Down Expand Up @@ -191,8 +191,7 @@ describe('Env var tests', () => {
expect(build?.name).toEqual(SAUCE_VISUAL_BUILD_NAME);
expect(build?.project).toBe(SAUCE_VISUAL_PROJECT);
expect(build?.branch).toBe(SAUCE_VISUAL_BRANCH);
// TODO: Enable this when API support arrives
//expect(build?.defaultBranch).toBe(SAUCE_VISUAL_DEFAULT_BRANCH);
expect(build?.defaultBranch).toBe(SAUCE_VISUAL_DEFAULT_BRANCH);
},
15 * 1000
);
Expand Down

0 comments on commit 4d56416

Please sign in to comment.