Hi @isaac-mercieca, thanks for the PR! Could you, please, add unit tests to cover the fix?

Codecov Report

Attention: Patch coverage is 58.33333% with 10 lines in your changes missing coverage. Please review.

Project coverage is 76.01%. Comparing base (49cebe5) to head (49a98c2).
Report is 167 commits behind head on master.

@@             Coverage Diff              @@
##             master     #367      +/-   ##
============================================
- Coverage     76.10%   76.01%   -0.10%     
- Complexity     1133     1136       +3     
============================================
  Files            75       75              
  Lines          5081     5103      +22     
  Branches        732      732              
============================================
+ Hits           3867     3879      +12     
- Misses          890      899       +9     
- Partials        324      325       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@igdianov I have added a test under BooksSchemaBuildTest to validate that the generated GraphQL schema includes a nested embeddable. While this validates the solution from a schema generation perspective which was initially failing, I do not know if an integration test should also be included. Seeing existing tests I noticed a test for EmbeddableIds which was approached differently from the rest.

@isaac-mercieca The schema test looks good. Please, include an integration test with the query using embedded attributes in the where criteria and selection. This will bring the required test coverage threshold to pass codecov checks