[V2V] Use full class name in task context data for conversion hosts

[djberg96]

Originally we set the resource_type to the full name, e.g. ManageIQ::Providers::Redhat::InfraManager::Host, so that we could take advantage of the SupportsFeature mixin. And so, we added a validation to force the issue.

However, that broke Rails polymorphic associations, so we changed the param internally to just the base name, e.g. host, in #18604.

Unfortunately, this also got saved in the context data for the associated task. The upshot is that when a user tries to use the "retry" button, which uses the context data to resubmit the request, there is a validation failure because the resource type is no longer the full name.

So, this PR restores the full resource type in the context_data of the associated task internally.

https://bugzilla.redhat.com/show_bug.cgi?id=1713394

[djberg96]

@mturley Please take a look.

[djberg96]

@miq-bot add_label bug, transformation

[djberg96]

@miq-bot add_label hammer/yes

[mturley]

@djberg96 I'm testing this locally and it looks like auth_user is also missing from the request_params. I think we need to include that explicitly too.

[mturley]

I think this is why it's missing from params: https://github.com/ManageIQ/manageiq-api/blob/5a0107f3c5852fa05a8f37c526abc2301673c8ca/app/controllers/api/conversion_hosts_controller.rb#L53

[djberg96]

@mturley I was never really sure why that was split out originally. Doesn't seem to be a good reason for it, but I'll just work around it for now.

[djberg96]

@mturley @fdupont-redhat Updated.

[miq-bot]

Checked commits https://github.com/djberg96/manageiq/compare/f11ff8209c7868a93ba48072e6c4f69250bc5144~...aa2e02e78494cad789732f51c5b427da4adb65ba with ruby 2.3.3, rubocop 0.69.0, haml-lint 0.20.0, and yamllint 1.10.0
2 files checked, 0 offenses detected
Everything looks fine. ⭐

[mturley]

👍 LGTM. Thanks @djberg96

[ghost]

👍 LGTM too.

[djberg96]

Closing in favor of ManageIQ/manageiq-api#600