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.

Sign up for GitHub

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

Jump to bottom

[Stack monitoring][Angular removal] Add unit tests #121149

Closed
Tracked by #112512
matschaffer opened this issue Dec 14, 2021 · 1 comment
Closed
Tracked by #112512

[Stack monitoring][Angular removal] Add unit tests #121149

matschaffer opened this issue Dec 14, 2021 · 1 comment
Labels
Feature:Stack Monitoring Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services

Comments

@matschaffer
Copy link
Contributor

matschaffer commented Dec 14, 2021 •
edited
Loading

As @estermv called out in #112512, the unit tests are a bit thin for Stack Monitoring.

We should come up with a list of what lacks testing (perhaps a coverage report?) and work it down.
@matschaffer matschaffer mentioned this issue Dec 14, 2021
Closed
3 tasks
@matschaffer matschaffer added Feature:Stack Monitoring Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services labels Dec 14, 2021
@smith
Copy link
Contributor

smith commented Apr 12, 2022

Closing this. Please write tests while working on the code!

@smith smith closed this as completed Apr 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Feature:Stack Monitoring Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@matschaffer @smith