dsccommunity / dsccommunity/SqlServerDsc
SqlServerDsc.Common: Add integration test for helper functions
Nobody has claimed this yet.
- Dominant language
- PowerShell
- Stars
- 385
- Forks
- 224
- PR merge metrics
- No merged PRs in 30d
Description
I suggest we look at adding integration tests for helper functions that are user throughout the module, like Connect-SQL. I think it would bring more quality to changes to actually test the helper functions as well. Not all, but those that seem necessary.
A change in a helper function for one resource could potentially break another resource, I could be hard to debug. It would also be easier to see all aspects of a change if writing integration tests. 🤔
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by locating shared helper functions in the SqlServerDsc module, including Connect-SQL, and identifying which resources use them. Review the existing integration-test conventions before selecting the helpers that need coverage. Done means integration tests cover the necessary shared helpers and help detect regressions across resources.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- powershell
- Domain
- testing
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100