Skip to content

Commit

Permalink
Merge 'test_using_timestamp: update expected errors' from Benny Halevy
Browse files Browse the repository at this point in the history
This mini-series updates the expected errors in `test/cql-pytest/test-timestamp.py`
to the ones changed in b7bbcdd.
Then, it renamed the test to `test_using_timestamp.py` so it would
run automatically with `test.py`.

Closes #14293

* github.com:scylladb/scylladb:
  cql-pytest: rename test-timestamp.py to test_using_timestamp.py
  cql-pytest: test-timestamp: test_key_writetime: update expected errors
  • Loading branch information
nyh committed Jun 19, 2023
2 parents 1c6c799 + b0bcad0 commit 25bbc42
Showing 1 changed file with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -86,7 +86,7 @@ def restrict_future_timestamp():
# test will need to be replaced by a test for the new feature instead of
# expecting an error message.
def test_key_writetime(cql, table1):
with pytest.raises(InvalidRequest, match='PRIMARY KEY part k'):
with pytest.raises(InvalidRequest, match='PRIMARY KEY part k|WRITETIME is not legal on partition key component k'):
cql.execute(f'SELECT writetime(k) FROM {table1}')
with pytest.raises(InvalidRequest, match='PRIMARY KEY part k'):
with pytest.raises(InvalidRequest, match='PRIMARY KEY part k|TTL is not legal on partition key component k'):
cql.execute(f'SELECT ttl(k) FROM {table1}')

0 comments on commit 25bbc42

Please sign in to comment.