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

[TECH DEBT] Create a fixture to read a dashboard query within a test #306

Open
aminmovahed-db opened this issue Aug 9, 2024 · 0 comments

Comments

@aminmovahed-db
Copy link

Maybe we want to implement a fixture to get the query to make sure we test that, something like:

  1. query = get_query("04_1_....sql")
  2. Replace database
  3. Run query
  4. Verify results

Originally posted by @JCZuurmond in databrickslabs/ucx#2205 (comment)

@JCZuurmond JCZuurmond changed the title Create a fixture to pass the query to be tested in the integration test functions [TECH DEBT] Create a fixture to read a dashboard query within a test Aug 12, 2024
@nfx nfx transferred this issue from databrickslabs/ucx Oct 9, 2024
JCZuurmond added a commit to databrickslabs/ucx that referenced this issue Oct 18, 2024
JCZuurmond added a commit to databrickslabs/ucx that referenced this issue Oct 21, 2024
JCZuurmond added a commit to databrickslabs/ucx that referenced this issue Oct 23, 2024
JCZuurmond added a commit to databrickslabs/ucx that referenced this issue Oct 23, 2024
JCZuurmond added a commit to databrickslabs/ucx that referenced this issue Oct 28, 2024
JCZuurmond added a commit to databrickslabs/ucx that referenced this issue Oct 28, 2024
JCZuurmond added a commit to databrickslabs/ucx that referenced this issue Oct 28, 2024
nfx pushed a commit to databrickslabs/ucx that referenced this issue Oct 29, 2024
## Changes
Add dashboard for tracking migration process, screenshot below as
comments.

Open changes:
- [x] Update install to replace UCX catalog in queries, call the value
to be replace `multiworkspaces`
- [x] Add counter for tables and views migrated
- [x] Use object serializer from #2743
- [x] Filter for latest scan only
- [x] Add overview of the (unique) failure messages
- [x] Reword "readiness"

### Linked issues

Resolves #2596
References databrickslabs/lsql#306
Requires: #3083 

### Functionality

- [x] added a new dashboard: migration-progress

### Tests

- [x] manually tested
- [x] added integration tests
- [x] verified on staging environment (screenshot attached)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

No branches or pull requests

1 participant