#4514 recursive task policy data
Merged by tkopecek. Opened by mikem.
mikem/koji recursive-task-policy-data  into  master

Download 4514.patch

Fixes https://pagure.io/koji/issue/4513

pretty please pagure-ci rebuild

@julian8628

it looks like this helper is not limited to get a "parent" task, so we can make it more common or use Task.getTaskInfo instead?

it looks like this helper is not limited to get a "parent" task, so we can make it more common or use Task.getTaskInfo instead?

Ah. This helper arose simply to make it easier to mock for the unit tests. It is simply splitting out the query code that was previously embedded in make_task. Of course this function will work on any task, but the hard-coded query fields are particular to the this particular case.

That said, there's really not much gain in the reduced query fields here. We could use Task.getInfo instead.

1 new commit added

  • avoid specialized helper function

:thumbsup:

Metadata Update from @mikem:
- Pull-request tagged with: testing-custom

Metadata Update from @mikem:
- Pull-request untagged with: testing-custom
- Pull-request tagged with: testing-basic

Metadata Update from @mfilip:
- Pull-request tagged with: testing-done

Commit 076f600d fixes this pull-request

Pull-Request has been merged by tkopecek

Metadata