Skip to content
This repository has been archived by the owner on Feb 22, 2023. It is now read-only.

Add message to handle wrong syntax when a hyphen(-) is not follow by a space #308

Closed
MKen-li opened this issue Oct 2, 2019 · 2 comments
Closed

Add message to handle wrong syntax when a hyphen(-) is not follow by a space #308

MKen-li opened this issue Oct 2, 2019 · 2 comments
Assignees
@jperata
Labels
bug Something isn't working sprint 44

Comments

@MKen-li
Copy link

MKen-li commented Oct 2, 2019 •
edited

Description:

We are showing false positives when a syntax error is not catch

Steps To Reproduce

  1. When executing a test like:
  • test: name
  • open guess the price:
    -prompt: "asdf"

Expected behavior

syntax error: a starting hyphen (-) inside a test has to be followed by a blank space

Actual behavior

The missing space between the hyphen and the prompt is not detected and the test passes.

Additional context

None.
@MKen-li MKen-li added bug Something isn't working upcoming labels Oct 2, 2019
@dmarvp dmarvp added sprint 44 and removed upcoming labels Oct 21, 2019
@jperata jperata mentioned this issue Nov 7, 2019
Merged
@jperata
Copy link
Contributor

jperata commented Nov 7, 2019

Screen Shot 2019-11-07 at 09 53 36

Can be tested on latest beta (2.4.18)

@MKen-li
Copy link
Author

MKen-li commented Nov 7, 2019

Marked as verified
Build = bespoken-tools@2.4.18

@MKen-li MKen-li closed this as completed Nov 7, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@jperata jperata

Labels
bug Something isn't working sprint 44
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@dmarvp @jperata @MKen-li