add bootstrap option

[romulodl]

Added the bootstrap option (issue #52)

After looking the issue and the tips on the pull request 64. I did quickly this peace of code and added this feature:

• if you add a file called bootstrap.php to the test folder, it will be automatically loaded;
• Exceptions will be raised if you pass a invalid, inexistent or not pass any file (only --bootstrap without the /path/to/file.php)

TODO: read the yaml file for a possible bootstrap configuration and load it.

I would like a review from you guys if the way I've done is ok or if I'm breaking any pattern that you could be using.

Thanks
Romulo

[romulodl]

There are bugs on these code, once I have time I will fix them. I just would like a review to know if I'm following the right path

[romulodl]

• Added the --bootstrap=path/to/file.php param
• If the bootstrap option is set up on phpspec.yml, try to load it
• if the path is empty, the file doesn't exist or the file isn't a ".php". It will return an InvalidArgumentException

Thanks
Romulo

[sivad77]

Any progress with this pull request?

[benja-M-1]

The construction of the option is wrong. The first argument is the name of the option, it should be something like

new InputOption('bootstrap', null, InputOption::VALUE_REQUIRED, 'Run a bootstrap file before start')

see http://api.symfony.com/2.2/Symfony/Component/Console/Input/InputOption.html#method___construct

[benja-M-1]

Actually this PR duplicates this one #64

[benja-M-1]

remove this line

[romulodl]

Looking at the code. I think it is better that the bootstrap option should be added to the run command, not directly into the Application class. But then I don't know what should be the behavior when someone runs ./bin/phpspec --bootstrap=file.php and the Application assumes that the run command was called.

[romulodl]

• Add the bootstrap to the Run Command.
• The bootstrap file will only be called when you run ./bin/phpspec run --bootstrap="file.php" and not will be called with ./bin/phpspec --bootstrap="file.php"