-
Notifications
You must be signed in to change notification settings - Fork 47
Allow to run tests with insiders build #88
Comments
@bpasero I noticed that setting |
Ah yeah good point. |
@joaomoreno would it be possible to get an endpoint in the form of: |
* Use Trusty image for build * Use XCode 9.1 * Bump vscode version to 1.1.8 * Update codecov ignored paths * Disable gcov on AppVeyor (causes long path issues) * Allow Linux builds to fail * Setting CODE_DOWNLOAD_URL won't help With all the tweaks the macOS builds seem to run more reliably then the Linux builds. However, the performance varies because of limites macOS capacity. This is covered in our GitHub setup already by not requiring the Travis build to pass for PRs. See also microsoft/vscode-extension-vscode#88
@joaomoreno that one downloads stable for me. How does the website get the latest insider build from https://code.visualstudio.com/insiders/ ? |
Oh https://vscode-update.azurewebsites.net/latest/win32/insider. There was an S too many. |
@joaomoreno thanks @guw interested to provide this as PR? I think we should just use the insider build when the extension declares " "engines": {
"vscode": "*"
} |
@guw yeah good point, it would probably only be useful on a branch where you want to run builds and tests explicitly against insiders. Alternatively we could introduce another environment variable to specify the build to be insiders. |
This change allows setting CODE_VERSION to 'insiders' which will trigger download of an Insiders build and using that for test execution. Fixes microsoft#88. Signed-off-by: Gunnar Wagenknecht <gunnar@wagenknecht.org>
Published as |
It's currently not possible to run VS Code Extension tests with an insiders build.
Looking at the code it seems that setting CODE_VERSION should do the trick. However, this doesn't work because of a hard coded "stable" in the generated download url. Also, setting
CODE_DOWNLOAD_URL
directly (eg., tohttps://vscode-update.azurewebsites.net/1.19.0-insider/linux-x64/insider
) won't help. The resulting executable name would becode-insiders
, which isn't currently supported.The text was updated successfully, but these errors were encountered: