Skip to content

Added dbal.path config parameter for test environment#1621

Merged
pjedrzejewski merged 1 commit intoSylius:masterfrom
alch:sqlite-path-config-parameter
Jun 21, 2014
Merged

Added dbal.path config parameter for test environment#1621
pjedrzejewski merged 1 commit intoSylius:masterfrom
alch:sqlite-path-config-parameter

Conversation

@alch
Copy link
Copy Markdown
Contributor

@alch alch commented Jun 17, 2014

Parameter %sylius.database.path% was added to the parameter.yml.dist
file so that at install time it is possible to choose the data file when
using pdo_sqlite as a persistence driver.
This is especially useful in test environment when using sqlite data
file on a fast/memory filesystem (such as tmpfs), to speed up test
execution.

The parameter defaults to null (~) not to interfere with standard
configuration.

Parameter %sylius.database.path% was added to the parameter.yml.dist
file so that at install time it is possible to choose the data file when
using pdo_sqlite as a persistence driver.
This is especially useful in test environment when using sqlite data
file on a fast/memory filesystem (such as tmpfs), to speed up test
execution.

The parameter defaults to null (~) not to interfere with standard
configuration.
@pjedrzejewski
Copy link
Copy Markdown
Contributor

Nice! Thank you Aldo! I will wait for the build - formality. :)

@alch
Copy link
Copy Markdown
Contributor Author

alch commented Jun 18, 2014

Whops, looks like travis ran out of time while behatting. https://travis-ci.org/Sylius/Sylius/jobs/27796269

Thanks for replying and keep up the great work 👍

@stloyd
Copy link
Copy Markdown
Contributor

stloyd commented Jun 21, 2014

👍

pjedrzejewski pushed a commit that referenced this pull request Jun 21, 2014
Added dbal.path config parameter for test environment
@pjedrzejewski pjedrzejewski merged commit e355129 into Sylius:master Jun 21, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants