use web artifacts for build step on ci

.ci/azure-pipelines-compat.yml

@@ -1,13 +1,13 @@
 parameters:
-  - name: Packages
+- name: Packages
-    type: object
+  type: object
-    default: {}
+  default: {}
-  - name: LinuxImage
+- name: LinuxImage
-    type: string
+  type: string
-    default: "ubuntu-latest"
+  default: "ubuntu-latest"
-  - name: DotNetSdkVersion
+- name: DotNetSdkVersion
-    type: string
+  type: string
-    default: 3.1.100
+  default: 3.1.100
 
 jobs:
   - job: CompatibilityCheck

.ci/azure-pipelines-main.yml

@@ -20,41 +20,33 @@ jobs:
         submodules: true
         persistCredentials: true
 
-      - task: CmdLine@2
+      - task: DownloadPipelineArtifact@2
-        displayName: "Clone Web Branch"
+        displayName: "Download Web Branch"
-        condition: and(succeeded(), or(contains(variables['Build.SourceBranch'], 'release'), contains(variables['Build.SourceBranch'], 'master')), eq(variables['BuildConfiguration'], 'Release'), in(variables['Build.Reason'], 'IndividualCI', 'BatchedCI', 'BuildCompletion'))
+        condition: in(variables['Build.Reason'], 'IndividualCI', 'BatchedCI', 'BuildCompletion')
         inputs:
-          script: "git clone --single-branch --branch $(Build.SourceBranchName) --depth=1 https://github.com/jellyfin/jellyfin-web.git $(Agent.TempDirectory)/jellyfin-web"
+          path: '$(Agent.TempDirectory)'
+          artifact: 'jellyfin-web-production'
+          source: 'specific'
+          project: 'Jellyfin Web'
+          pipeline: 'Build'
+          runBranch: variables['Build.SourceBranch']
 
-      - task: CmdLine@2
+      - task: DownloadPipelineArtifact@2
-        displayName: "Clone Web Target"
+        displayName: "Download Web Target"
-        condition: and(succeeded(), or(contains(variables['System.PullRequest.TargetBranch'], 'release'), contains(variables['System.PullRequest.TargetBranch'], 'master')), eq(variables['BuildConfiguration'], 'Release'), in(variables['Build.Reason'], 'PullRequest'))
+        condition: in(variables['Build.Reason'], 'PullRequest')
         inputs:
-          script: "git clone --single-branch --branch $(System.PullRequest.TargetBranch) --depth 1 https://github.com/jellyfin/jellyfin-web.git $(Agent.TempDirectory)/jellyfin-web"
+          path: '$(Agent.TempDirectory)'
+          artifact: 'jellyfin-web-production'
+          source: 'specific'
+          project: 'Jellyfin Web'
+          pipeline: 'Build'
+          runBranch: variables['System.PullRequest.TargetBranch']
 
-      - task: NodeTool@0
+      - task: ExtractFiles@1
-        displayName: "Install Node"
+        displayName: "Extract Web Client"
-        condition: and(succeeded(), or(contains(variables['System.PullRequest.TargetBranch'], 'release'), contains(variables['System.PullRequest.TargetBranch'], 'master'), contains(variables['Build.SourceBranch'], 'release'), contains(variables['Build.SourceBranch'], 'master')), eq(variables['BuildConfiguration'], 'Release'), in(variables['Build.Reason'], 'PullRequest', 'IndividualCI', 'BatchedCI', 'BuildCompletion'))
         inputs:
-          versionSpec: "12.x"
+          archiveFilePatterns: '$(Agent.TempDirectory)/*.zip'
-
+          destinationFolder: '$(Build.SourcesDirectory)/MediaBrowser.WebDashboard'
-      - task: CmdLine@2
-        displayName: "Build Web Client"
-        condition: and(succeeded(), or(contains(variables['System.PullRequest.TargetBranch'], 'release'), contains(variables['System.PullRequest.TargetBranch'], 'master'), contains(variables['Build.SourceBranch'], 'release'), contains(variables['Build.SourceBranch'], 'master')), eq(variables['BuildConfiguration'], 'Release'), in(variables['Build.Reason'], 'PullRequest', 'IndividualCI', 'BatchedCI', 'BuildCompletion'))
-        inputs:
-          script: yarn install
-          workingDirectory: $(Agent.TempDirectory)/jellyfin-web
-
-      - task: CopyFiles@2
-        displayName: "Copy Web Client"
-        condition: and(succeeded(), or(contains(variables['System.PullRequest.TargetBranch'], 'release'), contains(variables['System.PullRequest.TargetBranch'], 'master'), contains(variables['Build.SourceBranch'], 'release'), contains(variables['Build.SourceBranch'], 'master')), eq(variables['BuildConfiguration'], 'Release'), in(variables['Build.Reason'], 'PullRequest', 'IndividualCI', 'BatchedCI', 'BuildCompletion'))
-        inputs:
-          sourceFolder: $(Agent.TempDirectory)/jellyfin-web/dist
-          contents: "**"
-          targetFolder: $(Build.SourcesDirectory)/MediaBrowser.WebDashboard/jellyfin-web
-          cleanTargetFolder: true
-          overWrite: true
-          flattenFolders: false
 
       - task: UseDotNet@2
         displayName: "Update DotNet"

.ci/azure-pipelines-test.yml

@@ -1,26 +1,25 @@
 parameters:
-  - name: ImageNames
+- name: ImageNames
-    type: object
+  type: object
-    default:
+  default:
-      Linux: "ubuntu-latest"
+    Linux: "ubuntu-latest"
-      Windows: "windows-latest"
+    Windows: "windows-latest"
-      macOS: "macos-latest"
+    macOS: "macos-latest"
-  - name: TestProjects
+- name: TestProjects
-    type: string
+  type: string
-    default: "tests/**/*Tests.csproj"
+  default: "tests/**/*Tests.csproj"
-  - name: DotNetSdkVersion
+- name: DotNetSdkVersion
-    type: string
+  type: string
-    default: 3.1.100
+  default: 3.1.100
 
 jobs:
-  - job: MainTest
+  - job: Test
-    displayName: Main Test
+    displayName: Test
     strategy:
       matrix:
         ${{ each imageName in parameters.ImageNames }}:
           ${{ imageName.key }}:
             ImageName: ${{ imageName.value }}
-      maxParallel: 3
     pool:
       vmImage: "$(ImageName)"
     steps:
@@ -36,7 +35,7 @@ jobs:
           version: ${{ parameters.DotNetSdkVersion }}
 
       - task: DotNetCoreCLI@2
-        displayName: Run .NET Core CLI tests
+        displayName: 'Run CLI Tests'
         inputs:
           command: "test"
           projects: ${{ parameters.TestProjects }}
@@ -47,7 +46,7 @@ jobs:
 
       - task: Palmmedia.reportgenerator.reportgenerator-build-release-task.reportgenerator@4
         condition: and(succeeded(), eq(variables['Agent.OS'], 'Linux')) # !! THIS is for V1 only V2 will/should support merging
-        displayName: ReportGenerator (merge)
+        displayName: 'Run ReportGenerator'
         inputs:
           reports: "$(Agent.TempDirectory)/**/coverage.cobertura.xml"
           targetdir: "$(Agent.TempDirectory)/merged/"
@@ -56,7 +55,7 @@ jobs:
       ## V2 is already in the repository but it does not work "wrong number of segments" YAML error.
       - task: PublishCodeCoverageResults@1
         condition: and(succeeded(), eq(variables['Agent.OS'], 'Linux')) # !! THIS is for V1 only V2 will/should support merging
-        displayName: Publish Code Coverage
+        displayName: 'Publish Code Coverage'
         inputs:
           codeCoverageTool: "cobertura"
           #summaryFileLocation: '$(Agent.TempDirectory)/**/coverage.cobertura.xml' # !!THIS IS FOR V2

.ci/azure-pipelines.yml

@@ -1,12 +1,12 @@
 name: $(Date:yyyyMMdd)$(Rev:.r)
 
 variables:
-  - name: TestProjects
+- name: TestProjects
-    value: "tests/**/*Tests.csproj"
+  value: "tests/**/*Tests.csproj"
-  - name: RestoreBuildProjects
+- name: RestoreBuildProjects
-    value: "Jellyfin.Server/Jellyfin.Server.csproj"
+  value: "Jellyfin.Server/Jellyfin.Server.csproj"
-  - name: DotNetSdkVersion
+- name: DotNetSdkVersion
-    value: 3.1.100
+  value: 3.1.100
 
 pr:
   autoCancel: true