Commit ea722f6
[feat][backend]trace impl mult repo (#323)
* mult repo
* mult repo
* mult repo
* mult repo
* mult repo
* mult repo
* mult repo
* mult repo
* mult repo
* mult repo
* mult repo
* mult repo
* init lib
* init lib
* update otel lib
* update otel lib
* update otel lib
* update otel lib
* update otel lib
* update otel lib
* update IStorageProvider
* update IStorageProvider
* update IStorageProvider
* merge main
* fix
* update
* update
* test
* fix oapi
* anno callback
* anno callback
* anno callback
* anno callback
* anno callback
* update lib
* update lib
* update lib
* test
* test
* test
* fix panic
* 修复并补充backend/modules/observability目录下的UT
- 修复backend/modules/observability/infra/repo/ck/spans_test.go中的未定义QueryParam类型错误
- 修复backend/modules/observability/infra/repo/trace_test.go中的多个构建和运行时错误
- 创建backend/modules/observability/application/convertor/page_test.go用于测试新的convertor函数
- 创建backend/modules/observability/application/convertor/task/filter_test.go用于测试新的TaskFilters转换函数
- 修复mock文件中的包导入和类型引用错误
- 确保所有trace repo测试用例通过
主要修复内容包括:
- 修正结构体字段名称不匹配问题(spansDao → spanDaos)
- 添加缺失的mockStorageProvider实现
- 更新所有构造函数调用以使用正确的TraceRepoOption模式
- 修复mock期望以匹配实际方法签名
- 解决参数匹配问题
所有测试现已通过,确保代码库保持高质量和可靠性。
* 修复backend/modules/observability/domain下的UT问题
- 修复backend/modules/observability/domain/task/service/task_service_test.go中的构建错误
- 修复NewTaskServiceImpl构造函数调用,添加缺失的必需参数
- 创建stubIDGenerator、stubStorageProvider、stubTenantProvider实现
- 修复测试函数中直接使用未导出类型的问题
- 修复backend/modules/observability/domain/task/service/task_callback_test.go中的mock期望问题
- 添加缺失的GetTask方法期望
- 修复TaskCallbackServiceImpl初始化时缺少的taskRepo字段
- 修复backend/modules/observability/domain/trace/service/trace_service_test.go中的参数匹配问题
- 修复SearchTraceOApi测试中GetTrace参数期望不匹配问题
所有TaskServiceImpl、TaskCallbackServiceImpl和TraceServiceImpl相关的测试现已通过,确保代码库的稳定性和可靠性。
* update
* fix UT
* fix
* fix UT
* fix UT
* Change auto eval target type.
Change-Id: Idef45fe757a98f87c5f731934a47d11e57fcd953
* Change eval target type.
Change-Id: I92b5ca667d4bf5aac67ca7ccd069902b664c4713
* fix
* 修复并补充auto_evaluate_test.go的单元测试
- 修复了buildTestTask函数中缺失的SpanFilter字段导致的空指针解引用问题
- 修正了SpanFilter相关的类型定义错误
- 补充了buildEvalTargetParam函数的测试用例,覆盖不同平台类型
- 添加了NewAutoEvaluteProcessor构造函数测试
- 补充了错误处理路径的测试用例
- 整体测试覆盖率从74.3%提升到76.5%
- 关键函数buildEvalTargetParam覆盖率从66.7%提升到100%
* fix UT
* ad d UT
* remove anno param
* Add eval target builder.
Change-Id: If250f611f76eea274015338ab60f5182dafbfb91
* Add ctx.
Change-Id: I06e02341ff906c1606de20a6e3c4361d24d10bd4
* Fix ut.
Change-Id: I3630c4bcd80b1a49a519dce44e2ac5a59e0650d9
* Re push.
Change-Id: I74c5826df4f346ea0dfeeba1d96ba67df1d043b8
* add UT
* add UT
* fix UT
* fix UT
* fix UT
* fix UT
* fix UT
---------
Co-authored-by: zhaoxun.3233 <zhaoxun.3233@bytedance.com>
Co-authored-by: taoyifan89 <taoyifan@bytedance.com>1 parent e3c8972 commit ea722f6
File tree
64 files changed
+4982
-880
lines changed- backend
- api
- handler/coze/loop/apis
- kitex_gen/coze/loop/observability/domain/common
- modules/observability
- application
- convertor
- task
- trace
- domain
- component/storage
- mocks
- metric
- repo
- service
- task
- entity
- service
- taskexe
- processor
- tracehub
- trace
- entity
- loop_span
- otel
- repo
- service
- infra
- repo
- ck
- convertor
- mocks
- dao
- converter
- mocks
- storage
- lib
- otel
- open_inference
- idl/thrift/coze/loop/observability/domain
Some content is hidden
Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.
64 files changed
+4982
-880
lines changedSome generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
45 | 45 | | |
46 | 46 | | |
47 | 47 | | |
| 48 | + | |
48 | 49 | | |
49 | 50 | | |
50 | 51 | | |
| |||
216 | 217 | | |
217 | 218 | | |
218 | 219 | | |
| 220 | + | |
219 | 221 | | |
220 | 222 | | |
221 | 223 | | |
| |||
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | 2 | | |
3 | | - | |
4 | | - | |
5 | | - | |
| 3 | + | |
6 | 4 | | |
7 | 5 | | |
8 | 6 | | |
| |||
17 | 15 | | |
18 | 16 | | |
19 | 17 | | |
20 | | - | |
| 18 | + | |
21 | 19 | | |
22 | | - | |
| 20 | + | |
23 | 21 | | |
24 | 22 | | |
25 | 23 | | |
| |||
32 | 30 | | |
33 | 31 | | |
34 | 32 | | |
35 | | - | |
| 33 | + | |
36 | 34 | | |
37 | | - | |
| 35 | + | |
38 | 36 | | |
39 | 37 | | |
40 | | - | |
41 | | - | |
| 38 | + | |
42 | 39 | | |
43 | 40 | | |
44 | 41 | | |
| |||
77 | 74 | | |
78 | 75 | | |
79 | 76 | | |
80 | | - | |
| 77 | + | |
81 | 78 | | |
82 | 79 | | |
83 | 80 | | |
84 | 81 | | |
85 | | - | |
| 82 | + | |
86 | 83 | | |
87 | | - | |
| 84 | + | |
88 | 85 | | |
89 | 86 | | |
90 | | - | |
91 | | - | |
| 87 | + | |
| 88 | + | |
92 | 89 | | |
93 | 90 | | |
94 | 91 | | |
| |||
99 | 96 | | |
100 | 97 | | |
101 | 98 | | |
102 | | - | |
| 99 | + | |
103 | 100 | | |
104 | | - | |
| 101 | + | |
| 102 | + | |
| 103 | + | |
| 104 | + | |
| 105 | + | |
105 | 106 | | |
106 | 107 | | |
107 | 108 | | |
| |||
132 | 133 | | |
133 | 134 | | |
134 | 135 | | |
135 | | - | |
| 136 | + | |
136 | 137 | | |
137 | 138 | | |
138 | 139 | | |
139 | 140 | | |
140 | | - | |
| 141 | + | |
141 | 142 | | |
142 | | - | |
143 | | - | |
| 143 | + | |
144 | 144 | | |
145 | | - | |
| 145 | + | |
146 | 146 | | |
147 | | - | |
| 147 | + | |
148 | 148 | | |
149 | 149 | | |
150 | 150 | | |
| |||
238 | 238 | | |
239 | 239 | | |
240 | 240 | | |
241 | | - | |
242 | | - | |
243 | | - | |
| 241 | + | |
| 242 | + | |
| 243 | + | |
244 | 244 | | |
245 | 245 | | |
246 | 246 | | |
247 | 247 | | |
248 | | - | |
249 | | - | |
| 248 | + | |
| 249 | + | |
250 | 250 | | |
251 | | - | |
| 251 | + | |
252 | 252 | | |
253 | | - | |
254 | | - | |
255 | | - | |
256 | | - | |
257 | | - | |
| 253 | + | |
| 254 | + | |
| 255 | + | |
| 256 | + | |
| 257 | + | |
258 | 258 | | |
259 | 259 | | |
260 | 260 | | |
| |||
264 | 264 | | |
265 | 265 | | |
266 | 266 | | |
| 267 | + | |
| 268 | + | |
0 commit comments