Skip to content

Feature: Image asset table and test harness updates - #166

Merged
jbriones1 merged 17 commits into
mainfrom
feat_163_image-asset-table
Aug 17, 2026
Merged

Feature: Image asset table and test harness updates#166
jbriones1 merged 17 commits into
mainfrom
feat_163_image-asset-table

Conversation

@jbriones1

@jbriones1 jbriones1 commented Aug 17, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

closes #163

Description

Added image_assets table, its supporting database table, endpoints, and tests. Made improvements to the test harness.

Changes

  • Added an image_asset database table storing image ID, storage key, original filename, and creation timestamp.
  • Added admin-protected GET /image and POST /image endpoints to list and upload images.
  • Supports JPEG, PNG, and WebP file formats, limited to 8 megapixel images.
  • Uploaded images are stored under MEDIA_ROOT/images with uuid4 filenames and returns a public URL based on MEDIA_BASE_URL.
  • Added a script to import existing files from the media images directory into the database.
  • Added MEDIA_ROOT and MEDIA_BASE_URL environment variables.
  • Updated documentation about supported environment variables.
  • Added Pillow and multipart-upload dependencies.
  • Disabled permission dependencies in the development environment while retaining them in test and production.
  • Added unit and integration coverage for image validation, authorization, uploads, and failure cleanup.
  • Improved integration-test isolation using per-test transactions and savepoints, with related fixture/test corrections.

Fixes

  • Added a migration reducing computing ID columns from 32 to 8 characters.

jbriones1 added 17 commits August 15, 2026 23:33
@jbriones1
feat: add environment variables for media files
45167d8
@jbriones1
fix: add migration to update max computing ID length to 8
282578e
@jbriones1
feat: create image_asset migration
3b4ad41
@jbriones1
feat: add media environment variables and docs about env vars
6f4d53c
@jbriones1
feat: create image_asset migration
8acf259
@jbriones1
feat(image_asset): add endpoints
3fb7ba2
@jbriones1
dev: disabled permission dependencies on dev environments
4318b6f
@jbriones1
fix: add image_url and fix documentation
4ebcabb
@jbriones1
fix(image_asset): validator checks for supported types instead of PIL
84ab3c2
@jbriones1
docs: add a note about how pytest markers work
0347c68
@jbriones1
fix: remove redundant caught HTTPException
5c587b8
@jbriones1
tests(image_asset): add unit tests for validator
b8b1783
@jbriones1
fix: test environment set incorrectly
5ba6399
@jbriones1
feat(image_asset): integration tests and test harness improvements
f1b883a 
- tests now use the same database connection for clients
- tests now rollback instead of persisting their changes
- clients and database connections are function scoped
@jbriones1
fix: broken tests after fixture changes
db65db2
@jbriones1
feat: add script to scan the images directory and add them to db
181a5cf
@jbriones1
fix: workflows missing MEDIA_ROOT and MEDIA_BASE_URL
fb43b47
@jbriones1
jbriones1 merged commit 77ef22c into main Aug 17, 2026
3 checks passed
@jbriones1
jbriones1 deleted the feat_163_image-asset-table branch August 17, 2026 08:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Image Asset Table

1 participant

@jbriones1