Skip to content

Commit

Permalink
add tests
Browse files Browse the repository at this point in the history
  • Loading branch information
@dana-yaish
dana-yaish committed Mar 22, 2023
1 parent 42678fb commit 4aa264a
Show file tree
Hide file tree
Showing 5 changed files with 266 additions and 51 deletions.
44 changes: 22 additions & 22 deletions dist/index.js
View file
Original file line number Diff line number Diff line change
Expand Up @@ -22344,8 +22344,8 @@ const buildUploadExec = () => {
const workingDir = core.getInput('working-directory');
const plugin = core.getInput('plugin');
const exclude = core.getInput('exclude');
const execUploadArgs = [];
execUploadArgs.push('-n', `${name}`);
const uploadExecArgs = [];
uploadExecArgs.push('-n', `${name}`);
const uploadCommand = 'do-upload';
const uploadOptions = {};
uploadOptions.env = Object.assign(process.env, {
Expand All @@ -22368,77 +22368,77 @@ const buildUploadExec = () => {
uploadOptions.env.CODECOV_TOKEN = token;
}
if (dryRun) {
execUploadArgs.push('-d');
uploadExecArgs.push('-d');
}
if (envVarsArg.length) {
execUploadArgs.push('-e', envVarsArg.join(','));
uploadExecArgs.push('-e', envVarsArg.join(','));
}
if (failCi) {
execUploadArgs.push('-Z');
uploadExecArgs.push('-Z');
}
if (file) {
execUploadArgs.push('-f', `${file}`);
uploadExecArgs.push('-f', `${file}`);
}
if (files) {
files.split(',').map((f) => f.trim()).forEach((f) => {
execUploadArgs.push('-f', `${f}`);
uploadExecArgs.push('-f', `${f}`);
});
}
if (flags) {
flags.split(',').map((f) => f.trim()).forEach((f) => {
execUploadArgs.push('-F', `${f}`);
uploadExecArgs.push('-F', `${f}`);
});
}
if (overrideBranch) {
execUploadArgs.push('-B', `${overrideBranch}`);
uploadExecArgs.push('-B', `${overrideBranch}`);
}
if (overrideBuild) {
execUploadArgs.push('-b', `${overrideBuild}`);
uploadExecArgs.push('-b', `${overrideBuild}`);
}
if (overrideCommit) {
execUploadArgs.push('-C', `${overrideCommit}`);
uploadExecArgs.push('-C', `${overrideCommit}`);
}
else if (`${context.eventName}` == 'pull_request' ||
`${context.eventName}` == 'pull_request_target') {
execUploadArgs.push('-C', `${context.payload.pull_request.head.sha}`);
uploadExecArgs.push('-C', `${context.payload.pull_request.head.sha}`);
}
if (overridePr) {
execUploadArgs.push('-P', `${overridePr}`);
uploadExecArgs.push('-P', `${overridePr}`);
}
else if (`${context.eventName}` == 'pull_request_target') {
execUploadArgs.push('-P', `${context.payload.number}`);
uploadExecArgs.push('-P', `${context.payload.number}`);
}
if (rootDir) {
execUploadArgs.push('--network-root-folder', `${rootDir}`);
uploadExecArgs.push('--network-root-folder', `${rootDir}`);
}
if (searchDir) {
execUploadArgs.push('-s', `${searchDir}`);
uploadExecArgs.push('-s', `${searchDir}`);
}
if (slug) {
execUploadArgs.push('-r', `${slug}`);
uploadExecArgs.push('-r', `${slug}`);
}
if (workingDir) {
uploadOptions.cwd = workingDir;
}
if (plugin) {
execUploadArgs.push('--plugin', `${plugin}`);
uploadExecArgs.push('--plugin', `${plugin}`);
}
if (exclude) {
execUploadArgs.push('--exclude', `${exclude}`);
uploadExecArgs.push('--exclude', `${exclude}`);
}
if (uploaderVersion == '') {
uploaderVersion = 'latest';
}
return {
execUploadArgs,
uploadExecArgs,
uploadOptions,
failCi,
os,
uploaderVersion,
uploadCommand,
};
};
/* harmony default export */ const src_buildExec = (buildExec);


;// CONCATENATED MODULE: ./src/helpers.ts

Expand Down Expand Up @@ -24756,7 +24756,7 @@ var src_awaiter = (undefined && undefined.__awaiter) || function (thisArg, _argu

let failCi;
try {
const { execArgs, options, failCi, os, uploaderVersion, verbose } = src_buildExec();
const { execArgs, options, failCi, os, uploaderVersion, verbose } = buildExec();
const platform = getPlatform(os);
const filename = external_path_.join(__dirname, getUploaderName(platform));
external_https_.get(getBaseUrl(platform, uploaderVersion), (res) => {
Expand Down
2 changes: 1 addition & 1 deletion dist/index.js.map
View file

Large diffs are not rendered by default.

221 changes: 215 additions & 6 deletions src/buildExec.test.ts
View file
Original file line number Diff line number Diff line change
@@ -1,9 +1,14 @@
import * as github from '@actions/github';

import buildExec from './buildExec';
import {
buildCommitExec,
buildExec,
buildGeneralExec,
buildReportExec,
buildUploadExec,
} from './buildExec';

/* eslint-disable @typescript-eslint/no-var-requires */
const {version} = require('../package.json');
import {version} from '../package.json';

const context = github.context;

Expand All @@ -19,6 +24,7 @@ test('no arguments', () => {
if (context.eventName == 'pull_request') {
args.push('-C', `${context.payload.pull_request.head.sha}`);
}

expect(execArgs).toEqual(args);
expect(failCi).toBeFalsy();
});
Expand Down Expand Up @@ -131,7 +137,7 @@ describe('trim arguments after splitting them', () => {
];

test('files', () => {
const envs = {'files': './client-coverage.txt, ./lcov.info'};
const envs = {files: './client-coverage.txt, ./lcov.info'};

for (const [name, value] of Object.entries(envs)) {
process.env['INPUT_' + name.toUpperCase()] = value;
Expand All @@ -155,7 +161,7 @@ describe('trim arguments after splitting them', () => {
});

test('flags', () => {
const envs = {'flags': 'ios, mobile'};
const envs = {flags: 'ios, mobile'};

for (const [name, value] of Object.entries(envs)) {
process.env['INPUT_' + name.toUpperCase()] = value;
Expand All @@ -179,7 +185,7 @@ describe('trim arguments after splitting them', () => {
});

test('functionalities', () => {
const envs = {'functionalities': 'network, gcov'};
const envs = {functionalities: 'network, gcov'};

for (const [name, value] of Object.entries(envs)) {
process.env['INPUT_' + name.toUpperCase()] = value;
Expand All @@ -202,3 +208,206 @@ describe('trim arguments after splitting them', () => {
}
});
});

test('general args', () => {
const envs = {
url: 'https://codecov.enterprise.com',
verbose: 't',
};
for (const env of Object.keys(envs)) {
process.env['INPUT_' + env.toUpperCase()] = envs[env];
}

const args = buildGeneralExec();

expect(args).toEqual(
expect.arrayContaining([
'--enterprise-url',
'https://codecov.enterprise.com',
'-v',
]));

for (const env of Object.keys(envs)) {
delete process.env['INPUT_' + env.toUpperCase()];
}
});


test('upload args using context', () => {
const expectedArgs = [
'-n',
'',
];
const {uploadExecArgs, uploadCommand} = buildUploadExec();
if (context.eventName == 'pull_request') {
expectedArgs.push('-C', `${context.payload.pull_request.head.sha}`);
}
if (context.eventName == 'pull_request_target') {
expectedArgs.push('-P', `${context.payload.number}`);
}

expect(uploadExecArgs).toEqual(expectedArgs);
expect(uploadCommand).toEqual('do-upload');
});

test('upload args', () => {
const envs = {
'directory': 'coverage/',
'dry_run': 'true',
'env_vars': 'OS,PYTHON',
'fail_ci_if_error': 'true',
'file': 'coverage.xml',
'files': 'dir1/coverage.xml,dir2/coverage.xml',
'flags': 'test,test2',
'name': 'codecov',
'override_branch': 'thomasrockhu/test',
'override_build': '1',
'override_commit': '9caabca5474b49de74ef5667deabaf74cdacc244',
'override_pr': '2',
'root_dir': 'root/',
'slug': 'fakeOwner/fakeRepo',
'token': 'd3859757-ab80-4664-924d-aef22fa7557b',
'working-directory': 'src',
};
for (const env of Object.keys(envs)) {
process.env['INPUT_' + env.toUpperCase()] = envs[env];
}

const {uploadExecArgs, uploadCommand} = buildUploadExec();

expect(uploadExecArgs).toEqual(
expect.arrayContaining([
'-n',
'codecov',
'-d',
'-e',
'OS,PYTHON',
'-Z',
'-f',
'coverage.xml',
'-f',
'dir1/coverage.xml',
'-f',
'dir2/coverage.xml',
'-F',
'test',
'-F',
'test2',
'-B',
'thomasrockhu/test',
'-b',
'1',
'-C',
'9caabca5474b49de74ef5667deabaf74cdacc244',
'-P',
'2',
'--network-root-folder',
'root/',
'-s',
'coverage/',
'-r',
'fakeOwner/fakeRepo',
]));
expect(uploadCommand).toEqual('do-upload');
for (const env of Object.keys(envs)) {
delete process.env['INPUT_' + env.toUpperCase()];
}
});


test('report args', () => {
const envs = {
override_commit: '9caabca5474b49de74ef5667deabaf74cdacc244',
slug: 'fakeOwner/fakeRepo',
token: 'd3859757-ab80-4664-924d-aef22fa7557b',
};
for (const env of Object.keys(envs)) {
process.env['INPUT_' + env.toUpperCase()] = envs[env];
}

const {reportExecArgs, reportCommand} = buildReportExec();

expect(reportExecArgs).toEqual(
expect.arrayContaining([
'-C',
'9caabca5474b49de74ef5667deabaf74cdacc244',
'--slug',
'fakeOwner/fakeRepo',
]));
expect(reportCommand).toEqual('create-report');
for (const env of Object.keys(envs)) {
delete process.env['INPUT_' + env.toUpperCase()];
}
});


test('report args using context', () => {
const envs = {
token: 'd3859757-ab80-4664-924d-aef22fa7557b',
};
for (const env of Object.keys(envs)) {
process.env['INPUT_' + env.toUpperCase()] = envs[env];
}
const expectedArgs : string[] = [];
if (context.eventName == 'pull_request') {
expectedArgs.push('-C', `${context.payload.pull_request.head.sha}`);
}

const {reportExecArgs, reportCommand} = buildReportExec();

expect(reportExecArgs).toEqual(expectedArgs);
expect(reportCommand).toEqual('create-report');
for (const env of Object.keys(envs)) {
delete process.env['INPUT_' + env.toUpperCase()];
}
});


test('commit args', () => {
const envs = {
override_commit: '9caabca5474b49de74ef5667deabaf74cdacc244',
slug: 'fakeOwner/fakeRepo',
token: 'd3859757-ab80-4664-924d-aef22fa7557b',
override_branch: 'thomasrockhu/test',
override_pr: '2',
commit_parent: '83231650328f11695dfb754ca0f540516f188d27',
};
for (const env of Object.keys(envs)) {
process.env['INPUT_' + env.toUpperCase()] = envs[env];
}

const {commitExecArgs, commitCommand} = buildCommitExec();

expect(commitExecArgs).toEqual(
expect.arrayContaining([
'-C',
'9caabca5474b49de74ef5667deabaf74cdacc244',
'--slug',
'fakeOwner/fakeRepo',
'-B',
'thomasrockhu/test',
'--pr',
'2',
'--parent-sha',
'83231650328f11695dfb754ca0f540516f188d27',
]));
expect(commitCommand).toEqual('create-commit');
for (const env of Object.keys(envs)) {
delete process.env['INPUT_' + env.toUpperCase()];
}
});

test('commit args using context', () => {
const expectedArgs :string[] = [];

const {commitExecArgs, commitCommand} = buildCommitExec();
if (context.eventName == 'pull_request') {
expectedArgs.push('-C', `${context.payload.pull_request.head.sha}`);
}
if (context.eventName == 'pull_request_target') {
expectedArgs.push('-P', `${context.payload.number}`);
}

expect(commitExecArgs).toEqual(expectedArgs);
expect(commitCommand).toEqual('create-commit');
});

0 comments on commit 4aa264a

Please sign in to comment.