Merge pull request #708 from auscompgeek/datalog-get

Fix DataLogManager.getLog() trying to copy
robotpy · Mar 19, 2022 · c51b1fc · c51b1fc
2 parents cdf3c37 + dc920f0
commit c51b1fc
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 0 deletions.
diff --git a/gen/DataLogManager.yml b/gen/DataLogManager.yml
@@ -9,5 +9,6 @@ classes:
       Start:
       Log:
       GetLog:
+        return_value_policy: reference
       GetLogDir:
       LogNetworkTables:
diff --git a/tests/test_datalogmanager.py b/tests/test_datalogmanager.py
@@ -0,0 +1,10 @@
+import pathlib
+import wpilib
+
+
+def test_get_log(tmp_path: pathlib.Path):
+    log_dir = tmp_path / "wpilogs"
+    log_dir.mkdir()
+    wpilib.DataLogManager.start(str(log_dir))
+    log = wpilib.DataLogManager.getLog()
+    assert log is not None