Feat/amit test

[shachargiladi]

Proposed changes

Issue(s)

How to test or reproduce

Screenshots

Types of changes

• [ ] Bugfix (non-breaking change which fixes an issue)
• [ ] Improvement (non-breaking change which improves a current function)
• [ ] New feature (non-breaking change which adds functionality)
• [ ] Documentation update (if none of the other choices apply)

Checklist

• [ ] I have read the CONTRIBUTING doc
• [ ] I have signed the CLA
• [ ] Lint and unit tests pass locally with my changes
• [ ] I have added tests that prove my fix is effective or that my feature works (if applicable)
• [ ] I have added necessary documentation (if applicable)
• [ ] Any dependent changes have been merged and published in downstream modules

Further comments

[CLAassistant]

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}

[lgtm-com[bot]]

This pull request introduces 5 alerts when merging 567408ee45f9dc02d13ef5da0ffe3bdbbf4fb85a into 991d7156a9b2893e41e1ba8ab0f9916f1cc25746 - view on LGTM.com

new alerts:

• 5 for Unused variable, import, function or class

[lgtm-com[bot]]

This pull request introduces 8 alerts when merging 79963f0fe2528f6c5239deafa501f857fae8fd8f into 991d7156a9b2893e41e1ba8ab0f9916f1cc25746 - view on LGTM.com

new alerts:

• 8 for Unused variable, import, function or class