Flatten subcollections and enhance sweep scope to work with them

[Ladas]

Subcollections as concept are being ditched, everything will be flat now.

If we want to archive e.g. container_groups with their 'subcollections' containers, after we save them all we'll send scope:

sweep_scope: ["container_groups", "containers"]

If we want to archive it targeted way, so we've saved container_groups X and Y and all their containers we'll send scope:

:sweep_scope => {
        :container_groups  => [
          {:ems_ref => "container_group_ems_ref_1"},
          {:ems_ref => "container_group_ems_ref_2"},
        ],
        :containers => [
          {:container_group => persister.container_groups.lazy_find("container_group_ems_ref_1")},
          {:container_group => persister.container_groups.lazy_find("container_group_ems_ref_2")},
        ]
      }

The scope provided can contain any attributes on IC.

Depends on:

• [x] make serialization work
• [x] add last_seen_on to every DB table involved in refresh https://github.com/ManageIQ/topological_inventory-core/pull/126

[Ladas]

cc @slemrmartin

[miq-bot]

Checked commits https://github.com/Ladas/inventory_refresh/compare/9d60f6ec863011fdbeb7ad6c3359cdec938e1699~...e3ca0fac409a0e15851c0854c19a97d81da41e99 with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0 23 files checked, 7 offenses detected

lib/inventory_refresh/inventory_collection/builder.rb

• [ ] :exclamation: - Line 12, Col 23 - Layout/SpaceInsideArrayPercentLiteral - Use only a single space inside array percent literal.

lib/inventory_refresh/inventory_object.rb

• [ ] :exclamation: - Line 221, Col 9 - Layout/MultilineOperationIndentation - Align the operands of a condition in an if statement spanning multiple lines.
• [ ] :exclamation: - Line 225, Col 9 - Layout/MultilineOperationIndentation - Align the operands of a condition in an if statement spanning multiple lines.

lib/inventory_refresh/save_collection/sweeper.rb

• [ ] :exclamation: - Line 70, Col 38 - Rails/Blank - Use scope.blank? instead of scope.nil? || scope.empty?.

spec/persister/sweep_inactive_records_spec.rb

• [ ] :exclamation: - Line 432, Col 32 - Layout/ClosingParenthesisIndentation - Indent ) the same as the start of the line where ( is.
• [ ] :exclamation: - Line 502, Col 11 - Style/IdenticalConditionalBranches - Move InventoryRefresh::SaveInventory.sweep_inactive_records( persister.manager, persister.inventory_collections, persister.sweep_scope, refresh_state(2, time, persister.sweep_scope) ) out of the conditional.
• [ ] :exclamation: - Line 509, Col 11 - Style/IdenticalConditionalBranches - Move InventoryRefresh::SaveInventory.sweep_inactive_records( persister.manager, persister.inventory_collections, persister.sweep_scope, refresh_state(2, time, persister.sweep_scope) ) out of the conditional.