update_available updated to handle unwanted update

closes #32
betcode-org · Mar 11, 2017 · 61550ec · 61550ec
1 parent 13d779a
commit 61550ec
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 2 deletions.
diff --git a/betfairlightweight/__init__.py b/betfairlightweight/__init__.py
@@ -5,5 +5,5 @@
 
 
 __title__ = 'betfairlightweight'
-__version__ = '0.9.9'
+__version__ = '0.9.10'
 __author__ = 'Liam Pauling'
diff --git a/betfairlightweight/utils.py b/betfairlightweight/utils.py
@@ -102,5 +102,6 @@ def update_available(available, book_update, deletion_select):
                     updated = True
                     break
 
-        if not updated:
+        if not updated and book[deletion_select] != 0:
+            # handles betfair bug, http://forum.bdp.betfair.com/showthread.php?t=3351
             available.append(book)
diff --git a/tests/test_utils.py b/tests/test_utils.py
@@ -144,6 +144,14 @@ def test_update_available_new_replace(self):
         update_available(current, book_update, 2)
         assert current == expected
 
+        # tests handling of betfair bug, http://forum.bdp.betfair.com/showthread.php?t=3351
+        book_update = [[2, 0, 0], [1, 1.01, 9835.74], [0, 1.02, 1126.22]]
+        current = [[1, 1.01, 9835.74], [0, 1.02, 1126.22]]
+        expected = [[1, 1.01, 9835.74], [0, 1.02, 1126.22]]
+
+        update_available(current, book_update, 2)
+        assert current == expected
+
     def test_update_available_new_remove(self):
         book_update = [[27, 0]]
         current = [[27, 0.95], [13, 28.01], [1.02, 1157.21]]