feat: Add support for inserting only for DatastoreOperations

[cfranzen]

Implemented insert() and insertAll() on DatastoreOperations interface and correspondingly on DatastoreTemplate.

Closes #1024

[conventional-commit-lint-gcf]

🤖 I detect that the PR title and the commit message differ and there's only one commit. To use the PR title for the commit history, you can use Github's automerge feature with squashing, or use automerge label. Good luck human!

-- conventional-commit-lint bot
https://conventionalcommits.org/

[meltsufin]

Thanks for the contribution. It looks good overall, but there is a lot of code duplication with the existing save() method. Can you please improve that part?

[cfranzen]

Thanks for the contribution. It looks good overall, but there is a lot of code duplication with the existing save() method. Can you please improve that part?

Thanks for the review @meltsufin. I tried to apply all your suggestions to my change, especially reducing the code duplication. I am not completely happy with my solution for the tests: using parameterized tests and passing SAVE or INSERT as string. It works and duplicates nearly no code, but it is not that clean. Let me know when you have a better solution for that.

[meltsufin]

Thanks for the contribution. It looks good overall, but there is a lot of code duplication with the existing save() method. Can you please improve that part?

Thanks for the review @meltsufin. I tried to apply all your suggestions to my change, especially reducing the code duplication. I am not completely happy with my solution for the tests: using parameterized tests and passing SAVE or INSERT as string. It works and duplicates nearly no code, but it is not that clean. Let me know when you have a better solution for that.

Thanks for making the deduplication changes. It looks much better. I can't really think of a better way to clean up the tests, other than using an enum instead of the strings for @ValueSource. Let's do that and remove the unused method I pointed out, and this PR should be good to go!

[cfranzen]

Thanks for the contribution. It looks good overall, but there is a lot of code duplication with the existing save() method. Can you please improve that part?

Thanks for the review @meltsufin. I tried to apply all your suggestions to my change, especially reducing the code duplication. I am not completely happy with my solution for the tests: using parameterized tests and passing SAVE or INSERT as string. It works and duplicates nearly no code, but it is not that clean. Let me know when you have a better solution for that.

Thanks for making the deduplication changes. It looks much better. I can't really think of a better way to clean up the tests, other than using an enum instead of the strings for @ValueSource. Let's do that and remove the unused method I pointed out, and this PR should be good to go!

Good catch! Did not see the unused method. Test is using an enum instead of strings now.

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

85.0% Coverage
0.0% Duplication

[cfranzen]

@meltsufin Thanks for merging to main. Would it be possible to also merge my changes to the 3.x branch?

[meltsufin]

@meltsufin Thanks for merging to main. Would it be possible to also merge my changes to the 3.x branch?

Sure, I opened #1766.