Skip to content
Browse files

changing the name of settings.inc to saunter.inc

  • Loading branch information...
1 parent 0162881 commit 8a44f7237aeb141c497fd516f170433f23420d8f @adamgoucher adamgoucher committed Nov 2, 2011
View
4 README.md
@@ -140,7 +140,7 @@ With the class marked as abstract, PHP won't try to create an instance of it whi
Config Files
------------
-In the _conf_ directory there is a settings.inc.default file. Taking a page from the RoR playbook, this file should be copied and renamed to just settings.inc. The stacktrace you will get if you forget should remind you to do this. The reason settings.inc is not checked in is so you can have different configs across different locations (like individual jobs in the CI server) or same location (one per environment and managed via symlink).
+In the _conf_ directory there is a saunter.inc.default file. Taking a page from the RoR playbook, this file should be copied and renamed to just saunter.inc. The stacktrace you will get if you forget should remind you to do this. The reason saunter.inc is not checked in is so you can have different configs across different locations (like individual jobs in the CI server) or same location (one per environment and managed via symlink).
All system/framework-wide configurations should go in this file. Rather than _completely_ pollute the global namespace I put everything into a settings array.
@@ -155,7 +155,7 @@ Running your scripts locally or in the OnDemand cloud is simply a matter of sett
$GLOBALS['settings']['sauce.ondemand']
-to _true_ and adjusting for which OS and browser combination you desire. Unlike other PHPUnit integrations with Selenium, I don't suggest that you figure out how to iterate over browser strings for runner-base parallelization. Instead, create a job per OS/browser in your CI server and use its local settings.inc to configure things. This way when one of those jobs fail (and eventually one will) you don't need to change any code to troubleshoot it -- you just need to run that job.
+to _true_ and adjusting for which OS and browser combination you desire. Unlike other PHPUnit integrations with Selenium, I don't suggest that you figure out how to iterate over browser strings for runner-base parallelization. Instead, create a job per OS/browser in your CI server and use its local saunter.inc to configure things. This way when one of those jobs fail (and eventually one will) you don't need to change any code to troubleshoot it -- you just need to run that job.
Notice as well that in the intermediary class, the teardown method will set the OnDemand job name and status as well.
View
0 ...terPHP/Defaults/conf/settings.inc.default → SaunterPHP/Defaults/conf/saunter.inc.default
File renamed without changes.
View
0 examples/ebay/conf/settings.inc → examples/ebay/conf/saunter.inc
File renamed without changes.
View
0 examples/saucelabs/conf/settings.inc → examples/saucelabs/conf/saunter.inc
File renamed without changes.
View
0 examples/saucelabs/conf/settings.inc.default → examples/saucelabs/conf/saunter.inc.default
File renamed without changes.
View
2 package.xml
@@ -35,7 +35,7 @@
<file baseinstalldir="/" name="saucelabs.inc.default" role="php">
<tasks:replace from="@package_version@" to="version" type="package-info" />
</file>
- <file baseinstalldir="/" name="settings.inc.default" role="php">
+ <file baseinstalldir="/" name="saunter.inc.default" role="php">
<tasks:replace from="@package_version@" to="version" type="package-info" />
</file>
</dir>
View
4 saunter.php
@@ -14,7 +14,7 @@ function initialize($installed) {
if (! is_dir("conf")) {
mkdir("conf");
}
- copy($defaults . "/conf/settings.inc.default", "conf/settings.inc.default");
+ copy($defaults . "/conf/saunter.inc.default", "conf/settings.inc.default");
copy($defaults . "/conf/saucelabs.inc.default", "conf/saucelabs.inc.default");
copy($defaults . "/phpunit.xml", "phpunit.xml");
@@ -84,7 +84,7 @@ function copy_logfile(&$log_name) {
register_shutdown_function('copy_logfile', &$log_name);
-require_once 'conf/settings.inc';
+require_once 'conf/saunter.inc';
/****
* the code in this block is covered under the Creative Commons Attribution 3.0 Unported License

0 comments on commit 8a44f72

Please sign in to comment.
Something went wrong with that request. Please try again.