-
Notifications
You must be signed in to change notification settings - Fork 225
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
SqlLogin: Integration tests fails intermittently #1634
Comments
I think the problem is the tests run too fast and the build worker can't keep up sometimes. Something doesn't return to the correct state fast enough. It become better after adding SqlServerDsc/tests/TestHelpers/CommonTestHelper.psm1 Lines 395 to 406 in 56ebcfe
Maybe we should call |
…' type (#1652) - SqlLogin - Added functionality to throw exception if an update to the `LoginMustChangePassword` value on an existing SQL Login is attempted. This functionality is not supported by referenced, SQL Server Management Object (SMO), libraries and cannot be supported directly by this module. - Added integration tests to ensure that an added (or updated) `SqlLogin` can connect into a SQL instance once added (or updated). - Added integration tests to ensure that the default database connected to by a `SqlLogin` is the same as specified in the resource's `DefaultDatabase` property/parameter. - Reversed order in which `PasswordExpirationEnabled` and `PasswordPolicyEnforced` are updated within `SqlLogin` resource. `PasswordPolicyEnforced` is now updated first. - Added integration tests to assert `LoginPasswordExpirationEnabled`, `LoginPasswordPolicyEnforced` and `LoginMustChangePassword` properties/parameters are applied and updated correctly. Similar integration tests also added to ensure the password of the `SqlLogin` is updated if the password within the `SqlCredential` value/object is changed (issue #361, issue #1032, and issue #1050). - Updated `SqlLogin`, integration tests to make use of amended `Wait-ForIdleLcm`, helper function, `-Clear` switch usage to remove intermittent, integration test failures (issue #1634).
@johlju @SphenicPaul
|
Reopening this issue. We had hoped we squashed that one. Let's keep track if there is always line 93 in the |
When converting the unit and integration tests to Pester 5 and pushed new commits countless times (well a lot any way) I have not seen this issue, so I will auto-close this issue when the Pester 5 conversion PR is merged. |
The integration tests for SqlLogin fails intermittently.
Originally posted by @Fiander in #1631 (comment)
The text was updated successfully, but these errors were encountered: