fix: add log parser tests and fix bugs - #79

Open

abhi-0203 wants to merge 1 commit into

weilixiong:mainfrom

abhi-0203:fix/log-parser-tests

abhi-0203 commented Jul 25, 2026

Summary

Added comprehensive test suite for log parser and fixed bugs discovered during testing.

Changes

New Files

tools/tests/ - Complete test suite with fixtures
15 test cases covering JSON, text, nginx, and malformed logs
Test fixtures for all parser types

Bug Fixes

Fixed extract_service() to match hyphenated service names (e.g. auth-service)
Fixed NginxLogParser remote_user mapping (was using wrong regex group)
Fixed Python 3.11 f-string syntax error in build.py

Test Results

All 15 tests pass (27 subtests), covering:

JSON log parsing
Text log parsing
Nginx log parsing
Malformed log handling
Edge cases
Aggregator integration

Related

Bounty: 5 for adding independent log parser


          fix: add log parser tests and fix bugs

7729db7

- Added 15 test cases for log parser (JSON, text, nginx, malformed)
- Fixed extract_service() to match hyphenated service names
- Fixed NginxLogParser remote_user mapping
- Fixed Python 3.11 f-string syntax error in build.py
- All tests pass (27 subtests)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet