diff --git a/ckanext/odsh/tests_tpsh/test_validation.py b/ckanext/odsh/tests_tpsh/test_validation.py index d4273be8be748a4118003965756abb7d7d4787ab..2e3c5e7e44a9e6aee86bb62fb09bd50ed04082d7 100644 --- a/ckanext/odsh/tests_tpsh/test_validation.py +++ b/ckanext/odsh/tests_tpsh/test_validation.py @@ -63,15 +63,12 @@ class Test_validate_licenseAttributionByText(WithFrontendValidationMocks): data_mock.update({ ('license_id',): u'http://dcat-ap.de/def/licenses/dl-zero-de/2.0', }) - with nt.assert_raises(toolkit.Invalid) as err: - validate_licenseAttributionByText( - self.key_mock, - data_mock, - self.error_mock, - self.context_mock - ) - nt.assert_equal( - err.exception.error, 'licenseAttributionByText: text not allowed for this license') + validate_licenseAttributionByText( + self.key_mock, + data_mock, + self.error_mock, + self.context_mock + ) class Test_validate_extra_groupsFrontend(WithFrontendValidationMocks): diff --git a/ckanext/odsh/validation.py b/ckanext/odsh/validation.py index 54361f32d6f7c4420c2e7d1216e56582dfb6c601..5817fa644394f2f81254d67b66149493af1f065d 100644 --- a/ckanext/odsh/validation.py +++ b/ckanext/odsh/validation.py @@ -166,10 +166,6 @@ def validate_licenseAttributionByText(key, data, errors, context): raise toolkit.Invalid( 'licenseAttributionByText: empty not allowed') - if (not isByLicense) and hasAttribution: - raise toolkit.Invalid( - 'licenseAttributionByText: text not allowed for this license') - def _isByLicense(data): register = model.Package.get_license_register()