Skip to content

Commit

Permalink
allegro-internal/flex-roadmap#687 Added compression properties to nod…
Browse files Browse the repository at this point in the history
…e metadata
  • Loading branch information
nastassia-dailidava committed Jul 30, 2024
1 parent ef66026 commit e8138b5
Showing 1 changed file with 20 additions and 9 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,8 @@ class MetadataNodeGroupTest {
mergeSlashes = true,
pathWithEscapedSlashesAction = "KEEP_UNCHANGED"
)
private val defaultCompConfig = Compressor(false, 1)
private val compressionConfig = CompressionConfig(defaultCompConfig, defaultCompConfig)
@Test
fun `should assign to group with all dependencies`() {
// given
Expand All @@ -50,7 +52,8 @@ class MetadataNodeGroupTest {
proxySettings = ProxySettings().with(
serviceDependencies = serviceDependencies("a", "b", "c"),
allServicesDependencies = true
)
),
compressionConfig = compressionConfig
)
)
}
Expand All @@ -68,7 +71,8 @@ class MetadataNodeGroupTest {
ServicesGroup(
proxySettings = ProxySettings().with(serviceDependencies = setOf()),
pathNormalizationConfig = defaultNormalizationConfig,
communicationMode = XDS
communicationMode = XDS,
compressionConfig = compressionConfig
)
)
}
Expand All @@ -87,7 +91,8 @@ class MetadataNodeGroupTest {
ServicesGroup(
proxySettings = ProxySettings().with(serviceDependencies = serviceDependencies("a", "b", "c")),
pathNormalizationConfig = defaultNormalizationConfig,
communicationMode = XDS
communicationMode = XDS,
compressionConfig = compressionConfig
)
)
}
Expand All @@ -106,7 +111,8 @@ class MetadataNodeGroupTest {
AllServicesGroup(
communicationMode = ADS,
pathNormalizationConfig = defaultNormalizationConfig,
proxySettings = ProxySettings().with(allServicesDependencies = true)
proxySettings = ProxySettings().with(allServicesDependencies = true),
compressionConfig = compressionConfig
)
)
}
Expand All @@ -125,7 +131,8 @@ class MetadataNodeGroupTest {
ServicesGroup(
proxySettings = ProxySettings().with(serviceDependencies = serviceDependencies("a", "b", "c")),
pathNormalizationConfig = defaultNormalizationConfig,
communicationMode = ADS
communicationMode = ADS,
compressionConfig = compressionConfig
)
)
}
Expand All @@ -146,7 +153,8 @@ class MetadataNodeGroupTest {
// because service may define different settings for different dependencies (for example retry config)
communicationMode = ADS,
pathNormalizationConfig = defaultNormalizationConfig,
proxySettings = ProxySettings().with(serviceDependencies = serviceDependencies("a", "b", "c"))
proxySettings = ProxySettings().with(serviceDependencies = serviceDependencies("a", "b", "c")),
compressionConfig = compressionConfig
)
)
}
Expand All @@ -170,7 +178,8 @@ class MetadataNodeGroupTest {
proxySettings = ProxySettings().with(serviceDependencies = serviceDependencies("a", "b", "c")),
pathNormalizationConfig = defaultNormalizationConfig,
communicationMode = XDS,
serviceName = "app1"
serviceName = "app1",
compressionConfig = compressionConfig
)
)
}
Expand Down Expand Up @@ -210,7 +219,8 @@ class MetadataNodeGroupTest {
communicationMode = ADS,
serviceName = "app1",
pathNormalizationConfig = defaultNormalizationConfig,
proxySettings = addedProxySettings.with(serviceDependencies = serviceDependencies("a", "b"))
proxySettings = addedProxySettings.with(serviceDependencies = serviceDependencies("a", "b")),
compressionConfig = compressionConfig
)
)
}
Expand All @@ -232,7 +242,8 @@ class MetadataNodeGroupTest {
communicationMode = XDS,
serviceName = "app1",
pathNormalizationConfig = defaultNormalizationConfig,
proxySettings = addedProxySettings.with(allServicesDependencies = true)
proxySettings = addedProxySettings.with(allServicesDependencies = true),
compressionConfig = compressionConfig
)
)
}
Expand Down

0 comments on commit e8138b5

Please sign in to comment.