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

Fix openReportIfNecessary method when report path name is different t… #144

Merged
merged 1 commit into from
Nov 17, 2022

Conversation

nicholasray
Copy link
Contributor

…han the group name

The desktop-dev group was failing to write to/open the report because it was looking for a dektop-dev folder instead of a desktop-development folder. Since the report path is configurable and may differ from the group name, we need to look at the config in order to determine the report path.

Additionally:

  • Refactor the removeTestFolder method to have a relativePath param instead of passing the entire config.

…han the group name

The desktop-dev group was failing to write to/open the report in the
openReportIfNecessary method because it was looking for a `dektop-dev` folder
instead of a `desktop-development` folder.  Since the report path is
configurable and may differ from the group name, we need to look at the config
in order to determine the report path.

Additionally:

* Refactor the removeTestFolder method to have a relativePath param instead of
passing the entire config.
@nicholasray nicholasray merged commit 39c3845 into main Nov 17, 2022
@nicholasray nicholasray deleted the fix-desktop-dev-file-path branch November 17, 2022 01:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants