Mark test results classes as non test classes in init

Author: ian-sift

python/lib/sift_client/resources/__init__.py

@@ -176,6 +176,11 @@ async def main():
     FileAttachmentsAPI,
 )
 
+# These are not test classes, so we need to set __test__ to False to avoid pytest warnings.
+# Do this here because for some reason our docs generation doesn't like it when done in the classes themselves.
+TestResultsAPI.__test__ = False
+TestResultsAPIAsync.__test__ = False
+
 __all__ = [
     "AssetsAPI",
     "AssetsAPIAsync",

python/lib/sift_client/sift_types/__init__.py

@@ -163,6 +163,7 @@
 from sift_client.sift_types.test_report import (
     TestMeasurement,
     TestMeasurementCreate,
+    TestMeasurementUpdate,
     TestMeasurementType,
     TestReport,
     TestReportCreate,
@@ -173,6 +174,20 @@
     TestStepType,
 )
 
+# These are not test classes, so we need to set __test__ to False to avoid pytest warnings.
+# Do this here because for some reason our docs generation doesn't like it when done in the classes themselves.
+TestStepType.__test__ = False
+TestMeasurementType.__test__ = False
+TestMeasurement.__test__ = False
+TestMeasurementCreate.__test__ = False
+TestMeasurementUpdate.__test__ = False
+TestStatus.__test__ = False
+TestStep.__test__ = False
+TestStepCreate.__test__ = False
+TestReport.__test__ = False
+TestReportCreate.__test__ = False
+TestReportUpdate.__test__ = False
+
 __all__ = [
     "Asset",
     "AssetUpdate",

python/lib/sift_client/sift_types/test_report.py

@@ -43,7 +43,6 @@
 class TestStatus(Enum):
     """TestStatus enum."""
 
-    __test__ = False
     UNSPECIFIED = 0
     DRAFT = 1
     PASSED = 2
@@ -57,7 +56,6 @@ class TestStatus(Enum):
 class TestStepType(Enum):
     """TestStepType enum."""
 
-    __test__ = False
     UNSPECIFIED = 0
     SEQUENCE = 1
     GROUP = 2
@@ -68,7 +66,6 @@ class TestStepType(Enum):
 class TestMeasurementType(Enum):
     """TestMeasurementType enum."""
 
-    __test__ = False
     UNSPECIFIED = 0
     DOUBLE = 1
     STRING = 3
@@ -79,7 +76,6 @@ class TestMeasurementType(Enum):
 class TestStepBase(ModelCreateUpdateBase):
     """Base model for TestStepUpdate and TestStepCreate. Contains shared fields for all test steps. Update and create models differ mostly in what fields are required vs optional."""
 
-    __test__ = False
     parent_step_id: str | None = None
     description: str | None = None
     error_info: ErrorInfo | None = None
@@ -243,7 +239,6 @@ def __eq__(self, other: object) -> bool:
 class TestMeasurementBase(ModelCreateUpdateBase):
     """Base model for TestMeasurementUpdate and TestMeasurementCreate. Contains shared fields for all test measurements. Update and create models differ mostly in what fields are required vs optional."""
 
-    __test__ = False
     numeric_value: float | None = None
     string_value: str | None = None
     boolean_value: bool | None = None
@@ -330,7 +325,6 @@ def to_proto(self) -> TestMeasurementProto:
 class TestMeasurement(BaseType[TestMeasurementProto, "TestMeasurement"]):
     """TestMeasurement model representing a measurement in a test."""
 
-    __test__ = False
     measurement_type: TestMeasurementType
     name: str
     test_step_id: str
@@ -432,7 +426,6 @@ def update(
 class TestReportBase(ModelCreateUpdateBase):
     """Base model for TestReportUpdate and TestReportCreate. Contains shared fields for all test reports. Update and create models differ mostly in what fields are required vs optional."""
 
-    __test__ = False
     status: TestStatus | None = None
     metadata: dict[str, str | float | bool] | None = None
     serial_number: str | None = None
@@ -526,7 +519,6 @@ def _to_proto(self) -> ErrorInfoProto:
 class TestReport(BaseType[TestReportProto, "TestReport"], FileAttachmentsMixin):
     """TestReport model representing a test report."""
 
-    __test__ = False
     status: TestStatus
     name: str
     test_system_name: str