[WIP] Make provider's enable/disable methods protected

[slemrmartin]

ExtManagementSystem.enable! and disable! should be used only by parent EMS to children or by pause!/resume, not as public methods

Issue: #17489

depends on:

• Suspend providers #17452
• • Allow the suspension of cloud, infra and physical infra providers manageiq-ui-classic#4269
• • Use the new universal methods for suspending/resuming a provider manageiq-api#434

[miq-bot]

Checked commits slemrmartin/manageiq@ec3cb9e~...53dcf98 with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0
10 files checked, 6 offenses detected

app/models/ext_management_system.rb

• ❗ - Line 798, Col 27 - Style/SymbolProc - Pass &:enable! as an argument to each instead of a block.
• ❗ - Line 800, Col 27 - Style/SymbolProc - Pass &:disable! as an argument to each instead of a block.

app/models/zone.rb

• ❗ - Line 11, Col 3 - Rails/HasManyOrHasOneDependent - Specify a :dependent option.
• ❗ - Line 12, Col 3 - Rails/HasManyOrHasOneDependent - Specify a :dependent option.
• ❗ - Line 7, Col 3 - Rails/InverseOf - Specify an :inverse_of option.

spec/models/miq_queue_spec.rb

• ⚠️ - Line 374, Col 7 - Lint/UselessAssignment - Useless assignment to variable - msg.

[miq-bot]

This pull request is not mergeable. Please rebase and repush.