Qi4j

Qi4j tests do not take @Queryable(false) into account

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 1.2
  • Fix Version/s: 1.2
  • Component/s: Indexing
  • Labels:
    None
  • Environment:
    Any.

Description

Currently org.qi4j.test.indexing.model.entities.CatEntity is marked as @Queryable(false). However, org.qi4j.test.indexing.AbstractEntityFinderTest and org.qi4j.test.indexing.AbstractQueryTest expect CatEntity to be returned with results, which causes test errors in working implementations of indexing.

I've created a patch (attachment) for this which should make the tests take non-queryability of CatEntity into account.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Stanislav Muhametsin added a comment -

I added a patch where @Queryable annotation will get read also from super-types. This caused configuration composites to be indexed.

Show
Stanislav Muhametsin added a comment - I added a patch where @Queryable annotation will get read also from super-types. This caused configuration composites to be indexed.
Hide
Permalink
Rickard Öberg added a comment -

Applied patches

Show
Rickard Öberg added a comment - Applied patches

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: