Adds a test for issue #17 and fixes comment about ReshapePhase which …

…is now gone

phaser/phase.py

@@ -34,7 +34,6 @@ def __init__(self,
     def load_data(self, data):
         """ Call this method to pass record-oriented data to the Phase before calling 'run'
         Can be overridden to load data in a different structure.
-        Used in phaser's builtin phases - by regular Phase and ReshapePhase.
         Note that in normal operation, a Records object is passed in with Record objects and row numbers -
         however if a Phase is being used in tests, it makes testing a lot easier if load_data can take a
         raw list of dicts and row numbers get added.  """

tests/test_columns.py

@@ -109,6 +109,12 @@ def test_rename_passed_string():
     assert phase.headers == [col.name]
 
 
+@pytest.mark.skip("Todo - issue #17")
+def test_conflicting_renames():
+    with pytest.raises(PhaserError):
+        Phase(columns=[FloatColumn(name="Division", rename='div'), IntColumn(name="Divisor", rename='div')])
+
+
 def test_canonicalize_names():
     col1 = Column("Country of Origin")
     phase = Phase(columns=[col1])