[Security Solution][Case] Attach alerts to cases: Tests

[cnasikas]

Summary

This PR adds and improves tests for the attach alerts to cases feature.

Checklist

Delete any items that are not applicable to this PR.

• Unit or functional tests were updated or added to match the most common scenarios

For maintainers

• This was checked for breaking API changes and was labeled appropriately

[stephmilovic]

You should use waitFor here instead. See: https://kentcdodds.com/blog/common-mistakes-with-react-testing-library. You can also get rid of the wrapper.update. that article says do the side effect, then wrap the expect in waitFor:

  it('it toggles the switch', async () => {
    const wrapper = mount(<SyncAlertsSwitch disabled={false} />);

    wrapper.find('button[data-test-subj="sync-alerts-switch"]').first().simulate('click');
    await waitFor(() => {
      expect(wrapper.find('[data-test-subj="sync-alerts-switch"]').first().prop('checked')).toBe(
        false
      );
    });
  });

[cnasikas]

Thanks for the article!

[cnasikas]

@elasticmachine merge upstream

[kibanamachine]

💚 Build Succeeded

• continuous-integration/kibana-ci/pull-request
• Commit: 3936f23

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id	before	after	diff
securitySolution	8.5MB	8.5MB	+461.0B

History

• 💚 Build #97609 succeeded 2a48b78
• 💔 Build #97601 failed 18f1be5
• 💔 Build #97526 failed d725669
• 💚 Build #97010 succeeded 20984efa61bf59bf6bc66044f40fcfa91b9100c5
• 💚 Build #96631 succeeded 9527b4b94187ec858ccc9236174f3d4eed7fe82f

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

[stephmilovic]

thanks for making those changes. LGTM!