hjorslev / SteamPS

Module that utilizes PowerShell as a wrapper for SteamCMD and interacts with various Steam APIs.

https://www.powershellgallery.com/packages/SteamPS

MIT License

69 stars 14 forks source link

✅ Pester Tests #63

Closed hjorslev closed 3 months ago

hjorslev commented 4 months ago

PR Summary

This PR contains pester tests that aims improving the code coverage of SteamPS.

Context

Changes

[ ] Get-SteamFriendList
[ ] Get-SteamNews
[ ] Get-SteamPlayerBan
[ ] Get-SteamPlayerSummary
[ ] Get-SteamRecentlyPlayedGame

Checklist

[x] Pull Request has a meaningful title.
[ ] Summarized changes.
[ ] Pull Request is ready to merge & is not WIP.
[x] Added tests / only testable interactively.
- Make sure you add a new test if old tests do not effectively test the code changed.
[ ] Added documentation / opened issue to track adding documentation at a later date.

hjorslev commented 3 months ago

I'm going to merge the pull request as is. Coverage has been improved slightly. However, I believe there needs to be a cleanup in the various Steam Web API cmdlets so that they all return their output as PSObjects. Currently, they may return in different formats. For example, Resolve-VanityURL actually returns its output as a PSObject making the OutputFormat parameter useless.