Skip to content

Clean up dead code, improve test coverage, rename for clarity#10

Merged
kenichirow merged 2 commits intomainfrom
cleanup-dead-code
Feb 16, 2026
Merged

Clean up dead code, improve test coverage, rename for clarity#10
kenichirow merged 2 commits intomainfrom
cleanup-dead-code

Conversation

@kenichirow
Copy link
Owner

@kenichirow kenichirow commented Feb 16, 2026 •
edited
Loading

Summary

  • Fix runtime crashes (nil path, nil headers, host matching, HTTPoison dispatch)
  • Remove dead code (Handler, build_path_params_match, jason dependency, commented-out code)
  • Rename FaultInjector to ErrorInjector, roll/1 to sampled?/1
  • Improve test coverage (94.44%)
  • Fix CI, reorganize README

kenichirow and others added 2 commits February 16, 2026 20:25
@claude
Rename FaultInjector to ErrorInjector
8e69cf8 
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@claude
Improve test coverage and rename roll to sampled?
f3816ef 
- Add tests for ErrorInjector resp_delay, sampled?/1, catch-all match?/4, req_headers_match?/2
- Add Plug integration tests for SlowInjector and RejectInjector
- Rename roll/1 to sampled?/1 for clarity
- Translate test names from Japanese to English

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@kenichirow kenichirow merged commit 1ce2c74 into main Feb 16, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@kenichirow