Issue 41 in tapestry5-cayenne: Integration tests are flawed

[tapestry...@googlecode.com]

Status: New
Owner: ----
Labels: Type-Defect Priority-Medium

New issue 41 by rfrovarp...@gmail.com: Integration tests are flawed
http://code.google.com/p/tapestry5-cayenne/issues/detail?id=41

Looking at the integration tests for CommitAfter I have discovered that the
page for that test is flawed. By using the Persist annotation on the object
being tested, Tapestry's persistence is actually being tested instead of
Cayenne persistence.

By having setupRender always attempt to load from the DB, you actually test
Cayenne instead of Tapestry.

Attachments:
commitafterpage.patch 1.2 KB

[tapestry...@googlecode.com]

Updates:
Status: Accepted

Comment #1 on issue 41 by robert.z...@gmail.com: Integration tests are
flawed
http://code.google.com/p/tapestry5-cayenne/issues/detail?id=41

(No comment was entered for this change.)

[tapestry...@googlecode.com]

Updates:
Status: Fixed

Comment #2 on issue 41 by robert.z...@gmail.com: Integration tests are
flawed
http://code.google.com/p/tapestry5-cayenne/issues/detail?id=41

You actually need to have both @Persist and have the entity loaded at each
render. Technically, you don't /need/ @Persist, but it simplifies the page
logic greatly. In any event, this is now fixed in trunk.