removing the from nailgun import entities and moving target_sat, module_target_sat

[omkarkhatavkar]

Key Changes:

• Direct Imports Removed: We've cleaned up our codebase by eliminating direct imports from the nailgun package.

• Using target_sat, module_target_sat, class_target_sat: Modified the API calls and using the target_sat.api. pattern for all API calls.

[omkarkhatavkar]

trigger: test-robottelo

[omkarkhatavkar]

trigger: test-robottelo

[omkarkhatavkar]

@shweta83 @vsedmik @pondrejk @Griffin-Sullivan Can you please also take a look from the components side as it is touching multiple component tests.

[Gauravtalreja1]

@omkarkhatavkar Great job, thanks for this refactor ❤️
Could you add changes for updating fixtures, and setup part done by nailgun entities in CLI/UI/destructive/etc modules too? or are you intending to open a separate PR for each?

[shweta83]

@omkarkhatavkar Changes impacting Rocket team components are good. Ack for the rocket team tests except for @Gauravtalreja1 comment.

[omkarkhatavkar]

@omkarkhatavkar Great job, thanks for this refactor ❤️ Could you add changes for updating fixtures, and setup part done by nailgun entities in CLI/UI/destructive/etc modules too? or are you intending to open a separate PR for each?

@Gauravtalreja1 It will be the seperate PR's as It will be easy to test it out and also review will be smoother

[Griffin-Sullivan]

As far as I can tell, none of these components are owned by Platform. Everything looks good to me!

[ColeHiggins2]

Looks good to me! Nice work

[omkarkhatavkar]

Hi @JacobCallahan and @vijaysawant, please review. Once confirmed, we can merge without frequent rebases. Thanks!