Skip to content

Conversation

@ashishkumar468
Copy link
Collaborator

Description (required)

Added unit-test cases for FileMetadataUtils

What changes did you make and why?

Added unit-tests to cover all possible tags from preferences

@codecov-io
Copy link

Codecov Report

Merging #3253 into master will increase coverage by 0.05%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##           master   #3253      +/-   ##
=========================================
+ Coverage    6.15%    6.2%   +0.05%     
=========================================
  Files         270     270              
  Lines       11608   11608              
  Branches      906     906              
=========================================
+ Hits          714     720       +6     
+ Misses      10839   10833       -6     
  Partials       55      55
Impacted Files Coverage Δ
.../fr/free/nrw/commons/upload/FileMetadataUtils.java 72.72% <0%> (+54.54%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 88bb1d8...890a664. Read the comment docs.

@maskaravivek maskaravivek merged commit 01ed96b into commons-app:master Nov 30, 2019
@ashishkumar468 ashishkumar468 deleted the feature/filemetautils-test branch November 30, 2019 17:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants