removed event loop support from service helper entirely

Author: cartertinney

dev_utils/dev_utils/service_helper.py

@@ -12,25 +12,15 @@ def __init__(
         iothub_connection_string,
         eventhub_connection_string,
         eventhub_consumer_group,
-        event_loop=None,
         executor=None,
     ):
-        # Keep the passed-in loop as a fallback, but prefer the currently running loop when
-        # scheduling work so we never attach futures to a different loop than the caller.
-        self._event_loop_override = event_loop
         self._executor = executor or concurrent.futures.ThreadPoolExecutor()
         self._inner_object = ServiceHelperSync(
             iothub_connection_string, eventhub_connection_string, eventhub_consumer_group
         )
 
-    def _get_event_loop(self):
-        try:
-            return asyncio.get_running_loop()
-        except RuntimeError:
-            return self._event_loop_override or asyncio.get_event_loop()
-
     async def _run_in_executor(self, func, *args):
-        loop = self._get_event_loop()
+        loop = asyncio.get_running_loop()
         return await loop.run_in_executor(self._executor, func, *args)
 
     def set_identity(self, device_id, module_id):

tests/e2e/iothub_e2e/pytest.ini

@@ -5,8 +5,6 @@ junit_logging=all
 junit_family=xunit2
 junit_log_passing_tests=True
 asyncio_mode=auto
-asyncio_legacy_mode=true
-; asyncio_default_fixture_loop_scope = function
 addopts=
     --testdox
     --force-testdox

tests/e2e/provisioning_e2e/pytest.ini

@@ -2,7 +2,6 @@
 timeout=30
 testdox_format=plaintext
 asyncio_mode=auto
-asyncio_legacy_mode=true
 addopts=
     --testdox
-    --force-testdox
+    --force-testdox