Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Add printing messages for MatrixCanceledError and InfrastructureError #1043

Merged
merged 2 commits into from
Aug 24, 2020
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion test_runner/src/main/kotlin/ftl/Main.kt
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ import ftl.cli.firebase.test.ProvidedSoftwareCommand
import ftl.log.setDebugLogging
import ftl.util.readRevision
import ftl.util.readVersion
import ftl.util.withGlobalExceptionHandling
import ftl.run.exception.withGlobalExceptionHandling
import picocli.CommandLine

@CommandLine.Command(
Expand Down
2 changes: 1 addition & 1 deletion test_runner/src/main/kotlin/ftl/args/ArgsFileVisitor.kt
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
package ftl.args

import ftl.util.FlankGeneralError
import ftl.run.exception.FlankGeneralError
import java.io.IOException
import java.nio.file.FileSystems
import java.nio.file.FileVisitOption
Expand Down
4 changes: 2 additions & 2 deletions test_runner/src/main/kotlin/ftl/args/ArgsHelper.kt
Original file line number Diff line number Diff line change
Expand Up @@ -24,8 +24,8 @@ import ftl.shard.StringShards
import ftl.shard.createShardsByShardCount
import ftl.shard.shardCountByTime
import ftl.shard.stringShards
import ftl.util.FlankConfigurationError
import ftl.util.FlankGeneralError
import ftl.run.exception.FlankConfigurationError
import ftl.run.exception.FlankGeneralError
import ftl.util.FlankTestMethod
import ftl.util.assertNotEmpty
import java.io.File
Expand Down
2 changes: 1 addition & 1 deletion test_runner/src/main/kotlin/ftl/args/IosArgs.kt
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ package ftl.args

import com.google.common.annotations.VisibleForTesting
import ftl.ios.Xctestrun.findTestNames
import ftl.util.FlankConfigurationError
import ftl.run.exception.FlankConfigurationError
import ftl.util.FlankTestMethod

data class IosArgs(
Expand Down
4 changes: 2 additions & 2 deletions test_runner/src/main/kotlin/ftl/args/ValidateAndroidArgs.kt
Original file line number Diff line number Diff line change
Expand Up @@ -7,8 +7,8 @@ import ftl.android.UnsupportedModelId
import ftl.android.UnsupportedVersionId
import ftl.config.containsPhysicalDevices
import ftl.config.containsVirtualDevices
import ftl.util.FlankConfigurationError
import ftl.util.IncompatibleTestDimensionError
import ftl.run.exception.FlankConfigurationError
import ftl.run.exception.IncompatibleTestDimensionError
import java.io.File

fun AndroidArgs.validate() {
Expand Down
4 changes: 2 additions & 2 deletions test_runner/src/main/kotlin/ftl/args/ValidateCommonArgs.kt
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,8 @@ package ftl.args

import ftl.config.Device
import ftl.config.FtlConstants
import ftl.util.FlankConfigurationError
import ftl.util.FlankGeneralError
import ftl.run.exception.FlankConfigurationError
import ftl.run.exception.FlankGeneralError

fun CommonArgs.validate() {
assertProjectId()
Expand Down
4 changes: 2 additions & 2 deletions test_runner/src/main/kotlin/ftl/args/ValidateIosArgs.kt
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
package ftl.args

import ftl.ios.IosCatalog
import ftl.util.FlankConfigurationError
import ftl.util.IncompatibleTestDimensionError
import ftl.run.exception.FlankConfigurationError
import ftl.run.exception.IncompatibleTestDimensionError

fun IosArgs.validate() {
assertXcodeSupported()
Expand Down
4 changes: 2 additions & 2 deletions test_runner/src/main/kotlin/ftl/args/yml/YamlDeprecated.kt
Original file line number Diff line number Diff line change
Expand Up @@ -9,8 +9,8 @@ import com.fasterxml.jackson.databind.node.ObjectNode
import com.google.common.annotations.VisibleForTesting
import ftl.args.ArgsHelper.yamlMapper
import ftl.util.loadFile
import ftl.util.FlankConfigurationError
import ftl.util.FlankGeneralError
import ftl.run.exception.FlankConfigurationError
import ftl.run.exception.FlankGeneralError
import java.io.Reader
import java.nio.file.Files
import java.nio.file.Path
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ package ftl.args.yml
import com.fasterxml.jackson.databind.JsonNode
import com.fasterxml.jackson.dataformat.yaml.snakeyaml.error.MarkedYAMLException
import ftl.args.yml.errors.ConfigurationErrorMessageBuilder
import ftl.util.FlankConfigurationError
import ftl.run.exception.FlankConfigurationError

fun convertConfigurationErrorExceptions(missingParameterError: Exception, yaml: JsonNode?): Throwable {
val errorMessageBuilder = ConfigurationErrorMessageBuilder
Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
package ftl.cli.firebase.test

import ftl.args.yml.YamlDeprecated
import ftl.util.YmlValidationError
import ftl.run.exception.YmlValidationError
import java.nio.file.Path

fun processValidation(validationResult: String, shouldFix: Boolean, ymlPath: Path) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ package ftl.cli.firebase.test.android.configuration
import ftl.android.AndroidCatalog.getLocaleDescription
import ftl.args.AndroidArgs
import ftl.config.FtlConstants
import ftl.util.FlankConfigurationError
import ftl.run.exception.FlankConfigurationError
import picocli.CommandLine
import java.nio.file.Paths

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ package ftl.cli.firebase.test.android.models
import ftl.android.AndroidCatalog
import ftl.args.AndroidArgs
import ftl.config.FtlConstants
import ftl.util.FlankConfigurationError
import ftl.run.exception.FlankConfigurationError
import picocli.CommandLine
import java.nio.file.Paths

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ package ftl.cli.firebase.test.android.versions
import ftl.android.AndroidCatalog
import ftl.args.AndroidArgs
import ftl.config.FtlConstants
import ftl.util.FlankConfigurationError
import ftl.run.exception.FlankConfigurationError
import picocli.CommandLine
import java.nio.file.Paths

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ package ftl.cli.firebase.test.ios.configuration
import ftl.args.IosArgs
import ftl.config.FtlConstants
import ftl.ios.IosCatalog.getLocaleDescription
import ftl.util.FlankConfigurationError
import ftl.run.exception.FlankConfigurationError
import picocli.CommandLine
import java.nio.file.Paths

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ package ftl.cli.firebase.test.ios.models
import ftl.args.IosArgs
import ftl.config.FtlConstants
import ftl.ios.IosCatalog
import ftl.util.FlankConfigurationError
import ftl.run.exception.FlankConfigurationError
import picocli.CommandLine
import java.nio.file.Paths

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ package ftl.cli.firebase.test.ios.versions
import ftl.args.IosArgs
import ftl.config.FtlConstants
import ftl.ios.IosCatalog
import ftl.util.FlankConfigurationError
import ftl.run.exception.FlankConfigurationError
import picocli.CommandLine
import java.nio.file.Paths

Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
package ftl.cli.firebase.test.networkprofiles

import ftl.environment.networkProfileDescription
import ftl.util.FlankConfigurationError
import ftl.run.exception.FlankConfigurationError
import picocli.CommandLine

@CommandLine.Command(
Expand Down
4 changes: 2 additions & 2 deletions test_runner/src/main/kotlin/ftl/config/FtlConstants.kt
Original file line number Diff line number Diff line change
Expand Up @@ -16,8 +16,8 @@ import ftl.args.IosArgs
import ftl.gc.UserAuth
import ftl.http.HttpTimeoutIncrease
import ftl.util.BugsnagInitHelper.initBugsnag
import ftl.util.FlankConfigurationError
import ftl.util.FlankGeneralError
import ftl.run.exception.FlankConfigurationError
import ftl.run.exception.FlankGeneralError
import ftl.util.readRevision
import java.io.IOException
import java.nio.file.Path
Expand Down
2 changes: 1 addition & 1 deletion test_runner/src/main/kotlin/ftl/doctor/Doctor.kt
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ import ftl.args.ArgsHelper
import ftl.args.IArgs
import ftl.config.loadAndroidConfig
import ftl.config.loadIosConfig
import ftl.util.FlankConfigurationError
import ftl.run.exception.FlankConfigurationError
import ftl.util.loadFile
import java.io.Reader
import java.lang.StringBuilder
Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
package ftl.environment

import com.google.api.services.testing.model.Locale
import ftl.util.FlankGeneralError
import ftl.run.exception.FlankGeneralError

fun List<Locale>.getLocaleDescription(localeId: String) = findLocales(localeId)?.prepareDescription().orErrorMessage(localeId).plus("\n")

Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
package ftl.environment.android

import com.google.api.services.testing.model.AndroidModel
import ftl.util.FlankGeneralError
import ftl.run.exception.FlankGeneralError

fun List<AndroidModel>.getDescription(modelId: String) = findModel(modelId)?.prepareDescription().orErrorMessage(modelId)

Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
package ftl.environment.android

import com.google.api.services.testing.model.AndroidVersion
import ftl.util.FlankGeneralError
import ftl.run.exception.FlankGeneralError

fun List<AndroidVersion>.getDescription(versionId: String) = findVersion(versionId)?.prepareDescription().orErrorMessage(versionId)

Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
package ftl.environment.ios

import com.google.api.services.testing.model.IosModel
import ftl.util.FlankGeneralError
import ftl.run.exception.FlankGeneralError

fun List<IosModel>.getDescription(modelId: String) = findModel(modelId)?.prepareDescription().orErrorMessage(modelId)

Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
package ftl.environment.ios

import com.google.api.services.testing.model.IosVersion
import ftl.util.FlankGeneralError
import ftl.run.exception.FlankGeneralError

fun List<IosVersion>.getDescription(versionId: String) = findVersion(versionId)?.prepareDescription().orErrorMessage(versionId)

Expand Down
4 changes: 2 additions & 2 deletions test_runner/src/main/kotlin/ftl/filter/TestFilters.kt
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,8 @@ package ftl.filter

import com.linkedin.dex.parser.TestMethod
import ftl.config.FtlConstants
import ftl.util.FlankConfigurationError
import ftl.util.FlankGeneralError
import ftl.run.exception.FlankConfigurationError
import ftl.run.exception.FlankGeneralError
import java.io.IOException
import java.nio.file.Files
import java.nio.file.Paths
Expand Down
2 changes: 1 addition & 1 deletion test_runner/src/main/kotlin/ftl/gc/GcAndroidTestMatrix.kt
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ import ftl.gc.android.mapToDeviceFiles
import ftl.gc.android.setEnvironmentVariables
import ftl.gc.android.setupAndroidTest
import ftl.run.platform.android.AndroidTestConfig
import ftl.util.FlankGeneralError
import ftl.run.exception.FlankGeneralError
import ftl.util.join
import ftl.util.timeoutToSeconds

Expand Down
2 changes: 1 addition & 1 deletion test_runner/src/main/kotlin/ftl/gc/GcIosTestMatrix.kt
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ import com.google.api.services.testing.model.ToolResultsHistory
import ftl.args.IosArgs
import ftl.ios.Xctestrun
import ftl.ios.Xctestrun.toByteArray
import ftl.util.FlankGeneralError
import ftl.run.exception.FlankGeneralError
import ftl.util.ShardCounter
import ftl.util.join
import ftl.util.timeoutToSeconds
Expand Down
2 changes: 1 addition & 1 deletion test_runner/src/main/kotlin/ftl/gc/GcStorage.kt
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ import ftl.config.FtlConstants.GCS_PREFIX
import ftl.reports.xml.model.JUnitTestResult
import ftl.reports.xml.parseAllSuitesXml
import ftl.reports.xml.xmlToString
import ftl.util.FlankGeneralError
import ftl.run.exception.FlankGeneralError
import ftl.util.ProgressBar
import ftl.util.join
import java.io.File
Expand Down
2 changes: 1 addition & 1 deletion test_runner/src/main/kotlin/ftl/gc/GcTestMatrix.kt
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ package ftl.gc
import com.google.api.services.testing.model.CancelTestMatrixResponse
import com.google.api.services.testing.model.TestMatrix
import ftl.http.executeWithRetry
import ftl.util.FlankGeneralError
import ftl.run.exception.FlankGeneralError
import kotlinx.coroutines.Dispatchers
import kotlinx.coroutines.delay
import kotlinx.coroutines.withContext
Expand Down
8 changes: 4 additions & 4 deletions test_runner/src/main/kotlin/ftl/gc/GcToolResults.kt
Original file line number Diff line number Diff line change
Expand Up @@ -19,10 +19,10 @@ import ftl.config.FtlConstants.applicationName
import ftl.config.FtlConstants.httpCredential
import ftl.config.FtlConstants.httpTransport
import ftl.http.executeWithRetry
import ftl.util.FTLProjectError
import ftl.util.FlankGeneralError
import ftl.util.PermissionDenied
import ftl.util.ProjectNotFound
import ftl.run.exception.FTLProjectError
import ftl.run.exception.FlankGeneralError
import ftl.run.exception.PermissionDenied
import ftl.run.exception.ProjectNotFound

object GcToolResults {

Expand Down
4 changes: 2 additions & 2 deletions test_runner/src/main/kotlin/ftl/http/ExecuteWithRetry.kt
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,8 @@ package ftl.http
import com.google.api.client.googleapis.services.json.AbstractGoogleJsonClientRequest
import com.google.api.client.http.HttpResponseException
import ftl.config.FtlConstants
import ftl.util.PermissionDenied
import ftl.util.ProjectNotFound
import ftl.run.exception.PermissionDenied
import ftl.run.exception.ProjectNotFound
import kotlinx.coroutines.delay
import kotlinx.coroutines.runBlocking
import java.io.IOException
Expand Down
2 changes: 1 addition & 1 deletion test_runner/src/main/kotlin/ftl/ios/Parse.kt
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
package ftl.ios

import ftl.config.FtlConstants.isMacOS
import ftl.run.exception.FlankGeneralError
import ftl.util.Bash
import ftl.util.FlankGeneralError
import ftl.util.copyBinaryResource
import java.io.File

Expand Down
2 changes: 1 addition & 1 deletion test_runner/src/main/kotlin/ftl/ios/Xctestrun.kt
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ import com.dd.plist.NSArray
import com.dd.plist.NSDictionary
import com.dd.plist.NSString
import com.dd.plist.PropertyListParser
import ftl.util.FlankGeneralError
import ftl.run.exception.FlankGeneralError
import java.io.ByteArrayOutputStream
import java.io.File
import java.nio.file.Paths
Expand Down
10 changes: 5 additions & 5 deletions test_runner/src/main/kotlin/ftl/json/MatrixMap.kt
Original file line number Diff line number Diff line change
@@ -1,11 +1,11 @@
package ftl.json

import com.google.api.services.testing.model.TestMatrix
import ftl.util.FTLError
import ftl.util.FailedMatrixError
import ftl.util.IncompatibleTestDimensionError
import ftl.util.InfrastructureError
import ftl.util.MatrixCanceledError
import ftl.run.exception.FTLError
import ftl.run.exception.FailedMatrixError
import ftl.run.exception.IncompatibleTestDimensionError
import ftl.run.exception.InfrastructureError
import ftl.run.exception.MatrixCanceledError
import ftl.util.MatrixState

class MatrixMap(
Expand Down
2 changes: 1 addition & 1 deletion test_runner/src/main/kotlin/ftl/mock/MockServer.kt
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ import ftl.config.FtlConstants
import ftl.config.FtlConstants.JSON_FACTORY
import ftl.log.LogbackLogger
import ftl.util.Bash
import ftl.util.FlankGeneralError
import ftl.run.exception.FlankGeneralError
import ftl.util.StepOutcome.failure
import ftl.util.StepOutcome.flaky
import ftl.util.StepOutcome.inconclusive
Expand Down
3 changes: 2 additions & 1 deletion test_runner/src/main/kotlin/ftl/mock/TestArtifact.kt
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
package ftl.mock

import ftl.util.FlankGeneralError
import ftl.run.exception.FlankGeneralError
import okhttp3.OkHttpClient
import okhttp3.Request
import okhttp3.ResponseBody
Expand All @@ -14,6 +14,7 @@ import java.net.Socket
import java.nio.file.Files
import java.nio.file.Paths
import java.util.concurrent.TimeUnit
import kotlin.math.pow
import java.util.zip.ZipFile
import kotlin.math.pow

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ import com.google.api.services.toolresults.model.Step
import ftl.gc.GcToolResults
import ftl.json.SavedMatrix
import ftl.json.createSavedMatrix
import ftl.util.FTLError
import ftl.run.exception.FTLError
import ftl.util.timeoutToSeconds

data class TestOutcomeContext(
Expand Down
2 changes: 1 addition & 1 deletion test_runner/src/main/kotlin/ftl/reports/xml/JUnitXml.kt
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ import com.fasterxml.jackson.module.kotlin.KotlinModule
import ftl.reports.xml.model.JUnitTestResult
import ftl.reports.xml.model.JUnitTestSuite
import ftl.reports.xml.preprocesor.fixHtmlCodes
import ftl.util.FlankGeneralError
import ftl.run.exception.FlankGeneralError
import java.io.File
import java.nio.file.Files
import java.nio.file.Path
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ package ftl.reports.xml.model

import com.fasterxml.jackson.annotation.JsonInclude
import com.fasterxml.jackson.dataformat.xml.annotation.JacksonXmlProperty
import ftl.util.FlankGeneralError
import ftl.run.exception.FlankGeneralError
import java.util.Locale

data class JUnitTestSuite(
Expand Down
2 changes: 1 addition & 1 deletion test_runner/src/main/kotlin/ftl/run/DumpShards.kt
Original file line number Diff line number Diff line change
Expand Up @@ -4,9 +4,9 @@ import ftl.args.AndroidArgs
import ftl.args.IosArgs
import ftl.args.isInstrumentationTest
import ftl.run.common.prettyPrint
import ftl.run.exception.FlankConfigurationError
import ftl.run.model.AndroidMatrixTestShards
import ftl.run.platform.android.getAndroidMatrixShards
import ftl.util.FlankConfigurationError
import ftl.util.obfuscatePrettyPrinter
import java.nio.file.Files
import java.nio.file.Paths
Expand Down
Loading