Skip to content

Commit a46e996

Browse files
committed
Fix tests.
Signed-off-by: Michał Zientkiewicz <[email protected]>
1 parent 170ce57 commit a46e996

File tree

1 file changed

+12
-8
lines changed

1 file changed

+12
-8
lines changed

dali/c_api/operator_trace_test.cc

Lines changed: 12 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -145,11 +145,12 @@ TEST_P(OperatorTraceTest, OperatorTraceTest) {
145145
for (int i = 0; i < prefetch_depth; i++) {
146146
daliShareOutput(&h);
147147

148-
for (size_t i = 0; i < std::size(operator_under_test_names); ++i) {
149-
const char *operator_name = operator_under_test_names[i];
150-
const char *trace_name = operator_trace_names[i];
148+
for (size_t op = 0; op < std::size(operator_under_test_names); op++) {
149+
const char *operator_name = operator_under_test_names[op];
150+
const char *trace_name = operator_trace_names[op];
151151
EXPECT_EQ(daliHasOperatorTrace(&h, operator_name, "this_trace_does_not_exist"), 0);
152-
ASSERT_NE(daliHasOperatorTrace(&h, operator_name, trace_name), 0) << "name: " << trace_name;
152+
ASSERT_NE(daliHasOperatorTrace(&h, operator_name, trace_name), 0)
153+
<< "operator_name: " << operator_name << "\ntrace_name: " << trace_name;
153154

154155
EXPECT_EQ(std::string(daliGetOperatorTrace(&h, operator_name, trace_name)),
155156
make_string("test_value", iteration * prefetch_depth + i));
@@ -194,11 +195,13 @@ class OperatorTraceTestExternalInput : public OperatorTraceTest {
194195
pipeline_->AddOperator(OpSpec("PassthroughWithTraceOp")
195196
.AddArg("device", "cpu")
196197
.AddInput("OP_TRACE_IN_CPU", "cpu")
198+
.AddArg("trace_name", operator_trace_names[0])
197199
.AddOutput("PT_CPU", "cpu"),
198200
operator_under_test_names[0]);
199201
pipeline_->AddOperator(OpSpec("PassthroughWithTraceOp")
200202
.AddArg("device", "gpu")
201203
.AddInput("OP_TRACE_IN_GPU", "gpu")
204+
.AddArg("trace_name", operator_trace_names[1])
202205
.AddOutput("PT_GPU", "gpu"),
203206
operator_under_test_names[1]);
204207

@@ -262,11 +265,12 @@ TEST_P(OperatorTraceTestExternalInput, OperatorTraceTestExternalInput) {
262265
for (int i = 0; i < prefetch_depth; i++) {
263266
daliShareOutput(&h);
264267

265-
for (size_t i = 0; i < std::size(operator_under_test_names); i++) {
266-
const char *operator_name = operator_under_test_names[i];
267-
const char *trace_name = operator_trace_names[i];
268+
for (size_t op = 0; op < std::size(operator_under_test_names); op++) {
269+
const char *operator_name = operator_under_test_names[op];
270+
const char *trace_name = operator_trace_names[op];
268271
EXPECT_EQ(daliHasOperatorTrace(&h, operator_name, "this_trace_does_not_exist"), 0);
269-
ASSERT_NE(daliHasOperatorTrace(&h, operator_name, trace_name), 0) << "name: " << trace_name;
272+
ASSERT_NE(daliHasOperatorTrace(&h, operator_name, trace_name), 0)
273+
<< "operator_name: " << operator_name << "\ntrace_name: " << trace_name;
270274

271275
EXPECT_EQ(std::string(daliGetOperatorTrace(&h, operator_name, trace_name)),
272276
make_string("test_value", iteration * prefetch_depth + i));

0 commit comments

Comments
 (0)