Skip to content

Commit 660b195

Browse files
committed
Nans: modify a few tests
1 parent e78afc5 commit 660b195

File tree

2 files changed

+16
-6
lines changed

2 files changed

+16
-6
lines changed

integrations/acquisition/covidcast/test_csv_uploading.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -79,15 +79,17 @@ def test_uploading(self):
7979
f.write('tx,NA,0.2,20\n')
8080
f.write('wa,3,0.3,30\n')
8181

82-
# invalid, missing with an inf value and missing cols
82+
# invalid, missing with an inf value
8383
with open(source_receiving_dir + '/20200419_state_test_missing1.csv', 'w') as f:
8484
f.write('geo_id,val,se,sample_size,missing_val,missing_se,missing_sample_size\n')
8585
f.write(f'fl,inf,0.3,30,{Nans.UNKNOWN},{Nans.NOT_MISSING},{Nans.NOT_MISSING}\n')
8686

87+
# invalid, missing with an incorrect missing code
8788
with open(source_receiving_dir + '/20200419_state_test_missing2.csv', 'w') as f:
8889
f.write('geo_id,val,se,sample_size,missing_val,missing_se,missing_sample_size\n')
8990
f.write(f'tx,NA,0.2,20,{Nans.NOT_MISSING},{Nans.NOT_MISSING},{Nans.NOT_MISSING}\n')
9091

92+
# invalid, no missing with an incorrect missing code
9193
with open(source_receiving_dir + '/20200419_state_test_missing3.csv', 'w') as f:
9294
f.write('geo_id,val,se,sample_size,missing_val,missing_se,missing_sample_size\n')
9395
f.write(f'wa,3,0.3,30,{Nans.UNKNOWN},{Nans.NOT_MISSING},{Nans.NOT_MISSING}\n')

integrations/server/test_covidcast_endpoints.py

Lines changed: 13 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@
1212
import mysql.connector
1313
import requests
1414
import pandas as pd
15-
15+
from delphi_utils import Nans
1616

1717
# use the local instance of the Epidata API
1818
BASE_URL = "http://delphi_web_epidata/epidata/covidcast"
@@ -37,9 +37,9 @@ class CovidcastRow:
3737
lag: int = 0
3838
is_latest_issue: bool = True
3939
is_wip: bool = False
40-
# missing_value: Optional[int] = None
41-
# missing_std: Optional[int] = None
42-
# missing_sample_size: Optional[int] = None
40+
missing_value: int = Nans.NOT_MISSING
41+
missing_stderr: int = Nans.NOT_MISSING
42+
missing_sample_size: int = Nans.NOT_MISSING
4343

4444
def __str__(self):
4545
return f"""(
@@ -59,7 +59,10 @@ def __str__(self):
5959
{self.issue},
6060
{self.lag},
6161
{self.is_latest_issue},
62-
{self.is_wip}
62+
{self.is_wip},
63+
{self.missing_value},
64+
{self.missing_stderr},
65+
{self.missing_sample_size}
6366
)"""
6467

6568
@staticmethod
@@ -76,6 +79,9 @@ def from_json(json: Dict[str, Any]) -> "CovidcastRow":
7679
value=json["value"],
7780
stderr=json["stderr"],
7881
sample_size=json["sample_size"],
82+
missing_value=json["missing_value"],
83+
missing_stderr=json["missing_stderr"],
84+
missing_sample_size=json["missing_sample_size"]
7985
)
8086

8187
@property
@@ -138,6 +144,7 @@ def test_basic(self):
138144

139145
rows = [CovidcastRow(time_value=20200401 + i, value=i) for i in range(10)]
140146
first = rows[0]
147+
breakpoint()
141148
self._insert_rows(rows)
142149

143150
with self.subTest("validation"):
@@ -156,6 +163,7 @@ def test_trend(self):
156163
first = rows[0]
157164
last = rows[-1]
158165
ref = rows[num_rows // 2]
166+
breakpoint()
159167
self._insert_rows(rows)
160168

161169
out = self._fetch("/trend", signal=first.signal_pair, geo=first.geo_pair, date=last.time_value, window="20200401-20201212", basis=ref.time_value)

0 commit comments

Comments
 (0)