Project

General

Profile

Actions

Bug #16377

closed

Workflow fails when keep references in `defaults` are inaccessible even if not used by actual workflow.

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

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
CWL
Target version:
Story points:
-
Release relationship:
Auto

Subtasks 1 (0 open1 closed)

Task #16549: Review 16377-missing-defaultResolvedPeter Amstutz07/13/2020Actions
Actions #1

Updated by Peter Amstutz almost 4 years ago

  • Target version changed from 2020-06-03 Sprint to 2020-06-17 Sprint
Actions #2

Updated by Peter Amstutz almost 4 years ago

  • Target version changed from 2020-06-17 Sprint to 2020-07-01 Sprint
Actions #3

Updated by Peter Amstutz almost 4 years ago

  • Assigned To set to Peter Amstutz
Actions #4

Updated by Peter Amstutz almost 4 years ago

  • Category set to CWL
Actions #5

Updated by Peter Amstutz almost 4 years ago

  • Status changed from New to In Progress
Actions #6

Updated by Peter Amstutz almost 4 years ago

  • Target version changed from 2020-07-01 Sprint to 2020-07-15
Actions #7

Updated by Peter Amstutz almost 4 years ago

16377-missing-default @ 9c5923ce6ac1aa71d9a5ea72b4daffa129eef30b

developer-run-tests: #1954

It turns out this was already supposed to be handled, the bug was that exists() wasn't catching IOError(ENOENT) and turning it into "return False" as intended.

Actions #10

Updated by Peter Amstutz almost 4 years ago

  • Target version changed from 2020-07-15 to 2020-08-12 Sprint
Actions #11

Updated by Lucas Di Pentima almost 4 years ago

The fix looks good, but just in case, I dropped the fix, and rebased the tests to current master on a new branch, so I could run them on and see them fail. (ff0886642 - branch: 16377-test-check)
They didn't fail: developer-run-tests-remainder: #2045

Actions #12

Updated by Peter Amstutz almost 4 years ago

  • Status changed from In Progress to Resolved
Actions #13

Updated by Peter Amstutz over 3 years ago

  • Release set to 25
Actions

Also available in: Atom PDF