From 8eb9ee388dc243bdc2d60ff3ce0dae1bb06e99d6 Mon Sep 17 00:00:00 2001 From: Reid Miller Date: Thu, 7 Sep 2017 21:46:24 -0400 Subject: [PATCH 1/3] #141 Remove SPP from client_factory(...) until implemented. This would suggest to our users that the SPP client exists, where currently spp.py doesn't do anything. --- pyiso/__init__.py | 1 - pyiso/spp.py | 1 - 2 files changed, 2 deletions(-) diff --git a/pyiso/__init__.py b/pyiso/__init__.py index 368c732..d7596ce 100644 --- a/pyiso/__init__.py +++ b/pyiso/__init__.py @@ -39,7 +39,6 @@ 'PJM': {'class': 'PJMClient', 'module': 'pjm'}, 'PNM': {'class': 'SVERIClient', 'module': 'sveri'}, 'SASK': {'class': 'SaskPowerClient', 'module': 'sask'}, - 'SPP': {'class': 'SPPClient', 'module': 'spp'}, 'SPPC': {'class': 'NVEnergyClient', 'module': 'nvenergy'}, 'SRP': {'class': 'SVERIClient', 'module': 'sveri'}, 'TEPC': {'class': 'SVERIClient', 'module': 'sveri'}, diff --git a/pyiso/spp.py b/pyiso/spp.py index 036227d..d3ec2dd 100644 --- a/pyiso/spp.py +++ b/pyiso/spp.py @@ -20,7 +20,6 @@ def get_fuels(self, year=2014): 'WASTE': 'refuse', 'WIND': 'wind', } - else: return { 'COAL': 'coal', From e32dc6861ee2588e7ade5c1f3b3eafca30f0bad8 Mon Sep 17 00:00:00 2001 From: Reid Miller Date: Thu, 7 Sep 2017 21:47:22 -0400 Subject: [PATCH 2/3] #130 Deleted test_spp.py which consisted entirely of comments. We can create new test cases once an SPP client is implemented for #141. --- tests/test_spp.py | 18 ------------------ 1 file changed, 18 deletions(-) delete mode 100644 tests/test_spp.py diff --git a/tests/test_spp.py b/tests/test_spp.py deleted file mode 100644 index ef22059..0000000 --- a/tests/test_spp.py +++ /dev/null @@ -1,18 +0,0 @@ -from pyiso import client_factory -from unittest import TestCase - - -# class TestSPP(TestCase): -# def setUp(self): -# self.c = client_factory('SPP') - - -# def test_auth_keys(self): -# auth, idstr, headers = self.c.auth_keys() -# self.assertEqual(len(auth), 8) -# self.assertEqual(len(idstr), 12+50) -# self.assertIn('set-cookie', headers) - - # def test_fetch_csv(self): - # self.c.handle_options(data='gen') - # self.c.fetch_csv(*self.c.auth_keys()) From 2ec01b0d2a50ab377a54d302a53e91d6aa7ef279 Mon Sep 17 00:00:00 2001 From: Reid Miller Date: Fri, 8 Sep 2017 07:46:51 -0400 Subject: [PATCH 3/3] #141 Removed SPP from the client_factor unit tests. --- tests/unit/test_factory.py | 3 --- 1 file changed, 3 deletions(-) diff --git a/tests/unit/test_factory.py b/tests/unit/test_factory.py index 586130b..51d267c 100644 --- a/tests/unit/test_factory.py +++ b/tests/unit/test_factory.py @@ -17,9 +17,6 @@ def setUp(self): { 'name': 'MISO' }, - { - 'name': 'SPP', - }, { 'name': 'BPA', },