fix case insensitive cmp

Author: FredHappyface

licensecheck/checker.py

@@ -54,9 +54,9 @@ def check(
 		# Deal with --ignore-packages and --fail-packages
 		package.licenseCompat = False
 		packageName = package.name.upper()
-		if any(fnmatch(packageName, pattern) for pattern in ignore_packages):
+		if any(fnmatch(packageName, pattern.upper()) for pattern in ignore_packages):
 			package.licenseCompat = True
-		elif any(fnmatch(packageName, pattern) for pattern in fail_packages):
+		elif any(fnmatch(packageName, pattern.upper()) for pattern in fail_packages):
 			pass  # package.licenseCompat = False
 		# Else get compat with myLice
 		else:

licensecheck/license_matrix/__init__.py

@@ -41,6 +41,7 @@
 
 import csv
 from pathlib import Path
+from typing import Any
 
 from loguru import logger
 
@@ -109,7 +110,7 @@
 )
 
 
-def licenseLookup(licenseStr: str, ignoreLicenses: set[str] | None = None) -> L:
+def _licenseLookup(licenseStr: str, ignoreLicenses: set[str] | None = None) -> L:
 	"""
 	Identify a license from an uppercase string representation of a license.
 
@@ -118,22 +119,23 @@ def licenseLookup(licenseStr: str, ignoreLicenses: set[str] | None = None) -> L:
 	:return L: License represented by licenseStr
 	"""
 	licenseStr = licenseStr.upper()
+	ignoreLicenses_: set[str] = {x.upper() for x in ignoreLicenses or {}}
 	if len(licenseStr or "") < 1:
 		return L.NO_LICENSE
 
 	for liceStr, lice in termToLicense.items():
 		if liceStr.strip() in licenseStr:
 			return lice
 
-	if licenseStr not in (ignoreLicenses or ""):
+	if licenseStr not in (ignoreLicenses_ or ""):
 		logger.warning(f"'{licenseStr}' License not identified so falling back to UNKNOWN")
 	return L.UNKNOWN
 
 
 def _lst_licenseType(lice: str, ignoreLicenses: set[str] | None = None) -> list[L]:
 	if len(lice or "") < 1:
 		return [L.NO_LICENSE]
-	return [licenseLookup(str(x), ignoreLicenses) for x in lice.split(JOINS)]
+	return [_licenseLookup(str(x), ignoreLicenses) for x in lice.split(JOINS)]
 
 
 def licenseType(lice: str, ignoreLicenses: set[str] | None = None) -> set[L]:

pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "licensecheck"
-version = "2026.0.2"
+version = "2026.0.3"
 description = "Output the licenses used by dependencies and check if these are compatible with the project license"
 authors = [{ name = "FredHappyface" }]
 requires-python = ">=3.12"

tests/io/test_cli_main.py

@@ -1,7 +1,7 @@
 import pytest
 
-from licensecheck.models.config import LC_Config
 from licensecheck.io.cli import main
+from licensecheck.models.config import LC_Config
 
 
 @pytest.fixture
@@ -51,7 +51,7 @@ def test_main_success(
 def test_main_invalid_format(
 	config: LC_Config,
 	monkeypatch,
-):
+) -> None:
 	config.format = "does-not-exist"
 
 	monkeypatch.setattr(
@@ -106,7 +106,7 @@ def test_main_exit_code_zero_mode(
 def test_main_invalid_hidden_parameter(
 	config: LC_Config,
 	monkeypatch,
-):
+) -> None:
 	config.hide_output_parameters = {"NOT_A_FIELD"}
 
 	monkeypatch.setattr(
@@ -133,7 +133,7 @@ def test_main_valid_hidden_parameters(
 	config: LC_Config,
 	monkeypatch,
 	hidden: list[str],
-):
+) -> None:
 	config.hide_output_parameters = hidden
 
 	monkeypatch.setattr(
@@ -157,7 +157,7 @@ def test_main_valid_hidden_parameters(
 def test_main_passes_args_to_checker(
 	config: LC_Config,
 	monkeypatch,
-):
+) -> None:
 	called = {}
 
 	def fake_check(**kwargs):
@@ -190,7 +190,7 @@ def test_main_closes_output_file(
 	config: LC_Config,
 	monkeypatch,
 	tmp_path,
-):
+) -> None:
 	output = tmp_path / "out.txt"
 
 	config.file = str(output)

tests/test_checker.py

@@ -26,6 +26,12 @@ def mock_package_info_manager() -> PackageInfoManager:
 		(None, {"PACKAGE*"}, None, True),  # Globs are supported in fail_packages
 		(None, None, {"GPL-3.0"}, True),  # GPL-3.0 should be marked as incompatible
 		(None, None, {"GPL*"}, False),  # Globs are not supported on licenses!
+		({"package_b"}, None, None, False),  # Ignored package should not cause failure
+		({"package*"}, None, None, False),  # Globs are supported in ignore_packages
+		(None, {"package_a"}, None, True),  # PACKAGE_A in fail_packages should fail
+		(None, {"package*"}, None, True),  # Globs are supported in fail_packages
+		(None, None, {"gpl-3.0"}, True),  # GPL-3.0 should be marked as incompatible
+		(None, None, {"gpl*"}, False),  # Globs are not supported on licenses!
 	],
 )
 def test_check(

tests/test_license_matrix.py

@@ -43,7 +43,7 @@ def test_licenseLookup() -> None:
 	"""
 	licenses = []
 	for rawLicense in Path(f"{THISDIR}/data/rawLicenses.txt").read_text("utf-8").splitlines():
-		licenseName = license_matrix.licenseLookup(rawLicense)._name_
+		licenseName = license_matrix._licenseLookup(rawLicense)._name_
 		licenses.append(licenseName)
 
 	licenses.append("NO_LICENSE")
@@ -99,7 +99,7 @@ def test_whitelistedLicenseCompat() -> None:
 
 def test_warningsForIgnoredLicense(caplog: LogCaptureFixture) -> None:
 	zope = "ZOPE PUBLIC LICENSE"
-	license_matrix.licenseLookup(zope, set())
+	license_matrix._licenseLookup(zope, set())
 	assert any(
 		record.levelname == "WARNING"
 		and f"'{zope}' License not identified so falling back to UNKNOWN" in record.message
@@ -109,5 +109,11 @@ def test_warningsForIgnoredLicense(caplog: LogCaptureFixture) -> None:
 
 def test_warningsForIgnoredLicenseIgnored(caplog: LogCaptureFixture) -> None:
 	zope = "ZOPE PUBLIC LICENSE"
-	license_matrix.licenseLookup(zope, {zope})
+	license_matrix._licenseLookup(zope, {zope})
+	assert caplog.text == ""
+
+
+def test_warningsForIgnoredLicenseIgnored_lc(caplog: LogCaptureFixture) -> None:
+	zope = "ZOPE PUBLIC LICENSE"
+	license_matrix._licenseLookup(zope, {zope.lower()})
 	assert caplog.text == ""