gchq / Gaffer

A large-scale entity and relation database supporting aggregation of properties
Apache License 2.0
1.77k stars 353 forks source link

Add additional tests around merging schemas with intersections #2462

Closed t92549 closed 3 years ago

t92549 commented 3 years ago

2449 Allowed merging of schemas that overlap each other. More tests should be done around edge cases with aggregation to confirm we are happy with the behaviour before release 1.18.0.

Examples tests:

t92549 commented 3 years ago

Also improve variable names such as a: https://github.com/gchq/Gaffer/blob/768f83f392dd57ff4d27c537fe3612a866db26b7/store-implementation/federated-store/src/test/java/uk/gov/gchq/gaffer/federatedstore/FederatedStoreSchemaTest.java#L131