From 307ef8ead10f3c716ce3dd00d0405eddc7ed4d43 Mon Sep 17 00:00:00 2001
From: Adam Gilman <adam.gilman@gmail.com>
Date: Mon, 6 Nov 2017 19:06:55 +0000
Subject: [PATCH] Makes Creating SNS topics idempotent (#1324)

Make creating SNS topics idempotent

Closes #1323
---
 moto/sns/models.py                  |  9 ++++++---
 tests/test_sns/test_topics_boto3.py | 23 +++++++++++++++++++++++
 2 files changed, 29 insertions(+), 3 deletions(-)

diff --git a/moto/sns/models.py b/moto/sns/models.py
index 856255be..bf7c605e 100644
--- a/moto/sns/models.py
+++ b/moto/sns/models.py
@@ -193,9 +193,12 @@ class SNSBackend(BaseBackend):
         self.sms_attributes.update(attrs)
 
     def create_topic(self, name):
-        topic = Topic(name, self)
-        self.topics[topic.arn] = topic
-        return topic
+        candidate_topic = Topic(name, self)
+        if candidate_topic.arn in self.topics:
+            return self.topics[candidate_topic.arn]
+        else:
+            self.topics[candidate_topic.arn] = candidate_topic
+            return candidate_topic
 
     def _get_values_nexttoken(self, values_map, next_token=None):
         if next_token is None:
diff --git a/tests/test_sns/test_topics_boto3.py b/tests/test_sns/test_topics_boto3.py
index a9c2a290..495e63e7 100644
--- a/tests/test_sns/test_topics_boto3.py
+++ b/tests/test_sns/test_topics_boto3.py
@@ -31,6 +31,29 @@ def test_create_and_delete_topic():
     topics = topics_json["Topics"]
     topics.should.have.length_of(0)
 
+@mock_sns
+def test_create_topic_should_be_indempodent():
+    conn = boto3.client("sns", region_name="us-east-1")
+    topic_arn = conn.create_topic(Name="some-topic")['TopicArn']
+    conn.set_topic_attributes(
+        TopicArn=topic_arn,
+        AttributeName="DisplayName",
+        AttributeValue="should_be_set"
+    )
+    topic_display_name = conn.get_topic_attributes(
+        TopicArn=topic_arn
+    )['Attributes']['DisplayName']
+    topic_display_name.should.be.equal("should_be_set")
+
+    #recreate topic to prove indempodentcy
+    topic_arn = conn.create_topic(Name="some-topic")['TopicArn']
+    topic_display_name = conn.get_topic_attributes(
+        TopicArn=topic_arn
+    )['Attributes']['DisplayName']
+    
+    topic_display_name.should.be.equal("should_be_set")
+
+
 
 @mock_sns
 def test_get_missing_topic():