Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sa 3139 replace iwr with sdk #462

Merged
merged 31 commits into from
Feb 6, 2023

Conversation

kmaranionjc
Copy link
Contributor

@kmaranionjc kmaranionjc commented Feb 2, 2023

Issues

  • SA-3139 - SA-3139-Replace-Get-JCResults with SDK in Get-JCSystemApp

What does this solve?

Replace IWR with Get-JcSdkSystemInsight (SDK). Fixes output difference with system insights table and module output

Is there anything particularly tricky?

n/a

How should this be tested?

Run each of these commands and validate the results:

Get-JcSystemApp #Test with and without $Parallel

#SoftwareName

Get-JcSystemApp -softwareName “Microsoft Edge” 
Get-JcSystemApp -softwareName “Chess”  
Get-JcSystemApp -softwareName “firefox” 

#SoftwareName and OS

Get-JcSystemApp -softwareName “firefox” -systemOs “Linux” 
Get-JcSystemApp -softwareName “Chess” -systemOs “MacOs” 
Get-JcSystemApp -softwareName “Microsoft Edge” -systemOs “Windows” 

#SoftwareName and Id

Get-JcSystemApp -softwareName “Microsoft Edge” -systemId “Id”    #Windows 
Get-JcSystemApp -softwareName “firefox” -systemId “Id”  #Linux 
Get-JcSystemApp -softwareName “Chess” -systemId “Id” #Mac 

#SoftwareName and OS and version

Get-JcSystemApp -softwareName “Microsoft Edge” -systemOs “Windows” - softwareVersion “”
Get-JcSystemApp -softwareName “firefox” -systemOs “Linux” - softwareVersion “”
Get-JcSystemApp -softwareName “Chess” -systemOs “MacOs” - softwareVersion “”

#SoftwareName and Id and version

Get-JcSystemApp -softwareName “Microsoft Edge” -systemId “Id” - softwareVersion “” #Windows
Get-JcSystemApp -softwareName “firefox” -systemId “Id” - softwareVersion “” #Linux
Get-JcSystemApp -softwareName “Chess” -systemId “Id” - softwareVersion “” #MacOs

Test with and without $Parallel
#SoftwareName and Search. Test with and without $Parallel

Get-JcSystemApp -softwareName “microsoft Edge” -search
Get-JcSystemApp -softwareName “chess” -search
Get-JcSystemApp -softwareName “Firefox” -search

#SoftwareName and Os and Search

Get-JcSystemApp -softwareName “firefox” -systemOs “Linux” -search
Get-JcSystemApp -softwareName “safari” -systemOs “MacOs” -search
Get-JcSystemApp -softwareName “microsoft edge” -systemOs “Windows” -search

#SoftwareName and Id and Search

Get-JcSystemApp -softwareName “Microsoft Edge” -systemId “Id”    -search #Windows
Get-JcSystemApp -softwareName “firefox” -systemId “Id”  -search #Linux
Get-JcSystemApp -softwareName “Chess” -systemId “Id” -search #Mac

Screenshots

@kmaranionjc kmaranionjc self-assigned this Feb 2, 2023
@kmaranionjc kmaranionjc changed the base branch from master to SA-3067-Get-JCSystemApp February 2, 2023 19:51
@kmaranionjc kmaranionjc changed the base branch from SA-3067-Get-JCSystemApp to SA-3118_Get_JCSystemApp February 2, 2023 19:56
Copy link
Contributor

@jworkmanjc jworkmanjc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only change I have on the branch is the quick change to Get-JCSdk system to just return fewer fields

Copy link
Contributor

@gweinjc gweinjc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, I noticed some string manipulation that could possibly be reduced, but that's pretty nitpicky. Nice job!

… [skip ci]

Co-authored-by: Joe Workman <54448601+jworkmanjc@users.noreply.github.com>
@jworkmanjc
Copy link
Contributor

And last but not least, let's commit to changing our softwareName parameter to just Name and softwareVersion to just Version

Copy link
Contributor

@jworkmanjc jworkmanjc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the changes

@kmaranionjc kmaranionjc merged commit 90ed85e into SA-3118_Get_JCSystemApp Feb 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants