Skip to content

Commit c9dd54e

Browse files
committed
Address comments
1 parent 1f87ba4 commit c9dd54e

File tree

1 file changed

+45
-39
lines changed

1 file changed

+45
-39
lines changed

test/test-type-description-service.js

Lines changed: 45 additions & 39 deletions
Original file line numberDiff line numberDiff line change
@@ -85,49 +85,55 @@ describe('type description service test suite', function () {
8585
});
8686

8787
it('Test type description service configured by parameter', function (done) {
88-
exec(
89-
'ros2 param list /test_type_description_service',
90-
(error, stdout, stderr) => {
91-
if (error || stderr) {
92-
done(
93-
new Error(
94-
'Test type description service configured by parameter failed.'
95-
)
96-
);
97-
}
98-
if (stdout.includes('start_type_description_service')) {
99-
done();
100-
} else {
101-
done(
102-
new Error("'start_type_description_service' not found in stdout.")
103-
);
88+
setTimeout(() => {
89+
exec(
90+
'ros2 param list /test_type_description_service',
91+
(error, stdout, stderr) => {
92+
if (error || stderr) {
93+
done(
94+
new Error(
95+
`Test type description service configured by parameter failed. Error: ${error}, Stderr: ${stderr}`
96+
)
97+
);
98+
return;
99+
}
100+
if (stdout.includes('start_type_description_service')) {
101+
done();
102+
} else {
103+
done(
104+
new Error("'start_type_description_service' not found in stdout.")
105+
);
106+
}
104107
}
105-
}
106-
);
108+
);
109+
}, 1000);
107110
});
108111

109112
it('Test start_type_description_service parameter value', function (done) {
110-
exec(
111-
'ros2 param get /test_type_description_service start_type_description_service',
112-
(error, stdout, stderr) => {
113-
if (error || stderr) {
114-
done(
115-
new Error(
116-
'Test type description service configured by parameter failed.'
117-
)
118-
);
119-
}
120-
if (stdout.includes('Boolean value is: True')) {
121-
done();
122-
} else {
123-
console.log(stdout);
124-
done(
125-
new Error(
126-
"'start_type_description_service param value' not found in stdout."
127-
)
128-
);
113+
setTimeout(() => {
114+
exec(
115+
'ros2 param get /test_type_description_service start_type_description_service',
116+
(error, stdout, stderr) => {
117+
if (error || stderr) {
118+
done(
119+
new Error(
120+
`Test type description service configured by parameter failed. Error: ${error}, Stderr: ${stderr}`
121+
)
122+
);
123+
return;
124+
}
125+
if (stdout.includes('Boolean value is: True')) {
126+
done();
127+
} else {
128+
console.log(stdout);
129+
done(
130+
new Error(
131+
"'start_type_description_service param value' not found in stdout."
132+
)
133+
);
134+
}
129135
}
130-
}
131-
);
136+
);
137+
}, 1000);
132138
});
133139
});

0 commit comments

Comments
 (0)