Skip to content

Possible to enforce required coverage over certain lines? #14766

@asraa

Description

@asraa

Description
To validate that ENVOY_BUG conditions have code coverage, is it possible to issue warnings or a make a required check that a condition is tested for success and failure by leveraging the existing coverage report?

Motivation:

  • Encourages failure condition testing, especially since these are fatal in debug mode
  • Ensures that Envoy can continue safely even if hitting an ENVOY_BUG condition in release mode.

Difficulties:

  • With a normal coverage report, it's opaque whether the ENVOY_BUG condition passes or fails.

See
#14575

Metadata

Metadata

Assignees

No one assigned

    Labels

    area/securitystalestalebot believes this issue/PR has not been touched recently

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions