Fix test_thread_please_reaction_exists to use correct reaction name

[Copilot]

The test was querying for thread_please (underscore) while the config defines thread-please (hyphen), causing the test to fail.

Changes:

• Updated tests/test_automator_lambda.py line 246: reaction_configs.get('thread_please') → reaction_configs.get('thread-please')
• Updated docstring to match corrected reaction name

The config file (automator/config.yaml) correctly defines the reaction as thread-please to match Slack's emoji naming convention.

Original prompt

Problem

The test test_thread_please_reaction_exists is failing because it's looking for the wrong reaction name.

Actual Slack reaction: thread-please (with hyphen)
Test is looking for: thread_please (with underscore)

The config file correctly defines the reaction as thread-please to match the actual Slack reaction emoji, but the test file has an incorrect reference.

Solution

Update tests/test_automator_lambda.py to fix the test method test_thread_please_reaction_exists to look for thread-please instead of thread_please.

File to modify: tests/test_automator_lambda.py
Lines 244-253: In the test method test_thread_please_reaction_exists, change:

• Line 246: reaction_config = lambda_function.reaction_configs.get('thread_please')
  • Change to: reaction_config = lambda_function.reaction_configs.get('thread-please')

This will align the test with the actual Slack reaction name and resolve the failing test.

This pull request was created from Copilot chat.

---

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.