search

deis / workflow-e2e

End-to-end tests for Deis Workflow
MIT License
12 stars 18 forks source link

test config:set with multi-line values in container #23

Closed bacongobbler closed 8 years ago

bacongobbler commented 8 years ago

deis/deis#4844 deis/workflow#268

We have automated tests here, but we also need to ensure that the environment variable inside the container is properly escaped like it's done on Heroku.

mboersma commented 8 years ago

I think this is fixed in #78, but we should add another spec that specifically tests deis config:set with a private key as mentioned in deis/deis#4844 and verify.

mboersma commented 8 years ago

I've tested this with the specific data in deis/deis#4844 and I'm convinced that this test covers the same territory.