After the fact feedback from PR #558

[texasaggie97]

• This contribution adheres to CONTRIBUTING.md.

There was feedback that came in after #558 was completed. This handles that feedback.

What does this Pull Request accomplish?

• Rename new tests to better inform what they are doing
• Move calls that cause expected asserts and warnings to "assert" section
• Raise timeout to 10 seconds when waiting on semaphore

Why should this Pull Request be merged?

• Comments from Add tests to verify closing task in event callback does not crash or error #558

What testing has been done?

PR

[github-actions]

Test Results

    34 files  ±0      34 suites  ±0   47m 22s ⏱️ + 4m 2s
 2 248 tests ±0   1 910 ✅ ±0    338 💤 ±0  0 ❌ ±0 
39 690 runs  ±0  34 134 ✅ ±0  5 556 💤 ±0  0 ❌ ±0 

Results for commit 97c5eb9. ± Comparison against base commit 06da4f4.

This pull request removes 4 and adds 4 tests. Note that renamed tests count towards both.

tests.component.test_task_events ‑ test___done_event_registered___call_clear_in_callback___stop_close_in_callback_with_success_status[grpc_init_kwargs]
tests.component.test_task_events ‑ test___done_event_registered___call_clear_in_callback___stop_close_in_callback_with_success_status[library_init_kwargs]
tests.component.test_task_events ‑ test___every_n_samples_event_registered___call_clear_in_callback___stop_close_in_callback_with_success_status[grpc_init_kwargs]
tests.component.test_task_events ‑ test___every_n_samples_event_registered___call_clear_in_callback___stop_close_in_callback_with_success_status[library_init_kwargs]

tests.component.test_task_events ‑ test___done_event_registered___call_stop_close_in_callback___task_closed_with_success_status[grpc_init_kwargs]
tests.component.test_task_events ‑ test___done_event_registered___call_stop_close_in_callback___task_closed_with_success_status[library_init_kwargs]
tests.component.test_task_events ‑ test___every_n_samples_event_registered___call_stop_close_in_callback___task_closed_with_success_status[grpc_init_kwargs]
tests.component.test_task_events ‑ test___every_n_samples_event_registered___call_stop_close_in_callback___task_closed_with_success_status[library_init_kwargs]

This pull request removes 2 skipped tests and adds 2 skipped tests. Note that renamed tests count towards both.

tests.component.test_task_events ‑ test___done_event_registered___call_clear_in_callback___stop_close_in_callback_with_success_status[grpc_init_kwargs]
tests.component.test_task_events ‑ test___every_n_samples_event_registered___call_clear_in_callback___stop_close_in_callback_with_success_status[grpc_init_kwargs]

tests.component.test_task_events ‑ test___done_event_registered___call_stop_close_in_callback___task_closed_with_success_status[grpc_init_kwargs]
tests.component.test_task_events ‑ test___every_n_samples_event_registered___call_stop_close_in_callback___task_closed_with_success_status[grpc_init_kwargs]