chore: un-deprecate SignalGroup.signals (#288)

* chore: undeprecate signals

* test: fix test

Author: tlambert03

src/psygnal/_group.py

@@ -359,16 +359,16 @@ class MySignals(SignalGroup):
 
     @property
     def signals(self) -> Mapping[str, SignalInstance]:
-        """DEPRECATED: A mapping of signal names to SignalInstance instances."""
+        """A mapping of signal names to SignalInstance instances."""
         # TODO: deprecate this property
-        warnings.warn(
-            "Accessing the `signals` property on a SignalGroup is deprecated. "
-            "Use __iter__ to iterate over all signal names, and __getitem__ or getattr "
-            "to access signal instances. This will be an error in a future.",
-            FutureWarning,
-            stacklevel=2,
-        )
-        return self._psygnal_instances
+        # warnings.warn(
+        #     "Accessing the `signals` property on a SignalGroup is deprecated. "
+        #     "Use __iter__ to iterate over all signal names, and __getitem__ or "
+        #     "getattr to access signal instances. This will be an error in a future.",
+        #     FutureWarning,
+        #     stacklevel=2,
+        # )
+        return MappingProxyType(self._psygnal_instances)
 
     def __len__(self) -> int:
         """Return the number of signals in the group (not including the relay)."""

src/psygnal/_signal.py

@@ -664,7 +664,7 @@ def connect_setattr(
         """
         if maxargs is _NULL:
             warnings.warn(
-                "The default value of maxargs will change from `None` to `1` in"
+                "The default value of maxargs will change from `None` to `1` in "
                 "version 0.11. To silence this warning, provide an explicit value for "
                 "maxargs (`None` for current behavior, `1` for future behavior).",
                 FutureWarning,

tests/test_evented_model.py

@@ -69,9 +69,9 @@ class User(EventedModel):
 
     # test event system
     assert isinstance(user.events, SignalGroup)
-    with pytest.warns(FutureWarning):
-        assert "id" in user.events.signals
-        assert "name" in user.events.signals
+    # with pytest.warns(FutureWarning):
+    assert "id" in user.events.signals
+    assert "name" in user.events.signals
 
     # ClassVars are excluded from events
     assert "age" not in user.events