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

During rasa test, YamlValidationException shows temp path #7640

Closed
2 tasks
shubhvachher opened this issue Dec 29, 2020 · 5 comments · Fixed by #8217
Closed
2 tasks

During rasa test, YamlValidationException shows temp path #7640

shubhvachher opened this issue Dec 29, 2020 · 5 comments · Fixed by #8217
Assignees
Labels
area:rasa-oss/cli Issues focused on the rasa command-line-interface area:rasa-oss 🎡 Anything related to the open source Rasa framework effort:atom-squad/2 Label which is used by the Rasa Atom squad to do internal estimation of task sizes. feature:ux-cli+training-data Feature: Improve user experience with Rasa CLI and training data for developers type:bug 🐛 Inconsistencies or issues which will cause an issue or problem for users or implementors.

Comments

@shubhvachher
Copy link

shubhvachher commented Dec 29, 2020

Rasa version: 2.0.0

Rasa SDK version (if used & relevant): Not Used

Rasa X version (if used & relevant): Not Used

Python version: 3.6.8 (anaconda)

Operating system (windows, osx, ...): windows

Issue:

During rasa test if there is a YamlValidationException then the path for the file shown points to the temp directory.

Error (including full traceback):

YamlValidationException: Failed to validate 'C:\Users\USER\AppData\Local\Temp\tmpjvcoar7v\3c3f65c658cd4a41b3fb1703892fa328_test_happypaths_basic.yml'. Please make sure 
the file is correct and all mandatory parameters are specified. Here are the errors found during validation:
...

Command or request that led to error:

rasa test

You could test using this retrieval intent example from my test_happypaths_basic.yml:

stories:
- story: faq/restaurant_available_testing
  steps:
  - user: "Does the hotel have a restaurant?"
  - intent: faq/restaurant_available
  - action: utter_faq/restaurant_available

Definition of Done

  • evalute if copying to /temp is needed and drop it if not
  • correctly report path to actual test file
@shubhvachher shubhvachher added area:rasa-oss 🎡 Anything related to the open source Rasa framework type:bug 🐛 Inconsistencies or issues which will cause an issue or problem for users or implementors. labels Dec 29, 2020
@shubhvachher
Copy link
Author

I could submit a PR for this if reproducible! Seems straight forward on the face of it.

@sara-tagger
Copy link
Collaborator

Thanks for raising this issue, @rgstephens will get back to you about it soon✨

Please also check out the docs and the forum in case your issue was raised there too 🤗

@rgstephens
Copy link
Contributor

@shubhvachher Thanks for reporting. Yes, a PR would be appreciated!

@shubhvachher
Copy link
Author

Sure, I'm out travelling till Jan 3rd. I'll get to this Jan 4.

@wochinge
Copy link
Contributor

@shubham3582 Any news on this?

@wochinge wochinge added the area:rasa-oss/cli Issues focused on the rasa command-line-interface label Jan 29, 2021
@TyDunn TyDunn added the feature:ux-cli+training-data Feature: Improve user experience with Rasa CLI and training data for developers label Feb 17, 2021
@gausie gausie added the effort:atom-squad/2 Label which is used by the Rasa Atom squad to do internal estimation of task sizes. label Mar 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:rasa-oss/cli Issues focused on the rasa command-line-interface area:rasa-oss 🎡 Anything related to the open source Rasa framework effort:atom-squad/2 Label which is used by the Rasa Atom squad to do internal estimation of task sizes. feature:ux-cli+training-data Feature: Improve user experience with Rasa CLI and training data for developers type:bug 🐛 Inconsistencies or issues which will cause an issue or problem for users or implementors.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants