Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Create tests for PropertiesFileUtils #270

Open
wants to merge 2 commits into
base: development
Choose a base branch
from

Conversation

permission-error
Copy link

@permission-error permission-error commented Feb 12, 2024

Pull Request Description:

This PR introduces a comprehensive test suite for the PropertiesFileUtils class within the org.opendatakit.builder package. The PropertiesFileUtils class is responsible for handling operations related to properties files, including reading from and writing to CSV files.

Changes Made:

  • Implemented test cases to ensure robustness and reliability of the PropertiesFileUtils class.
  • Covered various scenarios including null inputs, IOExceptions, permission denied, and file mismatches.
  • Ensured appropriate handling of exceptions and edge cases.
  • Verified the behavior of the class under different conditions, ensuring it meets expected functionality and maintains integrity.

Test Coverage:

  • The test suite covers both positive and negative scenarios, ensuring that the PropertiesFileUtils class behaves as expected.
  • Tested methods for writing properties into CSV files (writePropertiesIntoCsv) and reading properties from CSV files (readPropertiesFromCsv).
  • Checked the handling of null inputs, empty table IDs, IOExceptions, and permission denied scenarios.
  • Verified the creation of KeyValueStoreEntries and KeyValueStoreEntry objects.

Purpose:

  • Enhance the reliability and maintainability of the PropertiesFileUtils class by establishing a robust testing framework.
  • Ensure that future changes or updates to the class do not introduce regressions.
  • Foster a culture of quality assurance and test-driven development within the project.

@permission-error permission-error marked this pull request as ready for review February 14, 2024 21:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant