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

Run multiple files in one folder #25

Closed
xiaoyu-wu566 opened this issue Sep 27, 2022 · 3 comments · Fixed by #26
Closed

Run multiple files in one folder #25

xiaoyu-wu566 opened this issue Sep 27, 2022 · 3 comments · Fixed by #26
Assignees

Comments

@xiaoyu-wu566
Copy link

I have a nested folder like testJmeter/tests, and there are multiple test cases in it. But when I use
uses: rbhadti94/apache-jmeter-action@v0.5.0
with:
testFilePath: testJmeter/tests/
outputReportsFolder: reports/
It will failed with An error occurred: The file /github/workspace/tests/.jmx doesn't exist or can't be opened and I checked the log. It says Using Test File Path testJmeter/tests (correct), Running test with tests/.jmx (I think something wrong here as you use BASEFILE_PATH=$(basename $TESTFILE_PATH), you remove the prefix path here).
Is this the issue or I did something wrong? Could you please help me with it. Thanks

@rbhadti94
Copy link
Owner

Hey @xiaoyu-wu566,

I see where the issue is - https://github.com/rbhadti94/apache-jmeter-action/blob/master/entrypoint.sh#L38

Will get a fix in this week. Thanks for pointing it out. Basically I have to loop recursively through folders rather than the top level.

@rbhadti94 rbhadti94 self-assigned this Sep 27, 2022
@xiaoyu-wu566
Copy link
Author

Hi Ravindra,
Thanks very much! I forked you code and just changed BASEFILE_PATH to TESTFILE_PATH in the for loop. Then everything looks correct.

rbhadti94 added a commit that referenced this issue Oct 8, 2022
@rbhadti94 rbhadti94 linked a pull request Oct 8, 2022 that will close this issue
rbhadti94 added a commit that referenced this issue Oct 8, 2022
rbhadti94 added a commit that referenced this issue Oct 8, 2022
rbhadti94 added a commit that referenced this issue Oct 8, 2022
rbhadti94 added a commit that referenced this issue Oct 8, 2022
rbhadti94 added a commit that referenced this issue Oct 8, 2022
rbhadti94 added a commit that referenced this issue Oct 21, 2022
rbhadti94 added a commit that referenced this issue Oct 21, 2022
rbhadti94 added a commit that referenced this issue Oct 21, 2022
@rbhadti94
Copy link
Owner

@xiaoyu-wu566 - v0.6.0 should contain this feature now.

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 a pull request may close this issue.

2 participants