Please cherry-pick:

- private methods should not have exceptions thrown because these cases cannot
  be unit-tested (or very hard)
- normally only public, package-level or protected methods throw exceptions
  and have made sure that e.g. null-references won't get through to private
  methods

Signed-off-by: Roland Häder <roland@mxchange.org>

diff --git a/src/java/org/mxchange/jcountry/model/data/JobsAdminCountrySingletonBean.java b/src/java/org/mxchange/jcountry/model/data/JobsAdminCountrySingletonBean.java
index c23bb568262460a50b94bce9c8d40d83e7fa309c..37ec1ad98b51dedcac048c4416f913078244d8a8 100644 (file)
--- a/src/java/org/mxchange/jcountry/model/data/JobsAdminCountrySingletonBean.java
+++ b/src/java/org/mxchange/jcountry/model/data/JobsAdminCountrySingletonBean.java
@@ -102,20 +102,6 @@ public class JobsAdminCountrySingletonBean extends BaseJobsEnterpriseBean implem
         * @return Whether the country was found
         */
        private boolean isCountryAdded (final Country country) {
-               if (null == country) {
-                       // Throw NPE
-                       throw new NullPointerException("country is null"); //NOI18N
-               } else if (country.getCountryCode().isEmpty()) {
-                       // Code is not set
-                       throw new IllegalArgumentException("country.countryCode is empty"); //NOI18N
-               } else if (country.getCountryI18nKey().isEmpty()) {
-                       // I18n key is not set
-                       throw new IllegalArgumentException("country.countryI18nKey is empty"); //NOI18N
-               } else if (country.getCountryId() != null) {
-                       // Should be null
-                       throw new IllegalArgumentException(MessageFormat.format("country.countryId is not null ({0})", country.getCountryId())); //NOI18N
-               }
-
                // Default is not found
                boolean isAdded = false;