Full key status report on initialization errors

[mbj]

Currently anima's initialization error reports are weak.

• For the first missing key an exception is raised, resulting in more debugging cycles for more missing keys.
• If an extra key is present an exception is raised for all extra keys.
• If a key is missing no extra keys are reported.

IMHO anima should report a "full" key-state report if ANY error condition is found. Generating the full report will result in more computation but efficiency on the error code path is not a primary objective.

[dkubb]

@mbj it sounds like you want an object that is responsible for validating the input attributes? You could pass in the attributes into the constructor, then call a command method to assert the keys are valid, and then call #to_hash on the result. If the keys were invalid it would raise an exception with the full error message.

[mbj]

@dkubb I think an internal method should be enough. But I cant forsee
the implementation at all detail levels.

On Sun, Nov 17, 2013 at 04:49:07PM -0800, Dan Kubb wrote:

@mbj it sounds like you want an object that is responsible for validating the input attributes? You could pass in the attributes into the constructor, then call a command method to assert the keys are valid, and then call #to_hash on the result. If the keys were invalid it would raise an exception with the full error message.

---

Reply to this email directly or view it on GitHub:
#3 (comment)

Markus Schirp

Phone: +49 201 / 360 379 14
Fax: +49 201 / 360 379 16
Web: schirp-dso.com
Email: mbj@schirp-dso.com
Twitter: twitter.com/m_b_j
OS-Code: github.com/mbj

Altendorferstrasse 44
D-45127 Essen

[elskwid]

This is blocking v1.0. @mbj am I reading this correctly? You want to have the error reported as single exception that states the combined error state of any/all missing keys and extra keys. Right?