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

docs: How to test and check test coverage #1514

Merged
merged 2 commits into from
Aug 13, 2020

Conversation

draychev
Copy link
Contributor

This PR adds documentation on how to write tests, what kind of tests we have, what frameworks we use for tests and how to check test coverage.

information so that others can review it.

Please mark with X for applicable areas.

  • New Functionality [ ]
  • Documentation [ X ]
  • Install [ ]
  • Control Plane [ ]
  • CLI Tool [ ]
  • Certificate Management [ ]
  • Networking [ ]
  • Metrics [ ]
  • SMI Policy [ ]
  • Security [ ]
  • Tests / CI System [ ]
  • 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 August 12, 2020 18:44
snehachhabria
snehachhabria previously approved these changes Aug 12, 2020
Copy link
Contributor

@michelleN michelleN left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

small nit: would you mind moving the images to docs/images or docs/assets just to keep the directory clean?

file, which serves as an entry point for the Ginkgo tests within the given package.

#### Unit Tests
The most rudimentary tests are the unit tests. We strive for test coverage above 80% where
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@draychev draychev merged commit 489b7aa into openservicemesh:main Aug 13, 2020
@draychev draychev deleted the view-test-coverage branch August 13, 2020 16:27
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.

4 participants