Fixed card duplication issue #2349

[matuspetro]

1. Removed condition that blocked image comparison
2. Changed comparison logic from ID-based to cardID-based to prevent duplicates on repeated import

[TheLastProject]

Added unit tests for Utils class

These tests don't seem relevant to this PR. Can you please split that into another PR, to keep this one focused on the change it is supposed to fix? That makes reviewing much easier.

[matuspetro]

@TheLastProject I removed tests from this PR.

[TheLastProject]

I'm a bit confused, you seem to just be checking if the existing and to-import card both have images and the same ID? That doesn't sound like it fixes images being detected as different when they aren't. Instead, you're now most likely overwriting images which aren't the same, which would be way worse! Unless I'm misreading your change somehow?

[matuspetro]

I tested every scenario I could think of. If you try it and find a bug, please let me know and I will fix it.

[matuspetro]

@TheLastProject here is a video of testing:
https://drive.google.com/file/d/1hFreqeMK3JsWd-ScrqaBVxhjhVKeZTM2/view?usp=sharing