From 2aa5d04ecf0a7633c9d8b08c05ade4dff025a41d Mon Sep 17 00:00:00 2001 From: Anish Umale Date: Thu, 1 Aug 2024 19:57:45 +0530 Subject: [PATCH] change test to conform to new response structure --- mathesar/tests/rpc/tables/test_t_base.py | 120 +++++++---------------- 1 file changed, 36 insertions(+), 84 deletions(-) diff --git a/mathesar/tests/rpc/tables/test_t_base.py b/mathesar/tests/rpc/tables/test_t_base.py index 082315b783..714c1e4af3 100644 --- a/mathesar/tests/rpc/tables/test_t_base.py +++ b/mathesar/tests/rpc/tables/test_t_base.py @@ -298,22 +298,6 @@ def mock_list_joinable_tables(_table_oid, conn, max_depth): 'join_path': [[[2254329, 2], [2254334, 1]]], 'multiple_results': False }, - { - 'base': 2254329, - 'depth': 1, - 'target': 2254334, - 'fkey_path': [[2254406, False]], - 'join_path': [[[2254329, 2], [2254334, 1]]], - 'multiple_results': False - }, - { - 'base': 2254329, - 'depth': 1, - 'target': 2254350, - 'fkey_path': [[2254411, False]], - 'join_path': [[[2254329, 3], [2254350, 1]]], - 'multiple_results': False - }, { 'base': 2254329, 'depth': 1, @@ -322,33 +306,25 @@ def mock_list_joinable_tables(_table_oid, conn, max_depth): 'join_path': [[[2254329, 3], [2254350, 1]]], 'multiple_results': False }], - 'target_table_info': [ - { - '2254334': 'Items', - 'columns': [{'1': {'name': 'id', 'type': 'integer'}}, - {'2': {'name': 'Barcode', 'type': 'text'}}, - {'3': {'name': 'Acquisition Date', 'type': 'date'}}, - {'5': {'name': 'Book', 'type': 'integer'}}] - }, - { - '2254350': 'Patrons', - 'columns': [{'1': {'name': 'id', 'type': 'integer'}}, - {'2': {'name': 'First Name', 'type': 'text'}}, - {'3': {'name': 'Last Name', 'type': 'text'}}] - }, - { - '2254334': 'Items', - 'columns': [{'1': {'name': 'id', 'type': 'integer'}}, - {'2': {'name': 'Barcode', 'type': 'text'}}, - {'3': {'name': 'Acquisition Date', 'type': 'date'}}, - {'5': {'name': 'Book', 'type': 'integer'}}] + 'target_table_info': { + '2254334': { + 'name': 'Items', + 'columns': { + '1': {'name': 'id', 'type': 'integer'}, + '2': {'name': 'Barcode', 'type': 'text'}, + '3': {'name': 'Acquisition Date', 'type': 'date'}, + '5': {'name': 'Book', 'type': 'integer'} + } }, - { - '2254350': 'Patrons', - 'columns': [{'1': {'name': 'id', 'type': 'integer'}}, - {'2': {'name': 'First Name', 'type': 'text'}}, - {'3': {'name': 'Last Name', 'type': 'text'}}] - }] + '2254350': { + 'name': 'Patrons', + 'columns': { + '1': {'name': 'id', 'type': 'integer'}, + '2': {'name': 'First Name', 'type': 'text'}, + '3': {'name': 'Last Name', 'type': 'text'} + } + } + } } expected_dict = { 'joinable_tables': [ @@ -360,22 +336,6 @@ def mock_list_joinable_tables(_table_oid, conn, max_depth): 'join_path': [[[2254329, 2], [2254334, 1]]], 'multiple_results': False }, - { - 'base': 2254329, - 'depth': 1, - 'target': 2254334, - 'fkey_path': [[2254406, False]], - 'join_path': [[[2254329, 2], [2254334, 1]]], - 'multiple_results': False - }, - { - 'base': 2254329, - 'depth': 1, - 'target': 2254350, - 'fkey_path': [[2254411, False]], - 'join_path': [[[2254329, 3], [2254350, 1]]], - 'multiple_results': False - }, { 'base': 2254329, 'depth': 1, @@ -384,33 +344,25 @@ def mock_list_joinable_tables(_table_oid, conn, max_depth): 'join_path': [[[2254329, 3], [2254350, 1]]], 'multiple_results': False }], - 'target_table_info': [ - { - '2254334': 'Items', - 'columns': [{'1': {'name': 'id', 'type': 'integer'}}, - {'2': {'name': 'Barcode', 'type': 'text'}}, - {'3': {'name': 'Acquisition Date', 'type': 'date'}}, - {'5': {'name': 'Book', 'type': 'integer'}}] - }, - { - '2254350': 'Patrons', - 'columns': [{'1': {'name': 'id', 'type': 'integer'}}, - {'2': {'name': 'First Name', 'type': 'text'}}, - {'3': {'name': 'Last Name', 'type': 'text'}}] + 'target_table_info': { + '2254334': { + 'name': 'Items', + 'columns': { + '1': {'name': 'id', 'type': 'integer'}, + '2': {'name': 'Barcode', 'type': 'text'}, + '3': {'name': 'Acquisition Date', 'type': 'date'}, + '5': {'name': 'Book', 'type': 'integer'} + } }, - { - '2254334': 'Items', - 'columns': [{'1': {'name': 'id', 'type': 'integer'}}, - {'2': {'name': 'Barcode', 'type': 'text'}}, - {'3': {'name': 'Acquisition Date', 'type': 'date'}}, - {'5': {'name': 'Book', 'type': 'integer'}}] - }, - { - '2254350': 'Patrons', - 'columns': [{'1': {'name': 'id', 'type': 'integer'}}, - {'2': {'name': 'First Name', 'type': 'text'}}, - {'3': {'name': 'Last Name', 'type': 'text'}}] - }] + '2254350': { + 'name': 'Patrons', + 'columns': { + '1': {'name': 'id', 'type': 'integer'}, + '2': {'name': 'First Name', 'type': 'text'}, + '3': {'name': 'Last Name', 'type': 'text'} + } + } + } } monkeypatch.setattr(tables.base, 'connect', mock_connect) monkeypatch.setattr(tables.base, 'list_joinable_tables', mock_list_joinable_tables)