From: Karl O. Pinc <kop@meme.com>
Date: Thu, 25 Oct 2018 19:16:59 +0000 (-0500)
Subject: Mov mock_logger setup into fixture
X-Git-Url: https://papio.biology.duke.edu/gitweb/?a=commitdiff_plain;h=2ceea645936a16e799f51dfc1fa7adfc34bc5075;p=pgwui_upload_core

Mov mock_logger setup into fixture
---

diff --git a/tests/views/test_upload.py b/tests/views/test_upload.py
index 678fd48..b1acf65 100644
--- a/tests/views/test_upload.py
+++ b/tests/views/test_upload.py
@@ -68,6 +68,10 @@ def isolate_upload_view(monkeypatch):
         monkeypatch.setattr(
             upload, 'TableUploadHandler', MockTableUploadHandler)
 
+        mock_logger = MockLogger()
+        monkeypatch.setattr(upload, 'log', mock_logger)
+        return mock_logger
+
     return run
 
 
@@ -78,11 +82,8 @@ def isolate_upload_view(monkeypatch):
 def test_upload_view_db_not_changed(monkeypatch, isolate_upload_view):
     '''When the db did not change nothing logs'''
 
-    mock_logger = MockLogger()
-    monkeypatch.setattr(upload, 'log', mock_logger)
-
     response = {'db_changed': False}
-    isolate_upload_view(response)
+    mock_logger = isolate_upload_view(response)
     result = upload.upload_view(None)
 
     assert result == response
@@ -92,9 +93,6 @@ def test_upload_view_db_not_changed(monkeypatch, isolate_upload_view):
 def test_upload_view_db_changed(monkeypatch, isolate_upload_view):
     '''When the db did change something logs'''
 
-    mock_logger = MockLogger()
-    monkeypatch.setattr(upload, 'log', mock_logger)
-
     response = {
         'csv_checked': form_constants.CHECKED,
         'db': 'somedb',
@@ -107,7 +105,7 @@ def test_upload_view_db_changed(monkeypatch, isolate_upload_view):
         'upload_null': form_constants.CHECKED,
         'user': 'someuser',
     }
-    isolate_upload_view(response)
+    mock_logger = isolate_upload_view(response)
     result = upload.upload_view(None)
 
     assert result == response