Skip to content

Commit 8675e90

Browse files
authored
Merge branch 'main' into add-query-integration-test
2 parents ff5447d + 8e0e9a7 commit 8675e90

File tree

100 files changed

+8614
-272
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

100 files changed

+8614
-272
lines changed

.gitignore

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -51,3 +51,4 @@ resource.syso
5151
.gocache
5252
test-results.json
5353
.metrics/
54+
.mockery.log

.mockery.yaml

Lines changed: 11 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
with-expecter: false
1+
with-expecter: true
22
disable-version-string: true
33
outpkg: "mocks"
44
dir: "{{.InterfaceDir}}/mocks/"
@@ -24,28 +24,28 @@ packages:
2424
SpanProcessor:
2525
github.com/jaegertracing/jaeger/crossdock/services:
2626
interfaces:
27-
AgentService:
27+
CollectorService:
2828
QueryService:
29-
github.com/jaegertracing/jaeger/pkg/cassandra:
29+
github.com/jaegertracing/jaeger/internal/storage/cassandra:
3030
config:
3131
unroll-variadic: false
3232
interfaces:
3333
Iterator:
3434
Query:
3535
Session:
36-
github.com/jaegertracing/jaeger/pkg/distributedlock:
36+
github.com/jaegertracing/jaeger/internal/distributedlock:
3737
config:
3838
all: true
39-
github.com/jaegertracing/jaeger/pkg/es:
39+
github.com/jaegertracing/jaeger/internal/storage/elasticsearch:
4040
config:
4141
all: true
42-
github.com/jaegertracing/jaeger/pkg/es/client:
42+
github.com/jaegertracing/jaeger/internal/storage/elasticsearch/client:
4343
config:
4444
all: true
45-
github.com/jaegertracing/jaeger/pkg/kafka/consumer:
45+
github.com/jaegertracing/jaeger/internal/storage/kafka/consumer:
4646
interfaces:
4747
Consumer:
48-
github.com/jaegertracing/jaeger/plugin/sampling/leaderelection:
48+
github.com/jaegertracing/jaeger/internal/leaderelection:
4949
config:
5050
all: true
5151
github.com/jaegertracing/jaeger/internal/storage/v1/grpc/shared:
@@ -83,4 +83,6 @@ packages:
8383
interfaces:
8484
CoreSpanReader:
8585
CoreSpanWriter:
86-
86+
github.com/jaegertracing/jaeger/internal/storage/v1/elasticsearch/dependencystore:
87+
interfaces:
88+
CoreDependencyStore:

Makefile

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -218,7 +218,8 @@ init-submodules:
218218
MOCKERY_FLAGS := --all --disable-version-string
219219
.PHONY: generate-mocks
220220
generate-mocks: $(MOCKERY)
221-
$(MOCKERY)
221+
find . -path '*/mocks/*' -name '*.go' -type f -delete
222+
$(MOCKERY) | tee .mockery.log
222223

223224
.PHONY: certs
224225
certs:

cmd/anonymizer/app/query/query_test.go

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -71,12 +71,15 @@ func newTestServer(t *testing.T) *testServer {
7171
lis, err := net.Listen("tcp", ":0")
7272
require.NoError(t, err)
7373

74-
var exited sync.WaitGroup
74+
var started, exited sync.WaitGroup
75+
started.Add(1)
7576
exited.Add(1)
7677
go func() {
78+
started.Done()
7779
assert.NoError(t, server.Serve(lis))
7880
exited.Done()
7981
}()
82+
started.Wait()
8083
t.Cleanup(func() {
8184
server.Stop()
8285
exited.Wait() // don't allow test to finish before server exits

cmd/collector/app/sanitizer/cache/mocks/ServiceAliasMappingExternalSource.go

Lines changed: 35 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

cmd/collector/app/sanitizer/cache/mocks/ServiceAliasMappingStorage.go

Lines changed: 63 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

cmd/ingester/app/consumer/mocks/Message.go

Lines changed: 143 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)