Rolled back suppression of KeyError

line_profiler/_line_profiler.pyx::LineProfiler.c_last_time
    No longer suppressing the `KeyError` when accessed on a thread
    without profiling data;
    now wrapping the `KeyError` with a more helpful message

tests/test_line_profiler.py::test_last_time()
    Updated to reflect the above change

Author: TTsangSC

line_profiler/_line_profiler.pyx

@@ -471,10 +471,19 @@ cdef class LineProfiler:
 
     @property
     def c_last_time(self):
+        """
+        Raises:
+            KeyError
+                If no profiling data is available on the current thread.
+        """
         try:
             return (<dict>self._c_last_time)[threading.get_ident()]
-        except KeyError:  # We haven't actually profiled anything yet
-            return {}
+        except KeyError as e:
+            # We haven't actually profiled anything yet
+            raise (KeyError('No profiling data on the current thread '
+                            '(`threading.get_ident()` = '
+                            f'{threading.get_ident()})')
+                   .with_traceback(e.__traceback__)) from None
 
     @property
     def code_map(self):

tests/test_line_profiler.py

@@ -86,20 +86,29 @@ def test_last_time():
     are consistent.
     """
     prof = LineProfiler()
+    with pytest.raises(KeyError, match='[Nn]o profiling data'):
+        prof.c_last_time
+
+    def get_last_time(prof, *, c=False):
+        try:
+            return getattr(prof, 'c_last_time' if c else 'last_time')
+        except KeyError:
+            return {}
 
     @prof
     def func():
-        return prof.c_last_time.copy(), prof.last_time.copy()
+        return (get_last_time(prof, c=True).copy(),
+                get_last_time(prof).copy())
 
     # These are always empty outside a profiling context
     # (hence the need of the above function to capture the transient
     # values)
-    assert not prof.c_last_time
-    assert not prof.last_time
+    assert not get_last_time(prof, c=True)
+    assert not get_last_time(prof)
     # Inside `func()`, both should get an entry therefor
     clt, lt = func()
-    assert not prof.c_last_time
-    assert not prof.last_time
+    assert not get_last_time(prof, c=True)
+    assert not get_last_time(prof)
     assert set(clt) == {hash(func.__wrapped__.__code__.co_code)}
     assert set(lt) == {func.__wrapped__.__code__}