| 82 | } |
| 83 | |
| 84 | func TestFrontendBadConfigFails(t *testing.T) { |
| 85 | f, err := New(Config{ |
| 86 | TraceByID: TraceByIDConfig{ |
| 87 | QueryShards: minQueryShards - 1, |
| 88 | }, |
| 89 | Search: SearchConfig{ |
| 90 | Sharder: SearchSharderConfig{ |
| 91 | ConcurrentRequests: defaultConcurrentRequests, |
| 92 | TargetBytesPerRequest: defaultTargetBytesPerRequest, |
| 93 | MostRecentShards: defaultMostRecentShards, |
| 94 | }, |
| 95 | SLO: testSLOcfg, |
| 96 | }, |
| 97 | }, nil, nil, nil, nil, "", fakeHTTPAuthMiddleware, nil, log.NewNopLogger(), nil) |
| 98 | assert.EqualError(t, err, "frontend query shards should be between 2 and 100000 (both inclusive)") |
| 99 | assert.Nil(t, f) |
| 100 | |
| 101 | f, err = New(Config{ |
| 102 | TraceByID: TraceByIDConfig{ |
| 103 | QueryShards: maxQueryShards + 1, |
| 104 | SLO: testSLOcfg, |
| 105 | }, |
| 106 | Search: SearchConfig{ |
| 107 | Sharder: SearchSharderConfig{ |
| 108 | ConcurrentRequests: defaultConcurrentRequests, |
| 109 | TargetBytesPerRequest: defaultTargetBytesPerRequest, |
| 110 | MostRecentShards: defaultMostRecentShards, |
| 111 | }, |
| 112 | SLO: testSLOcfg, |
| 113 | }, |
| 114 | }, nil, nil, nil, nil, "", fakeHTTPAuthMiddleware, nil, log.NewNopLogger(), nil) |
| 115 | assert.EqualError(t, err, "frontend query shards should be between 2 and 100000 (both inclusive)") |
| 116 | assert.Nil(t, f) |
| 117 | |
| 118 | f, err = New(Config{ |
| 119 | TraceByID: TraceByIDConfig{ |
| 120 | QueryShards: maxQueryShards, |
| 121 | SLO: testSLOcfg, |
| 122 | }, |
| 123 | Search: SearchConfig{ |
| 124 | Sharder: SearchSharderConfig{ |
| 125 | ConcurrentRequests: 0, |
| 126 | TargetBytesPerRequest: defaultTargetBytesPerRequest, |
| 127 | MostRecentShards: defaultMostRecentShards, |
| 128 | }, |
| 129 | SLO: testSLOcfg, |
| 130 | }, |
| 131 | }, nil, nil, nil, nil, "", fakeHTTPAuthMiddleware, nil, log.NewNopLogger(), nil) |
| 132 | assert.EqualError(t, err, "frontend search concurrent requests should be greater than 0") |
| 133 | assert.Nil(t, f) |
| 134 | |
| 135 | f, err = New(Config{ |
| 136 | TraceByID: TraceByIDConfig{ |
| 137 | QueryShards: maxQueryShards, |
| 138 | SLO: testSLOcfg, |
| 139 | }, |
| 140 | Search: SearchConfig{ |
| 141 | Sharder: SearchSharderConfig{ |