1. add issues=* handler to apply_issues_filter

2. add unittest for placeholder on issues

Author: LeonLu2

integrations/server/test_covidcast.py

@@ -73,6 +73,18 @@ def _insert_placeholder_set_four(self):
       for i in [4, 5, 6]
     ]
     self._insert_rows(rows)
+    return rows
+
+  def _insert_placeholder_set_five(self):
+    rows = [
+      self._make_placeholder_row(time_value=2000_01_01, value=i*1., stderr=i*10., sample_size=i*100., issue=2000_01_03+i)[0]
+      for i in [1, 2, 3]
+    ] + [
+      # issue intended to overlap with the issue above
+      self._make_placeholder_row(time_value=2000_01_01+i-3, value=i*1., stderr=i*10., sample_size=i*100., issue=2000_01_03+i-3)[0]
+      for i in [4, 5, 6]
+    ]
+    self._insert_rows(rows)
     return rows 
 
   def test_round_trip(self):
@@ -257,6 +269,26 @@ def test_time_values_wildcard(self):
       'message': 'success',
     })
 
+  def test_issues_wildcard(self):
+    """Select all issues with a wildcard query."""
+
+    # insert placeholder data
+    rows = self._insert_placeholder_set_five()
+    expected_issues = [
+      self.expected_from_row(r) for r in rows[:3]
+    ]
+
+    # make the request
+    response, _ = self.request_based_on_row(rows[0], issues="*")
+
+    self.maxDiff = None
+    # assert that the right data came back
+    self.assertEqual(response, {
+      'result': 1,
+      'epidata': expected_issues,
+      'message': 'success',
+    })
+
   def test_signal_wildcard(self):
     """Select all signals with a wildcard query."""
 

src/server/_query.py

@@ -475,8 +475,11 @@ def apply_lag_filter(self, history_table: str, lag: Optional[int]):
 
     def apply_issues_filter(self, history_table: str, issues: Optional[TimeValues]):
         if issues:
-            self.retable(history_table)
-            self.where_integers("issue", issues)
+            if issues == ["*"]:
+                self.retable(history_table)
+            else:
+                self.retable(history_table)
+                self.where_integers("issue", issues)
         return self
 
     def apply_as_of_filter(self, history_table: str, as_of: Optional[int]):

src/server/_validate.py

@@ -127,7 +127,7 @@ def _parse_range(part: str):
 def parse_date(s: str) -> int:
     # parses a given string in format YYYYMMDD or YYYY-MM-DD to a number in the form YYYYMMDD
     try:
-        if s is "*":
+        if s == "*":
             return s
         else:
             return int(s.replace("-", ""))