Skip to content

Commit 574942b

Browse files
committed
Update convert_unit
1 parent f1de5fd commit 574942b

File tree

1 file changed

+21
-6
lines changed

1 file changed

+21
-6
lines changed

tests/integration/test_unit_conversion.py

Lines changed: 21 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -220,8 +220,9 @@ def test_convert_unit_ur_injection():
220220
)
221221

222222

223+
@pobj_type
223224
@check_auto_index_casting_pobj
224-
def test_convert_unit_mapping(only_two_index_levels_pobj):
225+
def test_convert_unit_mapping(only_two_index_levels_pobj, pobj_type):
225226
pytest.importorskip("pint")
226227

227228
start = create_test_df(
@@ -240,6 +241,8 @@ def test_convert_unit_mapping(only_two_index_levels_pobj):
240241
& (start.index.get_level_values("run") == 0)
241242
].reset_index(["scenario", "run"], drop=True)
242243

244+
start = convert_to_desired_type(start, pobj_type)
245+
243246
# Don't convert W / m^2
244247
res = convert_unit(start, {"K": "degC", "ZJ": "J"})
245248

@@ -262,8 +265,9 @@ def test_convert_unit_mapping(only_two_index_levels_pobj):
262265
)
263266

264267

268+
@pobj_type
265269
@check_auto_index_casting_pobj
266-
def test_convert_series(only_two_index_levels_pobj):
270+
def test_convert_series(only_two_index_levels_pobj, pobj_type):
267271
pytest.importorskip("pint")
268272

269273
# Check that conversion works if user supplies a Series of target units
@@ -283,6 +287,8 @@ def test_convert_series(only_two_index_levels_pobj):
283287
& (start.index.get_level_values("run") == 0)
284288
].reset_index(["scenario", "run"], drop=True)
285289

290+
start = convert_to_desired_type(start, pobj_type)
291+
286292
desired_units = (
287293
start.loc[start.index.get_level_values("variable") != "temperature"]
288294
.reset_index("unit")["unit"]
@@ -309,8 +315,9 @@ def test_convert_series(only_two_index_levels_pobj):
309315
)
310316

311317

318+
@pobj_type
312319
@check_auto_index_casting_pobj
313-
def test_convert_series_all_rows(only_two_index_levels_pobj):
320+
def test_convert_series_all_rows(only_two_index_levels_pobj, pobj_type):
314321
pytest.importorskip("pint")
315322

316323
start = create_test_df(
@@ -329,6 +336,8 @@ def test_convert_series_all_rows(only_two_index_levels_pobj):
329336
& (start.index.get_level_values("run") == 0)
330337
].reset_index(["scenario", "run"], drop=True)
331338

339+
start = convert_to_desired_type(start, pobj_type)
340+
332341
desired_units = start.reset_index("unit")["unit"].replace(
333342
{"W / m^2": "ZJ / yr / m^2", "ZJ": "PJ"}
334343
)
@@ -353,8 +362,9 @@ def test_convert_series_all_rows(only_two_index_levels_pobj):
353362
)
354363

355364

365+
@pobj_type
356366
@check_auto_index_casting_pobj
357-
def test_convert_series_extra_rows(only_two_index_levels_pobj):
367+
def test_convert_series_extra_rows(only_two_index_levels_pobj, pobj_type):
358368
pytest.importorskip("pint")
359369

360370
start = create_test_df(
@@ -373,6 +383,8 @@ def test_convert_series_extra_rows(only_two_index_levels_pobj):
373383
& (start.index.get_level_values("run") == 0)
374384
].reset_index(["scenario", "run"], drop=True)
375385

386+
start = convert_to_desired_type(start, pobj_type)
387+
376388
desired_units = start.reset_index("unit")["unit"].replace(
377389
{"W / m^2": "ZJ / yr / m^2", "ZJ": "PJ"}
378390
)
@@ -403,8 +415,9 @@ def test_convert_series_extra_rows(only_two_index_levels_pobj):
403415
)
404416

405417

418+
@pobj_type
406419
@check_auto_index_casting_pobj
407-
def test_convert_unit_like_no_op(only_two_index_levels_pobj):
420+
def test_convert_unit_like_no_op(only_two_index_levels_pobj, pobj_type):
408421
start = create_test_df(
409422
variables=[
410423
("Cold", "mK"),
@@ -421,9 +434,11 @@ def test_convert_unit_like_no_op(only_two_index_levels_pobj):
421434
& (start.index.get_level_values("run") == 0)
422435
].reset_index(["scenario", "run"], drop=True)
423436

437+
start = convert_to_desired_type(start, pobj_type)
438+
424439
res = convert_unit_like(start, start)
425440

426-
pd.testing.assert_frame_equal(res, start)
441+
check_result(res, start)
427442

428443

429444
check_auto_index_casting_target = pytest.mark.parametrize(

0 commit comments

Comments
 (0)