Hi Fawad,

> On 12 Jun 2019, at 18:37, Fawad Ali <m.fawaadal...@gmail.com> wrote:

[snip]

> I took a look at the application-releasenotes and what I understand is
> that there are sample demo pages instead of functional tests.

This is not 100% correct. There are both, and the demo pages are used in the 
functional tests.

* Demo pages: 
https://github.com/xwiki-contrib/application-releasenotes/tree/master/application-releasenotes-demo
* Func tests: 
https://github.com/xwiki-contrib/application-releasenotes/tree/master/application-releasenotes-test
* Demo dep in the func test module: 
https://github.com/xwiki-contrib/application-releasenotes/blob/master/application-releasenotes-test/application-releasenotes-test-tests/pom.xml#L43

Thanks
-Vincent

> I personally
> think our Interactive Maps Application aligns well with that approach and
> we can have the same type of tests/demos. WDYT?
> 
> I would start preparing for a release for now and implement the tests once
> we have coordinated on how we do it.
> 
> Best,
> Fawad
> 
> 
> On Fri, Jun 7, 2019 at 10:29 PM Vincent Massol <vinc...@massol.net> wrote:
> 
>> Hi,
>> 
>>> On 7 Jun 2019, at 18:59, Stéphane Laurière <slauri...@xwiki.com> wrote:
>>> 
>>> Fawad, Caty, all,
>>> 
>>> I have a short comment about the tests:
>>> 
>>>> Hi Caty,
>>>> Thanks for the review.
>>>>   Maps/MapTesting/Maps/TestMap - I find it strange that the Maps space
>> is duplicated
>>>> This space exists only for testing. It won't be there for the real
>> application. I named them so that its easier to know which type of object
>> pages are located in them (for myself).
>>>>   MapTesting, Maps, Points - spaces don't have homepages. The users
>> will navigate to them, since they are present in breadcrumb. So what is the
>> plan? Simpler paths? or create Homepages for these types of entry?
>>>> Since we are in the beta stage now, the whole MapTesting space exists
>> for testing for developers. It would not be there once we have a stable
>> version ready for release.
>>> 
>>> Actually this raises a question, all the more as we also discussed the
>> importance of having automated functional tests earlier today on #xwiki
>> with Vincent. For automated testing, we will need sample data, and I'm
>> wondering where we should store this sample data (and possible scripts or
>> code for obtaining it). How do other projects deal with test data in such a
>> context? Is the test data stored in the same repository or in a distinct
>> one? I was looking for some Solr application test data but could not find
>> it yet. Note that we may consider the testing area as a set of demos
>> instead in some way, couldn't we? It would make sense to keep it (just like
>> if it's real test data), and to provide a navigation across these pages as
>> you suggest it, Caty.
>> 
>> For the Release Notes app, I also have some data for the tests. See the
>> demo module in https://github.com/xwiki-contrib/application-releasenotes
>> 
>> Thanks
>> -Vincent
>> 
>>> 
>>> Cheers
>>> 
>>> Stéphane
>>> 
>>> 
>>>>   Lots of pages that are not hidden. All technical pages needs to be
>> hidden.
>>>> Again, these pages are not technical and exist only for testing
>> purposes.
>>>>   A bit confusing that there are 2 search boxes for the maps, see
>> https://up1.xwikisas.com/#XngcOAZexsKE4DryH2i6zA   Yes, thanks for
>> pointing out. We need to move the search function directly inside the map.
>> I will look into it once I am done with the facets.
>>> 
>>>> For the search input, its in extremely beta stage. I am still trying
>> to figure out the macros I have borrowed from SolrSearchMacros. So it will
>> take time for me to make a more stable version of the facets. I hope I can
>> do that in due time. :)
>>>>   Regarding the facets, we need some more user friendly translations
>> and customizations for this kind of UI, see
>> https://up1.xwikisas.com/#TYj_9oLn84Mfp87VnwNgkw As discussed earlier
>> with Stephane, we are still having issues using the normal $facetDisplayer
>> so we are using a workaround for testing purposes that's why it looks like
>> this. More precisely, we are using the
>> #displaySearchFacetValues($facetValues) macro for now.
>>>> Best,
>>>> Fawad
>>> 
>>> 
>> 
>> 

Reply via email to