SqlDatabaseObjectPermission: Fixing issue with INSERT permissions

[markaugust]

Pull Request (PR) description

This Pull Request (PR) fixes the following issues

Adding a foreach loop when checking object permissions to avoid issues with INSERT permissions

• Fixes #2006

Task list

• [x] Added an entry to the change log under the Unreleased section of the file CHANGELOG.md. Entry should say what was changed and how that affects users (if applicable), and reference the issue being resolved (if applicable).
• [ ] Resource documentation updated in the resource's README.md.
• [ ] Resource parameter descriptions updated in schema.mof.
• [ ] Comment-based help updated, including parameter descriptions.
• [ ] Localization strings updated.
• [ ] Examples updated.
• [x] Unit tests updated. See DSC Community Testing Guidelines.
• [ ] Integration tests updated (where possible). See DSC Community Testing Guidelines.
• [ ] Code changes adheres to DSC Community Style Guidelines.

---

This change is 

[codecov[bot]]

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 94%. Comparing base (a5c29fe) to head (d926778).

Additional details and impacted files

[](https://app.codecov.io/gh/dsccommunity/SqlServerDsc/pull/2007?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dsccommunity) ```diff @@ Coverage Diff @@ ## main #2007 +/- ## ==================================== Coverage 94% 94% ==================================== Files 94 94 Lines 7919 7920 +1 ==================================== + Hits 7489 7490 +1 Misses 430 430 ``` | [Flag](https://app.codecov.io/gh/dsccommunity/SqlServerDsc/pull/2007/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dsccommunity) | Coverage Δ | | |---|---|---| | [unit](https://app.codecov.io/gh/dsccommunity/SqlServerDsc/pull/2007/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dsccommunity) | `94% <100%> (+<1%)` | :arrow_up: | | [Files](https://app.codecov.io/gh/dsccommunity/SqlServerDsc/pull/2007?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dsccommunity) | Coverage Δ | | |---|---|---| | [...ectPermission/DSC\_SqlDatabaseObjectPermission.psm1](https://app.codecov.io/gh/dsccommunity/SqlServerDsc/pull/2007?src=pr&el=tree&filepath=source%2FDSCResources%2FDSC_SqlDatabaseObjectPermission%2FDSC_SqlDatabaseObjectPermission.psm1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dsccommunity#diff-c291cmNlL0RTQ1Jlc291cmNlcy9EU0NfU3FsRGF0YWJhc2VPYmplY3RQZXJtaXNzaW9uL0RTQ19TcWxEYXRhYmFzZU9iamVjdFBlcm1pc3Npb24ucHNtMQ==) | `100% <100%> (ø)` | |

[markaugust]

@johlju I'm not familiar enough with Mocks and Unit test to know exactly what needs to be done here. Looking at the code block, I made some changes to the unit test, but let me know if that's not what you were meaning.

[johlju]

I will review this PR as soon as I have time. Will look over the changes to the test you made. Good work!

[johlju]

Unit tests looks okay, it correctly fails without the proposed change. I push a change to the integration test to try to mimic the scenario in the issue.

[johlju]

@markaugust great work on this!

[markaugust]

Thank you @johlju . Out of curiosity, is there a timeline on when those fixes will get moved into a full release branch, rather than just in a preview branch?

[johlju]

Usually when we (community) feel there should be a new full release. I pushed a release tag now, so should release if pipeline don't fail 😊