Updating Tests.ps1 to only run limited tests. · microsoft/ReportingServicesTools@3e26491 · GitHub

Commit

Updating Tests.ps1 to only run limited tests.

Browse files

Loading branch information

Parth Shah committed Sep 1, 2017

1 parent 4080ee7 commit 3e26491

Test.ps1

-Original file line number
+Diff line change
@@ -1 +1,2 @@
-    Invoke-Pester -Path .\Tests
+    Invoke-Pester -Path .\Tests\CatalogItems
+    Invoke-Pester -Path .\Tests\Utilities

3 comments on commit `3e26491`

Contributor

jzabroski commented on `3e26491` Aug 10, 2018

@jtarquino @parthsha Why did you make this change?

Member

jtarquino commented on `3e26491` Aug 12, 2018

Mostly because the admin tests changes the state of the machine and it make it very unstable in automation, having said that I think we should enable the security tests suite

Contributor

jzabroski commented on `3e26491` Aug 12, 2018 via email

I basically agree. It is hard to understand the Admin functions and know the right order to call all of them. Any chance of adding to the README.MD a tutorial on how you use it at Microsoft? I think that's a barrier for people like me wanting to write tests... it could be aggravating if we don't agree on how to configure reporting services.

On Sun, Aug 12, 2018, 1:16 PM Jaime Tarquino ***@***.***> wrote: Mostly because the admin tests changes the state of the machine and it make it very unstable in automation, having said that I think we should enable the security tests suite — You are receiving this because you commented. Reply to this email directly, view it on GitHub <3e26491#commitcomment-30047569>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAbT_V4lj_Ti7QxSq0sQNWtT4p6_NupVks5uQGLrgaJpZM4V4yg8> .

Please sign in to comment.