Remove unittest

[kfrey-idm]

No description provided.

[Copilot]

Pull request overview

This pull request removes the unittest module dependency across the test suite, converting test classes from unittest.TestCase subclasses to plain classes. The changes also update decorator patterns from unittest to pytest equivalents and remove unittest.main() calls from test files.

Changes:

• Removed import unittest statements from all test files
• Converted test classes from inheriting unittest.TestCase to plain classes
• Replaced unittest decorators with pytest equivalents (@unittest.skipIf → @pytest.mark.skipif, @unittest.skip → @pytest.mark.skip)
• Removed unittest.main() calls from test entry points

Reviewed changes

Copilot reviewed 23 out of 23 changed files in this pull request and generated 16 comments.

Show a summary per file

File	Description
tests/unittests/test_version.py	Removed unittest import and converted test class to plain class
tests/unittests/test_susceptibility_distribution.py	Removed unittest import and converted test class to plain class
tests/unittests/test_schema_imports.py	Removed unittest import and converted test class to plain class
tests/unittests/test_properties_and_attributes.py	Removed unittest import and converted test classes to plain classes
tests/unittests/test_mortality_distribution.py	Removed unittest import and converted test class to plain class
tests/unittests/test_migration_imports.py	Removed unittest import and converted test class to plain class
tests/unittests/test_individual_properties.py	Removed unittest import and converted test class to plain class
tests/unittests/test_fertility_distribution.py	Removed unittest import and converted test class to plain class
tests/unittests/test_demographics_imports.py	Removed unittest import and converted test class to plain class
tests/unittests/test_config_imports.py	Removed unittest import and converted test class to plain class
tests/unittests/test_age_distribution.py	Removed unittest import and converted test class to plain class
tests/test_weather_files.py	Removed unittest import and converted test classes to plain classes
tests/test_spatial_reports.py	Removed unittest import and converted test classes to plain classes
tests/test_serialization.py	Replaced unittest decorators with pytest equivalents and converted test classes
tests/test_property_reports.py	Removed unittest import and converted test classes to plain classes
tests/test_node.py	Removed unittest import, converted test class, and removed unittest.main() call
tests/test_migration.py	Removed unittest import, converted test class, and removed unittest.main() call
tests/test_demog_send_over_pipes.py	Replaced unittest decorators with pytest equivalents and converted test class
tests/test_demog.py	Removed unittest import and converted test classes to plain classes
tests/test_config_demog.py	Removed unittest import, converted test class, and removed unittest.main() call
tests/test_config.py	Removed unittest import, converted test classes, and removed unittest.main() call
tests/test_channel_reports.py	Replaced unittest decorator with pytest equivalent and converted test classes
tests/test_campaign_module.py	Removed unittest import, converted test class, and removed unittest.main() call

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

The code uses self.assertDictEqual() which is a unittest assertion method, but the class no longer inherits from unittest.TestCase. This will cause an AttributeError at runtime. Replace with pytest assertions like assert individual_property.to_dict() == {'Property': 'blah', 'Values': ['blah1', 'blah2']}.

[Copilot]

The pytest.mark.skipif decorator requires a condition and a reason parameter. The correct syntax is @pytest.mark.skipif(skip_tests, reason='Skipping old tests to focus on V6'). Missing the reason= keyword will cause the decorator to fail.

[Copilot]

The pytest.mark.skipif decorator requires the reason parameter to be passed as a keyword argument. Change to @pytest.mark.skipif(not support.SNAPPY_SUPPORT, reason='No Snappy [de]compression support.').

[Copilot]

The pytest.mark.skipif decorator requires the reason parameter to be passed as a keyword argument. Change to @pytest.mark.skipif(support.SNAPPY_SUPPORT, reason='If Snappy support, test should not raise a UserWarning').

[Copilot]

The pytest.mark.skipif decorator requires the reason parameter to be passed as a keyword argument. Change to @pytest.mark.skipif(skip_tests, reason='Skipping old tests to focus on V6').

[Copilot]

The code uses self.assertRaises() which is a unittest context manager, but the class no longer inherits from unittest.TestCase. This will cause an AttributeError at runtime. Replace with pytest's with pytest.raises(UserWarning):.

Suggested change

	with self.assertRaises(UserWarning):
	with pytest.raises(UserWarning):

[Copilot]

The pytest.mark.skipif decorator requires the reason parameter to be passed as a keyword argument. Change to @pytest.mark.skipif(thisplatform.startswith('win'), reason='Not valid on Windows').

[Copilot]

The pytest.mark.skipif decorator requires the reason parameter to be passed as a keyword argument. Change to @pytest.mark.skipif(thisplatform.startswith('win'), reason='Not valid on Windows').

[Copilot]

The pytest.mark.skipif decorator requires the reason parameter to be passed as a keyword argument. Change to @pytest.mark.skipif(thisplatform.startswith('win'), reason='Not valid on Windows').

Suggested change

	@pytest.mark.skipif(thisplatform.startswith("win"), "Not valid on Windows")
	@pytest.mark.skipif(thisplatform.startswith("win"), reason="Not valid on Windows")

[Copilot]

The pytest.mark.skip decorator requires the reason parameter to be passed as a keyword argument. Change to @pytest.mark.skip(reason='known issue').

Suggested change

	@pytest.mark.skip("known issue")
	@pytest.mark.skip(reason="known issue")