Skip to content

Provide descriptive error on invalid table reference #1627

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

Merged
feast-ci-bot merged 20 commits into from
Jun 21, 2021
Merged

Provide descriptive error on invalid table reference #1627

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
20 commits
Select commit Hold shift + click to select a range
1a91a33
Initial commit to catch nonexistent table
Jun 7, 2021
da7a712
simplify nonexistent BQ table test
Jun 8, 2021
f6a2f98
clean up table_exists exception
Jun 8, 2021
41e69a7
remove unneeded variable
Jun 8, 2021
af937c4
function name change to _assert_table_exists
Jun 8, 2021
215de4b
Initial commit to catch nonexistent table
Jun 7, 2021
caac9d0
simplify nonexistent BQ table test
Jun 8, 2021
2d2a484
clean up table_exists exception
Jun 8, 2021
c548ab6
function name change to _assert_table_exists
Jun 8, 2021
d5eb8af
fix lint errors and rebase
Jun 9, 2021
21a7631
Fix get_table(None) error
Jun 9, 2021
50831ef
custom exception for both missing file and BQ source
Jun 9, 2021
4e7a803
revert FileSource checks
Jun 10, 2021
2d99b05
Use DataSourceNotFoundException instead of subclassing
Jun 11, 2021
c513660
Moved assert_table_exists out of the BQ constructor to apply_total
Jun 14, 2021
617dde1
rename test and test asset
Jun 16, 2021
6b3975a
move validate logic back to data_source
Jun 17, 2021
458beab
fixed tests
Jun 19, 2021
5abc428
Set pytest.integration for tests that access BQ
Jun 20, 2021
4ec510d
Import pytest in failed test files
Jun 20, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Set pytest.integration for tests that access BQ 
Signed-off-by: Cody Lin <codyl@twitter.com>
  • Loading branch information
Cody Lin committed Jun 20, 2021
commit 5abc428d17b16d70d01154e76c96cb616e4b6cf0
2 changes: 2 additions & 0 deletions sdk/python/tests/test_cli_local.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@
from tests.online_read_write_test import basic_rw_test


@pytest.mark.integration
def test_workflow() -> None:
"""
Test running apply on a sample repo, and make sure the infra gets created.
Expand Down Expand Up @@ -78,6 +79,7 @@ def test_workflow() -> None:
assertpy.assert_that(result.returncode).is_equal_to(0)


@pytest.mark.integration
def test_non_local_feature_repo() -> None:
"""
Test running apply on a sample repo, and make sure the infra gets created.
Expand Down
2 changes: 2 additions & 0 deletions sdk/python/tests/test_online_retrieval.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@
from tests.cli_utils import CliRunner, get_example_repo


@pytest.mark.integration
def test_online() -> None:
"""
Test reading from the online store in local mode.
Expand Down Expand Up @@ -238,6 +239,7 @@ def test_online() -> None:
os.rename(store.config.registry + "_fake", store.config.registry)


@pytest.mark.integration
def test_online_to_df():
"""
Test dataframe conversion. Make sure the response columns and rows are
Expand Down
1 change: 1 addition & 0 deletions sdk/python/tests/test_partial_apply.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@
from tests.online_read_write_test import basic_rw_test


@pytest.mark.integration
def test_partial() -> None:
"""
Add another table to existing repo using partial apply API. Make sure both the table
Expand Down