henry3260 opened a new pull request, #11994:
URL: https://github.com/apache/gravitino/pull/11994
### What changes were proposed in this pull request?
- Add shared helpers in `IntegrationTestEnv` for authorization
configuration, external Gravitino detection, server start/restart behavior, and
metalake setup/cleanup.
- Add `AuthorizationIntegrationTestEnv` to share common authorization
integration test lifecycle.
- Update user, owner, group, and role integration tests to reuse the shared
setup helpers.
- Move group and role management tests onto the shared authorization
integration test base class.
### Why are the changes needed?
Several Python client authorization integration tests duplicated the same
authorization config setup, Gravitino server lifecycle handling, and metalake
setup/cleanup logic.
Centralizing this logic reduces duplicate test boilerplate and keeps the
authorization integration tests consistent while preserving their existing
behavior.
Related to #3767.
### Does this PR introduce *any* user-facing change?
No.
### How was this patch tested?
- `/private/tmp/gravitino-client-python-test-venv/bin/black
clients/client-python/tests/integration/integration_test_env.py
clients/client-python/tests/integration/test_user.py
clients/client-python/tests/integration/test_owner.py
clients/client-python/tests/integration/test_group_management.py
clients/client-python/tests/integration/test_role_management.py`
- `PYTHONPATH=. PYLINTHOME=/private/tmp/gravitino-pylint-cache
/private/tmp/gravitino-client-python-test-venv/bin/pylint --rcfile=pylintrc
--persistent=n tests/integration/integration_test_env.py
tests/integration/test_user.py tests/integration/test_owner.py
tests/integration/test_group_management.py
tests/integration/test_role_management.py`
- Result: `10.00/10`
- `PYTHONPATH=. /private/tmp/gravitino-client-python-test-venv/bin/python -m
py_compile tests/integration/integration_test_env.py
tests/integration/test_user.py tests/integration/test_owner.py
tests/integration/test_group_management.py
tests/integration/test_role_management.py`
- `git diff --check`
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
