we don't always install jetty.

[wfindley]

No description provided.

[gsaslis]

yep this looks like a pretty straightforward fix... 👍

[gsaslis]

@wfindley do you feel confident extending the tests to cover this case?

[wfindley]

What needs extending? It seems to me that the current tests are adequate.

[gsaslis]

well, if they were indeed adequate - there should be one that is failing exactly because of the bug you've brought up here ;)
So we need a new (unit) test that confirms that the /etc/init.d/ file is named after the app_name (and not always 'jetty' ;) )

[wfindley]

Oh, yeah I can do that. Chefspec?
On Dec 12, 2015 8:59 AM, "Yorgos Saslis" notifications@github.com wrote:

well, if they were indeed adequate - there should be one that is failing
exactly because of the bug you've brought up here ;)
So we need a new (unit) test that confirms that the /etc/init.d/ file is
named after the app_name (and not always 'jetty' ;) )

—
Reply to this email directly or view it on GitHub
#8 (comment)
.

[gsaslis]

yep - that would be cool ;)