Fix import for aggregated models

[ottaviano]

Q	A
Bug fix?	yes
New feature?	no
BC breaks?	no
Related Issue	Fix #337
Need Doc update	no

Describe your change

This PR fix the import command which actually skips aggregated entities (Aggregator model).

[chloelbn]

Hey @ottaviano !

Thanks a lot for your contribution. Would you mind adding some tests to your PR please?

[ottaviano]

Hi @chloelbn, sure, but it seems that the tests depend on real Algolia backend? How can I run the test on local without real ALGOLIA_APP_ID and ALGOLIA_API_KEY?

[ottaviano]

@chloelbn, is there a way to run the tests on local without a real Algolia API keys?

[chloelbn]

Hi @ottaviano, you can chose to run only the tests you created and not the whole test suite, so it won't eat up your units in your account. It's not ideal but shouldn't be an extreme rise in your operations. If you don't have an Algolia account, you can create a free one with a small dataset here. Don't hesitate to ping me again if it's not clear!

[wlasnapl]

Do you know when this will be merged and released?

[keichinger]

@chloelbn we're running into the same issue/bug. How can we help getting this PR merged and released?

[chloelbn]

Hi @keichinger, thanks for your offer to help! You can write tests to make this move forward :)

[ottaviano]

Hi @keichinger, sorry for the wait :(

@chloelbn, I saw that there is already a test for the command search:index and I improve it to really test the case of aggregation.
Could you review, please?

[keichinger]

@ottaviano no worries! Anyone could have helped you out of there :) Especially during the holidays.

I'm glad that you've found the time to finish up this PR ☺️ I was planning on having a closer look at this PR tomorrow in the morning :) Guess that's no longer needed.