@Taragolis thank you for the review -- all but mocking the hook property and client property should be all set. Pretty unsure how to best do that? Should I just test that the hook was not called and keep the client portion of the test the same?

It could be something like that.

        with mock.patch.object(EcsBaseOperator, "hook", new_callable=mock.PropertyMock) as m:
            mocked_hook = mock.MagicMock(name="FakeHook")
            mocked_client = mock.MagicMock(name="FakeBoto3Client")
            mocked_hook.conn = mocked_client
            m.return_value = mocked_hook

            assert op.client == mocked_client
            m.assert_called_once()

And after that you could remove test_hook_and_client because it not required anymore after that

Thank you for the example!

Need to fix Static Checks:

• you could run locally breeze static-checks {id-of-failed-check} --all-files e.g. breeze static-checks ruff --all-files
• Another option run it thought pre-commit pre-commit run {id-of-failed-check} --all-files e.g. pre-commit run ruff --all-files

You could have a look what failed in Tests / Static checks (pull_request) logs

Awesome work, congrats on your first merged pull request! You are invited to check our Issue Tracker for additional contributions.