Tests: Satisfy linter for test_tracking.py

crate · Sep 9, 2023 · 34a2b32 · 34a2b32
1 parent 5551db5
commit 34a2b32
Show file tree

Hide file tree

Showing 2 changed files with 23 additions and 34 deletions.
diff --git a/pyproject.toml b/pyproject.toml
@@ -165,7 +165,6 @@ select = [
 ]
 
 extend-exclude = [
-  "tests/test_tracking.py",
 ]
 
 

diff --git a/tests/test_tracking.py b/tests/test_tracking.py
@@ -1,24 +1,20 @@
 # Source: mlflow:tests/tracking/test_tracking.py
-import json
 import math
 import os
 import pathlib
 import re
 import shutil
-import tempfile
 import time
 import unittest
 import uuid
 from concurrent.futures import ThreadPoolExecutor
-from pathlib import Path
 from unittest import mock
 
-import pytest
-import sqlalchemy
-
 import mlflow
 import mlflow.db
 import mlflow.store.db.base_sql_model
+import pytest
+import sqlalchemy
 from mlflow import entities
 from mlflow.entities import (
     Experiment,
@@ -31,7 +27,6 @@
     ViewType,
     _DatasetSummary,
 )
-from mlflow.environment_variables import MLFLOW_TRACKING_URI
 from mlflow.exceptions import MlflowException
 from mlflow.protos.databricks_pb2 import (
     BAD_REQUEST,
@@ -41,13 +36,8 @@
     ErrorCode,
 )
 from mlflow.store.db.db_types import MSSQL, MYSQL, POSTGRES, SQLITE
-from mlflow.store.db.utils import (
-    _get_latest_schema_revision,
-    _get_schema_version,
-)
 from mlflow.store.tracking import SEARCH_MAX_RESULTS_DEFAULT
 from mlflow.store.tracking.dbmodels import models
-from mlflow.store.tracking.dbmodels.initial_models import Base as InitialBase
 from mlflow.store.tracking.dbmodels.models import (
     SqlDataset,
     SqlExperiment,
@@ -62,16 +52,16 @@
 )
 from mlflow.store.tracking.sqlalchemy_store import SqlAlchemyStore, _get_orderby_clauses
 from mlflow.utils import mlflow_tags
-from mlflow.utils.file_utils import TempDir
 from mlflow.utils.mlflow_tags import MLFLOW_DATASET_CONTEXT, MLFLOW_RUN_NAME
 from mlflow.utils.name_utils import _GENERATOR_PREDICATES
 from mlflow.utils.os import is_windows
 from mlflow.utils.time_utils import get_current_time_millis
 from mlflow.utils.uri import extract_db_type_from_uri
 
 from mlflow_cratedb.patch.mlflow.db_types import CRATEDB
+
 from .abstract import AbstractStoreTest
-from .util import invoke_cli_runner, assert_dataset_inputs_equal
+from .util import assert_dataset_inputs_equal
 
 DB_URI = "crate://crate@localhost/?schema=testdrive"
 ARTIFACT_URI = "artifact_folder"
@@ -182,7 +172,7 @@ def _get_query_to_reset_experiment_id(self):
         dialect = self.store._get_dialect()
         if dialect == POSTGRES:
             return "ALTER SEQUENCE experiments_experiment_id_seq RESTART WITH 1"
-        elif dialect == MYSQL:
+        elif dialect == MYSQL:  # noqa: RET505
             return "ALTER TABLE experiments AUTO_INCREMENT = 1"
         elif dialect == MSSQL:
             return "DBCC CHECKIDENT (experiments, RESEED, 0)"
@@ -1046,7 +1036,7 @@ def test_get_metric_history_paginated_request_raises(self):
             "`get_metric_history` API.",
         ):
             self.store.get_metric_history(
-                "fake_run", "fake_metric", max_results=50, page_token="42"
+                "fake_run", "fake_metric", max_results=50, page_token="42"  # noqa: S106
             )
 
     def test_log_null_metric(self):
@@ -1146,7 +1136,8 @@ def test_log_param_max_length_value(self):
         with pytest.raises(MlflowException, match="exceeded length"):
             self.store.log_param(run.info.run_id, entities.Param(tkey, "x" * 1000))
 
-    @pytest.mark.skip("[FIXME] ColumnValidationException[Validation failed for experiment_id: Updating a primary key is not supported]")
+    @pytest.mark.skip("[FIXME] ColumnValidationException"
+                      "[Validation failed for experiment_id: Updating a primary key is not supported]")
     def test_set_experiment_tag(self):
         exp_id = self._experiment_factory("setExperimentTagExp")
         tag = entities.ExperimentTag("tag0", "value0")
@@ -2918,29 +2909,28 @@ def test_get_metric_history_on_non_existent_metric_key(self):
     @pytest.mark.skip(reason="[FIXME] MaxBytesLengthExceededException[bytes can be at most 32766 in length; got 65535]")
     def test_insert_large_text_in_dataset_table(self):
         with self.store.engine.begin() as conn:
-            # cursor = conn.cursor()
             dataset_source = "a" * 65535  # 65535 is the max size for a TEXT column
             dataset_profile = "a" * 16777215  # 16777215 is the max size for a MEDIUMTEXT column
             conn.execute(
                 sqlalchemy.sql.text(
                     f"""
-                INSERT INTO datasets 
-                    (dataset_uuid, 
-                    experiment_id, 
-                    name, 
-                    digest, 
-                    dataset_source_type, 
-                    dataset_source, 
-                    dataset_schema, 
+                INSERT INTO datasets
+                    (dataset_uuid,
+                    experiment_id,
+                    name,
+                    digest,
+                    dataset_source_type,
+                    dataset_source,
+                    dataset_schema,
                     dataset_profile)
-                VALUES 
-                    ('test_uuid', 
-                    0, 
-                    'test_name', 
-                    'test_digest', 
-                    'test_source_type', 
+                VALUES
+                    ('test_uuid',
+                    0,
+                    'test_name',
+                    'test_digest',
+                    'test_source_type',
                     '{dataset_source}', '
-                    test_schema', 
+                    test_schema',
                     '{dataset_profile}')
                 """
                 )