Add VSTS CI for Windows#7536
Conversation
There was a problem hiding this comment.
at all or on system drive?
There was a problem hiding this comment.
They don't have a page file on the system drive
There was a problem hiding this comment.
We could find the page file location:
#Open the registry on multiple remote computers
$reg = [Microsoft.Win32.RegistryKey]::OpenRemoteBaseKey('LocalMachine',$server )
$RegKeyPath= "SYSTEM\CurrentControlSet\Control\Session Manager\Memory Management"
$pageFileKey=$reg.OpenSubKey($RegKeyPath)
$pageFileLocation=$pageFileKey.GetValue("ExistingPageFiles")There was a problem hiding this comment.
I wonder how we'll do remoting tests if VSTS, address is 0.0.0.0?
What about web cmdlet tests? Seems we need add [Feature] to a commit title.
There was a problem hiding this comment.
remoting tests are working
There was a problem hiding this comment.
At this point, only CI has been ported, not Feature tests. We should have a later pull request to enable daily builds and [Feature] etc
.gitattributes
Outdated
There was a problem hiding this comment.
just forcing the line endings to be LF which is what we expect
.vsts-ci/windows.yml
Outdated
There was a problem hiding this comment.
Re-evaluate if this is needed
build.psm1
Outdated
There was a problem hiding this comment.
We should just use USERPROFILE instead.
build.psm1
Outdated
There was a problem hiding this comment.
This can be moved to L#242
…ifferent sessions. (PowerShell#6884)
make sure TestFileCatalog always uses lf endings
0222fc4 to
45c8fb2
Compare
PR Summary
Access-denied test for Get-Item C:\windows\appcompat\Programs\Install -ErrorAction Stop, because the path does not always existAccess-denied test for Get-Item C:\windows\appcompat\Programs\Install -ErrorAction Stoptest in VSTS #7553Should give .sys file if the fullpath is specified with hidden and force parameter, because pagefile.sys doesn't always exist and other files don't meet test's requirement.Should give .sys file if the fullpath is specified with hidden and force parameterin VSTS #7554Test-Connectiontests for same reasons they failed on VSTS LinuxTest-FileCatalog should pass when catalog is in the same folder as files being tested, because the CmdLet does not work in that scenarioTest-FileCatalog should pass when catalog is in the same folder as files being testedpasses in other CI system and fails in VSTS #7556appveyor.psm1to work with VSTSHelpersRemoting.psm1New-RemoteSessionto work for CimSession (discovered an issue during the investigation)Test wildcard with drive relative directory pathto work when there are multiple drivesNew-CimSessionTests to requireAdmin.gitatttributesso files clone like they do on appveyorPR Checklist
.h,.cpp,.cs,.ps1and.psm1files have the correct copyright headerWIP:to the beginning of the title and remove the prefix when the PR is ready.[feature]if the change is significant or affects feature tests