-
-
Notifications
You must be signed in to change notification settings - Fork 3.2k
Improve hurl test script with arguments #36273
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
Improve hurl test script with arguments #36273
Conversation
test/hurl/run.sh
Outdated
| TEST_public=false | ||
| ;; | ||
| *) | ||
| COOKIE=$( echo "$arg" | grep -c -- '--cookiefile=' ) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
$(grep -c -- '--cookiefile=' <<< "$arg") saves an echo.
You can even write the following, bu tI think that may confuse.
$(<<< "$arg" grep -c -- '--cookiefile=' )
|
examples with --just= |
No because you add options starting with "--" - apparently some options overlap. I prefer to select tests without a -- option. Basically, I think we could do just with what I proposed and --cookie option. Select api and public in my PR is I did not mean to overlap though... |
It happens, and I think your approach is better. Will you add possibility for specifying where the cookie should be saved/reused? I could see some value in having it possible to skip one or some files in a subdir that you included - for when people run it by hand during development
|
|
I added the cookie option and some other ones. |
yeah that's why i ended up using grep and xargs. I even though about using grep -e for regex or perhaps grep -E for lines like "abc|foo|..." |
|
Using But I went with what was already done. |
|
replaced with #36291 |
QUAL: Improve hurl test script, some arguments now available
This PR introduces arguments to the hurl test script:
Example:
./run.sh --noapi --cookiefile=/tmp/cookie.jar