summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--git_trace2_event_log.py8
-rw-r--r--tests/test_git_trace2_event_log.py2
2 files changed, 5 insertions, 5 deletions
diff --git a/git_trace2_event_log.py b/git_trace2_event_log.py
index 0d4f23f4..0e5e9089 100644
--- a/git_trace2_event_log.py
+++ b/git_trace2_event_log.py
@@ -167,9 +167,9 @@ class EventLog(object):
167 repo_config = {k: v for k, v in config.items() if k.startswith('repo.')} 167 repo_config = {k: v for k, v in config.items() if k.startswith('repo.')}
168 self.LogConfigEvents(repo_config, 'def_param') 168 self.LogConfigEvents(repo_config, 'def_param')
169 169
170 def GetDataEventName(self, key): 170 def GetDataEventName(self, value):
171 """Returns the 'data-json' if the key is argv else returns 'data'.""" 171 """Returns 'data-json' if the value is an array else returns 'data'."""
172 return 'data-json' if key.endswith('sys.argv') else 'data' 172 return 'data-json' if value[0] == '[' and value[-1] == ']' else 'data'
173 173
174 def LogDataConfigEvents(self, config, prefix): 174 def LogDataConfigEvents(self, config, prefix):
175 """Append a 'data' event for each config key/value in |config| to the current log. 175 """Append a 'data' event for each config key/value in |config| to the current log.
@@ -182,7 +182,7 @@ class EventLog(object):
182 prefix: Prefix for each key that is logged. 182 prefix: Prefix for each key that is logged.
183 """ 183 """
184 for key, value in config.items(): 184 for key, value in config.items():
185 event = self._CreateEventDict(self.GetDataEventName(key)) 185 event = self._CreateEventDict(self.GetDataEventName(value))
186 event['key'] = f'{prefix}/{key}' 186 event['key'] = f'{prefix}/{key}'
187 event['value'] = value 187 event['value'] = value
188 self._log.append(event) 188 self._log.append(event)
diff --git a/tests/test_git_trace2_event_log.py b/tests/test_git_trace2_event_log.py
index 6131c7b3..7bd21e21 100644
--- a/tests/test_git_trace2_event_log.py
+++ b/tests/test_git_trace2_event_log.py
@@ -267,7 +267,7 @@ class EventLogTestCase(unittest.TestCase):
267 self.assertIn('value', event) 267 self.assertIn('value', event)
268 key = event['key'].removeprefix(f'{prefix_value}/') 268 key = event['key'].removeprefix(f'{prefix_value}/')
269 value = event['value'] 269 value = event['value']
270 self.assertEqual(self._event_log_module.GetDataEventName(key), event['event']) 270 self.assertEqual(self._event_log_module.GetDataEventName(value), event['event'])
271 self.assertTrue(key in config and value == config[key]) 271 self.assertTrue(key in config and value == config[key])
272 272
273 def test_error_event(self): 273 def test_error_event(self):