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

injector: Simplify tests and use YAML values closer to reality #1776

Merged
merged 1 commit into from
Sep 30, 2020

Conversation

draychev
Copy link
Contributor

@draychev draychev commented Sep 30, 2020

This PR is a step in preparation towards #1772, which adds new unit tests for the injector package. These are small tweaks in string literals, no functional code changes.

This PR is stacked on top of #1773
This PR is a small piece of #1772


Please mark with X for applicable areas.

  • New Functionality [ ]
  • Documentation [ ]
  • Install [ ]
  • Control Plane [ ]
  • CLI Tool [ ]
  • Certificate Management [ ]
  • Networking [ ]
  • Metrics [ ]
  • SMI Policy [ ]
  • Security [ ]
  • Tests / CI System [X]
  • Other [ ]

Please answer the following questions with yes/no.

  • Does this change contain code from or inspired by another project? If so, did you notify the maintainers and provide attribution? no

@draychev draychev requested a review from a team as a code owner September 30, 2020 22:26
@draychev draychev linked an issue Sep 30, 2020 that may be closed by this pull request
15 tasks
@draychev draychev merged commit 5096fe9 into openservicemesh:main Sep 30, 2020
@draychev draychev deleted the improve-injector-tests branch September 30, 2020 23:31
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.

(root) Improve test coverage (goal: 80%)
3 participants