approvals / approvals/ApprovalTests.LabVIEW

Better Error Reporting for Parameterized and Combination Tests

Open
#26 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
LabVIEW
Stars
5
Forks
3
PR merge metrics
No merged PRs in 30d

Description

Currently if you mistype the name of a control in a Parameterized or Combination Test you get error 1055 Reference not Valid.

It would be nice to catch that and instead throw a different error with a different message so that it's clear that there is a typo.

The change needs to be made in the following image at ~ probe number 23, if the reference is not in the map, then skip the Ctl Value Set and generate an error with the appropriate message and include the name of the not found control and list the keys of the map in the error message, just to make it even clearer that there is a typo.
Image

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start at the image's probe number 23, where the issue says the control reference is checked against a map. Verify the behavior when the reference is absent, then make the error identify the missing control and list the map keys; done means mistyped controls produce the clearer error instead of error 1055.

Written by the indexing model from the issue text.

Assessment

Domain
testing-qa
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
52/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.