This commit fixes the configuration of the base compute service test class. 
That class initialized entities taken from the injector in several methods: 
setupContext and initializeContext. The former was called before all tests, and 
the latter was called several times during tests, to check the behaviour of 
jclouds when using multiple contexts.

The issue was that the first method initialized the OpenSocketFinder from the 
first context, and that context is closed everytime the initializeContext 
method is invoked, leaving the OpenSocketFinder instance unusable, since 
closing its context closes the executor services associated to it.

This moves the initialization of all entities that depend on the injector to 
the initializeContext method, so they are always properly initialized, and also 
aligns the annotation of the setupContext method with the one on the base class.
You can view, comment on, or merge this pull request online at:

  https://github.com/jclouds/jclouds/pull/1088

-- Commit Summary --

  * Fix BaseComputeServiceLiveTest configuration

-- File Changes --

    M 
compute/src/test/java/org/jclouds/compute/internal/BaseComputeServiceLiveTest.java
 (6)

-- Patch Links --

https://github.com/jclouds/jclouds/pull/1088.patch
https://github.com/jclouds/jclouds/pull/1088.diff

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/1088

Reply via email to