Skip to content

Commit 1bea06a

Browse files
committed
fixup test
1 parent 4935eab commit 1bea06a

File tree

1 file changed

+15
-18
lines changed

1 file changed

+15
-18
lines changed

tests/test_ods_package.py

Lines changed: 15 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -553,7 +553,7 @@ def test_unknown_column_management(self):
553553
assert 'FlexiLocdefault_rename' in oed_saved.location.dataframe.columns
554554

555555
def test_setting_schema_analysis__is_valid(self):
556-
file_name = 'data/analysis_settings.json'
556+
file_name = 'analysis_settings.json'
557557
file_url = f'https://raw.githubusercontent.com/OasisLMF/OasisPiWind/{piwind_branch}/{file_name}'
558558
ods_analysis_setting = AnalysisSettingSchema()
559559
assert (ods_analysis_setting.schema is not None)
@@ -563,10 +563,10 @@ def test_setting_schema_analysis__is_valid(self):
563563
abs_dir.mkdir()
564564

565565
with urllib.request.urlopen(file_url) as response, \
566-
open(pathlib.Path(tmp_dir, 'data/analysis_settings.json'), 'wb') as out_file:
566+
open(pathlib.Path(tmp_dir, 'analysis_settings.json'), 'wb') as out_file:
567567
shutil.copyfileobj(response, out_file)
568568

569-
settings_fp = pathlib.Path(tmp_dir, 'data/analysis_settings.json')
569+
settings_fp = pathlib.Path(tmp_dir, 'analysis_settings.json')
570570
settings_dict = ods_analysis_setting.load(settings_fp)
571571
valid, errors = ods_analysis_setting.validate(settings_dict)
572572

@@ -575,7 +575,7 @@ def test_setting_schema_analysis__is_valid(self):
575575
self.assertEqual(settings_dict, ods_analysis_setting.get(settings_fp))
576576

577577
def test_setting_schema_analysis__is_invalid(self):
578-
file_name = 'data/analysis_settings.json'
578+
file_name = 'analysis_settings.json'
579579
file_url = f'https://raw.githubusercontent.com/OasisLMF/OasisPiWind/{piwind_branch}/{file_name}'
580580
ods_analysis_setting = AnalysisSettingSchema()
581581
assert (ods_analysis_setting.schema is not None)
@@ -585,10 +585,10 @@ def test_setting_schema_analysis__is_invalid(self):
585585
abs_dir.mkdir()
586586

587587
with urllib.request.urlopen(file_url) as response, \
588-
open(pathlib.Path(tmp_dir, 'data/analysis_settings.json'), 'wb') as out_file:
588+
open(pathlib.Path(tmp_dir, 'analysis_settings.json'), 'wb') as out_file:
589589
shutil.copyfileobj(response, out_file)
590590

591-
settings_fp = pathlib.Path(tmp_dir, 'data/analysis_settings.json')
591+
settings_fp = pathlib.Path(tmp_dir, 'analysis_settings.json')
592592
settings_dict = ods_analysis_setting.load(settings_fp)
593593

594594
# Insert errors
@@ -599,12 +599,9 @@ def test_setting_schema_analysis__is_invalid(self):
599599

600600
valid, errors = ods_analysis_setting.validate(settings_dict, raise_error=False)
601601
self.assertFalse(valid)
602-
self.assertEqual({
603-
'ri_summaries': ['id 1 is duplicated'],
604-
'analysis_settings_schema gul_output': ["'True' is not of type 'boolean'"],
605-
'analysis_settings_schema required': ["'gul_summaries' is a required property"],
606-
}, errors)
607-
602+
self.assertEqual({'gul_output': ["'True' is not of type 'boolean'"],
603+
'required': ["'gul_summaries' is a required property"],
604+
'ri_summaries': ['id 1 is duplicated']}, errors)
608605
with self.assertRaises(OdsException):
609606
ods_analysis_setting.validate(settings_dict)
610607

@@ -619,10 +616,10 @@ def test_setting_schema_model__is_valid(self):
619616
abs_dir.mkdir()
620617

621618
with urllib.request.urlopen(file_url) as response, \
622-
open(pathlib.Path(tmp_dir, 'data/model_settings.json'), 'wb') as out_file:
619+
open(pathlib.Path(tmp_dir, 'model_settings.json'), 'wb') as out_file:
623620
shutil.copyfileobj(response, out_file)
624621

625-
settings_fp = pathlib.Path(tmp_dir, 'data/model_settings.json')
622+
settings_fp = pathlib.Path(tmp_dir, 'model_settings.json')
626623
settings_dict = ods_model_setting.load(settings_fp)
627624
valid, errors = ods_model_setting.validate(settings_dict)
628625

@@ -641,10 +638,10 @@ def test_setting_schema_model__is_invalid(self):
641638
abs_dir.mkdir()
642639

643640
with urllib.request.urlopen(file_url) as response, \
644-
open(pathlib.Path(tmp_dir, 'data/model_settings.json'), 'wb') as out_file:
641+
open(pathlib.Path(tmp_dir, 'model_settings.json'), 'wb') as out_file:
645642
shutil.copyfileobj(response, out_file)
646643

647-
settings_fp = pathlib.Path(tmp_dir, 'data/model_settings.json')
644+
settings_fp = pathlib.Path(tmp_dir, 'model_settings.json')
648645
settings_dict = ods_model_setting.load(settings_fp)
649646

650647
# Insert errors
@@ -655,8 +652,8 @@ def test_setting_schema_model__is_invalid(self):
655652
valid, errors = ods_model_setting.validate(settings_dict, raise_error=False)
656653
self.assertFalse(valid)
657654
self.assertEqual({
658-
'model_settings_schema additionalProperties': ["Additional properties are not allowed ('model_setting', 'unknown_key' were unexpected)"],
659-
'model_settings_schema required': ["'model_settings' is a required property"]
655+
'additionalProperties': ["Additional properties are not allowed ('model_setting', 'unknown_key' were unexpected)"],
656+
'required': ["'model_settings' is a required property"]
660657
}, errors)
661658

662659
with self.assertRaises(OdsException):

0 commit comments

Comments
 (0)