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

[TSK-28] - Unit Tests and Improvements #17

Merged
merged 55 commits into from
Nov 23, 2024
Merged

[TSK-28] - Unit Tests and Improvements #17

merged 55 commits into from
Nov 23, 2024

Conversation

phrp720
Copy link
Owner

@phrp720 phrp720 commented Nov 20, 2024

This pull request includes multiple updates to the aw-sync-agent project, focusing on enhancing the Makefile commands, improving the documentation, and adding new unit tests. The most important changes include adding new service commands to the Makefile, updating the README to reflect these changes, and introducing new tests for various components of the project.

Makefile Enhancements:

  • Added new service commands (SERVICE-START, SERVICE-STOP, SERVICE-REMOVE, SERVICE-RESTART, SERVICE-STATUS) to manage the aw-sync-agent service on both Windows and Linux. [1] [2] [3]

Documentation Updates:

  • Updated README.md to include new sections for general and service commands, and to clarify the requirements and steps to run the agent. [1] [2] [3] [4]

Code Improvements:

  • Refactored checkpoint.go to use getter and setter methods for the checkpointFile variable. [1] [2]
  • Changed the sett struct to Setts and made related function names public in settings.go. [1] [2] [3]

New Unit Tests:

  • Added comprehensive unit tests for the checkpoint, cron, filter, and settings packages to ensure their functionality. [1] [2] [3] [4] [5]

@phrp720 phrp720 merged commit 32664a3 into master Nov 23, 2024
2 checks passed
@phrp720 phrp720 deleted the phillip/TSK-28 branch November 23, 2024 20:19
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