Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Identify possible overlay error message categories and apply error message template #69

Closed
cari-lynn opened this issue Jan 28, 2021 · 2 comments

Comments

@cari-lynn
Copy link
Contributor

What are "helpful error messages"? Ask yourself these five questions to start:

Does the error...

  1. focus my attention on the source of the problem?
  2. give all of the relevant facts (and none of the irrelevant ones)?
  3. consistently use a set of accessible terms?
  4. make clear what's wrong?
  5. help me figure out how to fix it?

What were the results from the initial round of user testing?

During the first rounds of user testing, we found that our proposed error messages had addressed questions 1-4, but lacked any guidance on how to fix the problem. Since we know that our proposed changes to the error messages are improvements (based on user tests), we think it would be smartest to implement these improvements iteratively, rather than waiting until we have the "perfect" error message.

How

During the user testing stage, we worked out a general template on how we want the new error messages to look, however we only drafted complete errors in three different scenarios.
Template Error Message:

 ytt: Error: Overlaying (in order: <overlay file>):
Expected:
 <overlay file>:<line failure> (<match type>, '<node value>')  to match <expected match number> item in
 <template file>:<line failure> (<match type> in '<parent of match>') but matched <actual match number>.

 <overlay file>:<line failure> 
 <context from overlay file consisting of:
   - document start (---)
   - parent tree down to line of failure
   - annotation matcher above the failure line
   - ellipsis (...) to skip lines such as siblings and unimportant context
  >

 <template file>:<line failure> 
 <context from the template file consisting of:
   - document start (---)
   - parent tree down to lines where the overlay matched
   - ellipsis (...) to skip lines such as siblings and unimportant context 
  >

Since an overlay can match via an expression, the domain of overlay error messages is vast, making it difficult to implement changes. In order to address this, a HackMD Doc was created with a subset of scenarios and proposed errors. This document will also act as the acceptance criteria for the following implementation story.

Acceptance Criteria

At this time, this document is not the source of truth, and we are looking identify more error scenarios. Once an example error message is fleshed out for each major category that is identified, there should be enough information for implementation to begin.

Notes

@cari-lynn
Copy link
Contributor Author

Issue moved to vmware-tanzu/carvel-ytt #287 via ZenHub

@cari-lynn
Copy link
Contributor Author

Closed this because it was opened in the incorrect repo

@github-actions github-actions bot added the carvel triage This issue has not yet been reviewed for validity label Jan 28, 2021
@cari-lynn cari-lynn removed the carvel triage This issue has not yet been reviewed for validity label Jan 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant