From b397aba955da89b035088e2749cf1b143c11be1f Mon Sep 17 00:00:00 2001
From: kryofly <kryofly@space>
Date: Sun, 4 Feb 2018 22:15:14 +0100
Subject: [PATCH] tests: document kludge

---
 freqtrade/tests/conftest.py     | 1 +
 freqtrade/tests/test_analyze.py | 7 +++++--
 2 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/freqtrade/tests/conftest.py b/freqtrade/tests/conftest.py
index 2b1d14268..edeb89a59 100644
--- a/freqtrade/tests/conftest.py
+++ b/freqtrade/tests/conftest.py
@@ -261,6 +261,7 @@ def ticker_history_without_bv():
     ]
 
 
+# FIX: Perhaps change result fixture to use BTC_UNITEST instead?
 @pytest.fixture
 def result():
     with open('freqtrade/tests/testdata/BTC_ETH-1.json') as data_file:
diff --git a/freqtrade/tests/test_analyze.py b/freqtrade/tests/test_analyze.py
index 41a6c1c2f..c936d288b 100644
--- a/freqtrade/tests/test_analyze.py
+++ b/freqtrade/tests/test_analyze.py
@@ -19,8 +19,11 @@ def test_dataframe_correct_columns(result):
 
 
 def test_dataframe_correct_length(result):
-    # no idea what this check truly does - should we just remove it?
-    assert len(result.index) == 14397
+    # FIX: no idea what this check truly does - should we just remove it?
+    # magic needs to change everytime testdata/* is updated.
+    # Perhaps change result fixture to use BTC_UNITEST instead?
+    magic = 14397
+    assert len(result.index) == magic
 
 
 def test_populates_buy_trend(result):