TST: add tests for objects the use PyDMPrimitiveWidget's RULE_PROPERTIES

Also adding these b/c RULE_PROPERTIES system is going to get reworked to work
with qt6/pyside6. (these tests should remain same while implementation
changes)

pydm/tests/widgets/test_drawing.py

@@ -27,6 +27,7 @@
     PyDMDrawingPolyline,
     PyDMDrawingIrregularPolygon,
 )
+from ...utilities import checkPyDMWidgetDerivedObjectProperties
 
 
 # --------------------
@@ -37,6 +38,8 @@
 # # -------------
 # # PyDMDrawing
 # # -------------
+
+
 @pytest.mark.parametrize(
     "deg, expected_qt_deg",
     [
@@ -84,6 +87,14 @@ def test_qt_to_deg(qt_deg, expected_deg):
     assert qt_to_deg(qt_deg) == expected_deg
 
 
+expected_drawing_properties = {
+    "Set Pen Color": ["penColor", QColor],
+    "Set Pen Style": ["penStyle", int],
+    "Set Pen Width": ["penWidth", float],
+    "Set Brush Color": ["brush", QBrush],
+}
+
+
 def test_pydmdrawing_construct(qtbot):
     """
     Test the construction of a PyDM base object.
@@ -97,6 +108,8 @@ def test_pydmdrawing_construct(qtbot):
         Window for widget testing
     """
     pydm_drawing = PyDMDrawing()
+    # should be ok to just test this on a single instance of the created object
+    assert checkPyDMWidgetDerivedObjectProperties(pydm_drawing, expected_drawing_properties)
     qtbot.addWidget(pydm_drawing)
 
     assert pydm_drawing.alarmSensitiveBorder is False
@@ -482,6 +495,7 @@ def test_pydmdrawingline_draw_item(qtbot, signals, alarm_sensitive_content):
         True if the widget will be redraw with a different color if an alarm is triggered; False otherwise
     """
     pydm_drawingline = PyDMDrawingLine(init_channel="fake://tst")
+    assert checkPyDMWidgetDerivedObjectProperties(pydm_drawingline, expected_drawing_properties)
     qtbot.addWidget(pydm_drawingline)
 
     pydm_drawingline.alarmSensitiveContent = alarm_sensitive_content

pydm/tests/widgets/test_label.py

@@ -9,6 +9,7 @@
 from ...widgets.label import PyDMLabel
 from ...widgets.base import PyDMWidget
 from ...widgets.display_format import parse_value_for_display, DisplayFormat
+from ...utilities import checkPyDMWidgetDerivedObjectProperties
 
 from qtpy.QtWidgets import QApplication, QStyleOption
 from qtpy.QtCore import Qt
@@ -18,6 +19,9 @@
 # --------------------
 
 
+expected_label_properties = {"Text": ["value_changed", str]}
+
+
 def test_construct(qtbot):
     """
     Test the basic instantiation of the widget.
@@ -33,6 +37,8 @@ def test_construct(qtbot):
         Window for widget testing
     """
     pydm_label = PyDMLabel()
+    assert checkPyDMWidgetDerivedObjectProperties(pydm_label, expected_label_properties)
+
     qtbot.addWidget(pydm_label)
 
     display_format_type = pydm_label.displayFormat

pydm/tests/widgets/test_related_display_button.py

@@ -6,7 +6,7 @@
 from qtpy.QtWidgets import QApplication
 from ...utilities.stylesheet import global_style
 from ...widgets.related_display_button import PyDMRelatedDisplayButton
-from ...utilities import IconFont
+from ...utilities import IconFont, checkPyDMWidgetDerivedObjectProperties
 
 test_ui_path = os.path.join(os.path.dirname(os.path.realpath(__file__)), "../test_data", "test.ui")
 test_ui_path_with_stylesheet = os.path.join(
@@ -17,6 +17,9 @@
 )
 
 
+expected_related_display_button_properties = {"Text": ["setText", str], "Filenames": ["filenames", list]}
+
+
 def test_old_display_filename_property(qtbot):
     # This test is mostly only checking that the related display button
     # doesn't totally explode when the old 'displayFilename' property is used.
@@ -25,6 +28,7 @@ def test_old_display_filename_property(qtbot):
     main_window.setWindowTitle("Related Display Button Test")
     qtbot.addWidget(main_window)
     button = PyDMRelatedDisplayButton(parent=main_window)
+    assert checkPyDMWidgetDerivedObjectProperties(button, expected_related_display_button_properties)
     with warnings.catch_warnings(record=True) as record:
         button.displayFilename = test_ui_path
     assert len(record) >= 1

pydm/tests/widgets/test_slider.py

@@ -7,6 +7,7 @@
 from qtpy.QtGui import QMouseEvent
 from ...widgets.slider import PyDMSlider, PyDMPrimitiveSlider
 from ...widgets.base import PyDMWidget
+from ...utilities import checkPyDMWidgetDerivedObjectProperties
 
 # Unit Tests for the PyDMPrimitiveSlider class
 
@@ -199,6 +200,9 @@ def test_getSliderPosition(slider_fixture, request):
 # Unit Tests for the PyDMSlider Widget
 
 
+expected_slider_properties = {"Set Step Size ": ["step_size", float]}
+
+
 def test_construct(qtbot):
     """
     Test the construction of the widget.
@@ -212,6 +216,7 @@ def test_construct(qtbot):
         Window for widget testing
     """
     pydm_slider = PyDMSlider()
+    assert checkPyDMWidgetDerivedObjectProperties(pydm_slider, expected_slider_properties)
     qtbot.addWidget(pydm_slider)
 
     assert pydm_slider.alarmSensitiveContent is True

pydm/tests/widgets/test_symbol_editor.py

@@ -6,6 +6,10 @@
 
 from ...widgets.symbol import PyDMSymbol
 from ...widgets.symbol_editor import SymbolEditor
+from ...utilities import checkPyDMWidgetDerivedObjectProperties
+
+
+expected_symbol_properties = {"Index": ["set_current_key", int]}
 
 
 def test_symbol_editor(qtbot, monkeypatch):
@@ -21,6 +25,8 @@ def test_symbol_editor(qtbot, monkeypatch):
     """
     # Create the base widget
     widget = PyDMSymbol()
+    assert checkPyDMWidgetDerivedObjectProperties(widget, expected_symbol_properties)
+
     qtbot.addWidget(widget)
 
     # Ensure that no rules are set

pydm/utilities/__init__.py

@@ -560,3 +560,40 @@ def get_clipboard_text() -> str:
         if text:
             return text
     return ""
+
+
+def checkPyDMWidgetDerivedObjectProperties(class_object, extra_properties):
+    """
+    Check that an object derived from PyDMWidget has the expected properties,
+    these being the base properties applied to PyDMWidget and any extra ones applied to the subclass.
+
+    Parameters
+    ----------
+    class_object : Type[PyDMWidget]
+        The object to check the properties of.
+
+    extra_properties : dict
+        Map of the additional properties applied to the class_object.
+    """
+
+    # properties that all PyDMWidget derived objects should all have
+    pydm_widget_props = {
+        "Enable": ["setEnabled", bool],
+        "Visible": ["setVisible", bool],
+        "Opacity": ["set_opacity", float],
+        "Position - X": ["setX", int],
+        "Position - Y": ["setY", int],
+    }
+
+    # combine the properties into one map
+    pydm_widget_props.update(extra_properties)
+
+    # check if object's properties are as expected
+    for key, value in pydm_widget_props.items():
+        if key not in class_object.RULE_PROPERTIES:
+            print("Missing property: ", key)
+            return False
+        if class_object.RULE_PROPERTIES[key] != value:
+            print(f"Mismatch for property '{key}': expected {value}, got {class_object.RULE_PROPERTIES[key]}")
+            return False
+    return True

pydm/widgets/related_display_button.py

@@ -22,8 +22,7 @@
 
 class PyDMRelatedDisplayButton(QPushButton, PyDMWidget, new_properties=_relatedDisplayRuleProperties):
     """
-    A QPushButton capable of opening a new Display at the same of at a
-    new window.
+    A QPushButton capable of opening a new Display in the same or a new window.
 
     Parameters
     ----------