TRUNK-5065: Add tests to AdministrationServiceImpl

[PKatGITHUB]

Description of what I changed

Issue I worked on

see https://issues.openmrs.org/browse/TRUNK-5065

Checklist: I completed these to help reviewers :)

• My pull request only contains ONE single commit
  (the number above, next to the 'Commits' tab is 1).

  No? -> read here on how to squash multiple commits into one

• My IDE is configured to follow the code style of this project.

  No? Unsure? -> configure your IDE, format the code and add the changes with git add . && git commit --amend

• I have added tests to cover my changes. (If you refactored
  existing code that was well tested you do not have to add tests)

  No? -> write tests and add them to this commit git add . && git commit --amend

• I ran mvn clean package right before creating this pull request and
  added all formatting changes to my commit.

  No? -> execute above command

• All new and existing tests passed.

  No? -> figure out why and add the fix to your commit. It is your responsibility to make sure your code works.

• My pull request is based on the latest changes of the master branch.

  No? Unsure? -> execute command git pull --rebase upstream master

[coveralls]

Coverage decreased (-0.08%) to 58.733% when pulling 31e64b3 on PKatGITHUB:TRUNK-5065 into bbca156 on openmrs:master.

[teleivo]

@PKatGITHUB awesome!!!

[PKatGITHUB]

Thank you @teleivo !!!

[PKatGITHUB]

@teleivo what next :) ?

[teleivo]

@PKatGITHUB I cleaned AdministrationServiceTest up a bit so tests should be bit easier to read now.
I added one test case for saveGlobalProperty when saving the global property for
GLOBAL_PROPERTY_LOCALE_ALLOWED_LIST

see
dc33df6

you could now add the case for

the method saveGlobalProperty when saving the global property for GLOBAL_PROPERTY_DEFAULT_LOCALE fails using this style/template :)

[PKatGITHUB]

Hey @teleivo thanks for all your support :)
I have created one more PR. Have a look at #2650