[SYNPY-1253] Check MD5 before upload to verify change in content

[BryanFauble]

Problem:

1. When there is a cache miss we never checked if the content of the file to be uploaded actually changed. As a result we would always upload the file to Synapse.

Solution:

1. If we need to upload a file to Synapse and the file already exists in Synapse and we got back an MD5 from Synapse compare the MD5 in Synapse to what the local file is - Is they are the same do not upload the file.

Testing:

1. Integration tests around this logic

[pep8speaks]

Hello @BryanFauble! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

• In the file synapseclient/client.py:

Line 1638:89: E501 line too long (99 > 88 characters)

Comment last updated at 2024-02-09 17:20:57 UTC

[BWMac]

Looks good! Just a few comments/questions

[sonarcloud]

Quality Gate passed

Issues
17 New issues

Measures
0 Security Hotspots
96.9% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

[thomasyu888]

🔥