-
Notifications
You must be signed in to change notification settings - Fork 4k
Tests for chef extension arm commands #2713
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
Conversation
Hi @NimishaS, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution! TTYL, AZPRBOT; |
@markcowl , @hovsepm, I am getting this error while running the tests in
|
I have specified the following environment variables:
Please let us know if we missed anything. |
@NimishaS This is most likely due to dependency mismatch ensure that the dependencies of your libraries are up to date and match the dependencies of the test framework. If you look at the dev branch, the test framework and other versions there are correct. |
@markcowl , tried the following approaches:
c) Also tried Any pointers that can help? |
@NimishaS Please pull from dev branch and update the following library Microsoft.Rest.ClientRuntime.Azure.Authentication (ver 2.2.8) and the test framework and then try to run your test. |
8af1676
to
f186742
Compare
[Trait(Category.AcceptanceType, Category.CheckIn)] | ||
public void TestChefExtensionBasic() | ||
{ | ||
ComputeTestController.NewInstance.RunPsTest("Test-SetChefExtensionBasic"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@markcowl , @shahabhijeet the library related issues got resolved. I am facing an issue at this line now.
The term 'Test-SetChefExtensionBasic' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the spelling of the name, or if a path was included, verify that the path is correct and try again.
I am missing some link. How is Test-SetChefExtensionBasic
made available here? Do I have to add some path to the PS file where this method is defined?
I am referring DiagnosticExtensionTests
, but I couldn't find any reference for Test-DiagnosticsExtensionBasic
method or DiagnosticExtensionTests.ps1
file added anywhere.
looks like you are missing |
Thanks for help @mwrock. My specs are failing at
And they fail on including |
@markcowl, we are currently facing an issue where the |
199bf63
to
50e4579
Compare
@markcowl , I have added the recorded tests response in the |
@NimishaS one of your tests is failing, could you take a look at it. Also could you confirm if you do not see this failure locally when you run the test? 5.937301s ✘ Microsoft.Azure.Commands.Compute.Test.ScenarioTests.ChefExtensionTests.TestSetChefExtensionBasic |
@shahabhijeet , I don't get this error while running the test locally in |
@NimishaS can you verify if you now have access to the CI server and can see the logs? |
@shahabhijeet I still can't access the CI server to see the logs |
@NimishaS Can you follow the below documentation And let me know if you can then run your tests locally and record any tests. |
@shahabhijeet , tests are executing successfully in
I have added
Can you please confirm if this has something to do with the contents of the file? |
Also not sure why Travis is looking for |
@shahabhijeet can you take a look? |
51ae717
to
eb90f48
Compare
@NimishaS This is because tests are running in parallel on the CI, so you cnanot count on the current directory. You can use AppDomain.Current to determine the test directory when the test suit is started, as here: https://github.com/Azure/azure-powershell/blob/dev/src/ResourceManager/KeyVault/Commands.KeyVault.Test/ScenarioTests/KeyVaultManagementTests.cs#L36 and pass this in to you powershell script for any ancillary files it requires |
ac460fe
to
5c52599
Compare
@markcowl thanks for help. CI is passing now. |
Test suite for #2587
w.r.t. #2638