fiftyone.geolocation 4.2.1 → 4.3.3

package/ci/common-ci/scripts/modules/51DPipelineModule/51DPipelineModule.Tests.ps1

@@ -0,0 +1 @@
+# Placeholder for future tests

package/ci/common-ci/scripts/modules/51DPipelineModule/51DPipelineModule.psm1

@@ -0,0 +1,87 @@
+<#
+  ===================== Pipeline =====================
+  .Description
+  This module contains functions that perform actions
+  on Azure Devops pipelines.
+  For more information, please read description of
+  each function.
+#>
+
+Using module 51DAuthorizationModule
+Using module 51DEnvironmentModule
+
+<#
+  .Description
+  Get all pipelines from a Team project.
+  
+  .Parameter TeamProjectName
+  Name ofthe repository team project
+  
+  .Outputs
+  pipelines object as returned from AzureDevops web request or $null.
+#>
+function Get-Pipelines {
+	param(
+		[Parameter(Mandatory)]
+		[string]$TeamProjectName
+	)
+	
+	# Obtain the list of pipelines to access pipeline id later.
+	$url = "$([EnvironmentHandler]::GetEnvironmentVariable($env:SYSTEM_TEAMFOUNDATIONCOLLECTIONURI))$($TeamProjectName)/_apis/pipelines?api-version=6.0-preview.1"
+	
+	$response = Invoke-WebRequest `
+	-URI $url `
+	-Headers @{
+		Authorization = "$([Authorization]::AuthorizationString)"
+	} `
+	-Method GET
+	
+	if (!$(Test-RestResponse `
+		-Response $response `
+		-ErrorMessage "Failed to get the list of pipelines.")){
+		return $null
+	}
+	
+	# This variables is read-only
+	$pipelines = $response.content | Out-String | ConvertFrom-Json
+	return $pipelines
+}
+
+<#
+  .Description
+  Get the Id number of a pipeline.
+  
+  .Inputs Name
+  Name of the pipeline.
+  
+  .Parameter TeamProjectName
+  Name ofthe repository team project
+  
+  .Outputs
+  Id number of the pipeline or $null if nothing found.
+#>
+function Get-PipelineId {
+	param (
+		[Parameter(Mandatory)]
+		[string]$Name,
+		[Parameter(Mandatory)]
+		[string]$TeamProjectName
+	)
+	
+	$pipelines = Get-Pipelines -TeamProjectName $TeamProjectName
+	if ($pipelines -eq $null) {
+		Write-Host "# ERROR: Failed to get pipelines of current project '$TeamProjectName'"
+		return $null
+	}
+
+	for ($i = 0; $i -lt $pipelines.count; $i++) {
+		if ($pipelines.value[$i].name -eq $Name) {
+			Write-Host "# ID $($pipelines.value[$i].id) found for pipeline $Name."
+			return $pipelines.value[$i].id
+		}
+	}
+	Write-Host "# No ID found for pipeline $Name."
+	return $null
+}
+
+Export-ModuleMember -Function Get-PipelineId

package/ci/common-ci/scripts/modules/51DPullRequestModule/51DPullRequestModule.Tests.ps1

@@ -0,0 +1,194 @@
+Describe 'Complete-CorrespondingPullRequest' {
+	It 'fail if is not a release pull request' {
+		# Set required environment variable
+		# Save current source branch env variable
+		$buildSrcBranch = $Env:BUILD_SOURCEBRANCH
+		$Env:BUILD_SOURCEBRANCH = "refs/pull/123/merge"
+		Mock Test-IsReleasePullRequest { return $false } `
+			-ModuleName 51DPullRequestModule
+		Mock Get-Content { return "{}" } `
+			-ModuleName 51DPullRequestModule
+		
+		$result = Complete-CorrespondingPullRequest -ConfigFile "afile"
+		# Unset required environment variable
+		$Env:BUILD_SOURCEBRANCH = $buildSrcBranch
+		
+		$result | Should -BeExactly $false
+	}
+	
+	It 'fail to update submodule references' {
+		# Set required environment variable
+		$buildSrcBranch = $Env:BUILD_SOURCEBRANCH
+		$Env:BUILD_SOURCEBRANCH = "refs/pull/123/merge"
+		Mock Test-IsReleasePullRequest { return $true } `
+			-ModuleName 51DPullRequestModule
+		Mock Update-SubmoduleReferences { return $false } `
+			-ModuleName 51DPullRequestModule
+		Mock Get-Content { return "{}" } `
+			-ModuleName 51DPullRequestModule
+		
+		$result = Complete-CorrespondingPullRequest -ConfigFile "afile"
+		# Unset required environment variable
+		$Env:BUILD_SOURCEBRANCH = $buildSrcBranch
+		
+		$result | Should -BeExactly $false
+	}
+}
+
+InModuleScope '51DPullRequestModule' {
+	Describe 'Complete-PullRequest' {
+		It 'Successfully complete without approval' {
+			# Mock get last merge commit
+			Mock Invoke-WebRequest {
+				return @{
+					StatusCode = 200
+				} 
+			} `
+			-ParameterFilter {
+				$Method -eq "GET"
+			} `
+			-ModuleName 51DPullRequestModule
+			# Mock complete pull request
+			Mock Invoke-WebRequest {
+				return @{
+					StatusCode = 200
+				} 
+			} `
+			-ParameterFilter {
+				$Method -eq "PATCH"
+			} `
+			-ModuleName 51DPullRequestModule
+			$result = Complete-PullRequest `
+				-RepositoryName "repo1" `
+				-PullRequestId '1234' `
+				-ApprovalRequired $false `
+				-TeamProjectName "testproj"
+			$result | Should -BeExactly $true
+		}
+		
+		It 'Successfully complete with approval' {
+			Mock Test-PullRequestVotes { return $true  } `
+				-ModuleName 51DPullRequestModule
+			Mock Test-PullRequestComments { return $true  } `
+				-ModuleName 51DPullRequestModule
+			# Mock get last merge commit
+			Mock Invoke-WebRequest {
+				return @{
+					StatusCode = 200
+				} 
+			} `
+			-ParameterFilter {
+				$Method -eq "GET"
+			} `
+			-ModuleName 51DPullRequestModule
+			# Mock complete pull request
+			Mock Invoke-WebRequest {
+				return @{
+					StatusCode = 200
+				} 
+			} `
+			-ParameterFilter {
+				$Method -eq "PATCH"
+			} `
+			-ModuleName 51DPullRequestModule
+			$result = Complete-PullRequest `
+				-RepositoryName "repo1" `
+				-PullRequestId '1234' `
+				-ApprovalRequired $true `
+				-TeamProjectName "testproj"
+			$result | Should -BeExactly $true
+		}
+		
+		It 'requires approval but missing votes' {
+			Mock Test-PullRequestVotes { return $false  } `
+				-ModuleName 51DPullRequestModule
+			Mock Test-PullRequestComments { return $true  } `
+				-ModuleName 51DPullRequestModule
+			# Mock get last merge commit
+			Mock Invoke-WebRequest {
+				return @{
+					StatusCode = 200
+				} 
+			} `
+			-ParameterFilter {
+				$Method -eq "GET"
+			} `
+			-ModuleName 51DPullRequestModule
+			# Mock complete pull request
+			Mock Invoke-WebRequest {
+				return @{
+					StatusCode = 200
+				} 
+			} `
+			-ParameterFilter {
+				$Method -eq "PATCH"
+			} `
+			-ModuleName 51DPullRequestModule
+			$result = Complete-PullRequest `
+				-RepositoryName "repo1" `
+				-PullRequestId '1234' `
+				-ApprovalRequired $true `
+				-TeamProjectName "testproj"
+			$result | Should -BeExactly $false
+		}
+		
+		It 'requires approval but comment unresolved' {
+			Mock Test-PullRequestVotes { return $false  } `
+				-ModuleName 51DPullRequestModule
+			Mock Test-PullRequestComments { return $true  } `
+				-ModuleName 51DPullRequestModule
+			# Mock get last merge commit
+			Mock Invoke-WebRequest {
+				return @{
+					StatusCode = 200
+				} 
+			} `
+			-ParameterFilter {
+				$Method -eq "GET"
+			} `
+			-ModuleName 51DPullRequestModule
+			# Mock complete pull request
+			Mock Invoke-WebRequest {
+				return @{
+					StatusCode = 200
+				} 
+			} `
+			-ParameterFilter {
+				$Method -eq "PATCH"
+			} `
+			-ModuleName 51DPullRequestModule
+			$result = Complete-PullRequest `
+				-RepositoryName "repo1" `
+				-PullRequestId '1234' `
+				-ApprovalRequired $true `
+				-TeamProjectName "testproj"
+			$result | Should -BeExactly $false
+		}
+	}
+	
+	Describe 'Start-ProcessPullRequest' {
+		It 'successfull complete pull request to main' {
+			Mock Get-PullRequestToMain { return New-Object -TypeName Object } `
+				-ModuleName 51DPullRequestModule
+			Mock Restart-TestBuild { return $true } `
+				-ModuleName 51DPullRequestModule
+			$result = Start-ProcessPullRequest `
+				-RepositoryName "repo1" `
+				-ReleaseBranch "release1" `
+				-TeamProjectName "testproj"
+			$result | Should -BeExactly $true
+		}
+		
+		It 'pull request to main already exist' {
+			Mock Get-PullRequestToMain { return $null } `
+				-ModuleName 51DPullRequestModule
+			Mock New-PullRequestToMain { return $true } `
+				-ModuleName 51DPullRequestModule
+			$result = Start-ProcessPullRequest `
+				-RepositoryName "repo1" `
+				-ReleaseBranch "release1" `
+				-TeamProjectName "testproj"
+			$result | Should -BeExactly $true
+		}
+	}
+}