Skip to content
This repository has been archived by the owner on Jan 22, 2025. It is now read-only.

Document ProgramTest::new and fix ProgramTest::add_program (backport #17754) #17767

Merged
merged 1 commit into from
Jun 6, 2021

Conversation

mergify[bot]
Copy link
Contributor

@mergify mergify bot commented Jun 6, 2021

This is an automatic backport of pull request #17754 done by Mergify.


Mergify commands and options

More conditions and actions can be found in the documentation.

You can also trigger Mergify actions by commenting on this pull request:

  • @Mergifyio refresh will re-evaluate the rules
  • @Mergifyio rebase will rebase this PR on its base branch
  • @Mergifyio update will merge the base branch into this PR
  • @Mergifyio backport <destination> will backport this PR on <destination> branch

Additionally, on Mergify dashboard you can:

  • look at your merge queues
  • generate the Mergify configuration with the config editor.

Finally, you can contact us on https://mergify.io/

* document ProgramTest::new

* simplify ProgramTest::new doc-string

* make ProgramTest::add_program noisier

`add_program` (and `new`, implicitly) now prints a warning when the user
supplies a bogus program name to a ProgramTest and invokes `test-bpf`.

Additionally, it is now impossible to ask for a regular `test` and for
the generated ProgramTest to load BPF code instead of native code.
Previously, this was caused by a precedence issue: BPF code would always
be preferred over native if the program name was valid, regardless of
user choice.

(cherry picked from commit 2aaf557)
@mergify mergify bot added the automerge Merge this Pull Request automatically once CI passes label Jun 6, 2021
@codecov
Copy link

codecov bot commented Jun 6, 2021

Codecov Report

Merging #17767 (9116cc6) into v1.6 (9413051) will decrease coverage by 0.3%.
The diff coverage is 80.9%.

@@            Coverage Diff            @@
##             v1.6   #17767     +/-   ##
=========================================
- Coverage    82.9%    82.6%   -0.4%     
=========================================
  Files         414      423      +9     
  Lines      114017   116768   +2751     
=========================================
+ Hits        94544    96470   +1926     
- Misses      19473    20298    +825     

@mergify mergify bot merged commit a29344e into v1.6 Jun 6, 2021
@mergify mergify bot deleted the mergify/bp/v1.6/pr-17754 branch June 6, 2021 05:56
@brooksprumo brooksprumo mentioned this pull request Aug 23, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
automerge Merge this Pull Request automatically once CI passes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant