Files
Tariq HasanandGitHub 9183dfd24f chore(scripts): add copyright boilerplate verification (#3359)
* chore(scripts): add copyright boilerplate verification system

Signed-off-by: tariq-hasan <[email protected]>

* chore(ci): add boilerplate verification to CI pipeline

Signed-off-by: tariq-hasan <[email protected]>

* chore: regenerate files with year-less copyright headers

Signed-off-by: tariq-hasan <[email protected]>

* refactor(boilerplate): simplify verification logic

Signed-off-by: tariq-hasan <[email protected]>

* chore(boilerplate): add missing copyright headers

Signed-off-by: tariq-hasan <[email protected]>

* ci(boilerplate): move verifier to code-quality workflow and run unit tests

Signed-off-by: tariq-hasan <[email protected]>

* feat(boilerplate): accept license headers with any year and remove year validation logic

Signed-off-by: tariq-hasan <[email protected]>

* feat(boilerplate): add copyright headers to generated Python API

Signed-off-by: tariq-hasan <[email protected]>

* refactor(boilerplate): use block-comment header for generated Go files

Signed-off-by: tariq-hasan <[email protected]>

* feat(boilerplate): verify copyright headers in Rust files

Signed-off-by: tariq-hasan <[email protected]>

---------

Signed-off-by: tariq-hasan <[email protected]>
2026-05-29 10:50:56 +00:00

158 lines
5.4 KiB
Python

# Copyright The Kubeflow Authors.
#
# Licensed under the Apache License, Version 2.0 (the "License");
# you may not use this file except in compliance with the License.
# You may obtain a copy of the License at
#
# http://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS,
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
# See the License for the specific language governing permissions and
# limitations under the License.
import os
import tempfile
from unittest.mock import MagicMock, patch
import pytest
import pkg.initializers.utils.utils as utils
from pkg.initializers.model.s3 import S3
# Test cases for config loading
@pytest.mark.parametrize(
"test_name, test_config, expected",
[
(
"Full config with credentials",
{
"storage_uri": "s3://models/path",
"ignore_patterns": ["*.log", "*.txt"],
"endpoint": "https://s3.amazonaws.com",
"access_key_id": "test_access_key",
"secret_access_key": "test_secret_key",
"region": "us-east-1",
"role_arn": "arn:aws:iam::123456789012:role/TestRole",
},
{
"storage_uri": "s3://models/path",
"ignore_patterns": ["*.log", "*.txt"],
"endpoint": "https://s3.amazonaws.com",
"access_key_id": "test_access_key",
"secret_access_key": "test_secret_key",
"region": "us-east-1",
"role_arn": "arn:aws:iam::123456789012:role/TestRole",
},
),
(
"Minimal config without credentials",
{"storage_uri": "s3://models/path"},
{
"storage_uri": "s3://models/path",
"ignore_patterns": ["*.msgpack", "*.h5", "*.bin", "*.pt", "*.pth"],
"endpoint": None,
"access_key_id": None,
"secret_access_key": None,
"region": None,
"role_arn": None,
},
),
],
)
def test_load_config(test_name, test_config, expected):
"""Test config loading with different configurations"""
print(f"Running test: {test_name}")
s3_model_instance = S3()
with patch.object(utils, "get_config_from_env", return_value=test_config):
s3_model_instance.load_config()
assert s3_model_instance.config.__dict__ == expected
print("Test execution completed")
@pytest.mark.parametrize(
"test_name, test_case",
[
(
"Successful download with credentials",
{
"config": {
"storage_uri": "s3://models/path/subpath",
"ignore_patterns": None,
"endpoint": "https://s3.amazonaws.com",
"access_key_id": "test_access_key",
"secret_access_key": "test_secret_key",
"region": "us-east-1",
"role_arn": "arn:aws:iam::123456789012:role/TestRole",
},
"expected_bucket": "models",
"expected_prefix": "path/subpath",
},
),
(
"Successful download without credentials",
{
"config": {
"storage_uri": "s3://models/path",
"ignore_patterns": None,
"endpoint": None,
"access_key_id": None,
"secret_access_key": None,
"region": None,
"role_arn": None,
},
"expected_bucket": "models",
"expected_prefix": "path",
},
),
],
)
def test_download_model(test_name, test_case):
"""Test model download with different configurations"""
print(f"Running test: {test_name}")
# Setup the S3 model instance
s3_model_instance = S3()
s3_model_instance.config = MagicMock(**test_case["config"])
# Create a temporary directory for downloads
with tempfile.TemporaryDirectory() as temp_dir:
model_path = os.path.join(temp_dir, "model")
mock_storage = MagicMock()
with (
patch(
"pkg.initializers.utils.opendal.S3Storage", return_value=mock_storage
),
patch.object(utils, "MODEL_PATH", model_path),
):
s3_model_instance.download_model()
# Verify S3Storage was created with correct parameters
from pkg.initializers.utils.opendal import S3Storage
S3Storage.assert_called_once_with(
bucket=test_case["expected_bucket"],
endpoint=test_case["config"]["endpoint"],
access_key_id=test_case["config"]["access_key_id"],
secret_access_key=test_case["config"]["secret_access_key"],
region=test_case["config"]["region"],
role_arn=test_case["config"]["role_arn"],
)
# Verify download was called with correct parameters
mock_storage.download.assert_called_once_with(
prefix=test_case["expected_prefix"],
destination_path=model_path,
ignore_patterns=test_case["config"]["ignore_patterns"],
)
print("Test execution completed")