From 2ac7f1838dfbe46246d91a7c2f63449b0362d92d Mon Sep 17 00:00:00 2001 From: Grant Moore Date: Tue, 8 Feb 2022 09:06:02 -0500 Subject: [PATCH] remove examples from static analysis --- .deepsource.toml | 4 ++-- ...ument_crud.py => example_document_crud.py} | 3 ++- .../{queries.py => example_queries.py} | 3 ++- .../{ion_loads.py => example_ion_loads.py} | 0 src/innoldb/tests/test_qldb.py | 24 +++++++++++++------ 5 files changed, 23 insertions(+), 11 deletions(-) rename scripts/examples/{document_crud.py => example_document_crud.py} (90%) rename scripts/examples/{queries.py => example_queries.py} (77%) rename scripts/examples/pyqldb/{ion_loads.py => example_ion_loads.py} (100%) diff --git a/.deepsource.toml b/.deepsource.toml index 09a66eb..dfe8ce9 100644 --- a/.deepsource.toml +++ b/.deepsource.toml @@ -1,7 +1,7 @@ version = 1 -test_patterns = ["**/test_*.py"] -exclude_patterns = ["**/test_*.py", "**/__init__.py"] +test_patterns = ["**/test_*.py", "**/example_*.py"] +exclude_patterns = ["**/test_*.py", "**/__init__.py", "**/example_*.py"] [[analyzers]] name = "test-coverage" diff --git a/scripts/examples/document_crud.py b/scripts/examples/example_document_crud.py similarity index 90% rename from scripts/examples/document_crud.py rename to scripts/examples/example_document_crud.py index a0a4125..0107713 100644 --- a/scripts/examples/document_crud.py +++ b/scripts/examples/example_document_crud.py @@ -1,8 +1,9 @@ -from innoldb.qldb import Document import os os.environ['LEDGER'] = 'innolab' +# Import needs to come after environment variable has been set! +from innoldb.qldb import Document doc = Document('table_name') diff --git a/scripts/examples/queries.py b/scripts/examples/example_queries.py similarity index 77% rename from scripts/examples/queries.py rename to scripts/examples/example_queries.py index fb0b590..a1bff90 100644 --- a/scripts/examples/queries.py +++ b/scripts/examples/example_queries.py @@ -1,8 +1,9 @@ -from innoldb.qldb import Query, Document import os os.environ['LEDGER'] = 'innolab' +# Import needs to come after environment variable has been set! +from innoldb.qldb import Query, Document query = Query('table_name').all() diff --git a/scripts/examples/pyqldb/ion_loads.py b/scripts/examples/pyqldb/example_ion_loads.py similarity index 100% rename from scripts/examples/pyqldb/ion_loads.py rename to scripts/examples/pyqldb/example_ion_loads.py diff --git a/src/innoldb/tests/test_qldb.py b/src/innoldb/tests/test_qldb.py index e6a9ea0..548bb70 100644 --- a/src/innoldb/tests/test_qldb.py +++ b/src/innoldb/tests/test_qldb.py @@ -58,25 +58,31 @@ def test_document_snapshot(mock_create_index, mock_create_table, mock_tables, mo assert document.test == 'prop' assert document.test2 == 'prop2' + @patch('qldb.Driver.driver') @patch('qldb.Driver.tables') @patch('qldb.Driver.create_table') @patch('qldb.Driver.create_index') -def test_document_snapshot_native_nesting_deserialization(mock_create_index, mock_create_table, mock_tables, mock_driver): +def test_document_snapshot_nested_deserialization(mock_create_index, mock_create_table, mock_tables, mock_driver): document = Document(table='table', ledger='ledger', snapshot={ - 'test': { 'test2': 'value' } + 'test': { + 'test2': 'value' + } }) assert mock_driver.call_count == 2 assert isinstance(document.test, Strut) assert document.test.test2 == 'value' + @patch('qldb.Driver.driver') @patch('qldb.Driver.tables') @patch('qldb.Driver.create_table') @patch('qldb.Driver.create_index') -def test_document_snapshot_native_nesting_deserialization_more(mock_create_index, mock_create_table, mock_tables, mock_driver): +def test_document_snapshot_nested_deserialization_more(mock_create_index, mock_create_table, mock_tables, mock_driver): document = Document(table='table', ledger='ledger', snapshot={ - 'test': { 'test2': 'value', 'test3': 'another value', 'test4': 'yet another value' } + 'test': { + 'test2': 'value', 'test3': 'another value', 'test4': 'yet another value' + } }) assert mock_driver.call_count == 2 assert isinstance(document.test, Strut) @@ -84,11 +90,12 @@ def test_document_snapshot_native_nesting_deserialization_more(mock_create_index assert document.test.test3 == 'another value' assert document.test.test4 == 'yet another value' + @patch('qldb.Driver.driver') @patch('qldb.Driver.tables') @patch('qldb.Driver.create_table') @patch('qldb.Driver.create_index') -def test_document_snapshot_native_nesting_one_layer(mock_create_index, mock_create_table, mock_tables, mock_driver): +def test_document_snapshot_nested_deserialization_one_layer(mock_create_index, mock_create_table, mock_tables, mock_driver): document = Document(table='table', ledger='ledger', snapshot={ 'test_1': { 'test_2': { @@ -101,11 +108,12 @@ def test_document_snapshot_native_nesting_one_layer(mock_create_index, mock_crea assert isinstance(document.test_1.test_2, Strut) assert document.test_1.test_2.test_3 == 45 + @patch('qldb.Driver.driver') @patch('qldb.Driver.tables') @patch('qldb.Driver.create_table') @patch('qldb.Driver.create_index') -def test_document_snapshot_native_nesting_one_layer_more(mock_create_index, mock_create_table, mock_tables, mock_driver): +def test_document_snapshot_nested_deserialization_one_layer_more(mock_create_index, mock_create_table, mock_tables, mock_driver): document = Document(table='table', ledger='ledger', snapshot={ 'test_1': { 'test_2': { @@ -122,11 +130,12 @@ def test_document_snapshot_native_nesting_one_layer_more(mock_create_index, mock assert document.test_1.test_2.test_4 == 100 assert document.test_1.test_2.test_5 == 'tester' + @patch('qldb.Driver.driver') @patch('qldb.Driver.tables') @patch('qldb.Driver.create_table') @patch('qldb.Driver.create_index') -def test_document_snapshot_native_nesting_one_layer_complex(mock_create_index, mock_create_table, mock_tables, mock_driver): +def test_document_snapshot_nested_deserialization_one_layer_complex(mock_create_index, mock_create_table, mock_tables, mock_driver): document = Document(table='table', ledger='ledger', snapshot={ 'test_1': { 'test_2': { @@ -150,6 +159,7 @@ def test_document_snapshot_native_nesting_one_layer_complex(mock_create_index, m assert document.test_1.test_6.test_7 == 'will it work?' assert document.test_1.test_8 == 'last but not least' + @patch('qldb.Driver.driver') @patch('qldb.Driver.tables') @patch('qldb.Driver.create_table')