Update UploadFileRemoteOperation.java to get access to etag from resp…

[vince-bourgmayer]

…onse header

• Add a private static final value which is the response header's name for etag.
• update 'RemoteOperationResult uploadFile(OwnCloudClient client)' to check for this header, and set value as RemoteOperationResult's data.

[codecov]

Codecov Report

Merging #870 (01f0cb2) into master (f8105d3) will increase coverage by 0.03%.
The diff coverage is 37.50%.

@@            Coverage Diff             @@
##           master     #870      +/-   ##
==========================================
+ Coverage   48.09%   48.13%   +0.03%     
==========================================
  Files         168      168              
  Lines        6481     6484       +3     
  Branches      846      847       +1     
==========================================
+ Hits         3117     3121       +4     
+ Misses       2908     2907       -1     
  Partials      456      456              

Impacted Files	Coverage Δ
...lib/resources/files/UploadFileRemoteOperation.java	52.08% <37.50%> (+0.47%)	⬆️
...wncloud/android/lib/resources/files/FileUtils.java	85.71% <0.00%> (+14.28%)	⬆️

[AlvaroBrey]

Looks fine, but since the RemoteOperationResult has result data, it should be parameterized.

For this:

• UploadFileRemoteOperation should extend RemoteOperation<String> rather than the raw type
• Variables and return types should be RemoteOperationResult<String> (or RemoteOperationResult<> where applicable) rather than the raw type.

[AlvaroBrey]

@vince-bourgmayer Please fix DCO (https://github.com/nextcloud/android-library/pull/870/checks?check_run_id=6127967707) and this will be ready to merge

[vince-bourgmayer]

@vince-bourgmayer Please fix DCO (https://github.com/nextcloud/android-library/pull/870/checks?check_run_id=6127967707) and this will be ready to merge

@AlvaroBrey DCO has been fixed

[nextcloud-android-bot]

SpotBugs (new)

Warning Type	Number
Bad practice Warnings	11
Correctness Warnings	36
Internationalization Warnings	6
Malicious code vulnerability Warnings	54
Multithreaded correctness Warnings	3
Performance Warnings	14
Security Warnings	1
Dodgy code Warnings	45
Total	170

SpotBugs (master)

Warning Type	Number
Bad practice Warnings	11
Correctness Warnings	36
Internationalization Warnings	6
Malicious code vulnerability Warnings	54
Multithreaded correctness Warnings	3
Performance Warnings	14
Security Warnings	1
Dodgy code Warnings	45
Total	170

[AlvaroBrey]

Thank you @vince-bourgmayer !