Fixes #38170 - Explicitly handle content_view params during AK create

[jeremylenz]

What are the changes introduced in this pull request?

The find_cve_for_single method in the activation keys controller handles the logic of finding the content view environment when you've passed in separate content_view and lifecycle_environment params. However, before this change it did not handle content_view param correctly; it ignored it. The effect was a false positive - the activation key would get created, but not with the content view you specified.

With this change, you should correctly get the error

Environment ID and content view ID must be provided together

Considerations taken when implementing this change?

What are the testing steps for this pull request?

Test hammer activation-key create
Also test the web UI and make sure nothing breaks there with creating activation keys.

[LadislavVasina1]

I can confirm that this patch fixes the issue. Tested with upstream packit build.
PRE-PATCH ❌

hammer activation-key create --name="testAK_1" --organization="testORG" --content-view="testCV"
Activation key created.

POST-PATCH ✅

# hammer activation-key create --name="testAK_1" --organization="testORG" --content-view="testCV"
Could not create the activation key:
  Environment ID and content view ID must be provided together

[sjha4]

Before PR:

[root@centos9-katello-devel2 hammer-cli-katello]# hammer activation-key create --content-view=cv1 --organization-id=1 --name=test
Activation key created. 🔴

[root@centos9-katello-devel2 hammer-cli-katello]# hammer activation-key update --content-view=cv1 --organization-id=1 --name=test
Activation key updated. 🔴

hammer activation-key create --lifecycle-environment=test --organization-id=1 --name=test
Could not create the activation key:
Environment ID and content view ID must be provided together 🟢

[root@centos9-katello-devel2 hammer-cli-katello]# hammer activation-key create --content-view=cv1 --organization-id=1 --name=test --lifecycle-environment=test
Could not create the activation key:
Couldn't find content view environment with content view ID '2' and environment ID '2' 🟢

With PR:

[root@centos9-katello-devel2 hammer-cli-katello]# hammer activation-key create --content-view=cv1 --organization-id=1 --name=test
Could not create the activation key:
Environment ID and content view ID must be provided together 🟢

root@centos9-katello-devel2 hammer-cli-katello]# hammer activation-key create --lifecycle-environment=test --organization-id=1 --name=test
Could not create the activation key:

Environment ID and content view ID must be provided together 🟢
[root@centos9-katello-devel2 hammer-cli-katello]# hammer activation-key create --content-view=cv1 --organization-id=1 --name=test --lifecycle-environment=test
Could not create the activation key:
Couldn't find content view environment with content view ID '2' and environment ID '2' 🟢

[root@centos9-katello-devel2 hammer-cli-katello]# hammer activation-key update --content-view=cv1 --organization-id=1 --name=test
Could not update the activation key:
Environment ID and content view ID must be provided together 🟢

Happy path works as expected in both cases:

[root@centos9-katello-devel2 hammer-cli-katello]# hammer activation-key update --content-view=cv1 --lifecycle-environment=test --organization-id=1 --name=test
Activation key updated. 🟢