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

which is messing up console/testsuite #15

Closed
swiffer opened this issue Feb 12, 2018 · 5 comments
Closed

which is messing up console/testsuite #15

swiffer opened this issue Feb 12, 2018 · 5 comments
Labels

Comments

@swiffer
Copy link
Contributor

swiffer commented Feb 12, 2018

Hey,

great plugin! Works without hassles. Using it as backup when doing auto-deployments and loaded it in bootstrap_cli.php

however on my local dev machine mysqldump etc are not available causing which to output the system path 5 times before running the testsuite. can this be surpressed ?

@mirko-pagliai
Copy link
Owner

Hello @swiffer .

causing which to output the system path 5 times before running the testsuite

I can not understand what you mean.
Can you post some examples?

@swiffer
Copy link
Contributor Author

swiffer commented Feb 13, 2018

code below is coming from cygwin on windows (using git bash)

matth@SWIFFER-T440S MINGW64 /c/Websites/git-repo (master)
$ phpunit
which: no mysql in (/mingw64/bin:/usr/bin:/c/Users/matth/bin:/c/ProgramData/Oracle/Java/javapath:/c/Program Files (x86)/Intel/iCLS Client:/c/Program Files/Intel/iCLS Client:/c/WINDOWS/system32:/c/WINDOWS:/c/WINDOWS/System32/Wbem:/c/WINDOWS/System32/WindowsPowerShell/v1.0:/c/Program Files/PHP:/c/ProgramData/ComposerSetup/bin:/c/Program Files/Git LFS:/c/Program Files (x86)/Intel/Intel(R) Management Engine Components/DAL:/c/Program Files/Intel/Intel(R) Management Engine Components/DAL:/c/Program Files (x86)/Intel/Intel(R) Management Engine Components/IPT:/c/Program Files/Intel/Intel(R) Management Engine Components/IPT:/c/Program Files/Microsoft SQL Server/110/Tools/Binn:/c/Program Files/Intel/WiFi/bin:/c/Program Files/Common Files/Intel/WirelessCommon:/c/Program Files/nodejs:/cmd:/c/Users/matth/AppData/Roaming/Composer/vendor/bin:/c/Users/matth/AppData/Local/Microsoft/WindowsApps:/c/Program Files/Microsoft VS Code/bin:/c/Program Files/Intel/WiFi/bin:/c/Program Files/Common Files/Intel/WirelessCommon:/c/Users/matth/AppData/Roaming/npm)

@swiffer
Copy link
Contributor Author

swiffer commented Feb 13, 2018

Output after checking exitcode and redirecting output:

matth@SWIFFER-T440S MINGW64 /c/Websites/git-repo (master)
$ bin/cake backup export
Connection: default
Driver: Mysql
---------------------------------------------------------------
`mysqldump` executable not available

@mirko-pagliai
Copy link
Owner

Yes, I understand that the executables are not present in your installation.
However I did not understand what you propose.

Do you want the tests to run without the executables? But they are essential to verify the correct functioning of the code

@mirko-pagliai
Copy link
Owner

Ah, ok, now I understand. Sorry. You're right.

mirko-pagliai pushed a commit that referenced this issue Feb 14, 2018
* Closes #15

* Update global_functions.php

* Update global_functions.php

* Update global_functions.php

* Update Driver.php

* fixed to work under Windows

* fixed for sniffer

* fixed for sniffer

* fixed for coverage
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants