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

Improved tests #6

Merged
merged 23 commits into from
Mar 6, 2024
Merged

Conversation

twerkmeister
Copy link
Collaborator

@twerkmeister twerkmeister commented Feb 1, 2024

can be merged after clarify command is fixed in rasa

  • fixed ordering in disambiguation/negation tests
  • reinstated add_card flow that was missing for disambiguation / negation tests
  • made some tests less strict when it comes to slots, removing checks for exact value sets when a range of similar values works
  • quoted yes and no messages in yaml as they might be translated to True/False if one is not careful with the parser

@twerkmeister twerkmeister changed the title fixed disambiguation and negation tests Improved tests Feb 14, 2024
@tabergma
Copy link
Collaborator

Is your branch up to date? Seems like the actions were not executed successfully. There are some recent changes for that on main.

@twerkmeister
Copy link
Collaborator Author

Ok seems there's one flaky test left and then the negation test needs the new clarify option sorting

@tabergma
Copy link
Collaborator

should we wait for the next rasa release and then update the rasa version? and should we move the flaky test maybe to a folder could "flaky"?

Copy link
Collaborator

@tabergma tabergma left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Just one minor suggestion 💯

.github/pull_request_template.md Outdated Show resolved Hide resolved
@twerkmeister twerkmeister merged commit 1435f9f into main Mar 6, 2024
3 checks passed
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.

2 participants