test: Add unit test for quality endpoint, improve message for not-implemented - Githubissues

amundsen-io / amundsen

Amundsen is a metadata driven application for improving the productivity of data analysts, data scientists and engineers when interacting with data.

https://www.amundsen.io/amundsen/

Apache License 2.0

4.39k stars 954 forks source link

test: Add unit test for quality endpoint, improve message for not-implemented #2127

Closed B-T-D closed 1 year ago

B-T-D commented 1 year ago

Description

Adds a simple test for the quality/v0.py endpoint, to have some basic test coverage.
Also updates the message returned when a quality checks client isn't configured to be more verbose and descriptive. Based on the analogous message in announcements/v0.py.

Motivation and Context

Test was similar to the one I wrote for the notice endpoint so seemed like an easy opportunity to improve test coverage.

How Has This Been Tested?

Ran the new unit tests locally.

Documentation

Didn't modify any documentation.

CheckList

[x] PR title addresses the issue accurately and concisely
[ ] Updates Documentation and Docstrings
[x] Adds tests
[ ] Adds instrumentation (logs, or UI events)