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

Add error message when forgetting a name and fix bug with expectfail not working with unit names with special characters #100

Merged
merged 2 commits into from
Dec 14, 2023

Conversation

AutonomicPerfectionist
Copy link
Contributor

Is this pull request associated with an issue(s)?
Fix #99

Description
This PR is pretty simple in that it adds a helpful error message to inform users when they forget to use the NAME keyword or provide a value for it. It also raises a CPP exception when encountering to prevent anything weird from happening later and to give the user a proper stacktrace. Tests for this were also added.

In the process of fixing this issue I discovered that the expectfail template did not work when the section name had spaces, because it was being inserted verbatim into a set() command without quotes. This was also fixed by adding bracket quotes around important substitutions, in both templates and in the expectfail_subprocess module.

@AutonomicPerfectionist AutonomicPerfectionist linked an issue Dec 13, 2023 that may be closed by this pull request
@ryanmrichard ryanmrichard merged commit e7f90b5 into master Dec 14, 2023
7 checks passed
@ryanmrichard ryanmrichard deleted the 99-better-error-message-for-forgetting-name branch December 14, 2023 16:00
Copy link

🚀 [bumpr] Bumped! New version:v0.1.14 Changes:v0.1.13...v0.1.14

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

Successfully merging this pull request may close these issues.

Better Error Message for Forgetting "NAME"
2 participants