search

frjaeger220 / google-guice

Automatically exported from code.google.com/p/google-guice
Apache License 2.0
0 stars 0 forks source link

OSGi Test Suite #443

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago 
The attached patch and binaries adds an OSGi test suite to Guice.

This test starts up an OSGi container using the new standard launch API and
installs the relevant bundles. It registers bindings for classes of varying
visibility and attempts to create instances of them. It then repeats, but
with method interception enabled for classes with enough visibility.

The aim is to check the OSGi manifest and test the code generation support.

At the moment you will see errors running the test, but I have patches to
fix all the errors which I will attach to the relevant issues (343 etc...)

Enjoy!

Original issue reported on code.google.com by mccu...@gmail.com on 31 Oct 2009 at 5:58

Attachments:

GoogleCodeExporter commented 9 years ago 
Passing to Jesse for review of the new test suite.

Original comment by mccu...@gmail.com on 2 Nov 2009 at 8:11

GoogleCodeExporter commented 9 years ago 
[assumes http://code.google.com/p/google-guice/issues/detail?id=439 has been 
applied]

Description
===========

The attached patch (GUICE_ISSUE_443_20100504.patch) adds OSGi container tests 
to the
googlecode section of the Guice tests. The primary testcase starts an embedded 
OSGi
container which is then used to test injection and AOP interception for various 
types
and methods of different visibility.

You should expect the new testcase to fail until all the reflection fixes from
http://code.google.com/p/google-guice/issues/detail?id=343 have been applied.

Original comment by mccu...@gmail.com on 4 May 2010 at 11:25

Attachments:

GoogleCodeExporter commented 9 years ago 
It looks like this patch is using 'lib/build/felix-2.0.5.jar'.  Could you attach
2.0.5 too? Thanks!

Original comment by sberlin on 5 May 2010 at 2:22

GoogleCodeExporter commented 9 years ago 
Done (just FYI this jar was downloaded from
http://felix.apache.org/site/downloads.cgi - sub-project name 'Main', it's also 
the
same as the 'bin/felix.jar' file found in the Apache Felix 2.0.5 distribution)

Original comment by mccu...@gmail.com on 5 May 2010 at 4:05

Attachments:

GoogleCodeExporter commented 9 years ago 
I committed the changes in r1158, but have one additional question/request -- 
is it
possible to rework the test somehow so that it doesn't rely on Ant having 
prebuilt
the jar?  The test fails if you run it from within Eclipse and ant hasn't built 
the
jars.  As it is now, I just added a warning into the failures that describes 
how the
test needs ant to create the jars first.  But it would be nice if it could 
build the
jar the same way ant does, so it doesn't need that.

If it's just not possible, I think the warning is OK... but it would be a good 
thing
to fix.

Original comment by sberlin on 9 May 2010 at 12:56

GoogleCodeExporter commented 9 years ago

Original comment by sberlin on 22 Feb 2011 at 1:52