summaryrefslogtreecommitdiff
path: root/postprocess
diff options
context:
space:
mode:
authorStephan Bergmann <sbergman@redhat.com>2017-10-26 18:21:47 +0200
committerStephan Bergmann <sbergman@redhat.com>2017-10-26 18:26:02 +0200
commit25313923b08018bd837cfe3fc99a5e141602cafc (patch)
treeaa6963f264748904c83be7c35d3869b4428fe99b /postprocess
parent4e5123fc592cc1a97f7f64b964f858b6f71540cb (diff)
Fix CppunitTest_services for constructor-based implementations...
(i.e., using constructor="..." in the .component file) that implement multiple services, but are not implemented as single-instance objects (i.e., always return different objects from that constructor function). In that case, the "factory" would be the object itself, and would thus be different for different instantiations of that implementation. erAck ran into such a case with new code he's writing. Change-Id: Ib134a4c6d1400b43a7df3cf6a115f8206fb80c93
Diffstat (limited to 'postprocess')
-rw-r--r--postprocess/qa/services.cxx7
1 files changed, 0 insertions, 7 deletions
diff --git a/postprocess/qa/services.cxx b/postprocess/qa/services.cxx
index 0ab677339d9d..78dee9ed3ad1 100644
--- a/postprocess/qa/services.cxx
+++ b/postprocess/qa/services.cxx
@@ -209,13 +209,6 @@ void Test::test() {
k = impls.insert(
std::make_pair(name, Implementation(j, servs)))
.first;
- } else {
- CPPUNIT_ASSERT_MESSAGE(
- (OString(
- "multiple implementations named \"" + msg(name)
- + "\"")
- .getStr()),
- bool(j == k->second.factory));
}
CPPUNIT_ASSERT_MESSAGE(
(OString(