curben
/
cloudflared-mirror
mirror of https://github.com/cloudflare/cloudflared.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

Correct h2mux tests on windows without CGO_ENABLED 

Compression isn't supported without cgo on windows environments so the h2mux tests need to ignore compression.
This commit is contained in:
Devin Carr 2022-03-21 18:37:53 -07:00
parent 6eeaf4be4b
commit cad3cd1834
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
h2mux/h2mux_test.go
View File

@ -823,7 +823,7 @@ func TestMultipleStreamsWithDictionaries(t *testing.T) {
} }
originMuxMetrics := muxPair.OriginMux.Metrics() originMuxMetrics := muxPair.OriginMux.Metrics()
if q > CompressionNone && originMuxMetrics.CompBytesBefore.Value() <= 10*originMuxMetrics.CompBytesAfter.Value() { if q > CompressionNone && CompressionIsSupported() && originMuxMetrics.CompBytesBefore.Value() <= 10*originMuxMetrics.CompBytesAfter.Value() {
t.Fatalf("Cross-stream compression is expected to give a better compression ratio") t.Fatalf("Cross-stream compression is expected to give a better compression ratio")
} }
} }
@ -952,7 +952,7 @@ func TestSampleSiteWithDictionaries(t *testing.T) {
} }
originMuxMetrics := muxPair.OriginMux.Metrics() originMuxMetrics := muxPair.OriginMux.Metrics()
if q > CompressionNone && originMuxMetrics.CompBytesBefore.Value() <= 10*originMuxMetrics.CompBytesAfter.Value() { if q > CompressionNone && CompressionIsSupported() && originMuxMetrics.CompBytesBefore.Value() <= 10*originMuxMetrics.CompBytesAfter.Value() {
t.Fatalf("Cross-stream compression is expected to give a better compression ratio") t.Fatalf("Cross-stream compression is expected to give a better compression ratio")
} }
} }
@ -985,8 +985,8 @@ func TestLongSiteWithDictionaries(t *testing.T) {
assert.NoError(t, errGroup.Wait()) assert.NoError(t, errGroup.Wait())
originMuxMetrics := muxPair.OriginMux.Metrics() originMuxMetrics := muxPair.OriginMux.Metrics()
if q > CompressionNone && originMuxMetrics.CompBytesBefore.Value() <= 10*originMuxMetrics.CompBytesAfter.Value() { if q > CompressionNone && CompressionIsSupported() && originMuxMetrics.CompBytesBefore.Value() <= 10*originMuxMetrics.CompBytesAfter.Value() {
t.Fatalf("Cross-stream compression is expected to give a better compression ratio") t.Fatalf("Cross-stream compression (%d) is expected to give a better compression ratio: %d <= %d", q, originMuxMetrics.CompBytesBefore.Value(), 10*originMuxMetrics.CompBytesAfter.Value())
} }
} }
} }