Skip to content

Commit

Permalink
Fix/skip ever-red oracle tests (#192)
Browse files Browse the repository at this point in the history
  • Loading branch information
KonstantAnxiety authored Dec 26, 2023
1 parent 7a0ddf2 commit 6c5031f
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 7 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -429,11 +429,6 @@ def test_month_ago_for_shorter_month(self, request, db, saved_connection_id, con
any_db_table_200 = make_table(db, rows=200)
request.addfinalizer(functools.partial(db.drop_table, any_db_table_200.table))

# FIXME
# if any_db.conn_type == CONNECTION_TYPE_ORACLE:
# # Oracle cannot add a month to 2021-01-31 (2021-02-31 doesn't exist)
# pytest.skip()

ds = create_basic_dataset(
api_v1=control_api,
connection_id=saved_connection_id,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,5 +8,11 @@ class TestOracleBasicComplexQueries(OracleDataApiTestBase, DefaultBasicComplexQu
test_params = RegulatedTestParams(
mark_features_skipped={
DefaultBasicComplexQueryTestSuite.feature_window_functions: "Native window functions are not implemented"
}
},
mark_tests_failed={
DefaultBasicComplexQueryTestSuite.test_month_ago_for_shorter_month: (
"FIXME: Oracle cannot add a month to 2021-01-31 (2021-02-31 doesn't exist;"
" db error: ORA-01839: date not valid for month specified)"
),
},
)
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,5 @@ class TestStringFunctionOracle(OracleTestBase, DefaultStringFunctionFormulaConne
supports_split_3 = False

def test_oracle_unicode(self, dbe: DbEvaluator) -> None:
assert os.environ.get("NLS_LANG") == ".AL32UTF8"
# assert dbe.db.scalar(sa.select([sa.literal('карл')])) == 'карл' # not working.
assert dbe.db.scalar(sa.literal("карл", type_=sa.sql.sqltypes.NCHAR())) == "карл"

0 comments on commit 6c5031f

Please sign in to comment.