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

simplify running unit tests #54

Closed
mrothauer opened this issue Apr 9, 2017 · 1 comment
Closed

simplify running unit tests #54

mrothauer opened this issue Apr 9, 2017 · 1 comment

Comments

@mrothauer
Copy link
Member

in database.php the $default config needs to be set to $test config before running unit tests - as most of them are integration tests with database access and only a few are real unit tests.

task
manage somehow that the db config is changed automatically as soon as unit tests are run - and change it back as well

hints
a lot of tests work with "simple browser" calls. simple browser is a php browser. changing the database always on simple test calls is not enough, because some cron jobs (order lists, invoicesd) also work with simple browser. maybe attaching a header in test context will be a solution worth trying

@mrothauer mrothauer changed the title simplify running tests simplify running unit tests Apr 9, 2017
@mrothauer mrothauer added this to planned issues in Roadmap v1.3 (Summer Release 2017) Apr 9, 2017
@mrothauer mrothauer self-assigned this Apr 12, 2017
@mrothauer mrothauer moved this from planned issues to in progress in Roadmap v1.3 (Summer Release 2017) Apr 12, 2017
@mrothauer mrothauer moved this from in progress to finished and merged into develop in Roadmap v1.3 (Summer Release 2017) Apr 17, 2017
@mrothauer
Copy link
Member Author

#57

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Roadmap v1.3 (Summer Release 2017)
finished and merged into develop
Development

No branches or pull requests

1 participant