forked from activist-org/i18n-check
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
activist-org#2 writing comprehensive tests for utility functions
- Loading branch information
1 parent
066d5a8
commit 5b40b6e
Showing
1 changed file
with
157 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,157 @@ | ||
import unittest | ||
import tempfile | ||
import json | ||
import os | ||
import pytest | ||
|
||
from i18n_check.utils import ( | ||
read_json_file, | ||
collect_files_to_check, | ||
is_valid_key, | ||
path_to_valid_key, | ||
filter_valid_key_parts, | ||
get_all_json_files, | ||
lower_and_remove_punctuation, | ||
read_files_to_dict, | ||
) | ||
|
||
|
||
class TestUtils(unittest.TestCase): | ||
def test_read_json_file(self): | ||
# Sample JSON data | ||
sample_data = {"name": "Test", "value": 123} | ||
|
||
# Create a temp JSON | ||
with tempfile.NamedTemporaryFile( | ||
mode="w", delete=False, encoding="utf-8", suffix=".json" | ||
) as temp_file: | ||
json.dump(sample_data, temp_file) | ||
temp_file_path = temp_file.name | ||
|
||
# Read the JSON file using the function | ||
result = read_json_file(temp_file_path) | ||
|
||
# Assertions | ||
assert isinstance(result, dict) | ||
assert result == sample_data | ||
|
||
def test_collect_files_to_check(self): | ||
with tempfile.TemporaryDirectory() as temp_dir: | ||
skip_dir = os.path.join(temp_dir, "skip_dir") | ||
os.makedirs(skip_dir) | ||
|
||
valid_file = os.path.join(temp_dir, "valid.txt") | ||
skipped_file = os.path.join(temp_dir, "skip.txt") | ||
file_in_skip_dir = os.path.join(skip_dir, "file_in_skip_dir.txt") | ||
|
||
with open(valid_file, "w") as f: | ||
f.write("test") | ||
with open(skipped_file, "w") as f: | ||
f.write("test") | ||
with open(file_in_skip_dir, "w") as f: | ||
f.write("test") | ||
|
||
result = collect_files_to_check( | ||
directory=temp_dir, | ||
file_types=[".txt"], | ||
directories_to_skip=["skip_dir"], | ||
files_to_skip=["skip.txt"], | ||
) | ||
|
||
assert valid_file in result | ||
assert skipped_file not in result | ||
assert file_in_skip_dir not in result | ||
|
||
def test_get_all_json_files(self): | ||
with tempfile.TemporaryDirectory() as temp_dir: | ||
json_file_1 = os.path.join(temp_dir, "file1.json") | ||
json_file_2 = os.path.join(temp_dir, "file2.json") | ||
non_json_file = os.path.join(temp_dir, "file.txt") | ||
|
||
with open(json_file_1, "w") as f: | ||
f.write("{}") | ||
with open(json_file_2, "w") as f: | ||
f.write("{}") | ||
with open(non_json_file, "w") as f: | ||
f.write("test") | ||
|
||
result = get_all_json_files(temp_dir, os.sep) | ||
|
||
assert json_file_1 in result | ||
assert json_file_2 in result | ||
assert non_json_file not in result | ||
|
||
def test_read_files_to_dict(self): | ||
with tempfile.TemporaryDirectory() as temp_dir: | ||
file1 = os.path.join(temp_dir, "file1.txt") | ||
file2 = os.path.join(temp_dir, "file2.txt") | ||
|
||
content1 = "Hello, world!" | ||
content2 = "Python testing." | ||
|
||
with open(file1, "w") as f: | ||
f.write(content1) | ||
with open(file2, "w") as f: | ||
f.write(content2) | ||
|
||
result = read_files_to_dict([file1, file2]) | ||
|
||
assert isinstance(result, dict) | ||
assert result[file1] == content1 | ||
assert result[file2] == content2 | ||
|
||
def test_is_valid_key(self): | ||
assert is_valid_key("valid.key") | ||
assert is_valid_key("valid_key") | ||
assert is_valid_key("validkey123") | ||
assert not is_valid_key("Invalid-Key") | ||
assert not is_valid_key("invalid key") | ||
assert not is_valid_key("invalid/key") | ||
|
||
|
||
@pytest.mark.parametrize( | ||
"input_path, expected_key", | ||
[ | ||
(os.path.join("user", "ProfileSettings"), "user.profile_settings"), | ||
(os.path.join("admin", "Config", "Settings"), "admin.config.settings"), | ||
(os.path.join("API", "v1", "UserData"), "api.v1.user_data"), | ||
( | ||
os.path.join("folder", "SubFolder", "FileName"), | ||
"folder.sub_folder.file_name", | ||
), | ||
(os.path.join("nested.[id]", "path", "File"), "nested.path.file"), | ||
], | ||
) | ||
def test_path_to_valid_key(input_path, expected_key): | ||
assert path_to_valid_key(input_path) == expected_key | ||
|
||
|
||
@pytest.mark.parametrize( | ||
"input_list, expected_output", | ||
[ | ||
(["word", "word_suffix"], ["word_suffix"]), | ||
(["abc", "def", "ghi"], ["abc", "def", "ghi"]), | ||
(["prefix", "suffix", "prefix_suffix"], ["prefix_suffix"]), | ||
(["AnItem"], ["AnItem"]), | ||
([], []), | ||
], | ||
) | ||
def test_filter_valid_key_parts(input_list, expected_output): | ||
assert filter_valid_key_parts(input_list) == expected_output | ||
|
||
|
||
@pytest.mark.parametrize( | ||
"input_list, expected_output", | ||
[ | ||
( | ||
r"Remove all Python's punctuation except the: !#$%\"&'()*+,-./:;<=>?@[\]^_`{|}~ Mark", | ||
"remove all pythons punctuation except the ! mark", | ||
) | ||
], | ||
) | ||
def test_lower_and_remove_punctuation(input_list, expected_output): | ||
assert lower_and_remove_punctuation(input_list) == expected_output | ||
|
||
|
||
if __name__ == "__main__": | ||
unittest.main() |