Branch: refs/heads/master Home: https://github.com/openSUSE/open-build-service Commit: 0cd05eeea172fe7f650d37275fedf0846ce874af https://github.com/openSUSE/open-build-service/commit/0cd05eeea172fe7f650d37... Author: Eduardo Navarro <enavarro@suse.com> Date: 2022-07-11 (Mon, 11 Jul 2022) Changed paths: M src/api/app/controllers/source_attribute_controller.rb M src/api/app/models/attrib.rb M src/api/app/models/project/embargo_handler.rb M src/api/spec/models/attrib_spec.rb M src/api/spec/models/project/embargo_handler_spec.rb M src/api/test/functional/maintenance_test.rb Log Message: ----------- Introduce validation of EmbargoDate attribute This means it will be no longer possible to introduce an invalid EmbargoDate attribute. Anyway, the step of checking for an embargo date when releasing packages still handles the case of invalid embargo dates, which were stored previously in the database. This PR doesn't contain any data migration to modify embargo dates according to the new validation rules. As a consequence of introducing this validation in the `Attrib` model, some specs were modified to make use of invalid `Attrib` objects. Regarding maintenance minitests, as the case of invalid EmbargoDate attributes is already tested in specs, the minitests were adapted to handle the case of valid EmbargoDate attributes. Commit: 8fee8bff3f4c0a46b217b430d1b8fb5c13182ca5 https://github.com/openSUSE/open-build-service/commit/8fee8bff3f4c0a46b217b4... Author: Eduardo J <enavarro@suse.com> Date: 2022-07-12 (Tue, 12 Jul 2022) Changed paths: M src/api/app/controllers/source_attribute_controller.rb M src/api/app/models/attrib.rb M src/api/app/models/project/embargo_handler.rb M src/api/spec/models/attrib_spec.rb M src/api/spec/models/project/embargo_handler_spec.rb M src/api/test/functional/maintenance_test.rb Log Message: ----------- Merge pull request #12736 from eduardoj/add_validation_to_attributes Introduce validation for EmbargoDate attribute Compare: https://github.com/openSUSE/open-build-service/compare/60ea5cb27247...8fee8b...