Skip to content

Commit

Permalink
ci: make presto hive tests to cover only chartData and sqljson (#17782)
Browse files Browse the repository at this point in the history
* chore: make presto hive tests to cover only chartData and sqljson

* fix: single quote

* fix: add eval for arguments

* fix: add double quotes on args

* chore: add @pytest.mark.sql_json_flow

* fix: pre-commit
  • Loading branch information
amitmiran137 committed Dec 31, 2021
1 parent 8ebec60 commit 4954d52
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 3 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/superset-python-presto-hive.yml
Original file line number Diff line number Diff line change
Expand Up @@ -80,7 +80,7 @@ jobs:
- name: Python unit tests (PostgreSQL)
if: steps.check.outcome == 'failure'
run: |
./scripts/python_tests.sh
./scripts/python_tests.sh -m 'chart_data_flow or sql_json_flow'
- name: Upload code coverage
if: steps.check.outcome == 'failure'
run: |
Expand Down Expand Up @@ -158,7 +158,7 @@ jobs:
- name: Python unit tests (PostgreSQL)
if: steps.check.outcome == 'failure'
run: |
./scripts/python_tests.sh
./scripts/python_tests.sh -m 'chart_data_flow or sql_json_flow'
- name: Upload code coverage
if: steps.check.outcome == 'failure'
run: |
Expand Down
3 changes: 2 additions & 1 deletion scripts/python_tests.sh
Original file line number Diff line number Diff line change
Expand Up @@ -31,4 +31,5 @@ superset db upgrade
superset init

echo "Running tests"
pytest --durations=0 --maxfail=1 --cov=superset $@

pytest --durations=0 --maxfail=1 --cov=superset "$@"
1 change: 1 addition & 0 deletions tests/integration_tests/sqllab_tests.py
Original file line number Diff line number Diff line change
Expand Up @@ -66,6 +66,7 @@
QUERY_3 = "SELECT * FROM birth_names LIMIT 10"


@pytest.mark.sql_json_flow
class TestSqlLab(SupersetTestCase):
"""Testings for Sql Lab"""

Expand Down

0 comments on commit 4954d52

Please sign in to comment.