feat: Analyze licenses or-ed together separately

Multi-licensed packages may specify their allowed licenses using an `or` designation. This commit adds support for parsing out those `or`-ed licenses as if they had been listed separately.

Author: thehale

liccheck/command_line.py

@@ -223,16 +223,16 @@ def check_one(license_str, license_rule="AUTHORIZED", as_regex=False):
 
  at_least_one_unauthorized = False
  count_authorized = 0
- for license in pkg["licenses"]:
-  lower = license.lower()
+ licenses = get_license_names(pkg["licenses"])
+ for license in licenses:
  if check_one(
- lower,
+ license,
  license_rule="UNAUTHORIZED",
  as_regex=as_regex,
  ):
  at_least_one_unauthorized = True
  if check_one(
- lower,
+ license,
  license_rule="AUTHORIZED",
  as_regex=as_regex,
  ):
@@ -247,7 +247,7 @@ def check_one(license_str, license_rule="AUTHORIZED", as_regex=False):
  )
  or (
  count_authorized
- and count_authorized == len(pkg["licenses"])
+ and count_authorized == len(licenses)
  and level is Level.PARANOID
  )
  ):
@@ -259,6 +259,14 @@ def check_one(license_str, license_rule="AUTHORIZED", as_regex=False):
 
  return Reason.UNKNOWN
 
+def get_license_names(licenses):
+ names = []
+ for license in licenses:
+ license = license.lower()
+ options = license.split(" or ")
+ for option in options:
+ names.append(option)
+ return names
 
 def find_parents(package, all, seen):
  if package in seen:

tests/test_check_package.py

@@ -25,6 +25,11 @@ def packages():
  "version": "1",
  "licenses": ["authorized 1", "unauthorized 1"],
  },
+ {
+ "name": "auth_one_or_unauth_one",
+ "version": "2",
+ "licenses": ["authorized 1 or unauthorized 1"],
+ },
  {
  "name": "unauth_one",
  "version": "2",
@@ -77,9 +82,9 @@ def packages():
 @pytest.mark.parametrize(
  ("level", "reasons"),
  [
- (Level.STANDARD, [OK, OK, OK, UNAUTH, OK, UNAUTH, OK, UNKNOWN]),
- (Level.CAUTIOUS, [OK, OK, UNAUTH, UNAUTH, OK, UNAUTH, OK, UNKNOWN]),
- (Level.PARANOID, [OK, OK, UNAUTH, UNAUTH, OK, UNAUTH, UNKNOWN, UNKNOWN]),
+ (Level.STANDARD, [OK, OK, OK, OK, UNAUTH, OK, UNAUTH, OK, UNKNOWN]),
+ (Level.CAUTIOUS, [OK, OK, UNAUTH, UNAUTH, UNAUTH, OK, UNAUTH, OK, UNKNOWN]),
+ (Level.PARANOID, [OK, OK, UNAUTH, UNAUTH, UNAUTH, OK, UNAUTH, UNKNOWN, UNKNOWN]),
  ],
  ids=[level.name for level in Level],
 )