Skip to content

WAR-1097: Support for * in path to loop through all project and suite files under directory#340

Merged
bjrr merged 8 commits into
developfrom
feature/WAR-1097
Jan 9, 2018
Merged

WAR-1097: Support for * in path to loop through all project and suite files under directory#340
bjrr merged 8 commits into
developfrom
feature/WAR-1097

Conversation

@venkat1077

@venkat1077 venkat1077 commented Jan 5, 2018

Copy link
Copy Markdown
Contributor

Add support to execute the warrior case/suite xmls matching the pattern(with asterisk) given in the path.
Now asterisk(*) is supported in (i) case_path of a suite & (ii) suite_path of a project

Execute 'wftests/warrior_tests/projects/pj_glob.xml' to test this feature at both suite and case level
Regression and test logs are in Jira ticket

@venkat1077 venkat1077 self-assigned this Jan 5, 2018
@bjrr bjrr removed the request for review from sugan19 January 6, 2018 01:38
@kaheichan

kaheichan commented Jan 8, 2018

Copy link
Copy Markdown
Contributor

Hi @venkat1077
In order to fully test the functionality, Let's build a testsuite like this
TS:
TC1-tc/sample_tc/*.xml
inside the dir, has 3 tc with 1 step - ci regression action-increase value (counter)
TC2-verify_tc-verify the counter value as 3

can apply the same logic to PJ-TS test

@venkat1077

Copy link
Copy Markdown
Contributor Author

Thanks @kaheichan
I have added tests to verify the fix as suggested.
Execute 'wftests/warrior_tests/projects/pj_glob.xml' & it will verify this feature at both suite and case level.

@kaheichan

kaheichan commented Jan 9, 2018

Copy link
Copy Markdown
Contributor

Put the test in Travis, verified regression log and travis log work

@bjrr bjrr merged commit 344d17d into develop Jan 9, 2018
@bjrr bjrr added this to the Warrior-3.7.0 milestone Jan 12, 2018
@venkat1077 venkat1077 deleted the feature/WAR-1097 branch February 6, 2018 11:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants