refining CountriesMergerTest
This commit is contained in:
@@ -7,7 +7,7 @@ class CountriesMerger:
|
||||
def mergeSrcIntoDst(src: pd.Series, dst: pd.Series):
|
||||
def merge(series):
|
||||
series = series.dropna()
|
||||
return sorted(series[0].union(series[1:]))
|
||||
return sorted(set().union(*series))
|
||||
return (pd
|
||||
.merge(dst, src, how='left', left_index=True, right_index=True)
|
||||
.apply(merge, axis='columns'))
|
||||
|
||||
@@ -8,8 +8,8 @@ class CountriesMergerTest(unittest.TestCase):
|
||||
|
||||
def test_mergeSrcIntoDst(self):
|
||||
# Given
|
||||
dstCountries = pd.Series({'NO72A': {'CountryA-1'}, 'EW096': {'CountryA-2'}}, name='dstCountries')
|
||||
srcCountries = pd.Series({'NO72A': 'CountryB-1', 'FS192': 'CountryB-2'}, name='srcCountries')
|
||||
dstCountries = pd.Series({'NO72A': ['CountryA-1'], 'EW096': ['CountryA-2']}, name='dstCountries')
|
||||
srcCountries = pd.Series({'NO72A': ['CountryB-1'], 'FS192': ['CountryB-2']}, name='srcCountries')
|
||||
|
||||
# When
|
||||
mergedCountries = CountriesMerger.mergeSrcIntoDst(src = srcCountries, dst = dstCountries)
|
||||
|
||||
Reference in New Issue
Block a user