From 4520bb74478258b0842bb2af69658cbb99d6884b Mon Sep 17 00:00:00 2001
From: Oliver Walters <oliver.henry.walters@gmail.com>
Date: Thu, 27 May 2021 16:36:26 +1000
Subject: [PATCH] PEP style fixes

---
 InvenTree/InvenTree/exchange.py |  2 +-
 InvenTree/InvenTree/views.py    |  3 +--
 InvenTree/common/models.py      |  1 -
 InvenTree/common/test_views.py  | 11 +++--------
 InvenTree/company/tests.py      | 13 -------------
 5 files changed, 5 insertions(+), 25 deletions(-)

diff --git a/InvenTree/InvenTree/exchange.py b/InvenTree/InvenTree/exchange.py
index dfbfff872c..0695e69f48 100644
--- a/InvenTree/InvenTree/exchange.py
+++ b/InvenTree/InvenTree/exchange.py
@@ -7,7 +7,7 @@ class InvenTreeExchange(SimpleExchangeBackend):
     """
     Backend for automatically updating currency exchange rates.
 
-    Uses the exchangerate.host service API 
+    Uses the exchangerate.host service API
     """
 
     name = "InvenTreeExchange"
diff --git a/InvenTree/InvenTree/views.py b/InvenTree/InvenTree/views.py
index 4189def492..4ac90bd722 100644
--- a/InvenTree/InvenTree/views.py
+++ b/InvenTree/InvenTree/views.py
@@ -26,7 +26,6 @@ from part.models import Part, PartCategory
 from stock.models import StockLocation, StockItem
 from common.models import InvenTreeSetting, ColorTheme
 from users.models import check_user_role, RuleSet
-from InvenTree.helpers import clean_decimal
 
 import InvenTree.tasks
 
@@ -775,7 +774,6 @@ class SettingsView(TemplateView):
         return ctx
 
 
-
 class CurrencyRefreshView(RedirectView):
 
     url = reverse_lazy("settings-currencies")
@@ -819,6 +817,7 @@ class CurrencySettingsView(TemplateView):
 
         return ctx
 
+
 class AppearanceSelectView(FormView):
     """ View for selecting a color theme """
 
diff --git a/InvenTree/common/models.py b/InvenTree/common/models.py
index 6cfe5915e0..236e48770f 100644
--- a/InvenTree/common/models.py
+++ b/InvenTree/common/models.py
@@ -14,7 +14,6 @@ from django.db import models, transaction
 from django.db.utils import IntegrityError, OperationalError
 from django.conf import settings
 
-import djmoney.settings
 from djmoney.models.fields import MoneyField
 from djmoney.contrib.exchange.models import convert_money
 from djmoney.contrib.exchange.exceptions import MissingRate
diff --git a/InvenTree/common/test_views.py b/InvenTree/common/test_views.py
index 8dc5830108..56a244ba0c 100644
--- a/InvenTree/common/test_views.py
+++ b/InvenTree/common/test_views.py
@@ -98,20 +98,15 @@ class SettingsViewTest(TestCase):
         Tests for a setting which has choices
         """
 
-        setting = InvenTreeSetting.get_setting_object('INVENTREE_DEFAULT_CURRENCY')
+        setting = InvenTreeSetting.get_setting_object('PURCHASEORDER_REFERENCE_PREFIX')
 
         # Default value!
-        self.assertEqual(setting.value, 'USD')
+        self.assertEqual(setting.value, 'PO')
 
         url = self.get_url(setting.pk)
 
         # Try posting an invalid currency option
-        data, errors = self.post(url, {'value': 'XPQaaa'}, valid=False)
-
-        self.assertIsNotNone(errors.get('value'), None)
-
-        # Try posting a valid currency option
-        data, errors = self.post(url, {'value': 'AUD'}, valid=True)
+        data, errors = self.post(url, {'value': 'Purchase Order'}, valid=True)
 
     def test_binary_values(self):
         """
diff --git a/InvenTree/company/tests.py b/InvenTree/company/tests.py
index 2c6e722440..b1e05efe14 100644
--- a/InvenTree/company/tests.py
+++ b/InvenTree/company/tests.py
@@ -11,9 +11,6 @@ from .models import Company, Contact, ManufacturerPart, SupplierPart
 from .models import rename_company_image
 from part.models import Part
 
-from InvenTree.exchange import InvenTreeManualExchangeBackend
-from djmoney.contrib.exchange.models import Rate
-
 
 class CompanySimpleTest(TestCase):
 
@@ -40,16 +37,6 @@ class CompanySimpleTest(TestCase):
         self.acme0002 = SupplierPart.objects.get(SKU='ACME0002')
         self.zerglphs = SupplierPart.objects.get(SKU='ZERGLPHS')
         self.zergm312 = SupplierPart.objects.get(SKU='ZERGM312')
-        
-        # Exchange rate backend
-        backend = InvenTreeManualExchangeBackend()
-        backend.update_rates(base_currency=backend.base_currency)
-
-        Rate.objects.create(
-            currency='AUD',
-            value='1.35',
-            backend_id=backend.name,
-        )
 
     def test_company_model(self):
         c = Company.objects.get(name='ABC Co.')