Home » Eclipse Projects » Equinox » ExtensionRegistryTestSupport
| ExtensionRegistryTestSupport [message #504074] |
Wed, 16 December 2009 22:52  |
 Ralf Ebert
Messages: 72
Registered: July 2009 |
Member |
|
|
This is a multi-part message in MIME format.
--------------000707050408040109060800
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit
I just tried to write a unit test that checks its class under test for correct
evaluation of extensions. Trying that, I found it a bit cumbersome to create an
IExtensionRegistry instance that is populated from plain text files outside of
an OSGi environment. It's possible, but IMHO it is not as easy as it should be.
Also, I couldn't find any unit tests for the extension registry itself that
could be used as a starting point - did I miss something here?
After finding out how this works, I wrote an ExtensionRegistryTestSupport class
that helps with setting up an IExtensionRegistry in unit tests. For example:
============================================================ ====
ExtensionRegistryTestSupport ext = new ExtensionRegistryTestSupport();
// create plugin.xml defining extension points on-the-fly
// as extension points need to defined for extensions to be visible
ext.contributeExtensionPoints("com.example",
new String[] { "test1", "test2", "test3" });
// contribute using streams or strings on behalf of a bundleId
ext.contribute("com.example.a", getClass().getResourceAsStream("plugin-a.xml"));
ext.contribute("com.example.b", getClass().getResourceAsStream("plugin-b.xml"));
extensionRegistry = ext.getExtensionRegistry();
============================================================ ====
I attached my implementation of the class. This needs to be polished and
important things might be missing. I'd appreciate your feedback:
- do you see value in such a test support class?
- did I miss something in my implementation?
- if you see value in it, where could such a class be provided so that Eclipse
project unit tests could use it? (I wrote this as part of a unit test for a
contribution to org.eclipse.ui.workbench I'm working on)
Greetings,
Ralf
--
http://www.ralfebert.de/blog/eclipsercp/
--------------000707050408040109060800
Content-Type: text/x-java;
name="ExtensionRegistryTestSupport.java"
Content-Transfer-Encoding: base64
Content-Disposition: inline;
filename="ExtensionRegistryTestSupport.java"
cGFja2FnZSBleHRlbnNpb25tb2NrczsKCmltcG9ydCBqYXZhLmlvLkJ5dGVB cnJheUlucHV0
U3RyZWFtOwppbXBvcnQgamF2YS5pby5JT0V4Y2VwdGlvbjsKaW1wb3J0IGph dmEuaW8uSW5w
dXRTdHJlYW07CgppbXBvcnQgb3JnLmVjbGlwc2UuY29yZS5ydW50aW1lLkFz c2VydDsKaW1w
b3J0IG9yZy5lY2xpcHNlLmNvcmUucnVudGltZS5JRXh0ZW5zaW9uUmVnaXN0 cnk7CmltcG9y
dCBvcmcuZWNsaXBzZS5jb3JlLnJ1bnRpbWUuUmVnaXN0cnlGYWN0b3J5Owpp bXBvcnQgb3Jn
LmVjbGlwc2UuY29yZS5ydW50aW1lLnNwaS5SZWdpc3RyeUNvbnRyaWJ1dG9y OwoKLyoqCiAq
IEV4dGVuc2lvblJlZ2lzdHJ5VGVzdFN1cHBvcnQgc2ltcGxpZmllcyB0aGUg Y3JlYXRpb24g
b2YgYW4KICoge0BsaW5rIElFeHRlbnNpb25SZWdpc3RyeX0gaW5zdGFuY2Ug Zm9yIHVuaXQg
dGVzdGluZyBwdXJwb3Nlcy4KICogRXh0ZW5zaW9uUmVnaXN0cnlUZXN0U3Vw cG9ydCBjcmVh
dGVzIGFuZCBtYW5hZ2VzIGFuIGV4dGVuc2lvbiByZWdpc3RyeQogKiBpbnN0 YW5jZSBhbmQg
YWxsb3dzIHRvIGNvbnRyaWJ1dGUgY29udGVudHMgb24gdGhlIGZseSB1c2lu ZyBpdHMgY29u
dHJpYnV0ZQogKiBtZXRob2RzLgogKi8KcHVibGljIGNsYXNzIEV4dGVuc2lv blJlZ2lzdHJ5
VGVzdFN1cHBvcnQgewoKCXByaXZhdGUgZmluYWwgSUV4dGVuc2lvblJlZ2lz dHJ5IGV4dGVu
c2lvblJlZ2lzdHJ5ID0gUmVnaXN0cnlGYWN0b3J5LmNyZWF0ZVJlZ2lzdHJ5 KG51bGwsIHRo
aXMsIHRoaXMpOwoKCS8qKgoJICogUmV0dXJucyB0aGUgZXh0ZW5zaW9uUmVn aXN0cnkgaW5z
dGFuY2UgbWFuYWdlZCBieSB0aGlzCgkgKiBFeHRlbnNpb25SZWdpc3RyeVRl c3RTdXBwb3J0
IG9iamVjdC4KCSAqLwoJcHVibGljIElFeHRlbnNpb25SZWdpc3RyeSBnZXRF eHRlbnNpb25S
ZWdpc3RyeSgpIHsKCQlyZXR1cm4gZXh0ZW5zaW9uUmVnaXN0cnk7Cgl9CgoJ LyoqCgkgKiBD
b250cmlidXRlcyB0aGUgY29udGVudHMgb2YgdGhlIGdpdmVuIHBsdWdpblht bFN0cmVhbSBv
biBiZWhhbGYgb2YKCSAqIGJ1bmRsZUlkLiBUaGUgZ2l2ZW4gcGx1Z2luWG1s U3RyZWFtIGlz
IGNsb3NlZCBhZnRlciBjb250cmlidXRpbmcuCgkgKi8KCXB1YmxpYyB2b2lk IGNvbnRyaWJ1
dGUoU3RyaW5nIGJ1bmRsZUlkLCBJbnB1dFN0cmVhbSBwbHVnaW5YbWxTdHJl YW0pIHsKCQlB
c3NlcnQuaXNOb3ROdWxsKGJ1bmRsZUlkLCAiYnVuZGxlSWQiKTsKCQlBc3Nl cnQuaXNOb3RO
dWxsKHBsdWdpblhtbFN0cmVhbSwgInBsdWdpblhtbFN0cmVhbSIpOwoJCXRy eSB7CgkJCVJl
Z2lzdHJ5Q29udHJpYnV0b3IgY29udHJpYnV0b3IgPSBuZXcgUmVnaXN0cnlD b250cmlidXRv
cihidW5kbGVJZCwgYnVuZGxlSWQsIG51bGwsIG51bGwpOwoJCQlleHRlbnNp b25SZWdpc3Ry
eS5hZGRDb250cmlidXRpb24ocGx1Z2luWG1sU3RyZWFtLCBjb250cmlidXRv ciwgZmFsc2Us
IG51bGwsIG51bGwsIHRoaXMpOwoJCX0gY2F0Y2ggKEV4Y2VwdGlvbiBlKSB7 CgkJCXRocm93
IG5ldyBSdW50aW1lRXhjZXB0aW9uKGUpOwoJCX0gZmluYWxseSB7CgkJCXRy eSB7CgkJCQlw
bHVnaW5YbWxTdHJlYW0uY2xvc2UoKTsKCQkJfSBjYXRjaCAoSU9FeGNlcHRp b24gZSkgewoJ
CQkJdGhyb3cgbmV3IFJ1bnRpbWVFeGNlcHRpb24oZSk7CgkJCX0KCQl9Cgl9 CgoJLyoqCgkg
KiBDb250cmlidXRlcyB0aGUgY29udGVudHMgb2YgdGhlIGdpdmVuIHBsdWdp blhtbFN0cmlu
ZyBvbiBiZWhhbGYgb2YKCSAqIGJ1bmRsZUlkLgoJICovCglwdWJsaWMgdm9p ZCBjb250cmli
dXRlKFN0cmluZyBidW5kbGVJZCwgU3RyaW5nIHBsdWdpblhtbFN0cmluZykg ewoJCWNvbnRy
aWJ1dGUoYnVuZGxlSWQsIG5ldyBCeXRlQXJyYXlJbnB1dFN0cmVhbShwbHVn aW5YbWxTdHJp
bmcuZ2V0Qnl0ZXMoKSkpOwoJfQoKCS8qKgoJICogQ3JlYXRlcyBhIHBsdWdp bi54bWwgdGhh
dCBkZWZpbmVzIHRoZSBnaXZlbiBleHRlbnNpb24gcG9pbnRzIG9uIHRoZSBm bHkKCSAqIGFu
ZCBjb250cmlidXRlcyBpdCBvbiBiZWhhbGYgb2YgYnVuZGxlSWQuCgkgKi8K CXB1YmxpYyB2
b2lkIGNvbnRyaWJ1dGVFeHRlbnNpb25Qb2ludHMoU3RyaW5nIGJ1bmRsZUlk LCBTdHJpbmdb
XSBleHRlbnNpb25Qb2ludE5hbWVzKSB7CgkJU3RyaW5nQnVpbGRlciBzdHIg PSBuZXcgU3Ry
aW5nQnVpbGRlcigpOwoJCXN0ci5hcHBlbmQoIjw/eG1sIHZlcnNpb249XCIx LjBcIiBlbmNv
ZGluZz1cIlVURi04XCI/PlxuIik7CgkJc3RyLmFwcGVuZCgiPD9lY2xpcHNl IHZlcnNpb249
XCIzLjBcIj8+XG4iKTsKCQlzdHIuYXBwZW5kKCI8cGx1Z2luPlxuIik7CgkJ Zm9yIChpbnQg
aSA9IDA7IGkgPCBleHRlbnNpb25Qb2ludE5hbWVzLmxlbmd0aDsgaSsrKSB7 CgkJCVN0cmlu
ZyBleHRQb2ludE5hbWUgPSBleHRlbnNpb25Qb2ludE5hbWVzW2ldOwoJCQlz dHIuYXBwZW5k
KFN0cmluZy5mb3JtYXQoIlx0PGV4dGVuc2lvbi1wb2ludCBpZD1cIiVzXCIg bmFtZT1cIiVz
XCIvPlxuIiwgbmV3IE9iamVjdFtdIHsgZXh0UG9pbnROYW1lLAoJCQkJCWV4 dFBvaW50TmFt
ZSB9KSk7CgkJfQoJCXN0ci5hcHBlbmQoIjwvcGx1Z2luPlxuIik7CgkJY29u dHJpYnV0ZShi
dW5kbGVJZCwgc3RyLnRvU3RyaW5nKCkpOwoJfQoKfQ==
--------------000707050408040109060800--
|
|
| |
| Re: ExtensionRegistryTestSupport [message #504237 is a reply to message #504074] |
Thu, 17 December 2009 14:50  |
Oleg Besedin
Messages: 41
Registered: July 2009 |
Member |
|
|
Hi Ralf,
The extension registry JUnits are in the
:pserver:anonymous@dev.eclipse.org:/cvsroot/eclipse
CVS repository under
org.eclipse.core.tests.runtime
(see, for example, packages org.eclipse.core.tests.internal.registry and
org.eclipse.core.tests.internal.registry.simple).
(Historically, extension registry was a part of the Platorm/Runtime and when
the registry moved to Equinox the tests stayed in the old location.)
There are several types of tests: the tests that check simple add/remove
functionality; tests that work based on the bundle contributions, and
session tests. If you end up with a different (or more straightforward) test
harness for the extension registry JUnits, you are very welcome to
contribute it. The best way to do that would be to open a Bugzilla
enhancement for Equinox and attach patches to it.
Sincerely,
Oleg Besedin
"Ralf Ebert" <info@ralfebert.de> wrote in message
news:hgc9a3$fg1$1@build.eclipse.org...
>I just tried to write a unit test that checks its class under test for
>correct
> evaluation of extensions. Trying that, I found it a bit cumbersome to
> create an
> IExtensionRegistry instance that is populated from plain text files
> outside of
> an OSGi environment. It's possible, but IMHO it is not as easy as it
> should be.
>
> Also, I couldn't find any unit tests for the extension registry itself
> that
> could be used as a starting point - did I miss something here?
>
> After finding out how this works, I wrote an ExtensionRegistryTestSupport
> class
> that helps with setting up an IExtensionRegistry in unit tests. For
> example:
>
> ============================================================ ====
> ExtensionRegistryTestSupport ext = new ExtensionRegistryTestSupport();
>
> // create plugin.xml defining extension points on-the-fly
> // as extension points need to defined for extensions to be visible
> ext.contributeExtensionPoints("com.example",
> new String[] { "test1", "test2", "test3" });
>
> // contribute using streams or strings on behalf of a bundleId
> ext.contribute("com.example.a",
> getClass().getResourceAsStream("plugin-a.xml"));
> ext.contribute("com.example.b",
> getClass().getResourceAsStream("plugin-b.xml"));
>
> extensionRegistry = ext.getExtensionRegistry();
> ============================================================ ====
>
> I attached my implementation of the class. This needs to be polished and
> important things might be missing. I'd appreciate your feedback:
>
> - do you see value in such a test support class?
> - did I miss something in my implementation?
> - if you see value in it, where could such a class be provided so that
> Eclipse
> project unit tests could use it? (I wrote this as part of a unit test for
> a
> contribution to org.eclipse.ui.workbench I'm working on)
>
> Greetings,
>
> Ralf
>
>
> --
> http://www.ralfebert.de/blog/eclipsercp/
>
>
>
>
------------------------------------------------------------ --------------------
> package extensionmocks;
>
> import java.io.ByteArrayInputStream;
> import java.io.IOException;
> import java.io.InputStream;
>
> import org.eclipse.core.runtime.Assert;
> import org.eclipse.core.runtime.IExtensionRegistry;
> import org.eclipse.core.runtime.RegistryFactory;
> import org.eclipse.core.runtime.spi.RegistryContributor;
>
> /**
> * ExtensionRegistryTestSupport simplifies the creation of an
> * {@link IExtensionRegistry} instance for unit testing purposes.
> * ExtensionRegistryTestSupport creates and manages an extension registry
> * instance and allows to contribute contents on the fly using its
> contribute
> * methods.
> */
> public class ExtensionRegistryTestSupport {
>
> private final IExtensionRegistry extensionRegistry =
> RegistryFactory.createRegistry(null, this, this);
>
> /**
> * Returns the extensionRegistry instance managed by this
> * ExtensionRegistryTestSupport object.
> */
> public IExtensionRegistry getExtensionRegistry() {
> return extensionRegistry;
> }
>
> /**
> * Contributes the contents of the given pluginXmlStream on behalf of
> * bundleId. The given pluginXmlStream is closed after contributing.
> */
> public void contribute(String bundleId, InputStream pluginXmlStream) {
> Assert.isNotNull(bundleId, "bundleId");
> Assert.isNotNull(pluginXmlStream, "pluginXmlStream");
> try {
> RegistryContributor contributor = new RegistryContributor(bundleId,
> bundleId, null, null);
> extensionRegistry.addContribution(pluginXmlStream, contributor, false,
> null, null, this);
> } catch (Exception e) {
> throw new RuntimeException(e);
> } finally {
> try {
> pluginXmlStream.close();
> } catch (IOException e) {
> throw new RuntimeException(e);
> }
> }
> }
>
> /**
> * Contributes the contents of the given pluginXmlString on behalf of
> * bundleId.
> */
> public void contribute(String bundleId, String pluginXmlString) {
> contribute(bundleId, new
> ByteArrayInputStream(pluginXmlString.getBytes()));
> }
>
> /**
> * Creates a plugin.xml that defines the given extension points on the fly
> * and contributes it on behalf of bundleId.
> */
> public void contributeExtensionPoints(String bundleId, String[]
> extensionPointNames) {
> StringBuilder str = new StringBuilder();
> str.append("<?xml version=\"1.0\" encoding=\"UTF-8\"?>\n");
> str.append("<?eclipse version=\"3.0\"?>\n");
> str.append("<plugin>\n");
> for (int i = 0; i < extensionPointNames.length; i++) {
> String extPointName = extensionPointNames[i];
> str.append(String.format("\t<extension-point id=\"%s\" name=\"%s\"/>\n",
> new Object[] { extPointName,
> extPointName }));
> }
> str.append("</plugin>\n");
> contribute(bundleId, str.toString());
> }
>
> }
|
|
|
| Re: ExtensionRegistryTestSupport [message #504660 is a reply to message #504237] |
Mon, 21 December 2009 23:00 |
Ralf Ebert
Messages: 72
Registered: July 2009 |
Member |
|
|
Hi Oleg,
thanks for your answer!
> The extension registry JUnits are in the
> :pserver:anonymous@dev.eclipse.org:/cvsroot/eclipse
> CVS repository under
> org.eclipse.core.tests.runtime
> (see, for example, packages org.eclipse.core.tests.internal.registry and
> org.eclipse.core.tests.internal.registry.simple).
>
> (Historically, extension registry was a part of the Platorm/Runtime and when
> the registry moved to Equinox the tests stayed in the old location.)
is this intentional or was there just no time to move them yet? I find tests
really hard to spot if they are not co-located with the code under test.
> There are several types of tests: the tests that check simple add/remove
> functionality; tests that work based on the bundle contributions, and
> session tests. If you end up with a different (or more straightforward) test
> harness for the extension registry JUnits, you are very welcome to
> contribute it. The best way to do that would be to open a Bugzilla
> enhancement for Equinox and attach patches to it.
The proposed class is not of great value for the test (harness) for the
extension registry itself. It is intended to be used in tests where you have
something under test that is driven by the ExtensionRegistry. It just makes it
easy to create and populate an ExtensionRegistry instance using a simple API.
I provided the bundle on GitHub: http://github.com/ralfebert/extensiontestsupport
If somebody should find this helpful generally, just drop me a note, then I will
contribute it via Bugzilla. It's licensed under EPL already.
Greetings,
Ralf
--
http://www.ralfebert.de/blog/eclipsercp/
|
|
|
Goto Forum:
Current Time: Thu Apr 25 09:11:23 GMT 2024
Powered by FUDForum. Page generated in 0.03002 seconds |
] 
Search
Help
Register
Login
Home
