[ 
https://issues.apache.org/jira/browse/ONAMI-91?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Mikhail Mazursky closed ONAMI-91.
---------------------------------


> Refactor TestConcurrentLazySingleton test
> -----------------------------------------
>
>                 Key: ONAMI-91
>                 URL: https://issues.apache.org/jira/browse/ONAMI-91
>             Project: Apache Onami
>          Issue Type: Improvement
>          Components: scopes
>    Affects Versions: scopes-1.0.0
>            Reporter: Mikhail Mazursky
>            Assignee: Jordan Zimmerman
>            Priority: Minor
>             Fix For: scopes-1.0.0
>
>
> - test with Executors.newSingleThreadExecutor() should shutdown the executor 
> in a finally block to avoid preventing JVM termination by active thread;
> - In the same test you can just wait on Future returned by executor;
> - I think it's better to always set timeout for waiting in such cases in 
> tests - like 1 minute. Just to avoid locking JVM in case of some bug in test.



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)

Reply via email to