mid ruff linting

Author: sneakers-the-rat

linkml_runtime/linkml_model/__init__.py

@@ -3,6 +3,11 @@
 from .meta import (
     AltDescription,
     AltDescriptionSource,
+    AnonymousClassExpression,
+    AnonymousEnumExpression,
+    AnonymousExpression,
+    AnonymousSlotExpression,
+    AnonymousTypeExpression,
     ClassDefinition,
     ClassDefinitionName,
     Definition,
@@ -90,4 +95,9 @@
     "AltDescription",
     "PermissibleValue",
     "PvFormulaOptions",
+    "AnonymousClassExpression",
+    "AnonymousEnumExpression",
+    "AnonymousExpression",
+    "AnonymousSlotExpression",
+    "AnonymousTypeExpression",
 ]

linkml_runtime/utils/permissiblevalueimpl.py

@@ -2,9 +2,9 @@
 from typing import Any, ClassVar, Optional, Union
 
 from rdflib import URIRef
-from utils.enumerations import EnumDefinitionImpl
 
 from linkml_runtime.utils.curienamespace import CurieNamespace
+from linkml_runtime.utils.enumerations import EnumDefinitionImpl
 from linkml_runtime.utils.metamodelcore import URI, URIorCURIE, empty_list
 from linkml_runtime.utils.yamlutils import YAMLRoot, extended_str
 

linkml_runtime/utils/ruleutils.py

@@ -136,12 +136,12 @@ def subclass_to_rules(
 
 
 def rule_subsumed_by_class(view: SchemaView, rule, cls: ClassDefinition):
-    induced_slots = view.class_induced_slots(cls.name)
+    # induced_slots = view.class_induced_slots(cls.name)
     return False
 
 
 def remove_redundant_rules(view: SchemaView, class_name: ClassDefinitionName):
-    induced_slots = view.class_induced_slots(class_name)
+    # induced_slots = view.class_induced_slots(class_name)
     cls = view.get_class(class_name)
     redundant_rules = []
     for rule in cls.rules:

linkml_runtime/utils/schemaops.py

@@ -22,7 +22,6 @@ def roll_up(sv: SchemaView, classes: CLASS_NAME_OR_LIST = None, mixins=True, is_
     dels = set()
     for cn in classes:
         c = sv.get_class(cn)
-        slots = []
         for d in sv.class_descendants(cn, reflexive=False, mixins=mixins, is_a=is_a):
             for sn in sv.class_slots(d):
                 s = sv.induced_slot(sn, class_name=d)
@@ -54,7 +53,6 @@ def roll_down(sv: SchemaView, classes: list[CLASS_NAME] = None, mixins=True, is_
     dels = set()
     for cn in classes:
         c = sv.get_class(cn)
-        slots = []
         for d in sv.class_ancestors(cn, reflexive=False, mixins=mixins, is_a=is_a):
             for sn in sv.class_slots(d):
                 s = sv.induced_slot(sn, class_name=d)
@@ -74,6 +72,5 @@ def roll_down(sv: SchemaView, classes: list[CLASS_NAME] = None, mixins=True, is_
             if is_a and c.is_a is not None:
                 del c.is_a
     for d in dels:
-        d_cls = sv.get_class(d)
         sv.delete_class(d)
     sv.set_modified()

pyproject.toml

@@ -113,7 +113,10 @@ select = [
   "UP", # pyupgrade
 ]
 force-exclude = true
-extend-ignore = ["E203"]
+extend-ignore = [
+    "E203",
+    "UP007", # until we drop 3.9
+]
 
 # -- Exclude files --
 extend-exclude = [
@@ -123,6 +126,9 @@ extend-exclude = [
     "notebooks/"
 ]
 
+[tool.ruff.per-file-ignores]
+"tests/**/**.py" = ["F841", "E501", "F842", "E741"] # I ain't fixing all that
+
 [tool.codespell]
 skip = '.git,*.pdf,*.svg,./tests,pyproject.toml,*.dill,poetry.lock'
 # Ignore table where words could be split across rows

tests/__init__.py

@@ -1,11 +1,14 @@
 import configparser
-import logging
 
 # Global testing control variables
 import os
 
 from tests.support.test_environment import MismatchAction
 
+__all__ = [
+    "MismatchAction",
+]
+
 # ---------------------------------------------------------------
 #                DO NOT change this file.
 # To change the default test harness settings:

tests/support/test_environment.py

@@ -22,7 +22,7 @@ def no_click_exit(_self, code=0):
 
 
 # This import has to occur here
-import click
+import click  # noqa: E402
 
 click.core.Context.exit = no_click_exit
 
@@ -209,8 +209,9 @@ def generate_single_file(
         :return: the generator output
         """
         # If no filter, default to identity function
-        if not filtr:
-            filtr = lambda s: s
+        if filtr is None:
+            def filtr(s):
+                return s
         filename = filename if isinstance(filename, list) else [filename]
         actual_file = self.root_temp_file_path(*filename) if use_testing_root else self.actual_path(*filename)
         expected_file = self.root_expected_path(*filename) if use_testing_root else self.expected_path(*filename)

tests/test_issues/test_linkml_runtime_issue_68.py

@@ -107,7 +107,7 @@ def test_issue_68(self):
 
         s3_induced_c2 = view.induced_slot("slot3", "class2")
         assert not s3_induced_c2.required
-        assert s3_induced_c2.description == None
+        assert s3_induced_c2.description is None
         assert s3_induced_c2.range == "class0"
 
         # mixins take priority over is-a

tests/test_linkml_model/test_linkml_files.py

@@ -6,8 +6,6 @@
 import pytest
 import requests
 
-HAVE_REQUESTS_CACHE = bool(find_spec("requests_cache"))
-
 from linkml_runtime.linkml_model.linkml_files import (
     GITHUB_IO_PATH_FOR,
     GITHUB_PATH_FOR,
@@ -21,6 +19,9 @@
     _Path,
 )
 
+HAVE_REQUESTS_CACHE = bool(find_spec("requests_cache"))
+
+
 EXPECTED_FORMATS = [
     (source, fmt) for source, fmt in product(Source, Format) if (fmt not in META_ONLY or source == Source.META)
 ]

tests/test_utils/test_schema_as_dict.py

@@ -5,7 +5,7 @@
 from linkml_runtime.linkml_model.meta import ClassDefinition
 from linkml_runtime.loaders.yaml_loader import YAMLLoader
 from linkml_runtime.utils.schema_as_dict import schema_as_dict, schema_as_yaml_dump
-from linkml_runtime.utils.schema_builder import ClassDefinition, SchemaBuilder, SlotDefinition
+from linkml_runtime.utils.schema_builder import SchemaBuilder, SlotDefinition
 from linkml_runtime.utils.schemaview import SchemaView
 from tests.test_utils import INPUT_DIR, OUTPUT_DIR