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.

Sign up for GitHub

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

Jump to bottom

test: adding more files testing #97

Merged
merged 11 commits into from
Feb 22, 2024
Merged

test: adding more files testing #97

merged 11 commits into from
Feb 22, 2024

Conversation

ethanholz
Copy link
Contributor

Adds more tests for file activities. Currently includes:

  • test(file): added fileSymlink tests
  • fix(file): fixed bug in fileSymlink
  • style(file): updated fileTouch to use types.Snprintf
  • test(file): added the initial fileTouch tests
  • fix(file): fixed incorrect variables, short-circuited test ops
  • test(file): added longer and more in depth fileTouch tests

@ethanholz
test(file): added fileSymlink tests
e328f5c
@ethanholz
fix(file): fixed bug in fileSymlink
e80d8d3 
- adds check if the symlink/file in name does not exist
@ethanholz
style(file): updated fileTouch to use types.Snprintf
ef9c5d5
@ethanholz
test(file): added the initial fileTouch tests
d85894f
@ethanholz
fix(file): fixed incorrect variables, short-circuited test ops
a732827 
- Synced mTime and aTime to use the same time.Now
- Fixed incorrectly named variable that caused unexpected behavior
- Addded checks for changes to when options were not provided
- Added case for when both times would be modified for test
@ethanholz
test(file): added longer and more in depth fileTouch tests
7659dd7
@ethanholz ethanholz self-assigned this Dec 19, 2023
@ethanholz
test(file): initial fileContent testing
b85a305
@ethanholz
test(file): validates case w/ two sources and one hash
73280be 
- This case currently fails but should not
@ethanholz
test(file): initial testing for simple case of fileManaged
9980784
@ethanholz
test(file): added a multi-source skip-verify
9f13ecc 
- Adds a case where a skip_verify option is passed for multiple sources
- Logic for this currently does not exist so test case is simple
@ethanholz
test(file): added test for managed external source
5f41734
@ethanholz ethanholz marked this pull request as ready for review February 16, 2024 19:39
@ethanholz ethanholz requested a review from taigrr February 16, 2024 19:39
@taigrr taigrr merged commit c7d1ee1 into gogrlx:master Feb 22, 2024
2 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@taigrr taigrr Awaiting requested review from taigrr

Assignees

@ethanholz ethanholz

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ethanholz @taigrr