chore: make event_name mandatory in Framework

ops/_private/harness.py

@@ -312,6 +312,7 @@ def __init__(
             self._charm_dir,
             self._meta,
             self._model,
+            '',
             juju_debug_at=self._juju_context.debug_at,
             # Harness tests will often have defer() usage without 'purging' the
             # deferred handler with reemit(), but still expect the next emit()

ops/framework.py

@@ -606,7 +606,7 @@ def __init__(
         charm_dir: Union[str, pathlib.Path],
         meta: 'charm.CharmMeta',
         model: 'Model',
-        event_name: Optional[str] = None,
+        event_name: str,
         juju_debug_at: Optional[Set[str]] = None,
         skip_duplicate_events: bool = True,
     ):

test/test_framework.py

@@ -73,6 +73,7 @@ def create_framework(
         charm_dir,
         meta=model._cache._meta if model else ops.CharmMeta(),
         model=model,  # type: ignore
+        event_name='',
         juju_debug_at=_JujuContext.from_dict(os.environ).debug_at,
     )
     request.addfinalizer(framework.close)
@@ -94,6 +95,7 @@ def test_deprecated_init(self, caplog: pytest.LogCaptureFixture):
                 None,  # type: ignore
                 None,  # type: ignore
                 None,  # type: ignore
+                '',
                 juju_debug_at=set(),
             )
         assert 'WARNING:ops.framework:deprecated: Framework now takes a Storage not a path' in [

test/test_helpers.py

@@ -112,6 +112,7 @@ class TestCharmEvents(ops.CharmEvents):
         tmpdir,
         meta,
         model,
+        'foo_event',
         juju_debug_at=_JujuContext.from_dict(os.environ).debug_at,
     )
 

test/test_storage.py

@@ -54,6 +54,7 @@ def create_framework(
             None,  # type: ignore
             None,  # type: ignore
             None,  # type: ignore
+            '',
             juju_debug_at=set(),
         )