You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@celix.apache.org by "Menno van der Graaf (JIRA)" <ji...@apache.org> on 2015/12/17 17:23:46 UTC
[jira] [Comment Edited] (CELIX-329) framework "service_" tests are
outdates, some small bugs in the sources
[ https://issues.apache.org/jira/browse/CELIX-329?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15062261#comment-15062261 ]
Menno van der Graaf edited comment on CELIX-329 at 12/17/15 4:23 PM:
---------------------------------------------------------------------
attached a patch that fixes the tests, as for the "service_registry" files the following prolems are fixed:
source:
-line 417: checking wrong status
-line 784: casting a "hash_map_pt" to "service_registration_pt"
-some small spelling mistakes
-some non exposed functions not declared as static
mock:
-out of date, many functions are only present in the mock after the refactor
and header:
-line 38: the values of the hashMap "registry.serviceRegistrations" are described as service registrations, while the values are actually arrayLists of registrations
-line 64: "serviceRegistry_findRegistration" is declared here, but there is no implementation, only the mock defenition remains
was (Author: mercotui):
attached a patch that fixes the tests, as for the "service_registry" files the following prolems are fixed:
source:
-line 417: checking wrong status
-line 784: casting a "hash_map_pt" to "service_registration_pt"
-some small spelling mistakes
-some non exposed functions not declared as static
mock:
-out of date, many functions are only present in the mock after the refactor
and header:
-line 38: the values of the hashMap "registry->serviceRegistrations" are described as service registrations, while the values are actually arrayLists of registrations
-line 64: "serviceRegistry_findRegistration" is declared here, but there is no implementation, only the mock defenition remains
> framework "service_" tests are outdates, some small bugs in the sources
> -----------------------------------------------------------------------
>
> Key: CELIX-329
> URL: https://issues.apache.org/jira/browse/CELIX-329
> Project: Celix
> Issue Type: Bug
> Components: Framework
> Reporter: Menno van der Graaf
> Attachments: service_refactor_tests.patch
>
>
> The tests for the "service_reference", "service_registry", and "service_tracker" are out of date (after the refactoring of the registry)
> there are also some small bugs in the "service_registry"
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)