-
Notifications
You must be signed in to change notification settings - Fork 1
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
Running concurrency #9
Comments
@djones8520 I think there is an open issue with regards to the config file switch, I wonder if it is reading your config file. Any way you can hoist your config up to the current directory you are running the tests from just to see if it changes anything? |
Same issue. I'm not sure if I'm misunderstanding what you mean about not being able to read the config file or not, but I think it is reading the file, because I do not see the concurrency options unless I put the new ConCurrencyPlugin line in the config file. |
I was thinking more the workers were unable to locate the config file. Not entirely sure what's going on. I will try testing against the latest version of peridot to see if anything has changed |
Just an fyi, we have peridot ~1.0 in composer, so we are not running the latest version. It's something I'm looking to update soon, so perhaps that would help. |
That should cover the latest I would think. |
1.17. I guess that is the latest version. In looking into our scope issue, I saw a issue/comment about 2.0, so I assumed 2.something was the latest version. |
Does the order of arguments matter? I've tried a few different combos, but a couple are:
vendor/bin/peridot -c tests/unit/peridot.php tests/unit --concurrent
vendor/bin/peridot -c tests/unit/peridot.php tests/unit --concurrent -p2
I get the message: Starting workers on 2/5 processes
But after that, nothing happens.
Thanks for any suggestions.
Edit: I just noticed the Usage that says the files should be after the options. I also tried the options first:
vendor/bin/peridot -c tests/unit/peridot.php --concurrent -p2 tests/unit
The text was updated successfully, but these errors were encountered: