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
Add subdom and subdir tests #74
Conversation
469da23
to
f14e15a
Compare
Signed-off-by: Kirtan Gajjar <kirtangajjar95@gmail.com>
Signed-off-by: Kirtan Gajjar <kirtangajjar95@gmail.com>
9116618
to
39a079d
Compare
*/ | ||
public function theSiteShouldBeMultisite( $site, $type ) | ||
{ | ||
$php_container = implode( explode( '.', $site ) ) . '_php_1'; |
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.
Instead of evaluating the name of the container and doing docker exec
, changing directory to the site root and then doing docker-compose exec
could be better and less error-prone.
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.
Yup. Good idea!
{ | ||
$this->commands = []; | ||
} | ||
|
||
/** | ||
* @AfterFeature | ||
*/ |
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.
Add the sites of this feature in cleanup
.
wp.test
wpsubdir.test
wpsubdom.test
need to be added in cleanup on completion.
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.
Yes. Didn't add it before as updated cleanup function was in another PR(which was yet to be merged)
fdf8abc
to
de01976
Compare
de01976
to
da66cb0
Compare
Signed-off-by: Kirtan Gajjar <kirtangajjar95@gmail.com>
Signed-off-by: Kirtan Gajjar kirtangajjar95@gmail.com