enhancement Success

User tests: Successful: Unsuccessful:

avatar mbabker
mbabker
12 Jan 2014

Presently, there are hard coded checks in the ReceivePullsHook class to do custom processing for the joomla-cms repo which is not optimal in that we should not be hard coding this type of activity into our structure. To allow for a more dynamic solution, this pull implements the framework's Event package into the hook controllers and registers the dispatcher globally via the application object.

avatar mbabker mbabker - open - 12 Jan 2014
avatar b2z
b2z - comment - 14 Jan 2014

I suppose that this cannot be tested? :)

avatar mbabker
mbabker - comment - 14 Jan 2014

The other day I was thinking of adding onto the CLI app a method to trigger the hook controllers since those effectively can't be tested without the data payload. Or we actually write unit tests for our code, but who does that? :-D

avatar mbabker mbabker - change - 15 Jan 2014
Labels Added: enhancement
avatar elkuku elkuku - reference | - 15 Jan 14
avatar elkuku elkuku - merge - 15 Jan 2014
avatar elkuku elkuku - close - 15 Jan 2014
avatar elkuku elkuku - change - 15 Jan 2014
Status New Closed
Closed_Date 0000-00-00 00:00:00 2014-01-15 15:33:47
avatar elkuku elkuku - close - 15 Jan 2014
avatar elkuku
elkuku - comment - 15 Jan 2014

So after me (and PHPStorm) looking at the code (which looks pretty cool)... let's see how it works on the server.

Having some kind of tests would definitely help but...

who does that?

:wink:

avatar mbabker mbabker - head_ref_deleted - 20 Jan 2014

Add a Comment

Login with GitHub to post a comment