Add all args (#1245)

* fix: add all args from cli * chore(release): bump to 4.0.1
2026-02-11 21:01:38 +00:00 · 2024-02-01 09:05:17 -08:00
parent 09686fcfcb
commit 1f9f5573d1
8 changed files with 219 additions and 104 deletions
--- a/src/buildExec.test.ts
+++ b/src/buildExec.test.ts
@@ -12,6 +12,7 @@ const context = github.context;

 test('general args', () => {
  const envs = {
+    codecov_yml_path: 'dev/codecov.yml',
    url: 'https://codecov.enterprise.com',
    verbose: 't',
  };
@@ -23,6 +24,8 @@ test('general args', () => {

  expect(args).toEqual(
      expect.arrayContaining([
+        '--codecov-yml-path',
+        'dev/codecov.yml',
        '--enterprise-url',
        'https://codecov.enterprise.com',
        '-v',
@@ -50,25 +53,38 @@ test('upload args using context', () => {

 test('upload args', () => {
  const envs = {
+    'codecov_yml_path': 'dev/codecov.yml',
+    'commit_parent': 'fakeparentcommit',
    'directory': 'coverage/',
+    'disable_file_fixes': 'true',
+    'disable_search': 'true',
    'dry_run': 'true',
    'env_vars': 'OS,PYTHON',
+    'exclude': 'node_modules/',
    'fail_ci_if_error': 'true',
    'file': 'coverage.xml',
    'files': 'dir1/coverage.xml,dir2/coverage.xml',
    'flags': 'test,test2',
+    'handle_no_reports_found': 'true',
+    'job_code': '32',
    'name': 'codecov',
+    'os': 'macos',
    'override_branch': 'thomasrockhu/test',
    'override_build': '1',
+    'override_build_url': 'https://example.com/build/2',
    'override_commit': '9caabca5474b49de74ef5667deabaf74cdacc244',
    'override_pr': '2',
+    'plugin': 'xcode',
    'plugins': 'pycoverage,compress-pycoverage',
+    'report_code': 'testCode',
    'root_dir': 'root/',
    'slug': 'fakeOwner/fakeRepo',
    'token': 'd3859757-ab80-4664-924d-aef22fa7557b',
+    'url': 'https://enterprise.example.com',
+    'use_legacy_upload_endpoint': 'true',
+    'verbose': 'true',
+    'version': '0.1.2',
    'working-directory': 'src',
-    'plugin': 'xcode',
-    'exclude': 'src',
  };
  for (const env of Object.keys(envs)) {
    process.env['INPUT_' + env.toUpperCase()] = envs[env];
@@ -76,11 +92,13 @@ test('upload args', () => {

  const {uploadExecArgs, uploadCommand} = buildUploadExec();
  const expectedArgs = [
-    '-n',
-    'codecov',
+    '--disable-file-fixes',
+    '--disable-search',
    '-d',
    '-e',
    'OS,PYTHON',
+    '--exclude',
+    'node_modules/',
    '-Z',
    '-f',
    'coverage.xml',
@@ -92,28 +110,36 @@ test('upload args', () => {
    'test',
    '-F',
    'test2',
+    '--handle-no-reports-found',
+    '--job-code',
+    '32',
+    '-n',
+    'codecov',
    '-B',
    'thomasrockhu/test',
    '-b',
    '1',
+    '--build-url',
+    'https://example.com/build/2',
    '-C',
    '9caabca5474b49de74ef5667deabaf74cdacc244',
    '-P',
    '2',
    '--plugin',
+    'xcode',
+    '--plugin',
    'pycoverage',
    '--plugin',
    'compress-pycoverage',
+    '--report-code',
+    'testCode',
    '--network-root-folder',
    'root/',
    '-s',
    'coverage/',
    '-r',
    'fakeOwner/fakeRepo',
-    '--plugin',
-    'xcode',
-    '--exclude',
-    'src',
+    '--legacy',
  ];

  expect(uploadExecArgs).toEqual(expectedArgs);
@@ -127,8 +153,10 @@ test('upload args', () => {
 test('report args', () => {
  const envs = {
    override_commit: '9caabca5474b49de74ef5667deabaf74cdacc244',
+    override_pr: 'fakePR',
    slug: 'fakeOwner/fakeRepo',
    token: 'd3859757-ab80-4664-924d-aef22fa7557b',
+    fail_ci_if_error: 'true',
  };
  for (const env of Object.keys(envs)) {
    process.env['INPUT_' + env.toUpperCase()] = envs[env];
@@ -136,13 +164,17 @@ test('report args', () => {

  const {reportExecArgs, reportCommand} = buildReportExec();

-  expect(reportExecArgs).toEqual(
-      expect.arrayContaining([
-        '-C',
-        '9caabca5474b49de74ef5667deabaf74cdacc244',
-        '--slug',
-        'fakeOwner/fakeRepo',
-      ]));
+  const expectedArgs = [
+    '-C',
+    '9caabca5474b49de74ef5667deabaf74cdacc244',
+    '-P',
+    'fakePR',
+    '--slug',
+    'fakeOwner/fakeRepo',
+    '-Z',
+  ];
+
+  expect(reportExecArgs).toEqual(expectedArgs);
  expect(reportCommand).toEqual('create-report');
  for (const env of Object.keys(envs)) {
    delete process.env['INPUT_' + env.toUpperCase()];
@@ -174,32 +206,34 @@ test('report args using context', () => {

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

  const {commitExecArgs, commitCommand} = buildCommitExec();
+  const expectedArgs = [
+    '--parent-sha',
+    '83231650328f11695dfb754ca0f540516f188d27',
+    '-B',
+    'thomasrockhu/test',
+    '-C',
+    '9caabca5474b49de74ef5667deabaf74cdacc244',
+    '--pr',
+    '2',
+    '--slug',
+    'fakeOwner/fakeRepo',
+    '-Z',
+  ];

-  expect(commitExecArgs).toEqual(
-      expect.arrayContaining([
-        '-C',
-        '9caabca5474b49de74ef5667deabaf74cdacc244',
-        '--slug',
-        'fakeOwner/fakeRepo',
-        '-B',
-        'thomasrockhu/test',
-        '--pr',
-        '2',
-        '--parent-sha',
-        '83231650328f11695dfb754ca0f540516f188d27',
-      ]));
+  expect(commitExecArgs).toEqual(expectedArgs);
  expect(commitCommand).toEqual('create-commit');
  for (const env of Object.keys(envs)) {
    delete process.env['INPUT_' + env.toUpperCase()];
--- a/src/buildExec.ts
+++ b/src/buildExec.ts
@@ -25,7 +25,7 @@ const buildCommitExec = () => {
  const overridePr = core.getInput('override_pr');
  const slug = core.getInput('slug');
  const token = core.getInput('token');
-
+  const failCi = isTrue(core.getInput('fail_ci_if_error'));

  const commitCommand = 'create-commit';
  const commitExecArgs = [];
@@ -69,16 +69,23 @@ const buildCommitExec = () => {
  if (slug) {
    commitExecArgs.push('--slug', `${slug}`);
  }
+  if (failCi) {
+    commitExecArgs.push('-Z');
+  }


  return {commitExecArgs, commitOptions, commitCommand};
 };

 const buildGeneralExec = () => {
+  const codecovYmlPath = core.getInput('codecov_yml_path');
  const url = core.getInput('url');
  const verbose = isTrue(core.getInput('verbose'));
  const args = [];

+  if (codecovYmlPath) {
+    args.push('--codecov-yml-path', `${codecovYmlPath}`);
+  }
  if (url) {
    args.push('--enterprise-url', `${url}`);
  }
@@ -90,8 +97,10 @@ const buildGeneralExec = () => {

 const buildReportExec = () => {
  const overrideCommit = core.getInput('override_commit');
+  const overridePr = core.getInput('override_pr');
  const slug = core.getInput('slug');
  const token = core.getInput('token');
+  const failCi = isTrue(core.getInput('fail_ci_if_error'));


  const reportCommand = 'create-report';
@@ -119,35 +128,54 @@ const buildReportExec = () => {
  ) {
    reportExecArgs.push('-C', `${context.payload.pull_request.head.sha}`);
  }
+  if (overridePr) {
+    reportExecArgs.push('-P', `${overridePr}`);
+  } else if (
+    `${context.eventName}` == 'pull_request_target'
+  ) {
+    reportExecArgs.push('-P', `${context.payload.number}`);
+  }
  if (slug) {
    reportExecArgs.push('--slug', `${slug}`);
  }
+  if (failCi) {
+    reportExecArgs.push('-Z');
+  }

  return {reportExecArgs, reportOptions, reportCommand};
 };

 const buildUploadExec = () => {
-  const envVars = core.getInput('env_vars');
+  const disableFileFixes = isTrue(core.getInput('disable_file_fixes'));
+  const disableSearch = isTrue(core.getInput('disable_search'));
  const dryRun = isTrue(core.getInput('dry_run'));
+  const envVars = core.getInput('env_vars');
+  const exclude = core.getInput('exclude');
  const failCi = isTrue(core.getInput('fail_ci_if_error'));
  const file = core.getInput('file');
  const files = core.getInput('files');
  const flags = core.getInput('flags');
+  const handleNoReportsFound = isTrue(core.getInput('handle_no_reports_found'));
+  const jobCode = core.getInput('job_code');
  const name = core.getInput('name');
  const os = core.getInput('os');
  const overrideBranch = core.getInput('override_branch');
  const overrideBuild = core.getInput('override_build');
+  const overrideBuildUrl = core.getInput('override_build_url');
  const overrideCommit = core.getInput('override_commit');
  const overridePr = core.getInput('override_pr');
+  const plugin = core.getInput('plugin');
  const plugins = core.getInput('plugins');
+  const reportCode = core.getInput('report_code');
  const rootDir = core.getInput('root_dir');
  const searchDir = core.getInput('directory');
  const slug = core.getInput('slug');
  const token = core.getInput('token');
  let uploaderVersion = core.getInput('version');
+  const useLegacyUploadEndpoint = isTrue(
+      core.getInput('use_legacy_upload_endpoint'),
+  );
  const workingDir = core.getInput('working-directory');
-  const plugin = core.getInput('plugin');
-  const exclude = core.getInput('exclude');

  const uploadExecArgs = [];
  const uploadCommand = 'do-upload';
@@ -169,21 +197,24 @@ const buildUploadExec = () => {
      envVarsArg.push(envVarClean);
    }
  }
-  if (name) {
-    uploadExecArgs.push(
-        '-n',
-        `${name}`,
-    );
-  }
  if (token) {
    uploadOptions.env.CODECOV_TOKEN = token;
  }
+  if (disableFileFixes) {
+    uploadExecArgs.push('--disable-file-fixes');
+  }
+  if (disableSearch) {
+    uploadExecArgs.push('--disable-search');
+  }
  if (dryRun) {
    uploadExecArgs.push('-d');
  }
  if (envVarsArg.length) {
    uploadExecArgs.push('-e', envVarsArg.join(','));
  }
+  if (exclude) {
+    uploadExecArgs.push('--exclude', `${exclude}`);
+  }
  if (failCi) {
    uploadExecArgs.push('-Z');
  }
@@ -200,12 +231,24 @@ const buildUploadExec = () => {
      uploadExecArgs.push('-F', `${f}`);
    });
  }
+  if (handleNoReportsFound) {
+    uploadExecArgs.push('--handle-no-reports-found');
+  }
+  if (jobCode) {
+    uploadExecArgs.push('--job-code', `${jobCode}`);
+  }
+  if (name) {
+    uploadExecArgs.push('-n', `${name}`);
+  }
  if (overrideBranch) {
    uploadExecArgs.push('-B', `${overrideBranch}`);
  }
  if (overrideBuild) {
    uploadExecArgs.push('-b', `${overrideBuild}`);
  }
+  if (overrideBuildUrl) {
+    uploadExecArgs.push('--build-url', `${overrideBuildUrl}`);
+  }
  if (overrideCommit) {
    uploadExecArgs.push('-C', `${overrideCommit}`);
  } else if (
@@ -221,11 +264,17 @@ const buildUploadExec = () => {
  ) {
    uploadExecArgs.push('-P', `${context.payload.number}`);
  }
+  if (plugin) {
+    uploadExecArgs.push('--plugin', `${plugin}`);
+  }
  if (plugins) {
    plugins.split(',').map((p) => p.trim()).forEach((p) => {
      uploadExecArgs.push('--plugin', `${p}`);
    });
  }
+  if (reportCode) {
+    uploadExecArgs.push('--report-code', `${reportCode}`);
+  }
  if (rootDir) {
    uploadExecArgs.push('--network-root-folder', `${rootDir}`);
  }
@@ -238,16 +287,12 @@ const buildUploadExec = () => {
  if (workingDir) {
    uploadOptions.cwd = workingDir;
  }
-  if (plugin) {
-    uploadExecArgs.push('--plugin', `${plugin}`);
-  }
-  if (exclude) {
-    uploadExecArgs.push('--exclude', `${exclude}`);
-  }
-
  if (uploaderVersion == '') {
    uploaderVersion = 'latest';
  }
+  if (useLegacyUploadEndpoint) {
+    uploadExecArgs.push('--legacy');
+  }

  return {
    uploadExecArgs,