Unit tests get_all_tables needs to be skipped because Pandas backend does not support list_databases

Open Beginner friendly
#1,788 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
2/5
Estimated time
1-3 hours
Newbie friendliness
72/100
Issue type
Bug
Clarity
Mostly clear
Activity status
Quiet
Tech stack
pandas, python
Domain
testing-qa

Research direction

Search the test suite for test_get_all_tables and inspect how backend-specific tests are skipped. Confirm the pandas client lacks list_databases, then skip this test for that backend and run the relevant unit-test module to verify the suite passes without masking other backends.

Written by the indexing model from the issue text.

Description

type: bug

Describe the bug
The test test_get_all_tables has to be skipped in Ibis 7.1 because there is no support for list_databases on a pandas client. Please see here.

In a future release when we support DuckDB for in memory tables, this unit test can be set up to list databases in duckDB.

What type of connections are you using for source and target?
Unit Test

Dominant language
Python
Stars
525
Forks
171
Avg merge
5d 15h
Merged PRs (30d)
4

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from GoogleCloudPlatform/professional-services-data-validator

All issues in GoogleCloudPlatform/professional-services-data-validator

Similar issues

More Python issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.