Task #986 (closed)
Opened 16 years ago
Closed 11 years ago
Remove client/ and port tests. was: Build BiltzAdapterContext for using RMI interface over Ice
Reported by: | jamoore | Owned by: | jamoore |
---|---|---|---|
Priority: | critical | Milestone: | Unscheduled |
Component: | API | Version: | 3.0-M1 |
Keywords: | testing, adapter, blitz, rmi, jboss | Cc: | jburel, dzmacdonald, jrswedlow, cxallan |
Resources: | n.a. | Referenced By: | n.a. |
References: | n.a. | Remaining Time: | n.a. |
Sprint: | n.a. |
Description
In order to not have to re-write all 700+ integration test methods, it may be more time-efficient to write an adapter to allow the RMI-based tests to run over Ice.
This would work in that an interceptor would receive the call, transfer the ome.model.* objects to omero.model.* objects, make the call, and then do the reverse transformation with the return value.
There is obviously some performance overhead, but in the tests this is unimportant.
The question is if this would also be useful as a mid-term solution for porting between JBoss and Blitz. If this works well, JBoss could become an optional feature as early as milestone:3.0-Beta4, if WebAdmin? is ported to WeBlitz? (WebLitz?? OmeroWeblitz??)
Change History (8)
comment:1 Changed 15 years ago by jmoore
- Cc callan added
comment:2 Changed 15 years ago by jmoore
- Milestone changed from OMERO-Beta4 to OMERO-Beta4.1
Won't be happening for 4.0. Pushing to 4.1 where we can look at build/testing stuff.
comment:3 Changed 15 years ago by jmoore
- Summary changed from Build BiltzAdapterContext for using RMI interface over Ice to Remove client/ and port tests. was: Build BiltzAdapterContext for using RMI interface over Ice
comment:4 Changed 15 years ago by jmoore
- Priority changed from major to critical
Won't have time to port all the tests for 4.1. But as soon as we do a general cleanup, it should be considered.
comment:5 Changed 14 years ago by jmoore
- Type changed from User Story to Task
comment:6 Changed 13 years ago by cxallan
Referencing ticket #1736 has changed sprint.
comment:7 Changed 12 years ago by jmoore
Referencing ticket #1736 has changed sprint.
comment:8 Changed 11 years ago by jamoore
- Resolution set to invalid
- Status changed from new to closed
#1106 chose to not use this as a mid-term solution. The question now is whether or not it's worth it for preserving our components/client tests or not. The implementation is 60-70% finished, but the last bit could obviously be the hardest. Most likely it's better to just convert them all. Also makes for better examples.
Is this something we could all do together?