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

Add Tests #18

Merged
merged 21 commits into from
Jun 2, 2024
Merged

Add Tests #18

merged 21 commits into from
Jun 2, 2024

Conversation

fullerzz
Copy link
Owner

@fullerzz fullerzz commented May 26, 2024 •
edited
Loading

Functions Requiring Tests in main.go

  • updateFileTags
  • searchFiles
  • getAvif
  • loadConfig
  • loadFileMetadata
  • writeFileMetadata
  • buildLink
  • saveFile
  • listFiles

Functions Requiring Tests in resizer.go

  • writeNewImg
  • createAltSizes

@fullerzz fullerzz self-assigned this May 26, 2024
fullerzz added 18 commits June 1, 2024 14:18
@fullerzz
added first test in main_test.go
48885b1
@fullerzz
added additional test
71eeda2
@fullerzz
added test for loading metadata
c5d250f
@fullerzz
added test for create alt sizes
157534a
@fullerzz
added testdata
b2bb2f6
@fullerzz
fixed asserts in TestLoadFileMetadata tests
32d409d
@fullerzz
wip: TestLoadMissingFileMetadata
bca0ab1
@fullerzz
completed TestLoadMissingFileMetadata
9f664c6
@fullerzz
added additional test data + test for saving file
0e32259
@fullerzz
added additional error logs
d9377a8
@fullerzz
additional logging of ops in saveFile
f4c9b99
@fullerzz
added precommit hooks
b66728b
@fullerzz
I think tests are failing in CI due to not using the local uploads dir
a2061d6
@fullerzz
creating uploads dir in CI before running tests
6f42372
@fullerzz
passing filePath instead of filename when calling createAltSizes from…
ab8e074 
… saveFile()
@fullerzz
added github action for golangci-lint
94051d1
@fullerzz
removed parameterized values from slog.Error message
3566dc4
@fullerzz
renamed file and job
454dbe4
@fullerzz fullerzz linked an issue Jun 2, 2024 that may be closed by this pull request
Add Tests #20
Open
11 tasks
@fullerzz fullerzz marked this pull request as ready for review June 2, 2024 20:23
@fullerzz fullerzz merged commit 4c6b3d0 into main Jun 2, 2024
4 checks passed
@fullerzz fullerzz deleted the feat/add-tests branch June 2, 2024 20:24
@fullerzz fullerzz mentioned this pull request Jun 23, 2024
Draft
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@fullerzz fullerzz

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add Tests
1 participant
@fullerzz