Skip to content

test(tools): add independent log parser fixtures (#5)#32

Open
yanyishuai wants to merge 1 commit into
weilixiong:mainfrom
yanyishuai:bounty/issue-5-log-parser-fixtures
Open

test(tools): add independent log parser fixtures (#5)#32
yanyishuai wants to merge 1 commit into
weilixiong:mainfrom
yanyishuai:bounty/issue-5-log-parser-fixtures

Conversation

@yanyishuai

@yanyishuai yanyishuai commented Jun 27, 2026

Copy link
Copy Markdown

Summary

Independent hand-written log parser fixtures + unit tests for tools/log_aggregator.py (issue #5).

Acceptance criteria (#5)

  • Independent fixtures: JSON, plain text, nginx-style lines (tools/tests/fixtures/log_parser/)
  • Tests verify timestamp, level, service/format, key fields
  • Malformed line case does not crash parsing
  • No production parser behavior change unless fixture exposes a bug
  • python build.py -m compliance diagnostic included (diagnostic/build-bf2147ac.logd)

Testing

python -m unittest tools.tests.test_log_parser_fixtures -v
python build.py -m compliance

Closes #5

Solana wallet for bounty payout: Do4v7foHJvRJLpRRoGaVPWX6DDEjX3yTK7J91gpwUQpE

@yanyishuai

Copy link
Copy Markdown
Author

Bounty submission ready for review. Independent parser fixtures + unit tests pass; build diagnostic included in PR. Happy to address feedback quickly.

@yanyishuai

Copy link
Copy Markdown
Author

Updated with acceptance criteria checklist, independent fixtures, and build diagnostic. Ready for review.

@yanyishuai

Copy link
Copy Markdown
Author

External bounty ready for maintainer review — acceptance criteria documented in PR body. Happy to address feedback immediately.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[$25 BOUNTY] [Python] Add independent log parser fixtures

1 participant