Branch: refs/heads/master
Home: https://github.com/openSUSE/open-build-service
Commit: 8e0e02940e544fbf02d0debdd54cedcf89898f0f
https://github.com/openSUSE/open-build-service/commit/8e0e02940e544fbf02d0de...
Author: Björn Geuken
Date: 2018-04-16 (Mon, 16 Apr 2018)
Changed paths:
M src/api/spec/cassettes/Webui_RequestController/POST_changerequest/with_valid_parameters/forwards.yml
M src/api/spec/controllers/webui/request_controller_spec.rb
Log Message:
-----------
[frontend] Fix test for forwarding bs requests
This test was failing with a MissingAction exception when the VCR cassettes
where not used. To fix this we had to ensure that the target package of
the forwarded request has different source content than the original
package.
In our test suite this test was still passing because of the recorded
cassettes.
Commit: 3f16463cbd2c8c03822c64c8a1554d6d1a57971e
https://github.com/openSUSE/open-build-service/commit/3f16463cbd2c8c03822c64...
Author: Björn Geuken
Date: 2018-04-16 (Mon, 16 Apr 2018)
Changed paths:
M src/api/app/controllers/webui/request_controller.rb
Log Message:
-----------
[frontend] Fix tracking of debug data for failed request forwarding
There were a few problems here:
* The 'action' variable has to be defined outside of the transaction.
Otherwise it would never be defined when catching exceptions.
* Since we might have multiple actions, it's not enough to track
only one of them.
* The validation message of the bs request already includes validation
messages of associated request actions.
Commit: f4ca0a179982d995cf38f326de8a3d68be460db3
https://github.com/openSUSE/open-build-service/commit/f4ca0a179982d995cf38f3...
Author: Christian Bruckmayer
Date: 2018-04-17 (Tue, 17 Apr 2018)
Changed paths:
M src/api/app/controllers/webui/request_controller.rb
M src/api/spec/cassettes/Webui_RequestController/POST_changerequest/with_valid_parameters/forwards.yml
M src/api/spec/controllers/webui/request_controller_spec.rb
Log Message:
-----------
Merge pull request #4869 from bgeuken/update_test_for_branch_forwarding
[frontend] Fix test for forwarding bs requests
Compare: https://github.com/openSUSE/open-build-service/compare/b19bfacb72d7...f4ca0a...