Skip to content

Commit 2529323

Browse files
committed
[dev] expose functions to query whether an event has moniloggers registered to it
1 parent d045a89 commit 2529323

File tree

6 files changed

+11
-11
lines changed

6 files changed

+11
-11
lines changed

build-and-test.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,5 +2,5 @@
22

33
rm dist/*
44
python3 -m build
5-
pip install dist/monilogger-0.0.3.tar.gz
5+
pip install dist/monilogger-0.0.4.tar.gz
66
python3 -m unittest

setup.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99
from setuptools import Extension, find_packages, setup
1010
from distutils import ccompiler
1111

12-
__version__ = "0.0.3"
12+
__version__ = "0.0.4"
1313

1414
# Convert distutils Windows platform specifiers to CMake -A arguments
1515
PLAT_TO_CMAKE = {

src/api/MoniLogger.cc

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -218,12 +218,12 @@ namespace MoniLogger
218218
}
219219
}
220220

221-
inline bool has_registered_moniloggers(size_t event)
221+
bool has_registered_moniloggers(size_t event)
222222
{
223223
return registered_moniloggers[event].empty();
224224
}
225225

226-
inline std::list<py::function> get_registered_moniloggers(size_t event)
226+
std::list<py::function> get_registered_moniloggers(size_t event)
227227
{
228228
return registered_moniloggers[event];
229229
}

src/api/monilogger/monilogger.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,4 +25,4 @@ def define_event(event, triggering_events):
2525
event_name = event
2626
else:
2727
event_name = event.__qualname__
28-
mnlg.define_event(event_name, triggering_events)
28+
mnlg.define_composite_event(event_name, triggering_events)

src/api/monilogger_module.cc

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -6,9 +6,9 @@ PYBIND11_MODULE(_monilogger, m) {
66
.def(py::init<>());
77
m.def("register", &MoniLogger::register_monilogger);
88
m.def("stop", &MoniLogger::unregister_monilogger);
9-
m.def("define_event", &MoniLogger::register_composite_event);
10-
m.def("define_basic_events", &MoniLogger::register_base_events);
11-
m.def("get_basic_events", &MoniLogger::get_base_events);
9+
m.def("define_composite_event", &MoniLogger::register_composite_event);
10+
m.def("define_base_events", &MoniLogger::register_base_events);
11+
m.def("get_base_events", &MoniLogger::get_base_events);
1212
m.def("emit_event", [](std::string event_name, std::shared_ptr<MoniLogger::MoniLoggerExecutionContext> scope)
1313
{
1414
MoniLogger::trigger(event_name, scope);

tests/tests.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -15,13 +15,13 @@ def test_undefined_event(self):
1515
def test_monilogger(ctx):
1616
self.result = self.result + 1
1717

18-
mnlg.define_basic_events({"MyEvent":0})
18+
mnlg.define_base_events({"MyEvent":0})
1919
mnlg.emit_event("MyEvent", mnlg.MoniLoggerExecutionContext())
2020

2121
self.assertEqual(self.result, 1)
2222

2323
def test_class_event(self):
24-
mnlg.define_basic_events({"MyOtherEvent":0})
24+
mnlg.define_base_events({"MyOtherEvent":0})
2525

2626
@monilogger.register(MyOtherEvent)
2727
def test_monilogger(ctx):
@@ -32,7 +32,7 @@ def test_monilogger(ctx):
3232
self.assertEqual(self.result, 1)
3333

3434
def test_composite_event(self):
35-
mnlg.define_basic_events({"MyEvent":0, "MyOtherEvent":1})
35+
mnlg.define_base_events({"MyEvent":0, "MyOtherEvent":1})
3636
monilogger.define_event("MyCompositeEvent", ["MyEvent", "MyOtherEvent"])
3737

3838
@monilogger.register("MyCompositeEvent")

0 commit comments

Comments
 (0)