Bugfix/scheduler/aws regressions

[maxhoheiser]

Fixing regressions for event bridge scheduler CRUD.
AWS Snapshot Tests are in this PR: localstack/localstack#12114

[bentsku]

Should this fix be added upstream in https://github.com/moto/moto? I think this is a valid candidate 😄

[maxhoheiser]

Should this fix be added upstream in https://github.com/moto/moto? I think this is a valid candidate 😄

Since I added the tests not in moto but in localstack I doubt that Bert will accept these fixes - and I want to unblock the customer asap - this is why I though to create the PR into our branch.

[bentsku]

Since I added the tests not in moto but in localstack I doubt that Bert will accept these fixes - and I want to unblock the customer asap - this is why I though to create the PR into our branch.

I see, but I think this is overhead for us to maintain, @viren-nadkarni will chime in, but every commit we add here is additional maintenance burden at every moto bump. The testing in moto is not too complicated, and adding those should be straightforward, hopefully.

[viren-nadkarni]

@bentsku is right, this must be submitted upstream. I'm happy to bump moto-ext as soon as it is merged.

[maxhoheiser]

submitted upstream in this PR: getmoto#8478