You are viewing a plain text version of this content. The canonical link for it is here.
Posted to derby-dev@db.apache.org by "ASF subversion and git services (JIRA)" <ji...@apache.org> on 2014/06/25 09:58:24 UTC
[jira] [Commented] (DERBY-6634) Improve test coverage of
SqlXmlUtil.java
[ https://issues.apache.org/jira/browse/DERBY-6634?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14043156#comment-14043156 ]
ASF subversion and git services commented on DERBY-6634:
--------------------------------------------------------
Commit 1605285 from [~knutanders] in branch 'code/trunk'
[ https://svn.apache.org/r1605285 ]
DERBY-6634: Improve test coverage of SqlXmlUtil.java
Remove dead code from the time when SqlXmlUtil implemented the
Formatable interface (before DERBY-3870).
> Improve test coverage of SqlXmlUtil.java
> ----------------------------------------
>
> Key: DERBY-6634
> URL: https://issues.apache.org/jira/browse/DERBY-6634
> Project: Derby
> Issue Type: Improvement
> Components: Test
> Affects Versions: 10.11.0.0
> Reporter: Knut Anders Hatlen
> Assignee: Knut Anders Hatlen
> Priority: Minor
> Attachments: d6634-1a-dead-code.diff
>
>
> The test coverage report for SqlXmlUtil indicates that we don't have tests for:
> - recompilation of XML queries used in triggers (the code looks dead, and can probably be removed)
> - serialization of sequences of atomic values returned by an XPath query (as far as I can see, we only support single atomic values and sequences of documents, so probably that's also code that can be removed, possibly replaced by an assert)
> - XMLEXISTS operations where the XPath query returns a scalar (after DERBY-6624, it will take the same code path as a query that returns a non-empty sequence, so it won't show up in the report anymore, but it's still useful to have a test case for it)
> - XMLQUERY operations where the XPath query returns a boolean (after DERBY-6624, the same code path will be taken for all data types, so it won't show up in the report anymore, but it's still useful to have a test case for it)
--
This message was sent by Atlassian JIRA
(v6.2#6252)