remove examples from static analysis

Makpar-Innovation-Laboratory · Feb 8, 2022 · 2ac7f18 · 2ac7f18
1 parent f39e07b
commit 2ac7f18
Show file tree

Hide file tree

Showing 5 changed files with 23 additions and 11 deletions.
diff --git 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 → scripts/examples/example_document_crud.py b/scripts/examples/document_crud.py → 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 → scripts/examples/example_queries.py b/scripts/examples/queries.py → 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 → scripts/examples/pyqldb/example_ion_loads.py b/scripts/examples/pyqldb/ion_loads.py → scripts/examples/pyqldb/example_ion_loads.py
diff --git a/src/innoldb/tests/test_qldb.py b/src/innoldb/tests/test_qldb.py
@@ -58,37 +58,44 @@ 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)
     assert document.test.test2 == 'value'
     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')