Stop using base_args in some cases to simplyfy, add pylint disables for tests, use faker for random test ids and values, etc

[hiroshinishio]

Summary

• Added # pylint: disable=unused-argument,too-many-instance-attributes to test files
• Removed base_args pattern from test files - use direct parameters instead
• Replaced hardcoded test values with faker-generated random values to avoid database conflicts
• Fixed token assignment order in issue_handler.py (moved from L168 to L116)

Files Changed

• services/github/comments/test_*.py - Rewritten with faker
• services/webhook/utils/test_create_pr_checkbox_comment.py - Rewritten with faker
• services/supabase/repositories/test_update_repository.py - Added pylint disable
• services/webhook/issue_handler.py - Fixed token usage before assignment

Social Media Post

GitAuto tests now use random values with faker library - no more database conflicts from identical test data, and cleaner test patterns without base_args dictionaries.