Added default topic arn to master config. Added tests to ensure pipeline

coursera / dataduct

DataPipeline for humans.

Other

252 stars 82 forks source link

Added default topic arn to master config. Added tests to ensure pipeline #241

Open p5k6 opened 8 years ago

p5k6 commented 8 years ago

defs override default config.

Reference issue #240

One note - the DEFAULT_TOPIC_ARN in dataduct/etl/etl_pipeline.py is defined within the class, rather than outside (as other vars are - such as S3_ETL_BUCKET), because

it keeps the global namespace a little cleaner
mocking is far easier, as the var is local to the class level at that point

coveralls commented 8 years ago

Coverage increased (+0.08%) to 64.343% when pulling 8c1584e4e91488b949f7bd2a2a2856ce9c4c7afd on p5k6:add_topic_arn_config into 05b81ea3049c1bf60208753a63ed0292d30e19b4 on coursera:develop.