6
0
mirror of https://github.com/FirebirdSQL/firebird-qa.git synced 2025-01-22 13:33:07 +01:00

Added/Updated tests\functional\database\create\test_02.py: Added (temporary ?) 'credentials = False' to prevent ISQL from using '-USER ... -PASS ...'. IMO, this is bug; see https://github.com/FirebirdSQL/firebird/issues/8385

This commit is contained in:
pavel-zotov 2025-01-13 14:44:35 +03:00
parent f869ab0158
commit ef33e8f8a6

View File

@ -5,60 +5,67 @@ ID: create-database-02
TITLE: Create database: non sysdba user TITLE: Create database: non sysdba user
DESCRIPTION: DESCRIPTION:
FBTEST: functional.database.create.02 FBTEST: functional.database.create.02
NOTES:
[13.01.2025] pzotov
Added (temporary ?) 'credentials = False' to prevent ISQL from using '-USER ... -PASS ...'.
This is needed since 6.0.0.570, otherwise we get (on attempting to create DB):
Statement failed, SQLSTATE = 28000
Your user name and password are not defined. Ask your database administrator to set up a Firebird login.
-Different logins in connect and attach packets - client library error
(IMO, this is bug; see https://github.com/FirebirdSQL/firebird/issues/8385)
""" """
import locale
from pathlib import Path
import pytest import pytest
from firebird.qa import * from firebird.qa import *
db = db_factory() db = db_factory()
act = python_act('db', substitutions = [('[ \t]+', ' ')])
test_script = """ tmp_user = user_factory('db', name='tmp$boss', password='123')
set wng off; test_db = temp_file('tmp4test.fdb')
set bail on;
create or alter user ozzy password 'osb' revoke admin role;
commit;
revoke all on all from ozzy;
commit;
-- ::: NB ::: do NOT miss specification of 'USER' or 'ROLE' clause in
-- GRANT | REVOKE CREATE DATABASE, between `to` and login! Otherwise:
-- Statement failed, SQLSTATE = 0A000
-- unsuccessful metadata update
-- -GRANT failed
-- -feature is not supported
-- -Only grants to USER or ROLE are supported for CREATE DATABASE
grant create database to USER ozzy;
-- ^^^^
grant drop database to USER ozzy;
-- ^^^^
commit;
create database 'localhost:$(DATABASE_LOCATION)tmp.ozzy$db$987456321.tmp' user 'OZZY' password 'osb';
set list on;
select
a.mon$user "Who am I ?"
,iif( m.mon$database_name containing 'tmp.ozzy$db$987456321.tmp' , 'YES', 'NO! ' || m.mon$database_name) "Am I on just created DB ?"
from mon$database m, mon$attachments a where a.mon$attachment_id = current_connection;
commit;
drop database;
connect '$(DSN)' user 'SYSDBA' password 'masterkey';
revoke create database from user ozzy;
revoke drop database from user ozzy;
drop user ozzy;
commit;
"""
act = isql_act('db', test_script)
expected_stdout = """
Who am I ? OZZY
Am I on just created DB ? YES
"""
@pytest.mark.version('>=3.0') @pytest.mark.version('>=3.0')
def test_1(act: Action): def test_1(act: Action, tmp_user: User, test_db: Path):
act.expected_stdout = expected_stdout
act.execute() test_script = f"""
set wng off;
set bail on;
connect '{act.db.dsn}' user {act.db.user} password '{act.db.password}';
revoke all on all from {tmp_user.name};
commit;
-- ::: NB ::: do NOT miss specification of 'USER' or 'ROLE' clause in
-- GRANT | REVOKE CREATE DATABASE, between `to` and login! Otherwise:
-- Statement failed, SQLSTATE = 0A000
-- unsuccessful metadata update
-- -GRANT failed
-- -feature is not supported
-- -Only grants to USER or ROLE are supported for CREATE DATABASE
grant create database to USER {tmp_user.name};
-- ^^^^
grant drop database to USER {tmp_user.name};
-- ^^^^
commit;
create database 'localhost:{test_db}' user {tmp_user.name} password '{tmp_user.password}';
set list on;
select
a.mon$user "Who am I ?"
,iif( m.mon$database_name containing '{test_db}' , 'YES', 'NO! ' || m.mon$database_name) "Am I on just created DB ?"
from mon$database m cross join mon$attachments a
where a.mon$attachment_id = current_connection;
commit;
drop database;
set echo on;
-- Done.
"""
act.expected_stdout = f"""
Who am I ? {tmp_user.name.upper()}
Am I on just created DB ? YES
-- Done.
"""
act.isql(switches=['-q'], input=test_script, connect_db=False, combine_output = True, credentials = False, io_enc = locale.getpreferredencoding())
assert act.clean_stdout == act.clean_expected_stdout assert act.clean_stdout == act.clean_expected_stdout