New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

followup #389 - Mockup unit tests for S3CompatibleCredentialsStorageIntegration.java #1145

Open

lefebsy opened this issue Mar 10, 2025 · 0 comments

Milestone

lefebsy commented Mar 10, 2025

Is it ok if I do UT only for config class and move to dockerized tests for credentials class ? I am really not good in the mock code area.
(I can easily add minio to regtests docker-compose and enrich "regtests/t_spark_sql/")

I'm OK with a followup PR to resolve the mock related unit tests. We can get some help from the community.

Originally posted by @flyrain in #389 (comment)

github-project-automation bot added this to Basic Kanban Board

flyrain added this to the 1.0.0 milestone

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment