Current State and Roadmap

[ravage84]

trafficstars

The current state of this plugin isn't mentioned anywhere. Also, there is no roadmap, as it seems.

For example, there hasn't been a single release, yet. Not even an alpha release.

[josegonzalez]

Good call. I think @markstory had some ideas/things he wanted to implement before an initial release. Happy to dig into those as long as they are not testing related (I'm bad at that sorry not sorry).

[markstory]

I would like to shore up some of the coverage gaps, and ensure we have at least a few end to end integration tests with one of the simple enqueue adapters before doing any releases.

As for a roadmap I can try to put something together later this week.

[markstory]

I had attempted to put the following in to the milestone description but GitHub butchers the formatting.

Initial release for the Queue Plugin.

Goals

• [x] Provide job consumer and Job classes to enable background job dispatching.
• [x] Provide bake templates for job classes.
• [x] Add Cake/ as top level namespace
• [x] Decide final package name.
• [x] Get to ~90% test coverage.
• [ ] Include integration tests with end to end job execution.
• [ ] Publish documentation.

[othercorey]

I request adding "Move under Cake/ namespace".

[josegonzalez]

I think I setup the plugin in the same way we have Migrations and Bake, neither of which have the Cake/ namespace. Feel free to change it up though if we've decided that official plugins should have that, core or not :)

[othercorey]

I'm not going to make a big issue out of this, but my point-of-view is that we shouldn't be pushing things into the top-level namespace just because. I'm not a fan of the argument "well you shouldn't be using two queue systems in the first place.". Unless we're implementing de facto queue in php, our library names are too common.

[ADmad]

I added Add Cake/ as top level namespace to markstory's comment above. Our TwigView plugin also has the same.

[josegonzalez]

Sounds good :)

[markstory]

The namespace changes are done now :smile:

[T0mats0]

Looks like on the list is 90% coverage. We're almost there. QueueExtension needs a bit more. We also need end to end? Without an integration test, I can't think of a way to to test QueueExtension without mocks. So what's left there?

We also talked about reducing static methods. I can imagine a lot here.

Documentation is left too. No idea what/how to write that up.

[markstory]

@T0mats0 I think an end to end test for a successful job dispatch and execution would be great to have.

The documentation has been deployed and auto updates on each merge. It hasn't been linked into the make book table of contents as this plugin is still early on, but the docs are available at https://book.cakephp.org/queue

[godgarden-dev]

I'm really looking forward to the release. Is it almost there? Thank you for all your wonderful OSS development.

[markstory]

I think we can do an initial release soon. The first batch of features are in place, and tests are much better than they were before.

[dereuromark]

Once this is done and docs are added, maybe also add it to https://github.com/FriendsOfCake/awesome-cakephp#queue for visibility?