-
Notifications
You must be signed in to change notification settings - Fork 45
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
Add support for at.exe .job files in scheduled tasks #249
Add support for at.exe .job files in scheduled tasks #249
Conversation
- Implement binary .job file parsing - Ensure consistency with XML task files by using the same record fields - Extend and enhance tests - Include missing docstrings and type hints for better documentation This change allows the tasks parser to handle .job files, providing compatibility with at.exe scheduled tasks.
Codecov Report
@@ Coverage Diff @@
## main #249 +/- ##
==========================================
+ Coverage 68.61% 68.81% +0.19%
==========================================
Files 222 225 +3
Lines 17036 17147 +111
==========================================
+ Hits 11690 11800 +110
- Misses 5346 5347 +1
Flags with carried forward coverage won't be shown. Click here to find out more.
... and 31 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
- Fixed docstrings - Better var naming - Identing - Fix struct field types
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This file is becoming a bit unwieldy, it might be a good idea to split out the structure parsing bits in a xml_tasks.py
and at_tasks.py
or something like that.
- Split tasks.py in three files and classes - Refactor to reduce code duplication - Various improvements
- Add more records to be tested - Add more fields to be tested - Improve test file with more fields used
…of github.com:fox-it/dissect.target into DIS-1558_Add_at.exe_job_files_scheduled_tasks_support
- Improve handling of padding - Fix delay bug in old code - Improve naming consistency - Improve exeption handling - Improve tests
- Move record descriptors to separate file - Implement some improvements - Fix some bugs
…of github.com:fox-it/dissect.target into DIS-1558_Add_at.exe_job_files_scheduled_tasks_support
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe an idea to move everything into a tasks
directory?
@pyrco what do you think?
That may be even cleaner indeed. I'd keep |
I'd remove the |
…of github.com:fox-it/dissect.target into DIS-1558_Add_at.exe_job_files_scheduled_tasks_support
Reviewed by @pyrco
(DIS-1558)