Open GoogleCodeExporter opened 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
[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:
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
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:
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
Original comment by sberlin
on 22 Feb 2011 at 1:52
Original issue reported on code.google.com by
mccu...@gmail.com
on 31 Oct 2009 at 5:58Attachments: