Clarify docs and tests around empty text files.

cloudfoundry-community / slack-notification-resource

Concourse CI resource for sending notifications to Slack.

MIT License

75 stars 82 forks source link

Clarify docs and tests around empty text files. #50

Closed jmcarp closed 6 years ago

jmcarp commented 6 years ago

Before #47, setting text_file to an empty or missing file resulted in no notification being sent. After #47, setting text_file to an empty file continued to result in no notification, but setting to an empty file sent a notification. This patch updates docs and tests to clarify the current expected behavior around empty and missing text files.

jmcarp commented 6 years ago

cc @drnic

drnic commented 6 years ago

Thx