Skip to content

Commit 5de5db4

Browse files
Remove unused getUpdatedPaths file and CI interaction
1 parent 7c7e1d7 commit 5de5db4

File tree

3 files changed

+22
-87
lines changed

3 files changed

+22
-87
lines changed

ci/azure-pipelines-extensions.yml

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -34,14 +34,14 @@ extends:
3434
param(
3535
[string] $fileName
3636
)
37-
$exceptions = @("*.gif", "*.svg", "*.png", "*.dll",
37+
$exceptions = @("*.gif", "*.svg", "*.png", "*.dll",
3838
"*\extensions\artifactengine\providers\typed-rest-client\httpclient.ts", "*\extensions\artifactengine\providers\typed-rest-client\util.ts",
3939
"*\extensions\bitbucket\src\tasks\downloadartifactsbitbucket\downloadbitbucket.js",
4040
"*\extensions\externaltfs\src\tasks\downloadartifactstfsgit\downloadtfgit.js")
4141
@($exceptions | ? { $fileName -ilike $_ }).Count -gt 0
4242
}
4343
Write-Host "Ensure there are no dev.azure.com and url.parse strings in the code."
44-
@("$(Build.Repository.LocalPath)\Extensions", "$(Build.Repository.LocalPath)\ServerTaskHelper", "$(Build.Repository.LocalPath)\TaskModules") | % {
44+
@("$(Build.Repository.LocalPath)\Extensions", "$(Build.Repository.LocalPath)\ServerTaskHelper", "$(Build.Repository.LocalPath)\TaskModules") | % {
4545
dir $_ -rec | % {
4646
if ($_.Attributes -eq "Directory") {
4747
return
@@ -72,8 +72,6 @@ extends:
7272
displayName: 'Use Node 10.24.1'
7373
inputs:
7474
versionSpec: 10.24.1
75-
- powershell: 'node getUpdatedPaths.js'
76-
displayName: 'PowerShell Script'
7775
- task: Npm@1
7876
displayName: 'npm install'
7977
inputs:
@@ -86,5 +84,4 @@ extends:
8684
displayName: 'gulp test'
8785
inputs:
8886
targets: test
89-
arguments: '--suite=ArtifactEngine'
90-
condition: or(contains(variables['UPDATEDAREAPATHS'], 'ArtifactEngine'),contains(variables['UPDATEDAREAPATHS'], 'package.json'))
87+
arguments: '--suite=ArtifactEngine'

getUpdatedPaths.js

Lines changed: 0 additions & 68 deletions
This file was deleted.

gulpfile.js

Lines changed: 19 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -705,33 +705,39 @@ gulp.task('testLib_NodeModules', gulp.series('testLib', function () {
705705
gulp.task('testResources', gulp.parallel('testLib_NodeModules', 'ps1tests', 'tstests', 'copyTestData'));
706706

707707
gulp.task("test", gulp.series("testResources", function(){
708-
process.env['TASK_TEST_TEMP'] =path.join(__dirname, _testTemp);
708+
process.env['TASK_TEST_TEMP'] = path.join(__dirname, _testTemp);
709+
709710
shell.rm('-rf', _testTemp);
710711
shell.mkdir('-p', _testTemp);
711712

712-
if (options.suite.indexOf("ArtifactEngine") >= 0 && options.e2e) {
713-
var suitePath = path.join(_testRoot, "Extensions/" + options.suite + "/**/*e2e.js");
713+
if (options.suite.indexOf("ArtifactEngine") >= 0 && options.e2e) {
714+
const suitePath = path.join(_testRoot, "Extensions/" + options.suite + "/**/*e2e.js");
714715
console.log(suitePath);
715-
var tfBuild = ('' + process.env['TF_BUILD']).toLowerCase() == 'true'
716-
return gulp.src([suitePath])
716+
const tfBuild = process.env['TF_BUILD'].toLowerCase() == 'true';
717+
return gulp
718+
.src([suitePath])
717719
.pipe(mocha({ reporter: 'spec', ui: 'bdd', useColors: !tfBuild }));
718720
}
719721

720722
if (options.suite.indexOf("ArtifactEngine") >= 0 && options.perf) {
721-
var suitePath = path.join(_testRoot, "Extensions/" + options.suite + "/**/*perf.js");
723+
const suitePath = path.join(_testRoot, "Extensions/" + options.suite + "/**/*perf.js");
722724
console.log(suitePath);
723-
var tfBuild = ('' + process.env['TF_BUILD']).toLowerCase() == 'true'
724-
return gulp.src([suitePath])
725+
const tfBuild = process.env['TF_BUILD'].toLowerCase() == 'true';
726+
return gulp
727+
.src([suitePath])
725728
.pipe(mocha({ reporter: 'spec', ui: 'bdd', useColors: !tfBuild }));
726729
}
727730

728-
var suitePath = path.join(_testRoot,"Extensions/" + options.suite + "/Tests/Tasks", options.suite + '/_suite.js');
731+
const suitePath = path.join(_testRoot,"Extensions/" + options.suite + "/Tests/Tasks", options.suite + '/_suite.js');
729732
console.log(suitePath);
730-
var suitePath2 = path.join(_testRoot, "Extensions/" + options.suite + "/**/*Tests.js");
733+
const suitePath2 = path.join(_testRoot, "Extensions/" + options.suite + "/**/*Tests.js");
731734
console.log(suitePath2);
732-
var tfBuild = ('' + process.env['TF_BUILD']).toLowerCase() == 'true'
733-
var ignorePath = "!" + path.join(_testRoot, "Extensions", "/**/UIContribution{,/**}");
734-
return gulp.src([ suitePath, suitePath2, ignorePath ], { allowEmpty: true })
735+
736+
const tfBuild = ('' + process.env['TF_BUILD']).toLowerCase() == 'true'
737+
const ignorePath = "!" + path.join(_testRoot, "Extensions", "/**/UIContribution{,/**}");
738+
739+
return gulp
740+
.src([ suitePath, suitePath2, ignorePath ], { allowEmpty: true })
735741
.pipe(mocha({ reporter: 'spec', ui: 'bdd', useColors: !tfBuild }));
736742
}));
737743

0 commit comments

Comments
 (0)