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

117 lines
3.9 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.
from unittest.mock import MagicMock, patch
import pytest
import pkg.initializers.utils.utils as utils
from pkg.initializers.model.huggingface import HuggingFace
# Test cases for config loading
@pytest.mark.parametrize(
"test_name, test_config, expected",
[
(
"Full config with token",
{"storage_uri": "hf://model/path", "access_token": "test_token"},
{
"storage_uri": "hf://model/path",
"ignore_patterns": ["*.msgpack", "*.h5", "*.bin", "*.pt", "*.pth"],
"access_token": "test_token",
},
),
(
"Minimal config without token",
{"storage_uri": "hf://model/path"},
{
"storage_uri": "hf://model/path",
"ignore_patterns": ["*.msgpack", "*.h5", "*.bin", "*.pt", "*.pth"],
"access_token": None,
},
),
],
)
def test_load_config(test_name, test_config, expected):
"""Test config loading with different configurations"""
print(f"Running test: {test_name}")
huggingface_model_instance = HuggingFace()
with patch.object(utils, "get_config_from_env", return_value=test_config):
huggingface_model_instance.load_config()
assert huggingface_model_instance.config.__dict__ == expected
print("Test execution completed")
@pytest.mark.parametrize(
"test_name, test_case",
[
(
"Successful download with token",
{
"config": {
"storage_uri": "hf://username/model-name",
"ignore_patterns": ["*.msgpack", "*.h5", "*.bin", "*.pt", "*.pth"],
"access_token": "test_token",
},
"should_login": True,
"expected_repo_id": "username/model-name",
},
),
(
"Successful download without token",
{
"config": {
"storage_uri": "hf://org/model-v1",
"ignore_patterns": ["*.msgpack", "*.h5", "*.bin", "*.pt", "*.pth"],
"access_token": None,
},
"should_login": False,
"expected_repo_id": "org/model-v1",
},
),
],
)
def test_download_model(test_name, test_case):
"""Test model download with different configurations"""
print(f"Running test: {test_name}")
huggingface_model_instance = HuggingFace()
huggingface_model_instance.config = MagicMock(**test_case["config"])
with patch("huggingface_hub.login") as mock_login, patch(
"huggingface_hub.snapshot_download"
) as mock_download:
# Execute download
huggingface_model_instance.download_model()
# Verify login behavior
if test_case["should_login"]:
mock_login.assert_called_once_with(test_case["config"]["access_token"])
else:
mock_login.assert_not_called()
# Verify download parameters
mock_download.assert_called_once_with(
repo_id=test_case["expected_repo_id"],
local_dir=utils.MODEL_PATH,
allow_patterns=["*.json", "*.safetensors", "*.model", "*.txt"],
ignore_patterns=test_case["config"]["ignore_patterns"],
)
print("Test execution completed")