Skip to content
This repository has been archived by the owner on Nov 1, 2023. It is now read-only.

Only keep coverage, libfuzzer and template commands #3428

Merged
merged 3 commits into from Aug 21, 2023

Conversation

tevoinea
Copy link
Member

@tevoinea tevoinea commented Aug 17, 2023

Summary of the Pull Request

What is this about?

Closes #3302

Removes the CLI commands for many of the local tasks as they can now be described via yaml template.

This is the new helptext for onefuzz-task local --help

Usage: onefuzz-task local [OPTIONS] [COMMAND]

Commands:
  coverage        execute a local-only coverage task
  libfuzzer-fuzz  execute a local-only libfuzzer fuzzing task
  template        uses the template to generate a run
  help            Print this message or the help of the given subcommand(s)

Options:
      --timeout <timeout>  The maximum running time in seconds
      --tui                Enable the terminal UI
  -h, --help               Print help

@tevoinea tevoinea marked this pull request as ready for review August 17, 2023 19:07
@tevoinea tevoinea enabled auto-merge (squash) August 21, 2023 12:59
@codecov-commenter
Copy link

codecov-commenter commented Aug 21, 2023

Codecov Report

Merging #3428 (05a27f3) into main (c0c5b83) will increase coverage by 3.35%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #3428      +/-   ##
==========================================
+ Coverage   33.59%   36.94%   +3.35%     
==========================================
  Files         300      164     -136     
  Lines       37125    21702   -15423     
==========================================
- Hits        12472     8018    -4454     
+ Misses      24653    13684   -10969     
Files Changed Coverage Δ
src/agent/onefuzz-task/src/local/cmd.rs 0.00% <ø> (ø)
src/agent/onefuzz-task/src/local/common.rs 0.00% <ø> (ø)
...c/agent/onefuzz-task/src/local/generic_analysis.rs 2.12% <ø> (+1.45%) ⬆️
...ent/onefuzz-task/src/local/generic_crash_report.rs 1.78% <ø> (+1.19%) ⬆️
.../agent/onefuzz-task/src/local/generic_generator.rs 2.17% <ø> (+1.56%) ⬆️
src/agent/onefuzz-task/src/local/libfuzzer.rs 2.63% <ø> (+1.92%) ⬆️
...t/onefuzz-task/src/local/libfuzzer_crash_report.rs 1.85% <ø> (+1.20%) ⬆️
...rc/agent/onefuzz-task/src/local/libfuzzer_merge.rs 2.43% <ø> (+1.47%) ⬆️
...ent/onefuzz-task/src/local/libfuzzer_regression.rs 1.96% <ø> (+1.33%) ⬆️
...ent/onefuzz-task/src/local/libfuzzer_test_input.rs 3.03% <ø> (+2.04%) ⬆️
... and 1 more

... and 135 files with indirect coverage changes

@tevoinea tevoinea merged commit a551709 into microsoft:main Aug 21, 2023
24 checks passed
@AdamL-Microsoft AdamL-Microsoft mentioned this pull request Aug 30, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cleanup the number of exposed commands
3 participants