[
https://issues.apache.org/jira/browse/SLING-10479?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17363013#comment-17363013
]
Christian Schneider commented on SLING-10479:
---------------------------------------------
[~sseifert] PR is done:
https://github.com/apache/sling-org-apache-sling-testing-osgi-mock/pull/8
> MockServiceRegistration should throw IllegalStateException if registration is
> already closed
> --------------------------------------------------------------------------------------------
>
> Key: SLING-10479
> URL: https://issues.apache.org/jira/browse/SLING-10479
> Project: Sling
> Issue Type: Bug
> Components: Apache Sling Testing Clients
> Affects Versions: Testing OSGi Mock 3.1.2
> Reporter: Christian Schneider
> Assignee: Christian Schneider
> Priority: Major
> Time Spent: 10m
> Remaining Estimate: 0h
>
> I tested some code that threw IllegalStateException when a service
> registration was unregistered twice.
> To test the behaviour automatically I created a unit test to reproduce the
> behaviour.
> Unfortunately it seems that when using OsgiContext the
> MockServiceRegistration I get when registering a service does not show the
> same behaviour as a real object.
> So I propose we also throw a IllegalStateException when unregister is called
> twice. I can provide a test case and fix for this.
> There also might be other cases like registration failed when we also would
> like to throw but I am not sure ho to implement this.
>
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
