Fixes #38547 - Add job template clone command to expose clone API endpoint

[Gauravtalreja1]

Description:

Exposeing the POST /api/job_templates/:id/clone API endpoint that was previously unavailable in the Hammer CLI

Test Results:

[root@foreman ~]# hammer job-template list | grep -i "ansible role"
233 | Ansible Roles - Ansible Default                            | Ansible Playbook           | Ansible  | job_template
234 | Ansible Roles - Install from Galaxy                        | Ansible Galaxy             | Ansible  | job_template
235 | Ansible Roles - Install from git                           | Ansible Roles Installation | Ansible  | job_template
[root@foreman ~]# hammer job-template clone --id 233 --name "Clone Ansible Roles - Ansible Default"
Job template cloned
[root@foreman ~]# hammer job-template list | grep -i "clone"
262 | Clone Ansible Roles - Ansible Default                      | Ansible Playbook           | Ansible  | job_template
[root@foreman ~]#  hammer job-template info --id 262
ID:                         262
Name:                       Clone Ansible Roles - Ansible Default
Job Category:               Ansible Playbook
Provider:                   Ansible
Type:                       job_template
Ansible Callback Enabled:   no
Ansible Check Mode Enabled: no
Description:

Inputs:

Locations:
    Default Location
    test_loc
Organizations:
    Default Organization
    test_org

[adamruzicka]

This should have the same interface as hammer template clone - --name (and --id) should identify the source template (the one being cloned), --new-name should be the name for the clone

[nacoool]

Tested the changes and everything works as expected

[ofedoren]

Here is the command Adam was referring to https://github.com/theforeman/hammer-cli-foreman/blob/master/lib/hammer_cli_foreman/template.rb#L190-L218

[stejskalleos]

One comment to address.
Redmine Check CI is failing, but looking at the git log, nobody bothers anymore about it :D

[stejskalleos]

The be bin/hammer template clone --new-name "clone_wars" --id 150 command uses --new-name param, we should do the same here, as mentioned by other reviewers.

[Gauravtalreja1]

@adamruzicka @stejskalleos Updated, Thank you! Could you please take another look at this PR?

[nacoool]

Tested using packit, Works as expected.

[stejskalleos]

@Gauravtalreja1 Rubocop CI is failing, please fix it so we can merge it.

[stejskalleos]

🍏 LGTM

hammer job-template clone --new-name "clone_wars" --id 150
Job template cloned