Thread: xml2: add test for coverage

xml2: add test for coverage

From
Dong Wook Lee
Date:
Hi, hackers
I made a small patch for xml2 to improve test coverage.
However, there was a problem using the functions below.

- xpath_number
- xpath_bool
- xpath_nodeset
- xpath_list

Do you have any advice on how to use this function correctly?
It would also be good to add an example of using the function to the document.

---
Regards,
DongWook Lee.

Attachment

Re: xml2: add test for coverage

From
Peter Eisentraut
Date:
On 23.08.22 03:38, Dong Wook Lee wrote:
> I made a small patch for xml2 to improve test coverage.
> However, there was a problem using the functions below.
> 
> - xpath_number
> - xpath_bool
> - xpath_nodeset
> - xpath_list
> 
> Do you have any advice on how to use this function correctly?
> It would also be good to add an example of using the function to the document.

I can confirm that these functions could use more tests and more 
documentation and examples.  But given that you registered a patch in 
the commit fest, it should be you who provides a patch to solve those 
issues.  Are you still working on this, or were you just looking for 
help on how to solve this?




Re: xml2: add test for coverage

From
vignesh C
Date:
On Fri, 25 Nov 2022 at 18:08, Peter Eisentraut
<peter.eisentraut@enterprisedb.com> wrote:
>
> On 23.08.22 03:38, Dong Wook Lee wrote:
> > I made a small patch for xml2 to improve test coverage.
> > However, there was a problem using the functions below.
> >
> > - xpath_number
> > - xpath_bool
> > - xpath_nodeset
> > - xpath_list
> >
> > Do you have any advice on how to use this function correctly?
> > It would also be good to add an example of using the function to the document.
>
> I can confirm that these functions could use more tests and more
> documentation and examples.  But given that you registered a patch in
> the commit fest, it should be you who provides a patch to solve those
> issues.  Are you still working on this, or were you just looking for
> help on how to solve this?

Hi DongWook Lee,

Are you planning to work on this and provide an updated patch, if you
are not planning to work on it, we can update the commitfest entry
accordingly.

Regards,
Vignesh



Re: xml2: add test for coverage

From
vignesh C
Date:
On Tue, 17 Jan 2023 at 17:06, vignesh C <vignesh21@gmail.com> wrote:
>
> On Fri, 25 Nov 2022 at 18:08, Peter Eisentraut
> <peter.eisentraut@enterprisedb.com> wrote:
> >
> > On 23.08.22 03:38, Dong Wook Lee wrote:
> > > I made a small patch for xml2 to improve test coverage.
> > > However, there was a problem using the functions below.
> > >
> > > - xpath_number
> > > - xpath_bool
> > > - xpath_nodeset
> > > - xpath_list
> > >
> > > Do you have any advice on how to use this function correctly?
> > > It would also be good to add an example of using the function to the document.
> >
> > I can confirm that these functions could use more tests and more
> > documentation and examples.  But given that you registered a patch in
> > the commit fest, it should be you who provides a patch to solve those
> > issues.  Are you still working on this, or were you just looking for
> > help on how to solve this?
>
> Hi DongWook Lee,
>
> Are you planning to work on this and provide an updated patch, if you
> are not planning to work on it, we can update the commitfest entry
> accordingly.

There has been no updates on this thread for some time, so this has
been switched as Returned with Feedback. Feel free to open it in the
next commitfest if you plan to continue on this.

Regards,
Vignesh