From e0433c47181dbcda535e385179cbb4ebf9e05069 Mon Sep 17 00:00:00 2001
From: Freddie Vargus <fvargus4@bu.edu>
Date: Mon, 3 Apr 2017 12:04:11 -0400
Subject: [PATCH] MAINT: Add better repr for transactions

Flake8
---
 tests/finance/test_transaction.py | 35 +++++++++++++++++++++++++++++++
 zipline/finance/transaction.py    | 14 +++++++++++++
 2 files changed, 49 insertions(+)
 create mode 100644 tests/finance/test_transaction.py

diff --git a/tests/finance/test_transaction.py b/tests/finance/test_transaction.py
new file mode 100644
index 00000000..161fa80b
--- /dev/null
+++ b/tests/finance/test_transaction.py
@@ -0,0 +1,35 @@
+#
+# Copyright 2017 Quantopian, Inc.
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+import pandas as pd
+from unittest import TestCase
+
+from zipline.assets import Equity
+from zipline.finance.transaction import Transaction
+
+
+class TransactionTestCase(TestCase):
+
+    def test_transaction_repr(self):
+        dt = pd.Timestamp('2017-01-01')
+
+        asset = Equity(1, exchange='test')
+        txn = Transaction(asset, amount=100, dt=dt, price=10, order_id=0)
+
+        expected = (
+            "Transaction(asset=Equity(1), dt=2017-01-01 00:00:00,"
+            " amount=100, price=10)"
+        )
+
+        self.assertEqual(repr(txn), expected)
diff --git a/zipline/finance/transaction.py b/zipline/finance/transaction.py
index 94b03300..adde43cc 100644
--- a/zipline/finance/transaction.py
+++ b/zipline/finance/transaction.py
@@ -35,6 +35,20 @@ class Transaction(object):
     def __getitem__(self, name):
         return self.__dict__[name]
 
+    def __repr__(self):
+        template = (
+            "{cls}(asset={asset}, dt={dt},"
+            " amount={amount}, price={price})"
+        )
+
+        return template.format(
+            cls=type(self).__name__,
+            asset=self.asset,
+            dt=self.dt,
+            amount=self.amount,
+            price=self.price
+        )
+
     def to_dict(self):
         py = copy(self.__dict__)
         del py['type']