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 for non-existing test folder, resulting in null listOfFiles #115

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

MrLurch81
Copy link

Purpose

When the default test folder (in build-artifacts, since Integration Studio 8.2.0) is not present, the plugin throws an exception. However, it should just return that no test cases were found, similar to when the test folder does exist but is empty.
Fixes the issue #114

Approach

Add a check to see if listOfFiles is not null before trying to read all files in listOfFiles.

@CLAassistant
Copy link

CLAassistant commented Feb 27, 2024

CLA assistant check
All committers have signed the CLA.

@MrLurch81
Copy link
Author

Hi @GDLMadushanka and @SanojPunchihewa,
I see that you have previously approved changes in this project. Any chance either of you could review this PR so it is no longer required to disable the maven tests?

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