← Back to team overview

zorba-coders team mailing list archive

Re: [Merge] lp:~zorba-coders/zorba/zorba_FOTS_driver into lp:zorba

 

Review: Needs Fixing

The implementation looks very clean. I poked around a bit and ran several random test sets.
The feeling I got is that it's not quite ready yet for merging. I would like to avoid waisting time when more people start using it and all run into the same issues.

Before we continue, I would like to understand why the following two problems exist. Especially the first one seems to be a candidate to fix many more tests.

- assert-serialization doesn't not seem to be implemented. Should it? I poked around and found many tests failing because of this (e.g. CondExpr002)
- It seems that all of the prod-OrderByClause tests fail. It's not clear whether this is a bug in my setting, the driver, or Zorba itself.

Some minor issues are:
- I find it very hard to read the output of a test run but maybe this will change if we have the stylesheet
- Rename fots_driver to fots
- We generally use xq instead of xqy for xquery files. Maybe we should do this here to.
- Rename zorba-fots-driver.xqy to fots-driver.xq
- How do I know whether my test run actually ran any tests? Sometimes I had a typo in the prefix and it didn't run anything but it's hard to see.
- Does the dependency output have to appear in every run?
-- 
https://code.launchpad.net/~zorba-coders/zorba/zorba_FOTS_driver/+merge/132980
Your team Zorba Coders is subscribed to branch lp:zorba.


Follow ups

References