Great.  Everything works now!

Sean

On 1/12/06, Bernd Bohmann <[EMAIL PROTECTED]> wrote:
> Hello Sean,
>
> I commit one missing change.
> The change was very easy you don't need a own MockObject for easymock :-)
>
>
> Have a nice day
>
> Bernd
>
>
> Sean Schofield schrieb:
> > Yes but it was causing all sorts of issues because I already applied
> > your earlier patch.  Do you have karma yet?  Go ahead and update my
> > latest commits.  If you don't have karma yet I can apply another
> > patch.
> >
> > Also there are still a few errors (not failures) in the commons tests.
> >  Maybe your latest patch solves this?
> >
> > Sean
> >
> > On 1/12/06, Bernd Bohmann <[EMAIL PROTECTED]> wrote:
> >
> >>Where is the MockFacesContext used?
> >>
> >>In MessageUtilTest MockFacesContext is not needed, have you received my
> >>last patch for commons?
> >>
> >>Bernd
> >>
> >>Sean Schofield schrieb:
> >>
> >>>I'm trying to run Bruno's patches which fix the unit tests (nice work
> >>>by the way!)  Anyways, one of the tests in commons requires
> >>>MockFacesContext which is in api.  The issue is that it can't find
> >>>MockFacesContext even with the commons-api dependency.  This is
> >>>because the mock stuff is in the *test* resources and so is not made
> >>>available to other modules with a dependency on api.
> >>>
> >>>The only solution I can think of is to move the MockFacesContext to
> >>>commons but that introduces a circular dependency issue.  We could
> >>>have two copies of the mock object I suppose ... (or one of them could
> >>>be a private inner class.)
> >>>
> >>>Ideas?
> >>>
> >>>Sean
> >>>
> >>
> >>--
> >>Dipl.-Ing. Bernd Bohmann - Atanion GmbH - Software Development
> >>Bismarckstr. 13, 26122 Oldenburg, http://www.atanion.com
> >>phone: +49 441 4082312, mobile: +49 173 8839471, fax: +49 441 4082333
> >>
> >
> >
>
> --
> Dipl.-Ing. Bernd Bohmann - Atanion GmbH - Software Development
> Bismarckstr. 13, 26122 Oldenburg, http://www.atanion.com
> phone: +49 441 4082312, mobile: +49 173 8839471, fax: +49 441 4082333
>

Reply via email to