Error handling and refactoring pre-undeploy-event-reg hook implementation

[sangeetha5491]

Description

Earlier , on aio app undeploy, if no actions were deployed, but the associated event registrations were still present, it would not undeploy the event registrations. This could happen if the actions were successfully undeployed but the event registrations failed.

This change fixes that behavior, by separating the events hook being called prior to the actions being undeployed.

Also, i have removed the post-deploy hook ( since i dont see a need right now, and can be added when required ) and added error handling to the pre-undeploy-events-reg

Related Issue

• Implement pre-undeploy-event-reg hook to undeploy event registrations aio-cli-plugin-events#65

Blocks

• Implement pre-undeploy-event-reg hook to undeploy event registrations aio-cli-plugin-events#65

Motivation and Context

handle undeploy for event registrations by calling the pre-undeploy-events-reg hook

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

• I have signed the Adobe Open Source CLA.
• My code follows the code style of this project.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have read the CONTRIBUTING document.
• I have added tests to cover my changes.
• All new and existing tests passed.

[codecov]

Codecov Report

Merging #694 (ae06a93) into master (b9cf971) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master      #694   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           57        57           
  Lines         3148      3146    -2     
  Branches       587       587           
=========================================
- Hits          3148      3146    -2     

Impacted Files	Coverage Δ
src/commands/app/undeploy.js	100.00% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[moritzraho]

ok lgtm makes sense to separate the events processing from the actions

[purplecabbage]

lgtm!