Feature/fix unit tests for #5262 ("Remove location" button for new uploads)

[TomerPacific]

Description (required)
#5262 introduces a useful feature, but makes two unit tests fail.
The two units tests that are failing are:

• testOnClickModifyLocation
• testOnStyleLoaded

Fixes #5291

What changes did you make and why?

Added a variable to mock the newly added removeLocationButton

Tests performed (required)

Tested unit tests on an emulated device with API level 29

Screenshots (for UI changes only)
N/A

[nicolas-raoul]

Thanks a lot for fixing the unit tests!

I just performed more real-world tests, and realized latitude/longitude are not actually removed from the EXIF of the picture. Hopefully this will be fixed soon and I will be able to merge. :-)

[TomerPacific]

@nicolas-raoul - Do you mind adding the hacktoberfet-accepted label?
Also, would you like me to help out with the rest of this feature?

[nicolas-raoul]

@TomerPacific

I added the label.
Sure, that would be wonderful! Feel free to create another branch, and pull from this branch into it.

[TomerPacific]

@nicolas-raoul - Can you elaborate on what the statement you made means:

I just performed more real-world tests, and realized latitude/longitude are not actually removed from the EXIF of the picture.

What exactly did you try and what exactly did you see (and how)?

Thanks

[nicolas-raoul]

Sure!
I just added reproduction steps at #5262 (comment)

[nicolas-raoul]

Please do not hesitate to comment at #5262 (comment) 🙂