From 024fb5a976f26b4de4e1744fa52f231a522c8003 Mon Sep 17 00:00:00 2001
From: Raphael Cohen <raphael.cohen@sekoia.fr>
Date: Tue, 16 Jun 2020 18:42:54 +0200
Subject: [PATCH] Fixes view links format

---
 arango/formatter.py |  4 +++-
 tests/test_view.py  | 11 +++++++++--
 2 files changed, 12 insertions(+), 3 deletions(-)

diff --git a/arango/formatter.py b/arango/formatter.py
index e508b721..a6d2974e 100644
--- a/arango/formatter.py
+++ b/arango/formatter.py
@@ -811,7 +811,9 @@ def format_view(body):  # pragma: no cover
     if 'writebufferSizeMax' in body:
         result['writebuffer_max_size'] = body['writebufferSizeMax']
     if 'links' in body:
-        result['links'] = [format_view_link(link) for link in body['links']]
+        result['links'] = {
+            name: format_view_link(link) for name, link in body['links'].items()
+        }
 
     return verify_format(body, result)
 
diff --git a/tests/test_view.py b/tests/test_view.py
index 07bf9724..5ea0e987 100644
--- a/tests/test_view.py
+++ b/tests/test_view.py
@@ -9,7 +9,7 @@
     ViewReplaceError,
     ViewUpdateError
 )
-from tests.helpers import assert_raises, generate_view_name
+from tests.helpers import assert_raises, generate_view_name, generate_col_name
 
 
 def test_view_management(db, bad_db, cluster):
@@ -18,15 +18,22 @@ def test_view_management(db, bad_db, cluster):
     bad_view_name = generate_view_name()
     view_type = 'arangosearch'
 
+    col_name = generate_col_name()
+    col = db.create_collection(
+        name=col_name
+    )
+
     result = db.create_view(
         view_name,
         view_type,
-        {'consolidationIntervalMsec': 50000}
+        {'consolidationIntervalMsec': 50000, "links": {col_name: {}}}
     )
     assert 'id' in result
     assert result['name'] == view_name
     assert result['type'] == view_type
     assert result['consolidation_interval_msec'] == 50000
+    assert col_name in result["links"]
+
     view_id = result['id']
 
     # Test create duplicate view