Skip to content

Skip SQL tests if tarantool version < 2.0.0 - #195

Merged
ylobankov merged 1 commit into
masterfrom
ylobankov/skip-or-run-sql-tests
Nov 15, 2021
Merged

ylobankov merged 1 commit into
masterfrom
ylobankov/skip-or-run-sql-tests

Conversation

@ylobankov

@ylobankov ylobankov commented Nov 11, 2021 •

Copy link
Copy Markdown
Contributor

Problem: all SQL-related tests fail with the following error
if tarantool < 2.0.0 is used:

tarantool.error.DatabaseError: (48, 'Unknown request type 11')

Tarantool supports SQL-related stuff only since the 2.0.0 version.
So this patch adds a special decorator that will skip the test if
tarantool version < 2.0.0 is used.

Fixes #194

@ylobankov
ylobankov force-pushed the ylobankov/skip-or-run-sql-tests branch 3 times, most recently from 051ca05 to 2b5ce9c Compare November 11, 2021 16:14
@ylobankov
ylobankov requested a review from Totktonada November 11, 2021 16:48
@ylobankov
ylobankov force-pushed the ylobankov/skip-or-run-sql-tests branch 2 times, most recently from 789478d to 2966ccf Compare November 12, 2021 11:09
Comment thread test/suites/lib/skip.py Resolved
Comment thread test/suites/test_dbapi.py Resolved
@Totktonada

Copy link
Copy Markdown
Contributor

Can I look at run logs on 1.10 somewhere?

@ylobankov
ylobankov force-pushed the ylobankov/skip-or-run-sql-tests branch 2 times, most recently from 06c3c14 to aa24697 Compare November 15, 2021 08:23
@ylobankov

Copy link
Copy Markdown
Contributor Author

Can I look at run logs on 1.10 somewhere?

For running tests against tarantool 1.10, I manually triggered the integration.yml workflow from tarantool repo against my test branch. The run log is available here.

Problem: all SQL-related tests fail with the following error
if tarantool < 2.0.0 is used:

    tarantool.error.DatabaseError: (48, 'Unknown request type 11')

Tarantool supports SQL-related stuff only since the 2.0.0 version.
So this patch adds a special decorator that will skip the test if
tarantool version < 2.0.0 is used.

Fixes #194

@Totktonada Totktonada left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@Totktonada
Totktonada force-pushed the ylobankov/skip-or-run-sql-tests branch from 8bd66d4 to 6de4c53 Compare November 15, 2021 16:50
@Totktonada

Totktonada commented Nov 15, 2021 •

Copy link
Copy Markdown
Contributor

(Squashed commits. Will push after CI tomorrow. Feel free to push yourself.)

@ylobankov
ylobankov merged commit 0f95f28 into master Nov 15, 2021
@ylobankov
ylobankov deleted the ylobankov/skip-or-run-sql-tests branch November 15, 2021 20:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

All SQL-related tests fail when tarantool < 2.0.0 is used

2 participants