mirror of
https://github.com/lrnselfreliance/wrolpi.git
synced 2026-10-26 19:01:39 -04:00
Inventories are a config-only MultiFileConfig (config/inventory/<slug>.yaml) and were not reachable by the admin config Restore UI, so there was no way to re-import a hand-edited/copied-in file or restore an inventory from its daily backup. Backend: - MultiFileConfig gains generic helpers: get_backup_dates(slug)/_get_backup_file (surface the daily config/backup/<subdir>/<slug>-DATE.yaml files) and reimport() (rebuild the in-memory store from disk; picks up hand-edits and copied-in files, drops files deleted on disk). backup_date is constrained to YYYYMMDD so it cannot traverse out of the backup directory. - InventoriesConfig.preview_restore/apply_restore: overwrite replaces the whole inventory; merge unions items by id (keeps current fields and existing items). The pre-restore state is backed up automatically, so a restore is undoable. - API: POST /inventory/reimport, GET /inventory/<slug>/backups, POST /inventory/<slug>/restore/preview, POST /inventory/<slug>/restore. Frontend: - New InventoryImportModal (opened from a Restore button in the Inventory toolbar): Re-import from disk, plus a dated-backup table with per-row Merge/Overwrite that previews the change before applying. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
131 lines
6.7 KiB
Python
131 lines
6.7 KiB
Python
import json
|
|
from http import HTTPStatus
|
|
|
|
import pytest
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_inventory_lifecycle_api(test_inventory_configs, async_client):
|
|
"""Create, list (full), rename, and delete an inventory via the API."""
|
|
request, response = await async_client.post('/api/inventory',
|
|
content=json.dumps(dict(name='Food Storage', type='food')))
|
|
assert response.status_code == HTTPStatus.CREATED, response.status_code
|
|
inventory = response.json['inventory']
|
|
slug = inventory['slug']
|
|
assert slug == 'food-storage'
|
|
assert inventory['fields'], 'food inventory should be seeded with default fields'
|
|
|
|
# GET / returns every inventory in full (fields + items) in one request.
|
|
request, response = await async_client.get('/api/inventory')
|
|
assert response.status_code == HTTPStatus.OK
|
|
assert response.json['inventories'][0]['name'] == 'Food Storage'
|
|
assert 'items' in response.json['inventories'][0]
|
|
|
|
# Rename via whole-inventory PUT; slug stays stable.
|
|
request, response = await async_client.put(f'/api/inventory/{slug}', content=json.dumps(dict(name='Pantry')))
|
|
assert response.status_code == HTTPStatus.OK, response.status_code
|
|
assert response.json['inventory']['name'] == 'Pantry'
|
|
assert response.json['inventory']['slug'] == slug
|
|
|
|
request, response = await async_client.delete(f'/api/inventory/{slug}')
|
|
assert response.status_code == HTTPStatus.NO_CONTENT, response.status_code
|
|
|
|
request, response = await async_client.get('/api/inventory')
|
|
assert len(response.json['inventories']) == 0
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_save_items_and_fields_api(food_inventory_factory, async_client):
|
|
"""Items and fields are persisted with a single whole-inventory PUT."""
|
|
slug = food_inventory_factory()
|
|
|
|
items = [
|
|
{'brand': 'Salty', 'name': 'Salt', 'item_size': '1', 'item_size_unit': 'lb', 'count': '5'},
|
|
{'brand': 'Ricey', 'name': 'Rice', 'count': '4'},
|
|
]
|
|
request, response = await async_client.put(f'/api/inventory/{slug}', content=json.dumps(dict(items=items)))
|
|
assert response.status_code == HTTPStatus.OK, response.json
|
|
saved_items = response.json['inventory']['items']
|
|
assert {i['name'] for i in saved_items} == {'Salt', 'Rice'}
|
|
assert all(isinstance(i['id'], int) for i in saved_items)
|
|
|
|
# Replace the field schema in the same kind of request.
|
|
new_fields = [
|
|
{'key': 'name', 'label': 'Name', 'type': 'text'},
|
|
{'key': 'location', 'label': 'Location', 'type': 'location'},
|
|
]
|
|
request, response = await async_client.put(f'/api/inventory/{slug}', content=json.dumps(dict(fields=new_fields)))
|
|
assert response.status_code == HTTPStatus.OK, response.json
|
|
assert [f['key'] for f in response.json['inventory']['fields']] == ['name', 'location']
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_put_version_conflict_api(food_inventory_factory, async_client):
|
|
"""A stale version is rejected with 409 instead of clobbering a newer save."""
|
|
slug = food_inventory_factory()
|
|
request, response = await async_client.get('/api/inventory')
|
|
inventory = next(i for i in response.json['inventories'] if i['slug'] == slug)
|
|
version = inventory['version']
|
|
|
|
request, response = await async_client.put(f'/api/inventory/{slug}',
|
|
content=json.dumps(dict(name='First', version=version)))
|
|
assert response.status_code == HTTPStatus.OK
|
|
|
|
request, response = await async_client.put(f'/api/inventory/{slug}',
|
|
content=json.dumps(dict(name='Stale', version=version)))
|
|
assert response.status_code == HTTPStatus.CONFLICT, response.status_code
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_missing_inventory_404(test_inventory_configs, async_client):
|
|
request, response = await async_client.put('/api/inventory/does-not-exist', content=json.dumps(dict(name='x')))
|
|
assert response.status_code == HTTPStatus.NOT_FOUND, response.status_code
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_backups_and_restore_api(food_inventory_factory, test_inventory_configs, async_client):
|
|
"""List backups, preview, and apply a merge restore via the API."""
|
|
from modules.inventory.defaults import default_fields
|
|
from wrolpi.common import write_config_data
|
|
config = test_inventory_configs
|
|
slug = food_inventory_factory(items=[{'id': 1, 'name': 'Rice', 'count': '4'}])
|
|
backup = config._get_backup_file(slug, '20260101')
|
|
backup.parent.mkdir(parents=True, exist_ok=True)
|
|
write_config_data(dict(version=9, slug=slug, name='Food Storage', type='food', fields=default_fields('food'),
|
|
items=[{'id': 1, 'name': 'Rice', 'count': '4'}, {'id': 2, 'name': 'Beans', 'count': '2'}]),
|
|
backup)
|
|
|
|
request, response = await async_client.get(f'/api/inventory/{slug}/backups')
|
|
assert response.status_code == HTTPStatus.OK
|
|
# Saving the seed items also wrote today's backup, so just assert our dated backup is listed.
|
|
assert '20260101' in response.json['dates']
|
|
|
|
request, response = await async_client.post(f'/api/inventory/{slug}/restore/preview',
|
|
content=json.dumps(dict(backup_date='20260101', mode='merge')))
|
|
assert response.status_code == HTTPStatus.OK, response.json
|
|
assert {i['id'] for i in response.json['preview']['add']} == {2}
|
|
|
|
request, response = await async_client.post(f'/api/inventory/{slug}/restore',
|
|
content=json.dumps(dict(backup_date='20260101', mode='merge')))
|
|
assert response.status_code == HTTPStatus.OK, response.json
|
|
assert {i['name'] for i in response.json['inventory']['items']} == {'Rice', 'Beans'}
|
|
|
|
# A bad mode is rejected.
|
|
request, response = await async_client.post(f'/api/inventory/{slug}/restore',
|
|
content=json.dumps(dict(backup_date='20260101', mode='bogus')))
|
|
assert response.status_code == HTTPStatus.BAD_REQUEST, response.status_code
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_reimport_api(food_inventory_factory, test_inventory_configs, async_client):
|
|
"""POST /reimport picks up an inventory file copied onto disk by hand."""
|
|
from modules.inventory.defaults import default_fields
|
|
from wrolpi.common import write_config_data
|
|
config = test_inventory_configs
|
|
food_inventory_factory(name='Food Storage')
|
|
write_config_data(dict(version=1, slug='tools-box', name='Tools Box', type='tool',
|
|
fields=default_fields('tool'), items=[]), config.get_file('tools-box'))
|
|
|
|
request, response = await async_client.post('/api/inventory/reimport')
|
|
assert response.status_code == HTTPStatus.OK, response.json
|
|
assert {i['slug'] for i in response.json['inventories']} == {'food-storage', 'tools-box'}
|