Bug 1600870

Summary: [Debt] Implement specs for transformation methods
Product: Red Hat CloudForms Management Engine Reporter: Fabien Dupont <fdupont>
Component: AutomateAssignee: Fabien Dupont <fdupont>
Status: CLOSED CURRENTRELEASE QA Contact: Yadnyawalk Tale <ytale>
Severity: low Docs Contact:
Priority: unspecified    
Version: 5.10.0CC: akarol, cpelland, fdupont, jprause, mfeifer, mkanoor, obarenbo, simaishi, smallamp, tfitzger
Target Milestone: GA   
Target Release: 5.10.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: v2v
Fixed In Version: 5.10.0.21 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2019-02-11 14:08:00 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: CFME Core Target Upstream Version:
Embargoed:
Bug Depends On: 1630783    
Bug Blocks:    

Description Fabien Dupont 2018-07-13 08:53:51 UTC
Over time, the Transformation namespace has been populated to implement new features, but no spec has been provided, increasing the overall technical debt of the project. This issue tracks the implementation of specs for this namespace, in order to reach full coverage.

Comment 6 CFME Bot 2018-10-19 20:31:32 UTC
New commit detected on ManageIQ/manageiq-content/hammer:

https://github.com/ManageIQ/manageiq-content/commit/ff0f2e10b527ea2ea4af84ac3841a58589d6342b
commit ff0f2e10b527ea2ea4af84ac3841a58589d6342b
Author:     Madhu Kanoor <mkanoor>
AuthorDate: Thu Sep 27 09:37:05 2018 -0400
Commit:     Madhu Kanoor <mkanoor>
CommitDate: Thu Sep 27 09:37:05 2018 -0400

    Merge pull request #421 from fdupont-redhat/v2v_refactor_and_specs_transformation_host

    Refactor and add specs for Transformation Hosts Common Utils

    (cherry picked from commit fcc47328a0ea2551904d9f58105c4c8e5fdaffb8)

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

 content/automate/ManageIQ/Transformation/Infrastructure/VM/vmwarews.class/__methods__/utils.rb | 58 +-
 content/automate/ManageIQ/Transformation/TransformationHosts/Common.class/__methods__/utils.rb | 81 +-
 spec/content/automate/ManageIQ/Transformation/TransformationHosts/Common.class/__methods__/utils_spec.rb | 314 +
 3 files changed, 379 insertions(+), 74 deletions(-)

Comment 10 Yadnyawalk Tale 2018-11-21 23:50:05 UTC
"This was a tracker BZ, so that we don't forget to implement specs.." 

Verifying with Comment #9 's justification, Thanks for reply @Fabien.