Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test: [TERR-53] Code Coverage for Grafana Network Map Panel Plugin Component Testing #45

Closed
wants to merge 3 commits into from

Conversation

sanchezelton
Copy link
Contributor

This PR adds code coverage reporting (through Istanbul) to component testing in the Network Map Grafana Plugin. It is limited to testing non-React components and is seperate from E2E testing.

Checklist

  • PR Title format: type: [TERR-XXX] Short description
    • type is one of: 'build', 'ci', 'chore', 'docs', 'feat', 'fix', 'perf', 'refactor', 'revert', 'style', 'test'
    • TERR-XXX refers to a Jira issue.
    • If this is a breaking change prefix the description with "BREAKING CHANGE:"
  • There is a related Jira issue for this pull request
  • Description should be a meaningful summary of thea changes you are proposing
  • For breaking changes prefix the title with "BREAKING CHANGE:" and include details in the description
  • This PR is as small as possible
  • This PR includes tests related to these changes, or existing tests provide coverage
  • This PR has updated documentation as appropriate

- updated coverage config
- added karma-coverage package by updated package.json
- added .code-workspace to .gitignore
…hub.com:esnet/grafana-esnet-networkmap-panel into feat/TERR-53-code-coverage
- added reporters to karma config
- refactored santize functions into utils module
- narrowed scope of coverage testing to non-d3, non-leaflet, non-pubsub code
- corrected imports of santize functions
- refactored render function to use resolvePath
- removed resolveObjPath fn from templates module
- removed topologyTools module
- added some jsdoc inlinedocs for utils module
- refactored util fn name from getUrlSearchParams to getUrlQueryParams
@sanchezelton
Copy link
Contributor Author

Closed due to changes improved upon and merged via PR-49: test: React Component Code Coverage and Error Validation

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant