From c4b9088bfcee05f39ccec8989eafa15b0fc69ddf Mon Sep 17 00:00:00 2001
From: Steven Cipriano <steven.cipriano@radius.com>
Date: Tue, 27 Jun 2017 11:31:43 -0700
Subject: [PATCH] Add support for recursive emr settings

- Updates _RecursiveDictRef to not implement __getitem__, avoiding errors when using recursive settings for an emr job flow
---
 moto/core/responses.py           |  3 +++
 tests/test_emr/test_emr_boto3.py | 17 ++++++++++++++++-
 2 files changed, 19 insertions(+), 1 deletion(-)

diff --git a/moto/core/responses.py b/moto/core/responses.py
index adad5d1de..82e9d4cad 100644
--- a/moto/core/responses.py
+++ b/moto/core/responses.py
@@ -414,6 +414,9 @@ class _RecursiveDictRef(object):
     def __getattr__(self, key):
         return self.dic.__getattr__(key)
 
+    def __getitem__(self, key):
+        return self.dic.__getitem__(key)
+
     def set_reference(self, key, dic):
         """Set the RecursiveDictRef object to keep reference to dict object
         (dic) at the key.
diff --git a/tests/test_emr/test_emr_boto3.py b/tests/test_emr/test_emr_boto3.py
index 830abdb85..237ff8bba 100644
--- a/tests/test_emr/test_emr_boto3.py
+++ b/tests/test_emr/test_emr_boto3.py
@@ -64,7 +64,18 @@ def test_describe_cluster():
     args['Configurations'] = [
         {'Classification': 'yarn-site',
          'Properties': {'someproperty': 'somevalue',
-                        'someotherproperty': 'someothervalue'}}]
+                        'someotherproperty': 'someothervalue'}},
+        {'Classification': 'nested-configs',
+         'Properties': {},
+         'Configurations': [
+             {
+                 'Classification': 'nested-config',
+                 'Properties': {
+                     'nested-property': 'nested-value'
+                 }
+             }
+         ]}
+    ]
     args['Instances']['AdditionalMasterSecurityGroups'] = ['additional-master']
     args['Instances']['AdditionalSlaveSecurityGroups'] = ['additional-slave']
     args['Instances']['Ec2KeyName'] = 'mykey'
@@ -87,6 +98,10 @@ def test_describe_cluster():
     config['Classification'].should.equal('yarn-site')
     config['Properties'].should.equal(args['Configurations'][0]['Properties'])
 
+    nested_config = cl['Configurations'][1]
+    nested_config['Classification'].should.equal('nested-configs')
+    nested_config['Properties'].should.equal(args['Configurations'][1]['Properties'])
+
     attrs = cl['Ec2InstanceAttributes']
     attrs['AdditionalMasterSecurityGroups'].should.equal(
         args['Instances']['AdditionalMasterSecurityGroups'])