opentelemetry-api: fix wrong conditional on duplicated events log provider warning

Author: xrmx

opentelemetry-api/src/opentelemetry/_events/__init__.py

@@ -192,7 +192,7 @@ def set_elp() -> None:
 
     did_set = _EVENT_LOGGER_PROVIDER_SET_ONCE.do_once(set_elp)
 
-    if log and did_set:
+    if log and not did_set:
         _logger.warning(
             "Overriding of current EventLoggerProvider is not allowed"
         )

opentelemetry-api/tests/events/test_event_logger_provider.py

@@ -11,13 +11,27 @@
 
 
 class TestGlobals(EventsGlobalsTest, unittest.TestCase):
-    def test_set_event_logger_provider(self):
+    @patch("opentelemetry._events._logger")
+    def test_set_event_logger_provider(self, logger_mock):
         elp_mock = Mock()
         # pylint: disable=protected-access
         self.assertIsNone(events._EVENT_LOGGER_PROVIDER)
         set_event_logger_provider(elp_mock)
         self.assertIs(events._EVENT_LOGGER_PROVIDER, elp_mock)
         self.assertIs(get_event_logger_provider(), elp_mock)
+        logger_mock.warning.assert_not_called()
+
+    @patch("opentelemetry._events._logger")
+    def test_set_event_logger_provider_will_warn_second_call(
+        self, logger_mock
+    ):
+        elp_mock = Mock()
+        set_event_logger_provider(elp_mock)
+        set_event_logger_provider(elp_mock)
+
+        logger_mock.warning.assert_called_once_with(
+            "Overriding of current EventLoggerProvider is not allowed"
+        )
 
     def test_get_event_logger_provider(self):
         # pylint: disable=protected-access