Ticket #9929 (closed: fixed)

Opened 6 years ago

Last modified 5 years ago

Add a more verbose message when we can't create an algorithm from history

Reported by: Samuel Jackson Owned by: Samuel Jackson
Priority: major Milestone: Release 3.3
Component: Framework Keywords:
Cc: Blocked By:
Blocking: Tester: Harry Jeffery

Description

If we try to create an algorithm from it's history record and it is a child algorithm whose workspace wasn't stored in the ADS we get a runtime error saying we can't create the property.

We should output a better message to developers in this situation.

Attachments

test_history_error.py (234 bytes) - added by Samuel Jackson 6 years ago.
Testing script

Change History

comment:1 Changed 6 years ago by Owen Arnold

  • Status changed from new to assigned

comment:2 Changed 6 years ago by Samuel Jackson

  • Status changed from assigned to inprogress

Refs #9929 Add a more informative error message.

Changeset: 67ac6316836fa4cb7d1d430e9bc65557a50d4f26

comment:3 Changed 6 years ago by Samuel Jackson

Refs #9929 Fix unused variable warning.

Changeset: 6be0b20f36a41da668c20a2f76755b220eb0a47a

comment:4 Changed 6 years ago by Samuel Jackson

  • Milestone changed from Backlog to Release 3.3

Changed 6 years ago by Samuel Jackson

Testing script

comment:5 Changed 6 years ago by Samuel Jackson

  • Status changed from inprogress to verify
  • Resolution set to fixed

To Test

Running the test script should reproduce the error for you. Check that these changes a more descriptive exception.

comment:6 Changed 6 years ago by Harry Jeffery

  • Status changed from verify to verifying
  • Tester set to Harry Jeffery

comment:7 Changed 6 years ago by Harry Jeffery

  • Status changed from verifying to closed

Merge remote-tracking branch 'origin/bugfix/9929_verbose_msg_create_algorithm'

Full changeset: 2cf789dfddac1932ee6b88648ff7282d467931b4

comment:8 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 10771

Note: See TracTickets for help on using tickets.