From: Carl Worth <cworth@cworth.org>
Date: Sat, 8 Jan 2022 08:25:01 +0000 (-0800)
Subject: Use empty list instead of empty tuple in a few places
X-Git-Url: https://git.cworth.org/git?a=commitdiff_plain;h=b89aeccacf69d84ace0bd0ac714608547037f161;p=turbot

Use empty list instead of empty tuple in a few places

I don't think the type actually mattered anywhere here, but it might have.
(Leave it to python and its annoying type unsafety to be unhelpful here.)
---

diff --git a/turbot/interaction.py b/turbot/interaction.py
index 86f438c..a6ef7d6 100644
--- a/turbot/interaction.py
+++ b/turbot/interaction.py
@@ -187,7 +187,7 @@ def edit_puzzle(turb, puzzle, trigger_id):
         solved = True
 
     solution_str = None
-    solution_list = puzzle.get("solution", ())
+    solution_list = puzzle.get("solution", [])
     if solution_list:
         solution_str = ", ".join(solution_list)
 
@@ -280,7 +280,7 @@ def edit_puzzle_submission(turb, payload, metadata):
         puzzle['status'] = 'solved'
     else:
         puzzle['status'] = 'unsolved'
-    puzzle['solution'] = ()
+    puzzle['solution'] = []
     solution = state['solution']['solution']['value']
     if solution:
         # Construct a list from a set to avoid any duplicates
@@ -1045,7 +1045,7 @@ def new_puzzle_submission(turb, payload, metadata):
     if rounds:
         puzzle['rounds'] = rounds
 
-    puzzle['solution'] = ()
+    puzzle['solution'] = []
     puzzle['status'] = 'unsolved'
 
     # Create a channel for the puzzle