Fix other Notice usage

Author: rabidpraxis

honeybadger/core.py

@@ -60,10 +60,7 @@ def wrap_excepthook(self, func):
         sys.excepthook = self.exception_hook
 
     def exception_hook(self, type, exception, exc_traceback):
-        notice = Notice(
-            exception=exception, thread_local=self.thread_local, config=self.config
-        )
-        self._send_notice(notice)
+        self.notify(exception=exception)
         self.existing_except_hook(type, exception, exc_traceback)
 
     def shutdown(self):

honeybadger/notice.py

@@ -49,11 +49,6 @@ def excluded_exception(self):
                 return True
         return False
 
-    def _get_thread_context(self):
-        if self.thread_local is None:
-            return {}
-        return getattr(self.thread_local, "context", {})
-
     def _construct_tags(self, tags):
         """
         Accepts either:

honeybadger/tests/test_core.py

@@ -100,6 +100,27 @@ def fake_send(notice):
     assert set(captured["tags"]) == {"from_ctx", "another_tag", "explicit"}
 
 
+def test_exception_hook_calls_notify(monkeypatch):
+    hb = Honeybadger()
+    captured = {}
+
+    def fake_send(notice=None, **kwargs):
+        captured["notified"] = notice
+        return "sent"
+
+    def fake_existing_except_hook(*args, **kwargs):
+        captured["hook"] = True
+
+    monkeypatch.setattr(hb, "_send_notice", fake_send)
+
+    exc = ValueError("fail!")
+    hb.wrap_excepthook(fake_existing_except_hook)
+    hb.exception_hook(ValueError, exc, None)
+
+    assert captured["notified"].exception == exc
+    assert captured["hook"] is True
+
+
 def test_threading():
     hb = Honeybadger()
     hb.configure(api_key="aaa", environment="development")  # Explicitly use development