Add print to confirm that the failed_test_stories.yml has been saved #1
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…asaHQ#3998) Co-authored-by: Henrike100 <henrikem100@gmail.com>
arthurTemporim
requested changes
Sep 20, 2020
rasa/cli/test.py
Outdated
| @@ -101,6 +102,7 @@ def run_core_test(args: argparse.Namespace) -> None: | |||
| else: | |||
| test_core_models(args.model, stories, output) | |||
|
|
|||
| print_info("Failed stories written to " + output + "/failed_test_stories.yml") | |||
There was a problem hiding this comment.
Is there another way to get failed_test_stories.yml value? placing the name by hand is not a good idea because if the file changes will be needed to change this log by hand.
|
Please, run the command |
|
I think for this contribution changing documentation will not be needed, but update the |
|
I made the changes you asked for. Please check if it is correct and if anything else is missing. |
arthurTemporim
approved these changes
Sep 23, 2020
Pull Request Test Coverage Report for Build 335845507
💛 - Coveralls |
RomuloSouza
pushed a commit
that referenced
this pull request
Dec 7, 2020
* backport fix * backport pika fix * pika fix backport * fix tests * test fixes * remove logger statement * Updated version of PikaMessageProcessor from the source branch * Small bugfix * changelog * prepared release of version 1.10.16 * Update CHANGELOG.rst Co-authored-by: Ella Rohm-Ensing <erohmensing@gmail.com> * add robots.txt and redirection headers * bump rasabaster version * Fix Pika multiprocessing for 1.10.x * Reformatting * Changelog entry * Updated tests * is_ready method is back * uninstall cached pydocstyle * also uninstall pydocstyle * uninstall in poetry env * Removing f-string * NoReturn -> None * prepared release of version 1.10.17 * Fix incorrect attribute for key prefix in RedisTrackerStore Fix RasaHQ#7306 * Add changelog entry * Remove the set-env command - 1.10.x (RasaHQ#7367) * Fix DynamoDB TrackerStore (RasaHQ#7340) * attempting to fix dynamo * fixed * fixed import * applied some more fixes * added deprecation warning for DynamoDB tracker store attribute * added changelog description for PR 7340 * fix formatting Co-authored-by: Tom Bocklisch <tom@rasa.com> * prepared release of version 1.10.18 (RasaHQ#7381) * Update docstrings * Update CHANGELOG.mdx * line length fix * Add print to confirm that the failed_test_stories.yml has been saved (RasaHQ#6793) * Add print to confirm that the failed_test_stories.yml has been saved (RasaHQ#3998) Co-authored-by: Henrike100 <henrikem100@gmail.com> * Change the way to show the file name * files formatted with black * add changelog * Update test core * Formatted with black * Update print Co-authored-by: mbslet <lmbs.geo@gmail.com> * Amend response schema to support missing top-level keys (RasaHQ#7232) * amend response schema to support missing top level keys * changelog entry * download mitie from github instead of s3 (RasaHQ#7434) Co-authored-by: melindaloubser1 <melinda.loubser@gmail.com> Co-authored-by: alwx <alwxndr@gmail.com> Co-authored-by: Roberto <43567378+rasabot@users.noreply.github.com> Co-authored-by: Melinda Loubser <32034278+melindaloubser1@users.noreply.github.com> Co-authored-by: Ella Rohm-Ensing <erohmensing@gmail.com> Co-authored-by: m-vdb <m.verger@rasa.com> Co-authored-by: Tobias Wochinger <t.wochinger@rasa.com> Co-authored-by: Duc-Minh Phan <alephvn@gmail.com> Co-authored-by: Federico Tedin <federicotdn@users.noreply.github.com> Co-authored-by: Tomasz Czekajlo <t.czekajlo@rasa.com> Co-authored-by: Tom Herold <tom.herold@scalableminds.com> Co-authored-by: Tom Bocklisch <tom@rasa.com> Co-authored-by: Henrique Martins <henrikem100@gmail.com> Co-authored-by: mbslet <lmbs.geo@gmail.com> Co-authored-by: pheel <philippe@mrbot.ai>
RochaCarla
pushed a commit
that referenced
this pull request
Nov 17, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Proposed changes:
results/failed_test_stories.yml, resolving issue Print out "Failed stories written to results/failed_stories.md" at the end of running rasa test core RasaHQ/rasa#3998Default path (results)
User specified path
Status (please check what you already did):
black(please check Readme for instructions)