Skip to content

Commit

Permalink
test(cli): fix check for arm64 build
Browse files Browse the repository at this point in the history
  • Loading branch information
bernardobridge committed Jun 27, 2024
1 parent 6516628 commit 26d59ab
Show file tree
Hide file tree
Showing 6 changed files with 36 additions and 11 deletions.
8 changes: 6 additions & 2 deletions packages/artillery/test/cloud-e2e/lambda/lambda-bom.test.js
Original file line number Diff line number Diff line change
@@ -1,12 +1,16 @@
const tap = require('tap');
const fs = require('fs');
const { $ } = require('zx');
const { getTestTags, generateTmpReportPath } = require('../../helpers');
const {
getTestTags,
generateTmpReportPath,
getImageArchitecture
} = require('../../helpers');

const tags = getTestTags(['type:acceptance']);

const A9_PATH = process.env.A9_PATH || 'artillery';
const ARCHITECTURE = process.env.HAS_ARM64_BUILD ? 'arm64' : 'x86_64';
const ARCHITECTURE = getImageArchitecture();

tap.before(async () => {
await $`${A9_PATH} -V`;
Expand Down
Original file line number Diff line number Diff line change
@@ -1,11 +1,15 @@
const tap = require('tap');
const fs = require('fs');
const { $ } = require('zx');
const { getTestTags, generateTmpReportPath } = require('../../helpers');
const {
getTestTags,
generateTmpReportPath,
getImageArchitecture
} = require('../../helpers');

const tags = getTestTags(['type:acceptance']);
const A9_PATH = process.env.A9_PATH || 'artillery';
const ARCHITECTURE = process.env.HAS_ARM64_BUILD ? 'arm64' : 'x86_64';
const ARCHITECTURE = getImageArchitecture();

tap.before(async () => {
await $`${A9_PATH} -V`;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,11 @@ const tap = require('tap');
const { $ } = require('zx');
const chalk = require('chalk');
const fs = require('fs');
const { generateTmpReportPath, getTestTags } = require('../../helpers');
const {
generateTmpReportPath,
getTestTags,
getImageArchitecture
} = require('../../helpers');

//NOTE: all these tests report to Artillery Dashboard to dogfood and improve visibility
const tags = getTestTags(['type:acceptance']);
Expand All @@ -13,7 +17,7 @@ tap.beforeEach(async (t) => {
});

const A9_PATH = process.env.A9_PATH || 'artillery';
const ARCHITECTURE = process.env.HAS_ARM64_BUILD ? 'arm64' : 'x86_64';
const ARCHITECTURE = getImageArchitecture();

tap.before(async () => {
await $`${A9_PATH} -V`;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,11 @@ const tap = require('tap');
const { $ } = require('zx');
const chalk = require('chalk');
const fs = require('fs');
const { generateTmpReportPath, getTestTags } = require('../../helpers');
const {
generateTmpReportPath,
getTestTags,
getImageArchitecture
} = require('../../helpers');

const tags = getTestTags(['type:acceptance']);
let reportFilePath;
Expand All @@ -12,7 +16,7 @@ tap.beforeEach(async (t) => {
});

const A9_PATH = process.env.A9_PATH || 'artillery';
const ARCHITECTURE = process.env.HAS_ARM64_BUILD ? 'arm64' : 'x86_64';
const ARCHITECTURE = getImageArchitecture();

tap.before(async () => {
await $`${A9_PATH} -V`;
Expand Down
8 changes: 6 additions & 2 deletions packages/artillery/test/cloud-e2e/lambda/lambda-smoke.test.js
Original file line number Diff line number Diff line change
@@ -1,7 +1,11 @@
const tap = require('tap');
const fs = require('fs');
const { $ } = require('zx');
const { getTestTags, generateTmpReportPath } = require('../../helpers');
const {
getTestTags,
generateTmpReportPath,
getImageArchitecture
} = require('../../helpers');

const tags = getTestTags(['type:acceptance']);

Expand All @@ -12,7 +16,7 @@ tap.beforeEach(async (t) => {
});

const A9_PATH = process.env.A9_PATH || 'artillery';
const ARCHITECTURE = process.env.HAS_ARM64_BUILD ? 'arm64' : 'x86_64';
const ARCHITECTURE = getImageArchitecture();

tap.before(async () => {
await $`${A9_PATH} -V`;
Expand Down
7 changes: 6 additions & 1 deletion packages/artillery/test/helpers/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -47,11 +47,16 @@ function getTestTags(additionalTags) {
return `${repoTag},${actorTag},${ciTag},${additionalTags.join(',')}`;
}

const getImageArchitecture = () => {
return process.env.HAS_ARM64_BUILD === 'true' ? 'arm64' : 'x86_64';
};

module.exports = {
execute,
deleteFile,
getRootPath,
returnTmpPath,
generateTmpReportPath,
getTestTags
getTestTags,
getImageArchitecture
};

0 comments on commit 26d59ab

Please sign in to comment.