Re: MockWebApplication and WicketTester no longer derived from WebApplication

2006-11-08 Thread Frank Bille

The problem was that WicketTester in src/main depended on DummyApplication
from src/test. Eclipse can cope with it (which is pretty annoying) but it
doesn't really make sense since only the stuff in src/main goes into our
binaries. When compiles it correctly and will then result in a compiletime
error because DummyApplication is unknown to WicketTester.

Frank

On 11/8/06, Juergen Donnerstag [EMAIL PROTECTED] wrote:


I've seen the fix and I don't understand why. Please correct me if I'm
wrong but you just replaced DummyApplication.java with an inline
DummyWebApplication. What was the reason for the change?

Juergen

On 11/7/06, Frank Bille [EMAIL PROTECTED] wrote:
 Juergen I really don't hope you are working on it, because I have just
 committed the fix :S

 Frank

 On 11/7/06, Frank Bille [EMAIL PROTECTED] wrote:
 
  Right now WicketTester (src/main) uses DummyApplication (src/test).
  Juergen are you looking at that or should I.
 
  Frank
 
  On 11/6/06, Igor Vaynberg [EMAIL PROTECTED] wrote:
   just making sure you knew about it to save you some work :)
  
   -igor
  
   On 11/6/06, Juergen Donnerstag [EMAIL PROTECTED] wrote:
   
I didn't like the patch. It only merged MockWebApplication and
WebTester into a single big class. The author assumed it works,
but as
the patch didn't include the modification necessary to the
existing
junit test cases, ...
   
Juergen
   
On 11/6/06, Igor Vaynberg [EMAIL PROTECTED] wrote:
 didnt someone already submit a patch that does this?

 -igor


 On 11/6/06, Juergen Donnerstag [EMAIL PROTECTED]
wrote:
 
  Currently MockWebApplication and WicketTester are derived from
  WebApplication which requires us to copy  paste code from
  MyApplication to MyWicketTester which is kind of ugly. I
changed
  MockWebApplication to delegate to an application instead. It
seems
  to
  work fine but most xxxExpectedResult.html files must be
changed.
  Example
 
  a
 
   
 
href=/WicketTester/WicketTester?wicket:interface=:0:link::ILinkListener
  wicket:id=linktest/a
  ---
  a
 
   
  
 
href=/DummyApplication/DummyApplication?wicket:interface=:0:link::ILinkListener
  wicket:id=linktest/a
 
  I guess that is ok, but I want to check with you first. Any
  thought?
 
  Juergen
 


   
  
  
 





Re: MockWebApplication and WicketTester no longer derived from WebApplication

2006-11-07 Thread Frank Bille

Right now WicketTester (src/main) uses DummyApplication (src/test).
Juergen are you looking at that or should I.

Frank

On 11/6/06, Igor Vaynberg [EMAIL PROTECTED] wrote:

just making sure you knew about it to save you some work :)

-igor

On 11/6/06, Juergen Donnerstag [EMAIL PROTECTED] wrote:

 I didn't like the patch. It only merged MockWebApplication and
 WebTester into a single big class. The author assumed it works, but as
 the patch didn't include the modification necessary to the existing
 junit test cases, ...

 Juergen

 On 11/6/06, Igor Vaynberg [EMAIL PROTECTED] wrote:
  didnt someone already submit a patch that does this?
 
  -igor
 
 
  On 11/6/06, Juergen Donnerstag [EMAIL PROTECTED] wrote:
  
   Currently MockWebApplication and WicketTester are derived from
   WebApplication which requires us to copy  paste code from
   MyApplication to MyWicketTester which is kind of ugly. I changed
   MockWebApplication to delegate to an application instead. It seems to
   work fine but most xxxExpectedResult.html files must be changed.
   Example
  
   a
  
 href=/WicketTester/WicketTester?wicket:interface=:0:link::ILinkListener
   wicket:id=linktest/a
   ---
   a
  

href=/DummyApplication/DummyApplication?wicket:interface=:0:link::ILinkListener
   wicket:id=linktest/a
  
   I guess that is ok, but I want to check with you first. Any thought?
  
   Juergen
  
 
 





MockWebApplication and WicketTester no longer derived from WebApplication

2006-11-06 Thread Juergen Donnerstag

Currently MockWebApplication and WicketTester are derived from
WebApplication which requires us to copy  paste code from
MyApplication to MyWicketTester which is kind of ugly. I changed
MockWebApplication to delegate to an application instead. It seems to
work fine but most xxxExpectedResult.html files must be changed.
Example

a href=/WicketTester/WicketTester?wicket:interface=:0:link::ILinkListener
wicket:id=linktest/a
---
a 
href=/DummyApplication/DummyApplication?wicket:interface=:0:link::ILinkListener
wicket:id=linktest/a

I guess that is ok, but I want to check with you first. Any thought?

Juergen


Re: MockWebApplication and WicketTester no longer derived from WebApplication

2006-11-06 Thread Johan Compagner

if it does the same in the end. go ahead.

johan


On 11/6/06, Juergen Donnerstag [EMAIL PROTECTED] wrote:


Currently MockWebApplication and WicketTester are derived from
WebApplication which requires us to copy  paste code from
MyApplication to MyWicketTester which is kind of ugly. I changed
MockWebApplication to delegate to an application instead. It seems to
work fine but most xxxExpectedResult.html files must be changed.
Example

a
href=/WicketTester/WicketTester?wicket:interface=:0:link::ILinkListener
wicket:id=linktest/a
---
a
href=/DummyApplication/DummyApplication?wicket:interface=:0:link::ILinkListener
wicket:id=linktest/a

I guess that is ok, but I want to check with you first. Any thought?

Juergen



Re: MockWebApplication and WicketTester no longer derived from WebApplication

2006-11-06 Thread Igor Vaynberg

didnt someone already submit a patch that does this?

-igor


On 11/6/06, Juergen Donnerstag [EMAIL PROTECTED] wrote:


Currently MockWebApplication and WicketTester are derived from
WebApplication which requires us to copy  paste code from
MyApplication to MyWicketTester which is kind of ugly. I changed
MockWebApplication to delegate to an application instead. It seems to
work fine but most xxxExpectedResult.html files must be changed.
Example

a
href=/WicketTester/WicketTester?wicket:interface=:0:link::ILinkListener
wicket:id=linktest/a
---
a
href=/DummyApplication/DummyApplication?wicket:interface=:0:link::ILinkListener
wicket:id=linktest/a

I guess that is ok, but I want to check with you first. Any thought?

Juergen