Project

General

Profile

Actions

Bug #15851

closed

[controller] returning 'null' instead of '[]' for empty result lists

Added by Peter Amstutz about 3 years ago. Updated about 3 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
-
Target version:
Start date:
11/21/2019
Due date:
% Done:

100%

Estimated time:
(Total: 0.00 h)
Story points:
-
Release relationship:
Auto

Description

federation is breaking here 22:42:23 /var/lib/cwl/stg7a0e7345-aadf-4b47-9dc1-253daca1b104/prepare.py
22:42:26 Traceback (most recent call last):
22:42:26 File "/var/lib/cwl/stg7a0e7345-aadf-4b47-9dc1-253daca1b104/prepare.py", line 32, in <module>
22:42:26 for c in cols["items"]:
22:42:26 TypeError: 'NoneType' object is not iterable

(09:22:33 AM) Fernando Monserrat: and run-cwl-test-xxx here 22:42:35 """, category=CWLToolDeprecationWarning)
22:42:35 ERROR Workflow error, try again with --debug for more information:
22:42:35 'items'

(09:24:34 AM) tetron: did we turn on the EnableBetaController around that time?

(09:31:33 AM) ward: tetron: Nov 6 17:06 UTC is when we set EnableBetaController14287: true on our dev clusters


Subtasks 1 (0 open1 closed)

Task #15859: Review 15851-empty-items-arrayResolvedPeter Amstutz11/21/2019

Actions
Actions #1

Updated by Peter Amstutz about 3 years ago

  • Description updated (diff)
Actions #2

Updated by Tom Clegg about 3 years ago

  • Assigned To set to Tom Clegg
Actions #5

Updated by Tom Clegg about 3 years ago

  • Subject changed from [controller] Possibly returning 'null' instead of '[]' for empty result lists to [controller] returning 'null' instead of '[]' for empty result lists
  • Status changed from New to In Progress
Actions #7

Updated by Peter Amstutz about 3 years ago

Tom Clegg wrote:

15851-empty-items-array @ 92d7d779d4f0415d09f536d286972bd953e7b566 -- https://ci.curoverse.com/view/Developer/job/developer-run-tests/1660/

Confirmed with manual test arv collection list --filters '[["uuid", "in", []]]' and switching between branches, LGTM.

Actions #8

Updated by Tom Clegg about 3 years ago

  • Status changed from In Progress to Resolved
  • % Done changed from 0 to 100
Actions #9

Updated by Peter Amstutz about 3 years ago

  • Release set to 22
Actions

Also available in: Atom PDF