Compare commits

...

9 commits

Author SHA1 Message Date
ijuanes
531b3a9048 docs: add separated frontend architecture options for mobile support
- Create docs/FRONTEND_OPTIONS.md with comprehensive analysis
- Explore 4 options for separated frontend:
  1. SPA with REST API (React/Vue/Svelte)
  2. Hybrid approach (mobile SPA + desktop server templates)
  3. HTMX for lightweight dynamic UI
  4. Native mobile app (React Native/Flutter/Capacitor)
- Include comparison matrix with pros/cons
- Provide implementation examples for each option
- Add responsive design guidelines
- Include API endpoints specification
- Recommend phased implementation roadmap
- No code changes - documentation only

This addresses mobile responsiveness while keeping backend intact.
2026-06-18 12:13:59 +01:00
ijuanes
1ad0e8bf13 docs: add PostgreSQL migration guide for future scalability
- Add section 9 to TECHNICAL_SPECIFICATIONS.md with:
  - Migration criteria (DB size, connections, users, transactions)
  - Proposed repository pattern architecture
  - Migration steps overview
  - Benefits of PostgreSQL

- Create docs/MIGRATION_TO_POSTGRES.md with comprehensive guide:
  - When to migrate (detailed thresholds)
  - Architecture overview (repository pattern)
  - Step-by-step migration process (10 steps)
  - Complete code examples:
    * Base repository interfaces
    * SQLite implementation
    * PostgreSQL implementation
    * Repository factory
    * Migration script
  - Docker Compose configuration
  - Rollback plan
  - Monitoring and maintenance
  - Performance optimization
  - Troubleshooting guide

This enables future migration when database grows beyond SQLite capabilities.
2026-06-17 17:22:50 +01:00
ijuanes
85978a710f docs: add testing documentation to README and RUNNING.md
- Add Testing section to README with:
  - How to run tests
  - Test structure overview
  - Key edge cases covered
  - Test configuration notes
- Add Running Tests section to RUNNING.md
- Document the 46 unit tests available
2026-06-17 17:10:12 +01:00
ijuanes
83ac67fea2 test: add comprehensive unit test suite
- Add tests/ directory with pytest configuration
- Add conftest.py with database fixtures (test_db, sample data)
- Add test_core.py: 21 tests for core operations (tablet, user, loan CRUD)
- Add test_edge_cases.py: 14 tests for edge cases (duplicates, invalid IDs, etc.)
- Add test_minimal_app.py: 11 tests for actual application functions
- Update pyproject.toml with pytest and coverage configuration
- Total: 46 tests covering core functionality and edge cases

Tests cover:
- Tablet CRUD operations
- User CRUD operations
- Loan/return workflows
- Duplicate prevention (serial numbers, identifications)
- Invalid ID handling
- Already loaned device prevention
- Non-existent loan handling
- Multiple loans per user (one-to-many relationship)
- Non-loanable device CRUD operations

To run: python3 -m pytest tests/ -v
2026-06-17 17:07:20 +01:00
ijuanes
d3d1fb1b32 feat(non-loanable-devices): implement non-loanable device management
- Add non_loanable_devices table to database schema
- Add web interface routes: /non_loanable_devices, /add_non_loanable_device, /edit_non_loanable_device, /delete_non_loanable_device
- Add CLI menu options 8-10 for non-loanable device management
- Add templates for non-loanable device CRUD operations
- Update README.md with new feature documentation
- Update REQUIREMENTS.md marking non-loanable devices as implemented
- Add gestiontablets.sh script

Implements: #porDefinir Registrar nuevos dispositivos no prestables
2026-06-09 23:54:38 +01:00
ijuanes
ac73e5dc93 docs: add technical specifications document
Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
2026-06-04 23:00:14 +01:00
ijuanes
5fd3aaf4c5 docs: add Spanish requirements document (platform/language agnostic)
Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
2026-06-04 10:12:18 +01:00
ijuanes
e9862f3d8d docs: add version control section to README
Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
2026-06-04 01:00:49 +01:00
ijuanes
ec79bc2ca3 feat(loans): add search and expandable past loans to user loans view
- Add search box to filter by user name, ID, tablet brand/model/serial
- Split loans into Current Loans (always visible) and Past Loans (collapsible)
- Past loans toggle with ▶/▼ indicator showing count
- All searchable data embedded in data-search attributes

Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
2026-06-04 01:00:17 +01:00
22 changed files with 4790 additions and 44 deletions

View file

@ -10,15 +10,16 @@ A simple SQLite-based system for managing tablet lending and returns.
- **Return Management**: Record when tablets are returned
- **History Tracking**: Complete history of all loans and returns
- **User Loans View**: See all tablets loaned to each user (demonstrates one-to-many relationship)
- **Non-Loanable Devices**: Track inventory devices that cannot be loaned (e.g., projectors, monitors)
- **Project Notes**: Markdown editor for development documentation
## Files
- `minimal_app.py` - Interactive command-line application
- `minimal_app.py` - Interactive command-line application (includes non-loanable device management)
- `test_app.py` - Test script that demonstrates functionality
- `tablets.db` - SQLite database (created automatically)
- `simple_app.py` - Web-based version (requires Flask)
- `app.py` - Alternative web version (requires Flask)
- `tablets.db` - SQLite database (created automatically, includes non_loanable_devices table)
- `simple_app.py` - Web-based version (requires Flask, includes non-loanable device management)
- `app.py` - Alternative web version (requires Flask, includes non-loanable device management)
## Quick Start
@ -144,6 +145,7 @@ python3 app.py
- **Loan Tablet**: Loan a tablet to a user (with search for large datasets)
- **User Loans**: View all tablets loaned to each user - demonstrates the **one-to-many relationship** (one user, multiple tablets)
- **Loan History**: Complete history of all loan and return transactions
- **Non-Loanable Devices**: Manage inventory of devices that cannot be loaned (add, edit, delete, view)
- **Project Management**: Markdown editor for development notes (saved to `notes_development/project_notes.md`)
## Database Backup
@ -153,6 +155,48 @@ To backup your data:
cp tablets.db tablets_backup_$(date +%Y%m%d).db
```
## Testing
The project includes a comprehensive unit test suite with **46 tests** covering core functionality and edge cases.
### Running Tests
```bash
# Install test dependencies (if not already installed)
source .venv/bin/activate
uv pip install pytest pytest-cov
# Run all tests
python3 -m pytest tests/ -v
# Run with coverage report
python3 -m pytest tests/ --cov=./ --cov-report=term-missing
```
### Test Structure
| File | Tests | Coverage |
|------|-------|----------|
| `tests/test_core.py` | 21 | Core operations (tablet, user, loan CRUD) |
| `tests/test_edge_cases.py` | 14 | Edge cases (duplicates, invalid IDs, etc.) |
| `tests/test_minimal_app.py` | 11 | Application function tests |
### Key Edge Cases Covered
- Loan a device that's already loaned (prevents double-loaning)
- Return a non-existent loan (handles gracefully)
- Duplicate serial numbers (rejected at database level)
- Duplicate user identifications (rejected at database level)
- Invalid tablet/user IDs (validated before operations)
- Multiple loans per user (one-to-many relationship)
- Loan-return-loan sequence (device lifecycle)
### Test Configuration
Tests use isolated temporary databases and automatically clean up after each test run. No impact on production data.
## License
This is a simple educational project. Feel free to use and modify it as needed.

221
REQUIREMENTS.md Normal file
View file

@ -0,0 +1,221 @@
# Requisitos del Sistema de Gestión de Préstamos de Tablets
## Descripción General
El sistema debe permitir la gestión del préstamo y devolución de dispositivos tablets entre un inventario y un conjunto de usuarios. El enfoque principal es el seguimiento del estado de cada dispositivo y su relación con los usuarios que los han tomado prestados.
---
## Requisitos Faltantes
### Gestión de dispositivos no prestables
**IMPLEMENTADO** - El sistema ahora permite:
- Registrar nuevos dispositivos no prestables en un inventario separado (tabla `non_loanable_devices`)
- Gestionar dispositivos no prestables a través de:
- Interfaz web (app.py): /non_loanable_devices, /add_non_loanable_device, /edit_non_loanable_device, /delete_non_loanable_device
- Interfaz CLI (minimal_app.py): Opciones 8, 9, 10 del menú
- La implementación utiliza una tabla separada en la misma base de datos con los siguientes campos:
- brand, model, serial_number (único), device_type, location, status, notes, purchase_date, purchase_cost
## Requisitos Funcionales
### Gestión de Dispositivos
El sistema debe permitir:
- Registrar nuevos dispositivos en el inventario
- Identificar cada dispositivo de manera única
- Almacenar información descriptiva de cada dispositivo (marca, modelo, número de serie)
- Consultar el estado actual de cada dispositivo (disponible, prestado)
### Gestión de Usuarios
El sistema debe permitir:
- Registrar nuevos usuarios
- Identificar cada usuario de manera única
- Almacenar información de contacto básica (nombre, identificación)
- Consultar la información de los usuarios registrados
### Gestión de Préstamos
El sistema debe permitir:
- Asignar un dispositivo disponible a un usuario
- Registrar la fecha y hora del préstamo
- Registrar la fecha y hora de la devolución
- Consultar el historial de préstamos de un dispositivo
- Consultar el historial de préstamos de un usuario
- Visualizar todos los dispositivos actualmente prestados
### Consulta y Reportes
El sistema debe permitir:
- Buscar dispositivos por cualquier atributo (marca, modelo, número de serie)
- Buscar usuarios por cualquier atributo (nombre, identificación)
- Filtrar préstamos por estado (activos, finalizados)
- Filtrar préstamos por dispositivo
- Filtrar préstamos por usuario
- Ver el historial completo de préstamos y devoluciones
---
## Modelo de Datos
### Entidades Principales
1. **Dispositivo**
- Identificador único
- Marca
- Modelo
- Número de serie (único)
- Estado (disponible, prestado)
2. **Usuario**
- Identificador único
- Nombre
- Identificación (única)
3. **Préstamo**
- Identificador único
- Dispositivo asociado
- Usuario asociado
- Fecha de préstamo
- Fecha de devolución (opcional)
- Estado (activo, finalizado)
### Relaciones
- Un **usuario** puede tener **múltiples préstamos** (relación uno a muchos)
- Un **dispositivo** solo puede estar en **un préstamo activo** a la vez
- Un **préstamo** relaciona exactamente un dispositivo con un usuario
- El historial de préstamos permite rastrear todos los movimientos de cada dispositivo
### Reglas de Negocio
1. Un dispositivo en estado "prestado" no puede ser prestado a otro usuario
2. Un préstamo finalizado debe registrar fecha de devolución
3. La identificación de dispositivos y usuarios debe ser única
4. Al devolver un dispositivo, este vuelve al estado "disponible"
---
## Flujos de Trabajo
### Registro de un Nuevo Dispositivo
1. El sistema solicita la información del dispositivo (marca, modelo, número de serie)
2. El sistema valida que el número de serie no esté registrado
3. El sistema registra el dispositivo con estado "disponible"
4. El sistema confirma el registro
### Registro de un Nuevo Usuario
1. El sistema solicita la información del usuario (nombre, identificación)
2. El sistema valida que la identificación no esté registrada
3. El sistema registra el usuario
4. El sistema confirma el registro
### Préstamo de un Dispositivo
1. El sistema muestra los dispositivos disponibles
2. El usuario selecciona un dispositivo
3. El sistema muestra los usuarios registrados
4. El usuario selecciona un usuario
5. El sistema registra el préstamo con:
- Dispositivo seleccionado
- Usuario seleccionado
- Fecha y hora actual
- Estado "activo"
6. El sistema actualiza el estado del dispositivo a "prestado"
7. El sistema confirma el préstamo
### Devolución de un Dispositivo
1. El sistema muestra los préstamos activos
2. El usuario selecciona un préstamo
3. El sistema registra la fecha y hora de devolución
4. El sistema actualiza el estado del préstamo a "finalizado"
5. El sistema actualiza el estado del dispositivo a "disponible"
6. El sistema confirma la devolución
### Consulta de Préstamos por Usuario
1. El sistema muestra la lista de usuarios
2. El usuario selecciona un usuario
3. El sistema muestra:
- Dispositivos actualmente prestados (préstamos activos)
- Historial de préstamos pasados (opcional: expandible)
4. El sistema permite buscar por cualquier campo
---
## Requisitos No Funcionales
### Usabilidad
- La interfaz debe permitir búsqueda eficiente en listas grandes (500+ elementos)
- La visualización de información debe ser clara y organizada
- Los estados deben ser fácilmente identificables
- Las acciones principales deben ser accesibles
### Escalabilidad
- El diseño debe soportar un crecimiento en el número de dispositivos y usuarios
- La búsqueda debe ser eficiente
- El historial debe ser consultable sin afectar el rendimiento
### Mantenibilidad
- La estructura de datos debe ser clara
- Las relaciones entre entidades deben estar bien definidas
- Los flujos de trabajo deben estar documentados
---
## Diseño de Interfaz (Conceptual)
### Vistas Principales
1. **Vista de Inventario**
- Lista de dispositivos
- Filtros por estado
- Búsqueda
2. **Vista de Usuarios**
- Lista de usuarios
- Búsqueda
3. **Vista de Préstamos Activos**
- Lista de préstamos en curso
- Filtros por dispositivo/usuario
- Acciones: devolver dispositivo
4. **Vista de Historial**
- Lista completa de préstamos
- Filtros por fecha, usuario, dispositivo, estado
- Detalles de cada préstamo
5. **Vista de Préstamos por Usuario**
- Lista de usuarios
- Para cada usuario: dispositivos actualmente prestados
- Opcional: historial de préstamos pasados (expandible)
### Elementos de Interacción
- Campos de búsqueda en todas las listas
- Botones de acción claros (Prestar, Devolver, Registrar)
- Indicadores visuales de estado (colores, iconos)
- Confirmación de acciones críticas
---
## Notas de Implementación
Este documento describe los requisitos a nivel de diseño. La implementación específica (lenguaje de programación, base de datos, framework, arquitectura) queda a criterio del equipo de desarrollo y puede variar según las necesidades técnicas.
Los detalles técnicos como:
- Tecnologías específicas
- Estructura de base de datos física
- APIs
- Autenticación
- Despliegue
Deberán ser definidos en la fase de diseño técnico y pueden evolucionar sin afectar los requisitos funcionales descritos aquí.

View file

@ -48,10 +48,22 @@ pkill -f "python app.py"
- **Port**: 5000 (configurable in app.py)
- **Virtual Environment**: `.venv/` (created with uv)
## Troubleshooting
If you have issues:
1. Check if the server is running: `ps aux | grep app.py`
2. Check the database: `sqlite3 tablets.db`
3. Restart the server: `source .venv/bin/activate && python app.py`
## Running Tests
Enjoy managing your tablets! 📱💻
The project includes 46 unit tests for verifying functionality:
```bash
# Run all tests
python3 -m pytest tests/ -v
# Run with coverage
python3 -m pytest tests/ --cov=./ --cov-report=term-missing
```
Tests cover:
- Core CRUD operations for tablets, users, and loans
- Edge cases (duplicates, invalid IDs, already loaned devices)
- Application function testing
- Non-loanable device management
All tests use isolated temporary databases and do not affect production data.

200
TECHNICAL_SPECIFICATIONS.md Normal file
View file

@ -0,0 +1,200 @@
# Especificaciones Técnicas
Documento complementario a `REQUIREMENTS.md` con propuestas concretas de implementación técnica.
---
## 1. Concurrencia en Base de Datos
| Opción | Descripción | Ventajas | Desventajas |
|--------|-------------|---------|-------------|
| **SQLite + WAL** | Modo Write-Ahead Logging | Zero-config, embebido, buena para <100 conexiones | Limitado a 1 escritor simultáneo |
| **PostgreSQL** | Base de datos relacional | Concurrencia completa, transacciones ACID | Requiere servidor |
| **Transacciones** | Aislamiento READ COMMITTED | Previene lecturas sucias | Bloqueos temporales |
| **Locking optimista** | Versión de fila en actualizaciones | Sin bloqueos | Conflictos requieren reintento |
**Recomendación**: SQLite con WAL para prototipos, PostgreSQL para producción. Usar transacciones para operaciones críticas (préstamo/devolución).
---
## 2. Autenticación Básica
| Opción | Descripción | Complejidad |
|--------|-------------|-------------|
| **Sesiones (Flask)** | Cookies firmadas | Baja |
| **JWT** | Tokens sin estado | Media |
| **OAuth 2.0** | Delegación a proveedores | Alta |
| **Basic Auth** | HTTP Basic Authentication | Mínima |
**Flujo mínimo**:
```
Login (user/pass) → Session/JWT → Middleware verifica rol → Acceso a endpoints
```
**Roles sugeridos**:
- `admin`: CRUD completo, gestión de usuarios
- `staff`: Préstamos/devoluciones
- `viewer`: Solo consulta
---
## 3. Separación Frontend/Backend
| Arquitectura | Descripción | Stack Ejemplo |
|--------------|-------------|---------------|
| **Monolítica + Templates** | Servidor renderiza HTML | Flask+Jinja2, Django |
| **API + SPA** | Backend API, frontend dinámico | FastAPI + React/Vue |
| **API + SSR** | Backend API + renderizado servidor | Next.js + API |
| **Microservicios** | Servicios independientes | Backend API + Frontend estático |
**Para este proyecto**:
- **Opción A**: Flask + Jinja2 (simple, suficiente para requisitos)
- **Opción B**: FastAPI + React (escalable, separable)
---
## 4. Búsqueda Eficiente (500+ elementos)
| Opción | Descripción | Implementación |
|--------|-------------|----------------|
| **Índices DB** | B-tree en columnas buscadas | `CREATE INDEX idx_brand ON tablets(brand)` |
| **Búsqueda full-text** | Búsqueda en texto | SQLite FTS5, PostgreSQL tsvector |
| **Filtrado cliente** | JavaScript filtra datos | Ideal para <2000 elementos |
| **Paginación** | Dividir resultados | `LIMIT 50 OFFSET 0` |
**Recomendación**: Índices en `serial_number`, `brand`, `model`, `user.name`, `user.identification`.
---
## 5. Estructura de Proyecto Propuesta
```
project/
├── backend/
│ ├── models/ # Entidades (Device, User, Loan)
│ ├── services/ # Lógica de negocio
│ ├── repositories/ # Acceso a datos
│ ├── routes/ # Endpoints/API
│ └── app.py # Configuración
├── frontend/ # Opcional para opción B
│ ├── public/
│ └── src/
├── migrations/ # Control de cambios DB
├── tests/
└── docs/
```
---
## 6. Decisiones Clave por Tomar
| Decisión | Opciones | Impacto |
|----------|----------|---------|
| Base de datos | SQLite / PostgreSQL | Concurrencia, despliegue |
| Autenticación | Sesiones / JWT / Ninguna | Seguridad, complejidad |
| Frontend | Templates / SPA / Ninguno | Experiencia usuario |
| Despliegue | Local / Docker / Cloud | Escalabilidad |
---
## 7. Ejemplo de Implementación Mínima (Pseudocódigo)
```python
# modelos.py
class Device:
id: int (PK)
serial_number: str (unique)
status: str (available/loaned)
loans: List[Loan] (one-to-many)
class User:
id: int (PK)
identification: str (unique)
loans: List[Loan] (one-to-many)
class Loan:
id: int (PK)
device_id: int (FK)
user_id: int (FK)
loan_date: datetime
return_date: datetime (nullable)
status: str (active/returned)
# servicios/loan_service.py
def loan_device(device_id: int, user_id: int) -> Loan:
with transaction(): # Atomicidad
device = get_device_for_update(device_id)
if device.status != 'available':
raise DeviceNotAvailableError()
loan = Loan.create(
device_id=device_id,
user_id=user_id,
loan_date=now(),
status='active'
)
device.status = 'loaned'
return loan
```
---
## 8. Consideraciones Adicionales
- **Validaciones**: Únicas (serial_number, identification) a nivel de base de datos
- **Auditoría**: Campos `created_at`, `updated_at` en todas las entidades
- **Backups**: Script para exportar base de datos periódicamente
- **Logging**: Registrar préstamos/devoluciones para auditoría
- **Versionado API**: `/api/v1/...` para compatibilidad futura
- **Pruebas**: Cubrir flujos críticos (préstamo con dispositivo no disponible, devolución de préstamo inexistente)
---
## 9. Migración a PostgreSQL (Futuro)
### Contexto
El sistema actualmente utiliza **SQLite** como base de datos embebida, ideal para prototipos y aplicaciones pequeñas. Sin embargo, SQLite tiene limitaciones para escalar.
### Criterios para Migración
Considerar migrar a **PostgreSQL** cuando:
- Tamaño DB > 1GB
- Más de 50 conexiones simultáneas
- Más de 200 usuarios activos
- Más de 500 transacciones por minuto
- Necesidad de múltiples servidores
### Arquitectura Propuesta
Usar un patrón de **Repository** para abstraer la base de datos:
```
backend/
├── repositories/
│ ├── base_repository.py # Interfaz abstracta
│ ├── sqlite_repository.py # Implementación SQLite
│ └── postgres_repository.py # Implementación PostgreSQL
└── config/
└── database.py # Fábrica de repositorios
```
### Schema PostgreSQL
El schema es similar al de SQLite pero con tipos de datos más específicos y índices adicionales para rendimiento.
### Pasos para Migración
1. Instalar dependencias: `pip install psycopg2-binary sqlalchemy alembic`
2. Configurar PostgreSQL y crear base de datos
3. Ejecutar script de migración: `python scripts/migrate_to_postgres.py`
4. Cambiar configuración: `DB_TYPE=postgres`
5. Iniciar aplicación
### Beneficios
- Concurrencia ilimitada (múltiples escritores)
- Escalabilidad a miles de conexiones
- Mejor rendimiento con índices
- Seguridad integrada (autenticación, roles)
- Backups automáticos
- Replicación y clustering

109
app.py
View file

@ -63,6 +63,22 @@ def init_db():
)
''')
# Create non_loanable_devices table for devices that cannot be loaned
cursor.execute('''
CREATE TABLE IF NOT EXISTS non_loanable_devices (
id INTEGER PRIMARY KEY AUTOINCREMENT,
brand TEXT NOT NULL,
model TEXT NOT NULL,
serial_number TEXT UNIQUE NOT NULL,
device_type TEXT NOT NULL,
location TEXT,
status TEXT DEFAULT 'available',
notes TEXT,
purchase_date TEXT,
purchase_cost REAL
)
''')
conn.commit()
@app.route('/')
@ -319,6 +335,99 @@ def user_loans():
return render_template('user_loans.html', users_with_loans=user_loans_list)
@app.route('/non_loanable_devices')
def non_loanable_devices():
"""Show all non-loanable devices"""
with get_db() as conn:
cursor = conn.cursor()
cursor.execute("SELECT * FROM non_loanable_devices ORDER BY device_type, brand, model")
devices = cursor.fetchall()
return render_template('non_loanable_devices.html', devices=devices)
@app.route('/add_non_loanable_device', methods=['GET', 'POST'])
def add_non_loanable_device():
"""Add a new non-loanable device to inventory"""
if request.method == 'POST':
brand = request.form['brand']
model = request.form['model']
serial_number = request.form['serial_number']
device_type = request.form['device_type']
location = request.form['location']
notes = request.form['notes']
purchase_date = request.form.get('purchase_date', '')
purchase_cost = request.form.get('purchase_cost', '')
try:
with get_db() as conn:
cursor = conn.cursor()
cursor.execute('''
INSERT INTO non_loanable_devices
(brand, model, serial_number, device_type, location, status, notes, purchase_date, purchase_cost)
VALUES (?, ?, ?, ?, ?, 'available', ?, ?, ?)
''', (brand, model, serial_number, device_type, location, notes, purchase_date, purchase_cost))
conn.commit()
flash('Non-loanable device added successfully!', 'success')
except sqlite3.IntegrityError:
flash('Error: Serial number already exists!', 'error')
return redirect(url_for('non_loanable_devices'))
return render_template('add_non_loanable_device.html')
@app.route('/edit_non_loanable_device/<int:device_id>', methods=['GET', 'POST'])
def edit_non_loanable_device(device_id):
"""Edit a non-loanable device"""
with get_db() as conn:
cursor = conn.cursor()
cursor.execute("SELECT * FROM non_loanable_devices WHERE id = ?", (device_id,))
device = cursor.fetchone()
if not device:
flash('Error: Device not found!', 'error')
return redirect(url_for('non_loanable_devices'))
if request.method == 'POST':
brand = request.form['brand']
model = request.form['model']
serial_number = request.form['serial_number']
device_type = request.form['device_type']
location = request.form['location']
status = request.form['status']
notes = request.form['notes']
purchase_date = request.form.get('purchase_date', '')
purchase_cost = request.form.get('purchase_cost', '')
try:
cursor.execute('''
UPDATE non_loanable_devices SET
brand = ?, model = ?, serial_number = ?, device_type = ?,
location = ?, status = ?, notes = ?, purchase_date = ?, purchase_cost = ?
WHERE id = ?
''', (brand, model, serial_number, device_type, location, status, notes, purchase_date, purchase_cost, device_id))
conn.commit()
flash('Device updated successfully!', 'success')
return redirect(url_for('non_loanable_devices'))
except sqlite3.IntegrityError:
flash('Error: Serial number already exists!', 'error')
return render_template('edit_non_loanable_device.html', device=device)
@app.route('/delete_non_loanable_device/<int:device_id>')
def delete_non_loanable_device(device_id):
"""Delete a non-loanable device"""
with get_db() as conn:
cursor = conn.cursor()
cursor.execute("DELETE FROM non_loanable_devices WHERE id = ?", (device_id,))
conn.commit()
flash('Device deleted successfully!', 'success')
return redirect(url_for('non_loanable_devices'))
if __name__ == '__main__':
# Initialize database
init_db()

1191
docs/FRONTEND_OPTIONS.md Normal file

File diff suppressed because it is too large Load diff

View file

@ -0,0 +1,987 @@
# PostgreSQL Migration Guide
This document describes how to migrate the Tablet Management System from SQLite to PostgreSQL when the database grows beyond SQLite's capabilities.
## When to Migrate
Consider migrating to PostgreSQL when you encounter any of these scenarios:
| Metric | SQLite Limit | PostgreSQL | Migration Trigger |
|--------|--------------|------------|-------------------|
| Database Size | ~10GB max | Unlimited | >1GB |
| Concurrent Writers | 1 | Thousands | >50 simultaneous |
| Active Users | <100 | Millions | >200 |
| Transactions/min | <100 | 100K+ | >500 |
| Servers | Single machine | Cluster | Multiple servers |
| High Availability | No | Yes | Required |
| Backup Strategy | Manual | Automated | Automated needed |
## Architecture Overview
The migration uses a **Repository Pattern** to abstract the database layer, allowing both SQLite and PostgreSQL to work seamlessly.
```
project/
├── backend/
│ ├── config/
│ │ ├── __init__.py
│ │ ├── settings.py # Database configuration
│ │ └── database.py # Repository factory
│ ├── repositories/
│ │ ├── __init__.py
│ │ ├── base_repository.py # Abstract base classes
│ │ ├── sqlite_repo.py # SQLite implementation
│ │ └── postgres_repo.py # PostgreSQL implementation
│ └── app.py # Main application (unchanged)
├── migrations/ # Alembic migrations
│ └── versions/
│ └── initial_schema.py
├── scripts/
│ └── migrate_to_postgres.py # Migration script
└── docker-compose.yml # Optional Docker setup
```
## Step 1: Install Dependencies
```bash
# For development
pip install psycopg2-binary sqlalchemy alembic
# For production (more efficient)
pip install psycopg2 sqlalchemy alembic
```
## Step 2: Set Up PostgreSQL
### Option A: Local Installation
```bash
# Ubuntu/Debian
sudo apt update
sudo apt install postgresql postgresql-contrib
# Create database and user
sudo -u postgres psql
```
In PostgreSQL shell:
```sql
CREATE DATABASE tablet_management;
CREATE USER tablet_user WITH PASSWORD 'your_secure_password';
GRANT ALL PRIVILEGES ON DATABASE tablet_management TO tablet_user;
ALTER USER tablet_user CREATEDB;
\q
```
### Option B: Docker (Recommended for Development)
```bash
# Start PostgreSQL container
docker run --name tablet-db -e POSTGRES_PASSWORD=your_password -e POSTGRES_USER=tablet_user -e POSTGRES_DB=tablet_management -p 5432:5432 -d postgres:16-alpine
# Or use docker-compose (see docker-compose.yml)
docker-compose up -d postgres
```
## Step 3: Configure Environment
Create a `.env` file:
```bash
# Database configuration
DB_TYPE=postgres # or 'sqlite'
DB_URL=postgresql://tablet_user:your_password@localhost:5432/tablet_management
# For SQLite (fallback)
DB_PATH=tablets.db
```
Or set environment variables:
```bash
export DB_TYPE=postgres
export DB_URL=postgresql://tablet_user:your_password@localhost:5432/tablet_management
```
## Step 4: Create Repository Abstraction
### Base Repository (Abstract Interface)
```python
# backend/repositories/base_repository.py
from abc import ABC, abstractmethod
from typing import Optional, List
from datetime import datetime
class BaseTabletRepository(ABC):
@abstractmethod
def get_by_id(self, tablet_id: int) -> Optional[dict]:
pass
@abstractmethod
def get_by_serial(self, serial: str) -> Optional[dict]:
pass
@abstractmethod
def get_all(self, status: Optional[str] = None) -> List[dict]:
pass
@abstractmethod
def add(self, brand: str, model: str, serial_number: str, notes: Optional[str] = None) -> dict:
pass
@abstractmethod
def update_status(self, tablet_id: int, status: str) -> bool:
pass
@abstractmethod
def delete(self, tablet_id: int) -> bool:
pass
class BaseUserRepository(ABC):
@abstractmethod
def get_by_id(self, user_id: int) -> Optional[dict]:
pass
@abstractmethod
def get_by_identification(self, identification: str) -> Optional[dict]:
pass
@abstractmethod
def get_all(self) -> List[dict]:
pass
@abstractmethod
def add(self, name: str, email: Optional[str], phone: Optional[str], identification: str) -> dict:
pass
class BaseLoanRepository(ABC):
@abstractmethod
def get_by_id(self, loan_id: int) -> Optional[dict]:
pass
@abstractmethod
def get_active_by_tablet(self, tablet_id: int) -> Optional[dict]:
pass
@abstractmethod
def get_by_user(self, user_id: int) -> List[dict]:
pass
@abstractmethod
def get_all(self, status: Optional[str] = None) -> List[dict]:
pass
@abstractmethod
def create(self, tablet_id: int, user_id: int) -> dict:
pass
@abstractmethod
def return_loan(self, loan_id: int) -> bool:
pass
class BaseNonLoanableDeviceRepository(ABC):
@abstractmethod
def get_by_id(self, device_id: int) -> Optional[dict]:
pass
@abstractmethod
def get_all(self) -> List[dict]:
pass
@abstractmethod
def add(self, brand: str, model: str, serial_number: str, device_type: str,
location: Optional[str] = None, notes: Optional[str] = None,
purchase_date: Optional[str] = None, purchase_cost: Optional[float] = None) -> dict:
pass
@abstractmethod
def update(self, device_id: int, **kwargs) -> bool:
pass
@abstractmethod
def delete(self, device_id: int) -> bool:
pass
```
### SQLite Implementation
```python
# backend/repositories/sqlite_repo.py
import sqlite3
from typing import Optional, List
from .base_repository import (
BaseTabletRepository, BaseUserRepository,
BaseLoanRepository, BaseNonLoanableDeviceRepository
)
class SQLiteTabletRepository(BaseTabletRepository):
def __init__(self, db_path: str = 'tablets.db'):
self.db_path = db_path
self._init_db()
def _init_db(self):
conn = sqlite3.connect(self.db_path)
cursor = conn.cursor()
cursor.execute('''
CREATE TABLE IF NOT EXISTS tablets (
id INTEGER PRIMARY KEY AUTOINCREMENT,
brand TEXT NOT NULL,
model TEXT NOT NULL,
serial_number TEXT UNIQUE NOT NULL,
status TEXT DEFAULT 'available',
notes TEXT
)
''')
conn.commit()
conn.close()
def get_by_id(self, tablet_id: int) -> Optional[dict]:
conn = sqlite3.connect(self.db_path)
conn.row_factory = sqlite3.Row
cursor = conn.cursor()
cursor.execute("SELECT * FROM tablets WHERE id = ?", (tablet_id,))
row = cursor.fetchone()
conn.close()
return dict(row) if row else None
def get_by_serial(self, serial: str) -> Optional[dict]:
conn = sqlite3.connect(self.db_path)
conn.row_factory = sqlite3.Row
cursor = conn.cursor()
cursor.execute("SELECT * FROM tablets WHERE serial_number = ?", (serial,))
row = cursor.fetchone()
conn.close()
return dict(row) if row else None
def get_all(self, status: Optional[str] = None) -> List[dict]:
conn = sqlite3.connect(self.db_path)
conn.row_factory = sqlite3.Row
cursor = conn.cursor()
query = "SELECT * FROM tablets"
params = ()
if status:
query += " WHERE status = ?"
params = (status,)
cursor.execute(query, params)
results = [dict(row) for row in cursor.fetchall()]
conn.close()
return results
def add(self, brand: str, model: str, serial_number: str, notes: Optional[str] = None) -> dict:
conn = sqlite3.connect(self.db_path)
conn.row_factory = sqlite3.Row
cursor = conn.cursor()
cursor.execute('''
INSERT INTO tablets (brand, model, serial_number, status, notes)
VALUES (?, ?, ?, 'available', ?)
''', (brand, model, serial_number, notes))
conn.commit()
tablet_id = cursor.lastrowid
cursor.execute("SELECT * FROM tablets WHERE id = ?", (tablet_id,))
row = cursor.fetchone()
conn.close()
return dict(row)
def update_status(self, tablet_id: int, status: str) -> bool:
conn = sqlite3.connect(self.db_path)
cursor = conn.cursor()
cursor.execute("UPDATE tablets SET status = ? WHERE id = ?", (status, tablet_id))
conn.commit()
changed = cursor.rowcount > 0
conn.close()
return changed
def delete(self, tablet_id: int) -> bool:
conn = sqlite3.connect(self.db_path)
cursor = conn.cursor()
cursor.execute("DELETE FROM tablets WHERE id = ?", (tablet_id,))
conn.commit()
deleted = cursor.rowcount > 0
conn.close()
return deleted
# Similar implementations for SQLiteUserRepository, SQLiteLoanRepository, etc.
```
### PostgreSQL Implementation
```python
# backend/repositories/postgres_repo.py
import psycopg2
from psycopg2 import sql
from psycopg2.extras import DictCursor
from typing import Optional, List
from .base_repository import (
BaseTabletRepository, BaseUserRepository,
BaseLoanRepository, BaseNonLoanableDeviceRepository
)
class PostgreSQLTabletRepository(BaseTabletRepository):
def __init__(self, connection_string: str):
self.connection_string = connection_string
self._init_db()
def _get_connection(self):
return psycopg2.connect(self.connection_string)
def _init_db(self):
conn = self._get_connection()
cursor = conn.cursor()
cursor.execute('''
CREATE TABLE IF NOT EXISTS tablets (
id SERIAL PRIMARY KEY,
brand VARCHAR(100) NOT NULL,
model VARCHAR(100) NOT NULL,
serial_number VARCHAR(50) UNIQUE NOT NULL,
status VARCHAR(20) DEFAULT 'available',
notes TEXT,
created_at TIMESTAMP DEFAULT NOW(),
updated_at TIMESTAMP DEFAULT NOW()
)
''')
cursor.execute('''
CREATE INDEX IF NOT EXISTS idx_tablets_serial
ON tablets(serial_number)
''')
cursor.execute('''
CREATE INDEX IF NOT EXISTS idx_tablets_status
ON tablets(status)
''')
conn.commit()
cursor.close()
conn.close()
def get_by_id(self, tablet_id: int) -> Optional[dict]:
conn = self._get_connection()
cursor = conn.cursor(cursor_factory=DictCursor)
cursor.execute("SELECT * FROM tablets WHERE id = %s", (tablet_id,))
row = cursor.fetchone()
conn.close()
return dict(row) if row else None
def get_by_serial(self, serial: str) -> Optional[dict]:
conn = self._get_connection()
cursor = conn.cursor(cursor_factory=DictCursor)
cursor.execute("SELECT * FROM tablets WHERE serial_number = %s", (serial,))
row = cursor.fetchone()
conn.close()
return dict(row) if row else None
def get_all(self, status: Optional[str] = None) -> List[dict]:
conn = self._get_connection()
cursor = conn.cursor(cursor_factory=DictCursor)
query = "SELECT * FROM tablets"
params = ()
if status:
query += " WHERE status = %s"
params = (status,)
cursor.execute(query, params)
results = [dict(row) for row in cursor.fetchall()]
conn.close()
return results
def add(self, brand: str, model: str, serial_number: str, notes: Optional[str] = None) -> dict:
conn = self._get_connection()
cursor = conn.cursor(cursor_factory=DictCursor)
cursor.execute('''
INSERT INTO tablets (brand, model, serial_number, status, notes)
VALUES (%s, %s, %s, 'available', %s)
RETURNING *
''', (brand, model, serial_number, notes))
row = cursor.fetchone()
conn.commit()
conn.close()
return dict(row)
def update_status(self, tablet_id: int, status: str) -> bool:
conn = self._get_connection()
cursor = conn.cursor()
cursor.execute(
"UPDATE tablets SET status = %s, updated_at = NOW() WHERE id = %s",
(status, tablet_id)
)
conn.commit()
changed = cursor.rowcount > 0
conn.close()
return changed
def delete(self, tablet_id: int) -> bool:
conn = self._get_connection()
cursor = conn.cursor()
cursor.execute("DELETE FROM tablets WHERE id = %s", (tablet_id,))
conn.commit()
deleted = cursor.rowcount > 0
conn.close()
return deleted
# Similar implementations for PostgreSQLUserRepository, PostgreSQLLoanRepository, etc.
```
## Step 5: Create Repository Factory
```python
# backend/config/database.py
import os
from backend.repositories.sqlite_repo import (
SQLiteTabletRepository, SQLiteUserRepository,
SQLiteLoanRepository, SQLiteNonLoanableDeviceRepository
)
from backend.repositories.postgres_repo import (
PostgreSQLTabletRepository, PostgreSQLUserRepository,
PostgreSQLLoanRepository, PostgreSQLNonLoanableDeviceRepository
)
from backend.repositories.base_repository import (
BaseTabletRepository, BaseUserRepository,
BaseLoanRepository, BaseNonLoanableDeviceRepository
)
class DatabaseConfig:
def __init__(self):
self.db_type = os.getenv('DB_TYPE', 'sqlite')
self.db_url = os.getenv('DB_URL', '')
self.db_path = os.getenv('DB_PATH', 'tablets.db')
@property
def is_postgres(self) -> bool:
return self.db_type == 'postgres'
def get_tablet_repository() -> BaseTabletRepository:
config = DatabaseConfig()
if config.is_postgres:
return PostgreSQLTabletRepository(config.db_url)
else:
return SQLiteTabletRepository(config.db_path)
def get_user_repository() -> BaseUserRepository:
config = DatabaseConfig()
if config.is_postgres:
return PostgreSQLUserRepository(config.db_url)
else:
return SQLiteUserRepository(config.db_path)
def get_loan_repository() -> BaseLoanRepository:
config = DatabaseConfig()
if config.is_postgres:
return PostgreSQLLoanRepository(config.db_url)
else:
return SQLiteLoanRepository(config.db_path)
def get_non_loanable_device_repository() -> BaseNonLoanableDeviceRepository:
config = DatabaseConfig()
if config.is_postgres:
return PostgreSQLNonLoanableDeviceRepository(config.db_url)
else:
return SQLiteNonLoanableDeviceRepository(config.db_path)
```
## Step 6: Update Application to Use Repositories
Modify your application to use the repository pattern:
```python
# In your app.py or service layer
from backend.config.database import (
get_tablet_repository, get_user_repository,
get_loan_repository, get_non_loanable_device_repository
)
# Instead of direct SQLite calls:
tablet_repo = get_tablet_repository()
user_repo = get_user_repository()
loan_repo = get_loan_repository()
# Example: Loan a tablet
def loan_tablet(tablet_id: int, user_id: int):
# Get repositories
tablet_repo = get_tablet_repository()
user_repo = get_user_repository()
loan_repo = get_loan_repository()
# Validate
tablet = tablet_repo.get_by_id(tablet_id)
if not tablet:
raise ValueError("Tablet not found")
if tablet['status'] != 'available':
raise ValueError("Tablet not available")
user = user_repo.get_by_id(user_id)
if not user:
raise ValueError("User not found")
# Check for active loan
active_loan = loan_repo.get_active_by_tablet(tablet_id)
if active_loan:
raise ValueError("Tablet already loaned")
# Create loan
loan = loan_repo.create(tablet_id, user_id)
# Update tablet status
tablet_repo.update_status(tablet_id, 'loaned')
return loan
```
## Step 7: Create Migration Script
```python
# scripts/migrate_to_postgres.py
#!/usr/bin/env python3
"""
Migration script from SQLite to PostgreSQL
"""
import sqlite3
import psycopg2
from psycopg2.extras import DictCursor
import argparse
from tqdm import tqdm
import os
def create_postgres_tables(conn):
"""Create all tables in PostgreSQL"""
cursor = conn.cursor()
# Tablets
cursor.execute('''
CREATE TABLE IF NOT EXISTS tablets (
id SERIAL PRIMARY KEY,
brand VARCHAR(100) NOT NULL,
model VARCHAR(100) NOT NULL,
serial_number VARCHAR(50) UNIQUE NOT NULL,
status VARCHAR(20) DEFAULT 'available',
notes TEXT,
created_at TIMESTAMP DEFAULT NOW(),
updated_at TIMESTAMP DEFAULT NOW()
)
''')
# Users
cursor.execute('''
CREATE TABLE IF NOT EXISTS users (
id SERIAL PRIMARY KEY,
name VARCHAR(100) NOT NULL,
email VARCHAR(255),
phone VARCHAR(20),
identification VARCHAR(50) UNIQUE NOT NULL,
created_at TIMESTAMP DEFAULT NOW(),
updated_at TIMESTAMP DEFAULT NOW()
)
''')
# Loans
cursor.execute('''
CREATE TABLE IF NOT EXISTS loans (
id SERIAL PRIMARY KEY,
tablet_id INTEGER NOT NULL REFERENCES tablets(id) ON DELETE RESTRICT,
user_id INTEGER NOT NULL REFERENCES users(id) ON DELETE RESTRICT,
loan_date TIMESTAMP NOT NULL DEFAULT NOW(),
return_date TIMESTAMP,
status VARCHAR(20) DEFAULT 'active',
created_at TIMESTAMP DEFAULT NOW()
)
''')
# Non-loanable devices
cursor.execute('''
CREATE TABLE IF NOT EXISTS non_loanable_devices (
id SERIAL PRIMARY KEY,
brand VARCHAR(100) NOT NULL,
model VARCHAR(100) NOT NULL,
serial_number VARCHAR(50) UNIQUE NOT NULL,
device_type VARCHAR(50) NOT NULL,
location VARCHAR(100),
status VARCHAR(20) DEFAULT 'available',
notes TEXT,
purchase_date DATE,
purchase_cost DECIMAL(10,2),
created_at TIMESTAMP DEFAULT NOW(),
updated_at TIMESTAMP DEFAULT NOW()
)
''')
# Indexes for performance
cursor.execute('CREATE INDEX IF NOT EXISTS idx_tablets_serial ON tablets(serial_number)')
cursor.execute('CREATE INDEX IF NOT EXISTS idx_tablets_status ON tablets(status)')
cursor.execute('CREATE INDEX IF NOT EXISTS idx_tablets_brand ON tablets(brand)')
cursor.execute('CREATE INDEX IF NOT EXISTS idx_users_identification ON users(identification)')
cursor.execute('CREATE INDEX IF NOT EXISTS idx_loans_tablet ON loans(tablet_id)')
cursor.execute('CREATE INDEX IF NOT EXISTS idx_loans_user ON loans(user_id)')
cursor.execute('CREATE INDEX IF NOT EXISTS idx_loans_status ON loans(status)')
conn.commit()
def migrate_table(conn_sqlite, conn_pg, table_name: str, pg_create_table: str):
"""Generic migration for a table"""
cursor_sqlite = conn_sqlite.cursor()
cursor_pg = conn_pg.cursor()
# Get all data from SQLite
cursor_sqlite.execute(f"SELECT * FROM {table_name}")
rows = cursor_sqlite.fetchall()
if not rows:
print(f"No data to migrate for {table_name}")
return
# Get column names
column_names = [desc[0] for desc in cursor_sqlite.description]
# Prepare INSERT statement
columns = ', '.join(column_names)
placeholders = ', '.join(['%s'] * len(column_names))
insert_sql = f"INSERT INTO {table_name} ({columns}) VALUES ({placeholders}) ON CONFLICT DO NOTHING"
# Migrate data
for row in tqdm(rows, desc=f"Migrating {table_name}"):
cursor_pg.execute(insert_sql, row)
conn_pg.commit()
print(f"✓ Migrated {len(rows)} rows from {table_name}")
def migrate_all(sqlite_path: str, pg_url: str):
"""Migrate all data from SQLite to PostgreSQL"""
print("Starting migration from SQLite to PostgreSQL...")
# Connect to SQLite
conn_sqlite = sqlite3.connect(sqlite_path)
# Connect to PostgreSQL
conn_pg = psycopg2.connect(pg_url)
try:
# Create tables
print("Creating PostgreSQL tables...")
create_postgres_tables(conn_pg)
# Migrate each table
migrate_table(conn_sqlite, conn_pg, 'tablets', '')
migrate_table(conn_sqlite, conn_pg, 'users', '')
migrate_table(conn_sqlite, conn_pg, 'loans', '')
migrate_table(conn_sqlite, conn_pg, 'non_loanable_devices', '')
print("\n✓ Migration completed successfully!")
print(f" SQLite: {sqlite_path}")
print(f" PostgreSQL: {pg_url}")
except Exception as e:
conn_pg.rollback()
print(f"\n✗ Migration failed: {e}")
raise
finally:
conn_sqlite.close()
conn_pg.close()
if __name__ == '__main__':
parser = argparse.ArgumentParser(description='Migrate from SQLite to PostgreSQL')
parser.add_argument('--sqlite', default='tablets.db', help='SQLite database path')
parser.add_argument('--postgres', required=True, help='PostgreSQL connection URL')
args = parser.parse_args()
migrate_all(args.sqlite, args.postgres)
```
## Step 8: Run Migration
```bash
# Test the migration first (dry run)
python scripts/migrate_to_postgres.py --sqlite tablets.db --postgres postgresql://tablet_user:password@localhost:5432/tablet_management_test
# Verify data in test database
psql -U tablet_user -d tablet_management_test -c "SELECT COUNT(*) FROM tablets;"
# When ready, migrate to production
python scripts/migrate_to_postgres.py --sqlite tablets.db --postgres postgresql://tablet_user:password@localhost:5432/tablet_management
```
## Step 9: Switch to PostgreSQL
```bash
# Update environment variables
export DB_TYPE=postgres
export DB_URL=postgresql://tablet_user:password@localhost:5432/tablet_management
# Restart application
python app.py
```
## Step 10: Verify and Monitor
```bash
# Check application logs for errors
# Monitor database connections
psql -U tablet_user -d tablet_management -c "SELECT COUNT(*) FROM tablets;"
# Check active connections
psql -U postgres -c "SELECT * FROM pg_stat_activity WHERE datname = 'tablet_management';"
```
## Rollback Plan
If something goes wrong:
1. **Immediate rollback:**
```bash
# Switch back to SQLite
export DB_TYPE=sqlite
export DB_PATH=tablets.db
python app.py
```
2. **Data verification:**
```bash
# Compare counts
sqlite3 tablets.db "SELECT COUNT(*) FROM tablets;"
psql -U tablet_user -d tablet_management -c "SELECT COUNT(*) FROM tablets;"
```
3. **Backup PostgreSQL data:**
```bash
pg_dump -U tablet_user -d tablet_management > postgres_backup_$(date +%Y%m%d).sql
```
## Docker Compose (Optional)
For easy deployment with Docker:
```yaml
# docker-compose.yml
version: '3.8'
services:
postgres:
image: postgres:16-alpine
container_name: tablet_db
environment:
POSTGRES_DB: tablet_management
POSTGRES_USER: tablet_user
POSTGRES_PASSWORD: ${POSTGRES_PASSWORD:-changeme}
ports:
- "5432:5432"
volumes:
- postgres_data:/var/lib/postgresql/data
healthcheck:
test: ["CMD-SHELL", "pg_isready -U tablet_user -d tablet_management"]
interval: 5s
timeout: 5s
retries: 5
restart: unless-stopped
app:
build: .
container_name: tablet_app
environment:
DB_TYPE: postgres
DB_URL: postgresql://tablet_user:${POSTGRES_PASSWORD:-changeme}@postgres:5432/tablet_management
ports:
- "5000:5000"
depends_on:
postgres:
condition: service_healthy
restart: unless-stopped
volumes:
postgres_data:
```
Start with Docker:
```bash
docker-compose up -d
```
## Benefits of PostgreSQL
### Performance
- **Concurrency:** Multiple writers simultaneously (no lock contention)
- **Indexing:** Advanced index types (B-tree, Hash, GiST, GIN, BRIN)
- **Query Optimization:** Advanced query planner
- **Connection Pooling:** Built-in support
### Scalability
- **Vertical:** Handles large datasets efficiently
- **Horizontal:** Read replicas, partitioning, sharding
- **Connections:** Supports thousands of concurrent connections
### Reliability
- **ACID Compliance:** Full transaction support
- **Point-in-Time Recovery:** Restore to any moment
- **Replication:** Master-slave, synchronous, asynchronous
- **Backups:** `pg_dump`, `pg_basebackup`, continuous archiving
### Security
- **Authentication:** Multiple methods (password, MD5, SCRAM, LDAP, Kerberos)
- **Authorization:** Role-based access control (RBAC)
- **Row-Level Security:** Policies for fine-grained access
- **Encryption:** SSL, at-rest encryption
### Features
- **JSON Support:** Native JSON/JSONB data type
- **Full-Text Search:** Advanced text search capabilities
- **Arrays:** Store arrays of values
- **Custom Types:** Create your own data types
- **Triggers:** Automatic actions on events
- **Stored Procedures:** Server-side functions
## Monitoring PostgreSQL
### Basic Queries
```sql
-- Active connections
SELECT * FROM pg_stat_activity WHERE datname = 'tablet_management';
-- Table sizes
SELECT table_name, pg_size_pretty(pg_total_relation_size(table_name))
FROM information_schema.tables WHERE table_schema = 'public';
-- Index usage
SELECT indexrelname, idx_scan, idx_tup_read, idx_tup_fetch
FROM pg_stat_user_indexes;
-- Slow queries (requires pg_stat_statements extension)
SELECT query, total_time, calls, mean_time
FROM pg_stat_statements ORDER BY mean_time DESC LIMIT 10;
```
### Enable pg_stat_statements
```sql
-- In PostgreSQL
CREATE EXTENSION pg_stat_statements;
-- Then in postgresql.conf
shared_preload_libraries = 'pg_stat_statements'
pg_stat_statements.track = all
```
## Maintenance Tasks
### Regular Maintenance
```bash
# Vacuum (reclaim space, update statistics)
vacuumdb -U tablet_user -d tablet_management --analyze
# Reindex (rebuild indexes)
reindexdb -U tablet_user -d tablet_management
```
### Backup Strategy
```bash
# Daily backup
pg_dump -U tablet_user -d tablet_management > /backups/tablet_management_$(date +%Y%m%d).sql
# Compressed backup
pg_dump -U tablet_user -d tablet_management | gzip > /backups/tablet_management_$(date +%Y%m%d).sql.gz
# Continuous archiving (WAL)
# In postgresql.conf:
wal_level = replica
archive_mode = on
archive_command = 'test ! -f /backups/wal/%f && cp %p /backups/wal/%f'
```
## Performance Optimization
### Configuration Tuning
```conf
# postgresql.conf recommendations
shared_buffers = 4GB # 25% of total RAM
work_mem = 16MB # For complex sorts
maintenance_work_mem = 512MB # For VACUUM, index creation
effective_cache_size = 12GB # 75% of total RAM
random_page_cost = 1.1 # SSD: 1.1, HDD: 4.0
max_worker_processes = 8 # Number of CPU cores
max_parallel_workers_per_gather = 4 # Parallel query workers
max_connections = 200 # Expected max connections
```
### Index Optimization
```sql
-- Add indexes for common queries
CREATE INDEX idx_loans_user_status ON loans(user_id, status);
CREATE INDEX idx_loans_date_range ON loans(loan_date, return_date);
-- Partial index for active loans
CREATE INDEX idx_loans_active ON loans(tablet_id) WHERE status = 'active';
-- Composite index for user loans
CREATE INDEX idx_loans_user_tablet ON loans(user_id, tablet_id);
```
## Troubleshooting
### Common Issues
**Connection refused:**
```bash
# Check if PostgreSQL is running
sudo systemctl status postgresql
# Check port
netstat -tuln | grep 5432
```
**Authentication failed:**
```bash
# Verify user and password
psql -U tablet_user -d tablet_management -h localhost
# Check pg_hba.conf
sudo nano /etc/postgresql/16/main/pg_hba.conf
```
**Database does not exist:**
```bash
# Create database
createdb -U postgres tablet_management
```
**Permission denied:**
```sql
-- Grant permissions
GRANT ALL PRIVILEGES ON DATABASE tablet_management TO tablet_user;
GRANT ALL ON SCHEMA public TO tablet_user;
```
### Logs
```bash
# PostgreSQL logs
sudo tail -f /var/log/postgresql/postgresql-16-main.log
# Application logs
journalctl -u tablet_management -f
```
## Conclusion
Migrating from SQLite to PostgreSQL provides:
- Better performance at scale
- True concurrency
- Enhanced reliability
- Advanced features
- Production-ready infrastructure
The repository pattern ensures a smooth transition with minimal code changes, and the migration script automates the data transfer process.

1
gestiontablets.sh Normal file
View file

@ -0,0 +1 @@
vibe --resume fa1bb230

View file

@ -44,6 +44,22 @@ def init_db():
)
''')
# Create non_loanable_devices table
cursor.execute('''
CREATE TABLE IF NOT EXISTS non_loanable_devices (
id INTEGER PRIMARY KEY AUTOINCREMENT,
brand TEXT NOT NULL,
model TEXT NOT NULL,
serial_number TEXT UNIQUE NOT NULL,
device_type TEXT NOT NULL,
location TEXT,
status TEXT DEFAULT 'available',
notes TEXT,
purchase_date TEXT,
purchase_cost REAL
)
''')
conn.commit()
conn.close()
@ -202,6 +218,59 @@ def show_loan_history():
conn.close()
def add_non_loanable_device(brand, model, serial_number, device_type, location='', notes='', purchase_date='', purchase_cost=None):
"""Add a new non-loanable device to inventory"""
conn = sqlite3.connect('tablets.db')
cursor = conn.cursor()
try:
cursor.execute('''
INSERT INTO non_loanable_devices
(brand, model, serial_number, device_type, location, status, notes, purchase_date, purchase_cost)
VALUES (?, ?, ?, ?, ?, 'available', ?, ?, ?)
''', (brand, model, serial_number, device_type, location, notes, purchase_date, purchase_cost))
conn.commit()
print(f"✓ Added non-loanable device: {brand} {model} ({serial_number}) - Type: {device_type}")
except sqlite3.IntegrityError:
print(f"✗ Error: Serial number {serial_number} already exists")
finally:
conn.close()
def show_non_loanable_devices():
"""Show all non-loanable devices"""
conn = sqlite3.connect('tablets.db')
cursor = conn.cursor()
cursor.execute("SELECT id, brand, model, serial_number, device_type, location, status FROM non_loanable_devices")
devices = cursor.fetchall()
print("\n=== Non-Loanable Devices ===")
if devices:
for device in devices:
print(f"ID: {device[0]}, Type: {device[4]}, {device[1]} {device[2]} ({device[3]}), Location: {device[5] or 'N/A'}, Status: {device[6]}")
else:
print("No non-loanable devices")
conn.close()
def delete_non_loanable_device(device_id):
"""Delete a non-loanable device"""
conn = sqlite3.connect('tablets.db')
cursor = conn.cursor()
cursor.execute("DELETE FROM non_loanable_devices WHERE id = ?", (device_id,))
conn.commit()
if cursor.rowcount > 0:
print(f"✓ Non-loanable device {device_id} deleted")
else:
print(f"✗ Error: Device {device_id} not found")
conn.close()
def main():
"""Main menu"""
init_db()
@ -218,9 +287,12 @@ def main():
print("5. Show Available Tablets")
print("6. Show Active Loans")
print("7. Show Loan History")
print("8. Exit")
print("8. Add Non-Loanable Device")
print("9. Show Non-Loanable Devices")
print("10. Delete Non-Loanable Device")
print("11. Exit")
choice = input("Enter your choice (1-8): ")
choice = input("Enter your choice (1-11): ")
if choice == '1':
print("\n=== Add Tablet ===")
@ -269,6 +341,28 @@ def main():
show_loan_history()
elif choice == '8':
print("\n=== Add Non-Loanable Device ===")
brand = input("Brand: ")
model = input("Model: ")
serial_number = input("Serial Number: ")
device_type = input("Device Type (e.g., projector, monitor): ")
location = input("Location (optional): ")
notes = input("Notes (optional): ")
add_non_loanable_device(brand, model, serial_number, device_type, location, notes)
elif choice == '9':
show_non_loanable_devices()
elif choice == '10':
print("\n=== Delete Non-Loanable Device ===")
show_non_loanable_devices()
device_id = input("Enter Device ID to delete: ")
try:
delete_non_loanable_device(int(device_id))
except ValueError:
print("✗ Error: Invalid ID format")
elif choice == '11':
print("Goodbye!")
break

View file

@ -5,3 +5,33 @@ description = "Add your description here"
readme = "README.md"
requires-python = ">=3.14"
dependencies = []
[tool.pytest.ini_options]
testpaths = ["tests"]
python_files = "test_*.py"
python_classes = "Test*"
python_functions = "test_*"
verbose = 1
addopts = "-v"
[tool.coverage.run]
source = ["."]
omit = [
"*/tests/*",
"*/.venv/*",
"*/__pycache__/*",
"*/.git/*",
"*/templates/*",
]
[tool.coverage.report]
exclude_lines = [
"pragma: no cover",
"def __repr__",
"raise NotImplementedError",
"if __name__ == .__main__.:",
"if TYPE_CHECKING:",
]
[tool.coverage.html]
directory = "htmlcov"

View file

@ -62,6 +62,22 @@ def init_db():
)
''')
# Create non_loanable_devices table
cursor.execute('''
CREATE TABLE IF NOT EXISTS non_loanable_devices (
id INTEGER PRIMARY KEY AUTOINCREMENT,
brand TEXT NOT NULL,
model TEXT NOT NULL,
serial_number TEXT UNIQUE NOT NULL,
device_type TEXT NOT NULL,
location TEXT,
status TEXT DEFAULT 'available',
notes TEXT,
purchase_date TEXT,
purchase_cost REAL
)
''')
conn.commit()
class SimpleHTTPRequestHandler(BaseHTTPRequestHandler):

View file

@ -0,0 +1,63 @@
{% extends "base.html" %}
{% block content %}
<h2>Add Non-Loanable Device</h2>
<p>Add a device that will be tracked in inventory but cannot be loaned to users (e.g., projectors, monitors, etc.)</p>
<form method="POST" action="/add_non_loanable_device">
<div class="form-group">
<label for="device_type">Device Type:</label>
<select id="device_type" name="device_type" required>
<option value="">Select device type...</option>
<option value="projector">Projector</option>
<option value="monitor">Monitor</option>
<option value="laptop">Laptop (non-loanable)</option>
<option value="printer">Printer</option>
<option value="camera">Camera</option>
<option value="audio">Audio Equipment</option>
<option value="network">Network Equipment</option>
<option value="other">Other</option>
</select>
</div>
<div class="form-group">
<label for="brand">Brand:</label>
<input type="text" id="brand" name="brand" required>
</div>
<div class="form-group">
<label for="model">Model:</label>
<input type="text" id="model" name="model" required>
</div>
<div class="form-group">
<label for="serial_number">Serial Number:</label>
<input type="text" id="serial_number" name="serial_number" required>
</div>
<div class="form-group">
<label for="location">Location:</label>
<input type="text" id="location" name="location">
</div>
<div class="form-group">
<label for="purchase_date">Purchase Date:</label>
<input type="date" id="purchase_date" name="purchase_date">
</div>
<div class="form-group">
<label for="purchase_cost">Purchase Cost:</label>
<input type="number" id="purchase_cost" name="purchase_cost" step="0.01">
</div>
<div class="form-group">
<label for="notes">Notes:</label>
<textarea id="notes" name="notes"></textarea>
</div>
<div class="form-group">
<button type="submit" class="btn">Add Device</button>
<a href="/non_loanable_devices" class="btn">Cancel</a>
</div>
</form>
{% endblock %}

View file

@ -144,6 +144,7 @@
<a href="/loan_tablet">Loan Tablet</a>
<a href="/history">Loan History</a>
<a href="/user_loans">User Loans</a>
<a href="/non_loanable_devices">Non-Loanable Devices</a>
<a href="/project_management">Project Management</a>
</div>

View file

@ -0,0 +1,71 @@
{% extends "base.html" %}
{% block content %}
<h2>Edit Non-Loanable Device</h2>
<form method="POST" action="/edit_non_loanable_device/{{ device.id }}">
<div class="form-group">
<label for="device_type">Device Type:</label>
<select id="device_type" name="device_type" required>
<option value="projector" {% if device.device_type == 'projector' %}selected{% endif %}>Projector</option>
<option value="monitor" {% if device.device_type == 'monitor' %}selected{% endif %}>Monitor</option>
<option value="laptop" {% if device.device_type == 'laptop' %}selected{% endif %}>Laptop (non-loanable)</option>
<option value="printer" {% if device.device_type == 'printer' %}selected{% endif %}>Printer</option>
<option value="camera" {% if device.device_type == 'camera' %}selected{% endif %}>Camera</option>
<option value="audio" {% if device.device_type == 'audio' %}selected{% endif %}>Audio Equipment</option>
<option value="network" {% if device.device_type == 'network' %}selected{% endif %}>Network Equipment</option>
<option value="other" {% if device.device_type == 'other' %}selected{% endif %}>Other</option>
</select>
</div>
<div class="form-group">
<label for="brand">Brand:</label>
<input type="text" id="brand" name="brand" value="{{ device.brand }}" required>
</div>
<div class="form-group">
<label for="model">Model:</label>
<input type="text" id="model" name="model" value="{{ device.model }}" required>
</div>
<div class="form-group">
<label for="serial_number">Serial Number:</label>
<input type="text" id="serial_number" name="serial_number" value="{{ device.serial_number }}" required>
</div>
<div class="form-group">
<label for="location">Location:</label>
<input type="text" id="location" name="location" value="{{ device.location or '' }}">
</div>
<div class="form-group">
<label for="status">Status:</label>
<select id="status" name="status" required>
<option value="available" {% if device.status == 'available' %}selected{% endif %}>Available</option>
<option value="in_use" {% if device.status == 'in_use' %}selected{% endif %}>In Use</option>
<option value="maintenance" {% if device.status == 'maintenance' %}selected{% endif %}>Maintenance</option>
<option value="retired" {% if device.status == 'retired' %}selected{% endif %}>Retired</option>
</select>
</div>
<div class="form-group">
<label for="purchase_date">Purchase Date:</label>
<input type="date" id="purchase_date" name="purchase_date" value="{{ device.purchase_date or '' }}">
</div>
<div class="form-group">
<label for="purchase_cost">Purchase Cost:</label>
<input type="number" id="purchase_cost" name="purchase_cost" step="0.01" value="{{ device.purchase_cost or '' }}">
</div>
<div class="form-group">
<label for="notes">Notes:</label>
<textarea id="notes" name="notes">{{ device.notes or '' }}</textarea>
</div>
<div class="form-group">
<button type="submit" class="btn">Update Device</button>
<a href="/non_loanable_devices" class="btn">Cancel</a>
</div>
</form>
{% endblock %}

View file

@ -0,0 +1,43 @@
{% extends "base.html" %}
{% block content %}
<div class="section">
<h2>Non-Loanable Devices Inventory</h2>
<p>These devices are tracked in inventory but cannot be loaned to users.</p>
<a href="/add_non_loanable_device" class="btn">Add Non-Loanable Device</a>
{% if devices %}
<table>
<thead>
<tr>
<th>Type</th>
<th>Brand</th>
<th>Model</th>
<th>Serial Number</th>
<th>Location</th>
<th>Status</th>
<th>Actions</th>
</tr>
</thead>
<tbody>
{% for device in devices %}
<tr>
<td>{{ device.device_type }}</td>
<td>{{ device.brand }}</td>
<td>{{ device.model }}</td>
<td>{{ device.serial_number }}</td>
<td>{{ device.location or '-' }}</td>
<td>{{ device.status }}</td>
<td>
<a href="/edit_non_loanable_device/{{ device.id }}" class="btn">Edit</a>
<a href="/delete_non_loanable_device/{{ device.id }}" class="btn btn-danger" onclick="return confirm('Are you sure you want to delete this device?')">Delete</a>
</td>
</tr>
{% endfor %}
</tbody>
</table>
{% else %}
<p>No non-loanable devices registered.</p>
{% endif %}
</div>
{% endblock %}

View file

@ -7,45 +7,98 @@
This page demonstrates the many-to-many relationship between users and tablets via the loans table.
</p>
<div class="user-loans-container">
<!-- Search Box -->
<div class="search-container">
<input type="text" id="searchInput" placeholder="Search by user name, identification, tablet brand, model, or serial number..."
onkeyup="filterUsers()">
<button onclick="clearSearch()" class="btn btn-clear">Clear</button>
</div>
<div class="user-loans-container" id="userLoansContainer">
{% for user_data in users_with_loans %}
{% set user = user_data.user %}
{% set loans = user_data.loans %}
{% set active_loans = loans|selectattr('status', 'equalto', 'active')|list %}
{% set returned_loans = loans|selectattr('status', 'equalto', 'returned')|list %}
<div class="user-card">
<div class="user-card"
data-search="{{ user.name|lower }} {{ user.identification|lower }} {% for loan in loans %}{{ loan.brand|lower }} {{ loan.model|lower }} {{ loan.serial_number|lower }} {% endfor %}">
<div class="user-header">
<h3>{{ user.name }} ({{ user.identification }})</h3>
<span class="loan-count">{{ loans|length }} tablet{{ 's' if loans|length != 1 else '' }} loaned</span>
<span class="loan-count">
{{ active_loans|length }} active, {{ returned_loans|length }} past
</span>
</div>
{% if loans %}
<table class="loans-table">
<thead>
<tr>
<th>Tablet</th>
<th>Serial Number</th>
<th>Loan Date</th>
<th>Return Date</th>
<th>Status</th>
</tr>
</thead>
<tbody>
{% for loan in loans %}
<tr class="loan-row status-{{ loan.status }}">
<td>{{ loan.brand }} {{ loan.model }}</td>
<td>{{ loan.serial_number }}</td>
<td>{{ loan.loan_date }}</td>
<td>{{ loan.return_date or '-' }}</td>
<td>
<span class="status-badge status-{{ loan.status }}">
{{ loan.status }}
</span>
</td>
{% if active_loans %}
<div class="loans-section current-loans">
<h4>Current Loans</h4>
<table class="loans-table">
<thead>
<tr>
<th>Tablet</th>
<th>Serial Number</th>
<th>Loan Date</th>
<th>Status</th>
</tr>
{% endfor %}
</tbody>
</table>
{% else %}
</thead>
<tbody>
{% for loan in active_loans %}
<tr class="loan-row status-{{ loan.status }}">
<td>{{ loan.brand }} {{ loan.model }}</td>
<td>{{ loan.serial_number }}</td>
<td>{{ loan.loan_date }}</td>
<td>
<span class="status-badge status-{{ loan.status }}">
{{ loan.status }}
</span>
</td>
</tr>
{% endfor %}
</tbody>
</table>
</div>
{% endif %}
{% if returned_loans %}
<div class="loans-section past-loans">
<h4>
<button class="expand-toggle" onclick="togglePastLoans(this)">
▶ Past Loans ({{ returned_loans|length }})
</button>
</h4>
<div class="past-loans-content" style="display: none;">
<table class="loans-table">
<thead>
<tr>
<th>Tablet</th>
<th>Serial Number</th>
<th>Loan Date</th>
<th>Return Date</th>
<th>Status</th>
</tr>
</thead>
<tbody>
{% for loan in returned_loans %}
<tr class="loan-row status-{{ loan.status }}">
<td>{{ loan.brand }} {{ loan.model }}</td>
<td>{{ loan.serial_number }}</td>
<td>{{ loan.loan_date }}</td>
<td>{{ loan.return_date or '-' }}</td>
<td>
<span class="status-badge status-{{ loan.status }}">
{{ loan.status }}
</span>
</td>
</tr>
{% endfor %}
</tbody>
</table>
</div>
</div>
{% endif %}
{% if not active_loans and not returned_loans %}
<p class="no-loans">No loans recorded for this user.</p>
{% endif %}
</div>
@ -56,6 +109,32 @@
{% endif %}
</div>
<script>
function filterUsers() {
const search = document.getElementById('searchInput').value.toLowerCase();
const cards = document.querySelectorAll('.user-card');
cards.forEach(card => {
const searchText = card.getAttribute('data-search') || '';
card.style.display = searchText.includes(search) ? '' : 'none';
});
}
function clearSearch() {
document.getElementById('searchInput').value = '';
filterUsers();
}
function togglePastLoans(button) {
const content = button.closest('.loans-section').querySelector('.past-loans-content');
const isExpanded = content.style.display !== 'none';
content.style.display = isExpanded ? 'none' : 'block';
const count = button.textContent.match(/\d+/)[0];
button.textContent = isExpanded ? '▶ Past Loans (' + count + ')' : '▼ Past Loans (' + count + ') ';
}
</script>
<style>
.relationship-note {
background-color: #e8f5e9;
@ -65,6 +144,23 @@
border-left: 4px solid #4CAF50;
}
.search-container {
margin-bottom: 20px;
display: flex;
gap: 10px;
}
.search-container input {
flex: 1;
padding: 10px;
border: 1px solid #ddd;
border-radius: 4px;
}
.btn-clear {
background-color: #999;
}
.user-loans-container {
display: flex;
flex-direction: column;
@ -98,6 +194,28 @@
font-style: italic;
}
.loans-section {
margin-top: 15px;
}
.loans-section h4 {
margin: 0 0 10px 0;
color: #555;
}
.expand-toggle {
background: none;
border: none;
color: #4CAF50;
cursor: pointer;
font-size: 14px;
padding: 0;
}
.expand-toggle:hover {
text-decoration: underline;
}
.loans-table {
width: 100%;
border-collapse: collapse;
@ -120,7 +238,7 @@
}
.loan-row.status-returned {
background-color: #e8f5e9;
background-color: #f5f5f5;
}
.status-badge {
@ -137,7 +255,7 @@
}
.status-badge.status-returned {
background-color: #4CAF50;
background-color: #9E9E9E;
color: white;
}

3
tests/__init__.py Normal file
View file

@ -0,0 +1,3 @@
"""
Unit tests for Tablet Management System
"""

181
tests/conftest.py Normal file
View file

@ -0,0 +1,181 @@
"""
Pytest configuration and fixtures for Tablet Management System tests
"""
import pytest
import sqlite3
import os
import sys
from datetime import datetime
# Add project root to path for imports
sys.path.insert(0, os.path.dirname(os.path.dirname(os.path.abspath(__file__))))
@pytest.fixture
def test_db_path():
"""Path to test database"""
return 'test_tablets.db'
@pytest.fixture
def init_test_db(test_db_path):
"""Initialize a fresh test database with schema"""
# Remove existing test database if it exists
if os.path.exists(test_db_path):
os.remove(test_db_path)
conn = sqlite3.connect(test_db_path)
cursor = conn.cursor()
# Create tables
cursor.execute('''
CREATE TABLE IF NOT EXISTS tablets (
id INTEGER PRIMARY KEY AUTOINCREMENT,
brand TEXT NOT NULL,
model TEXT NOT NULL,
serial_number TEXT UNIQUE NOT NULL,
status TEXT DEFAULT 'available',
notes TEXT
)
''')
cursor.execute('''
CREATE TABLE IF NOT EXISTS users (
id INTEGER PRIMARY KEY AUTOINCREMENT,
name TEXT NOT NULL,
email TEXT,
phone TEXT,
identification TEXT UNIQUE
)
''')
cursor.execute('''
CREATE TABLE IF NOT EXISTS loans (
id INTEGER PRIMARY KEY AUTOINCREMENT,
tablet_id INTEGER NOT NULL,
user_id INTEGER NOT NULL,
loan_date TEXT NOT NULL,
return_date TEXT,
status TEXT DEFAULT 'active',
FOREIGN KEY (tablet_id) REFERENCES tablets (id),
FOREIGN KEY (user_id) REFERENCES users (id)
)
''')
cursor.execute('''
CREATE TABLE IF NOT EXISTS non_loanable_devices (
id INTEGER PRIMARY KEY AUTOINCREMENT,
brand TEXT NOT NULL,
model TEXT NOT NULL,
serial_number TEXT UNIQUE NOT NULL,
device_type TEXT NOT NULL,
location TEXT,
status TEXT DEFAULT 'available',
notes TEXT,
purchase_date TEXT,
purchase_cost REAL
)
''')
conn.commit()
conn.close()
yield test_db_path
# Cleanup: remove test database
if os.path.exists(test_db_path):
os.remove(test_db_path)
@pytest.fixture
def db_conn(init_test_db):
"""Get a database connection to the test database"""
conn = sqlite3.connect(init_test_db)
conn.row_factory = sqlite3.Row
yield conn
conn.close()
@pytest.fixture
def sample_tablets(db_conn):
"""Insert sample tablets into test database"""
cursor = db_conn.cursor()
tablets = [
('Samsung', 'Galaxy Tab S7', 'SN001', 'available'),
('Apple', 'iPad Pro', 'SN002', 'available'),
('Lenovo', 'Tab P11', 'SN003', 'available'),
('Microsoft', 'Surface Pro', 'SN004', 'loaned'),
]
for brand, model, serial, status in tablets:
cursor.execute('''
INSERT INTO tablets (brand, model, serial_number, status)
VALUES (?, ?, ?, ?)
''', (brand, model, serial, status))
db_conn.commit()
return tablets
@pytest.fixture
def sample_users(db_conn):
"""Insert sample users into test database"""
cursor = db_conn.cursor()
users = [
('John Doe', 'john@example.com', '1234567890', 'ID001'),
('Jane Smith', 'jane@example.com', '0987654321', 'ID002'),
('Bob Johnson', 'bob@example.com', '5551234567', 'ID003'),
]
for name, email, phone, identification in users:
cursor.execute('''
INSERT INTO users (name, email, phone, identification)
VALUES (?, ?, ?, ?)
''', (name, email, phone, identification))
db_conn.commit()
return users
@pytest.fixture
def sample_loans(db_conn, sample_tablets, sample_users):
"""Insert sample loans into test database"""
cursor = db_conn.cursor()
# Get tablet and user IDs
cursor.execute("SELECT id FROM tablets WHERE serial_number = 'SN004'")
loaned_tablet = cursor.fetchone()
cursor.execute("SELECT id FROM users WHERE identification = 'ID001'")
user1 = cursor.fetchone()
cursor.execute("SELECT id FROM users WHERE identification = 'ID002'")
user2 = cursor.fetchone()
if loaned_tablet and user1:
# Active loan for SN004 to user1
loan_date = datetime.now().strftime('%Y-%m-%d %H:%M:%S')
cursor.execute('''
INSERT INTO loans (tablet_id, user_id, loan_date, status)
VALUES (?, ?, ?, 'active')
''', (loaned_tablet['id'], user1['id'], loan_date))
if user2:
# Returned loan for SN001 to user2
loan_date = datetime.now().strftime('%Y-%m-%d %H:%M:%S')
return_date = datetime.now().strftime('%Y-%m-%d %H:%M:%S')
cursor.execute('''
INSERT INTO loans (tablet_id, user_id, loan_date, return_date, status)
VALUES (?, ?, ?, ?, 'returned')
''', (1, user2['id'], loan_date, return_date))
db_conn.commit()
@pytest.fixture
def populated_db(db_conn, sample_tablets, sample_users, sample_loans):
"""Database with all sample data loaded"""
return db_conn

2
tests/requirements.txt Normal file
View file

@ -0,0 +1,2 @@
pytest==8.3.2
pytest-cov==5.0.0

481
tests/test_core.py Normal file
View file

@ -0,0 +1,481 @@
"""
Unit tests for core tablet management functions
Tests loan logic, validation, and database operations
"""
import pytest
import sqlite3
from datetime import datetime
import sys
import os
# Import the functions from minimal_app (they work with any db connection)
sys.path.insert(0, os.path.dirname(os.path.dirname(os.path.abspath(__file__))))
def get_tablet_count(conn):
"""Helper to get tablet count"""
cursor = conn.cursor()
cursor.execute("SELECT COUNT(*) FROM tablets")
return cursor.fetchone()[0]
def get_user_count(conn):
"""Helper to get user count"""
cursor = conn.cursor()
cursor.execute("SELECT COUNT(*) FROM users")
return cursor.fetchone()[0]
def get_loan_count(conn):
"""Helper to get loan count"""
cursor = conn.cursor()
cursor.execute("SELECT COUNT(*) FROM loans")
return cursor.fetchone()[0]
class TestTabletOperations:
"""Tests for tablet CRUD operations"""
def test_add_tablet_success(self, db_conn):
"""Test adding a new tablet successfully"""
cursor = db_conn.cursor()
# Add a tablet
cursor.execute('''
INSERT INTO tablets (brand, model, serial_number, status)
VALUES (?, ?, ?, 'available')
''', ('TestBrand', 'TestModel', 'TEST001'))
db_conn.commit()
# Verify it was added
cursor.execute("SELECT * FROM tablets WHERE serial_number = 'TEST001'")
tablet = cursor.fetchone()
assert tablet is not None
assert tablet['brand'] == 'TestBrand'
assert tablet['model'] == 'TestModel'
assert tablet['serial_number'] == 'TEST001'
assert tablet['status'] == 'available'
def test_add_tablet_duplicate_serial(self, db_conn):
"""Test that duplicate serial numbers are rejected"""
cursor = db_conn.cursor()
# Add first tablet
cursor.execute('''
INSERT INTO tablets (brand, model, serial_number, status)
VALUES (?, ?, ?, 'available')
''', ('Brand1', 'Model1', 'DUP001'))
db_conn.commit()
# Try to add duplicate
with pytest.raises(sqlite3.IntegrityError):
cursor.execute('''
INSERT INTO tablets (brand, model, serial_number, status)
VALUES (?, ?, ?, 'available')
''', ('Brand2', 'Model2', 'DUP001'))
db_conn.commit()
def test_tablet_status_update(self, db_conn):
"""Test updating tablet status"""
cursor = db_conn.cursor()
# Add a tablet
cursor.execute('''
INSERT INTO tablets (brand, model, serial_number, status)
VALUES (?, ?, ?, 'available')
''', ('Brand', 'Model', 'STATUS001'))
db_conn.commit()
# Update status
cursor.execute("UPDATE tablets SET status = 'loaned' WHERE serial_number = 'STATUS001'")
db_conn.commit()
# Verify update
cursor.execute("SELECT status FROM tablets WHERE serial_number = 'STATUS001'")
status = cursor.fetchone()['status']
assert status == 'loaned'
class TestUserOperations:
"""Tests for user CRUD operations"""
def test_add_user_success(self, db_conn):
"""Test adding a new user successfully"""
cursor = db_conn.cursor()
cursor.execute('''
INSERT INTO users (name, identification)
VALUES (?, ?)
''', ('Test User', 'TESTID001'))
db_conn.commit()
cursor.execute("SELECT * FROM users WHERE identification = 'TESTID001'")
user = cursor.fetchone()
assert user is not None
assert user['name'] == 'Test User'
assert user['identification'] == 'TESTID001'
def test_add_user_duplicate_identification(self, db_conn):
"""Test that duplicate user identifications are rejected"""
cursor = db_conn.cursor()
# Add first user
cursor.execute('''
INSERT INTO users (name, identification)
VALUES (?, ?)
''', ('User1', 'DUPID001'))
db_conn.commit()
# Try to add duplicate
with pytest.raises(sqlite3.IntegrityError):
cursor.execute('''
INSERT INTO users (name, identification)
VALUES (?, ?)
''', ('User2', 'DUPID001'))
db_conn.commit()
def test_user_with_contact_info(self, db_conn):
"""Test adding user with email and phone"""
cursor = db_conn.cursor()
cursor.execute('''
INSERT INTO users (name, email, phone, identification)
VALUES (?, ?, ?, ?)
''', ('Contact User', 'test@email.com', '1234567890', 'CONTACT001'))
db_conn.commit()
cursor.execute("SELECT * FROM users WHERE identification = 'CONTACT001'")
user = cursor.fetchone()
assert user['email'] == 'test@email.com'
assert user['phone'] == '1234567890'
class TestLoanOperations:
"""Tests for loan operations - the core business logic"""
def test_loan_tablet_success(self, populated_db):
"""Test loaning an available tablet to a user"""
cursor = populated_db.cursor()
# Get an available tablet and user
cursor.execute("SELECT id FROM tablets WHERE status = 'available' LIMIT 1")
tablet = cursor.fetchone()
cursor.execute("SELECT id FROM users LIMIT 1")
user = cursor.fetchone()
assert tablet is not None
assert user is not None
tablet_id = tablet['id']
user_id = user['id']
# Loan the tablet
loan_date = datetime.now().strftime('%Y-%m-%d %H:%M:%S')
cursor.execute("UPDATE tablets SET status = 'loaned' WHERE id = ?", (tablet_id,))
cursor.execute('''
INSERT INTO loans (tablet_id, user_id, loan_date, status)
VALUES (?, ?, ?, 'active')
''', (tablet_id, user_id, loan_date))
populated_db.commit()
# Verify tablet status changed
cursor.execute("SELECT status FROM tablets WHERE id = ?", (tablet_id,))
status = cursor.fetchone()['status']
assert status == 'loaned'
# Verify loan was created
cursor.execute("SELECT * FROM loans WHERE tablet_id = ? AND user_id = ?", (tablet_id, user_id))
loan = cursor.fetchone()
assert loan is not None
assert loan['status'] == 'active'
assert loan['return_date'] is None
def test_loan_already_loaned_tablet(self, populated_db):
"""Test that loaning an already loaned tablet fails gracefully"""
cursor = populated_db.cursor()
# Get a loaned tablet (SN004 should be loaned from sample data)
cursor.execute("SELECT id FROM tablets WHERE status = 'loaned' LIMIT 1")
tablet = cursor.fetchone()
cursor.execute("SELECT id FROM users LIMIT 1")
user = cursor.fetchone()
if tablet and user:
tablet_id = tablet['id']
user_id = user['id']
# Try to loan it again (should check status first)
cursor.execute("SELECT status FROM tablets WHERE id = ?", (tablet_id,))
status = cursor.fetchone()['status']
# This should be 'loaned', so we shouldn't be able to loan it
assert status == 'loaned'
# The application logic should prevent this
# In the actual app, this would be checked before inserting
# Here we verify the status is still loaned
cursor.execute("SELECT status FROM tablets WHERE id = ?", (tablet_id,))
final_status = cursor.fetchone()['status']
assert final_status == 'loaned'
def test_return_tablet_success(self, populated_db):
"""Test returning a loaned tablet"""
cursor = populated_db.cursor()
# Get an active loan
cursor.execute("SELECT * FROM loans WHERE status = 'active' LIMIT 1")
loan = cursor.fetchone()
if loan:
loan_id = loan['id']
tablet_id = loan['tablet_id']
# Return the tablet
return_date = datetime.now().strftime('%Y-%m-%d %H:%M:%S')
cursor.execute('''
UPDATE loans SET status = 'returned', return_date = ? WHERE id = ?
''', (return_date, loan_id))
cursor.execute("UPDATE tablets SET status = 'available' WHERE id = ?", (tablet_id,))
populated_db.commit()
# Verify loan status changed
cursor.execute("SELECT status, return_date FROM loans WHERE id = ?", (loan_id,))
updated_loan = cursor.fetchone()
assert updated_loan['status'] == 'returned'
assert updated_loan['return_date'] is not None
# Verify tablet status changed
cursor.execute("SELECT status FROM tablets WHERE id = ?", (tablet_id,))
tablet_status = cursor.fetchone()['status']
assert tablet_status == 'available'
def test_return_nonexistent_loan(self, db_conn):
"""Test returning a loan that doesn't exist"""
cursor = db_conn.cursor()
# Try to return a non-existent loan
cursor.execute("SELECT tablet_id FROM loans WHERE id = ? AND status = 'active'", (9999,))
loan = cursor.fetchone()
# Should be None since loan doesn't exist
assert loan is None
class TestEdgeCases:
"""Tests for edge cases and error conditions"""
def test_loan_to_nonexistent_user(self, db_conn):
"""Test loaning to a user that doesn't exist"""
cursor = db_conn.cursor()
# Add a tablet
cursor.execute('''
INSERT INTO tablets (brand, model, serial_number, status)
VALUES (?, ?, ?, 'available')
''', ('Brand', 'Model', 'EDGE001'))
db_conn.commit()
cursor.execute("SELECT id FROM tablets WHERE serial_number = 'EDGE001'")
tablet = cursor.fetchone()
# Try to loan to non-existent user (ID 9999)
# Note: SQLite doesn't enforce foreign keys by default unless we enable it
# The application should validate this at the application level
# For now, we verify that the user doesn't exist
cursor.execute("SELECT id FROM users WHERE id = 9999")
user = cursor.fetchone()
assert user is None # User doesn't exist
# In a real app with FK enforcement, this would raise IntegrityError
# For SQLite without FK enforcement, we just verify the user check
def test_loan_nonexistent_tablet(self, db_conn):
"""Test loaning a tablet that doesn't exist"""
cursor = db_conn.cursor()
# Add a user
cursor.execute('''
INSERT INTO users (name, identification)
VALUES (?, ?)
''', ('Test User', 'EDGEID001'))
db_conn.commit()
cursor.execute("SELECT id FROM users WHERE identification = 'EDGEID001'")
user = cursor.fetchone()
# Try to loan non-existent tablet (ID 9999)
# Note: SQLite doesn't enforce foreign keys by default unless we enable it
# The application should validate this at the application level
# For now, we verify that the tablet doesn't exist
cursor.execute("SELECT id FROM tablets WHERE id = 9999")
tablet_check = cursor.fetchone()
assert tablet_check is None # Tablet doesn't exist
# In a real app with FK enforcement, this would raise IntegrityError
# For SQLite without FK enforcement, we just verify the tablet check
def test_empty_database_operations(self, db_conn):
"""Test operations on empty database"""
cursor = db_conn.cursor()
# Query empty tables
cursor.execute("SELECT COUNT(*) FROM tablets")
tablet_count = cursor.fetchone()[0]
assert tablet_count == 0
cursor.execute("SELECT COUNT(*) FROM users")
user_count = cursor.fetchone()[0]
assert user_count == 0
cursor.execute("SELECT COUNT(*) FROM loans")
loan_count = cursor.fetchone()[0]
assert loan_count == 0
def test_multiple_loans_same_user(self, db_conn):
"""Test that one user can have multiple loans (one-to-many relationship)"""
cursor = db_conn.cursor()
# Add user
cursor.execute('''
INSERT INTO users (name, identification)
VALUES (?, ?)
''', ('MultiLoan User', 'MULTI001'))
db_conn.commit()
cursor.execute("SELECT id FROM users WHERE identification = 'MULTI001'")
user = cursor.fetchone()
# Add multiple tablets
for i in range(3):
cursor.execute('''
INSERT INTO tablets (brand, model, serial_number, status)
VALUES (?, ?, ?, 'available')
''', (f'Brand{i}', f'Model{i}', f'MULTI{i:03d}'))
db_conn.commit()
# Loan all tablets to the same user
cursor.execute("SELECT id FROM tablets WHERE serial_number LIKE 'MULTI%'")
tablets = cursor.fetchall()
loan_date = datetime.now().strftime('%Y-%m-%d %H:%M:%S')
for tablet in tablets:
cursor.execute("UPDATE tablets SET status = 'loaned' WHERE id = ?", (tablet['id'],))
cursor.execute('''
INSERT INTO loans (tablet_id, user_id, loan_date, status)
VALUES (?, ?, ?, 'active')
''', (tablet['id'], user['id'], loan_date))
db_conn.commit()
# Verify user has multiple loans
cursor.execute("SELECT COUNT(*) FROM loans WHERE user_id = ?", (user['id'],))
loan_count = cursor.fetchone()[0]
assert loan_count == 3
def test_serial_number_uniqueness_across_tables(self, db_conn):
"""Test that serial numbers are unique within their respective tables"""
cursor = db_conn.cursor()
# Add tablet with serial number
cursor.execute('''
INSERT INTO tablets (brand, model, serial_number, status)
VALUES (?, ?, ?, 'available')
''', ('Brand', 'Model', 'UNIQUE001'))
db_conn.commit()
# Add non-loanable device with same serial number (should be allowed - different tables)
cursor.execute('''
INSERT INTO non_loanable_devices
(brand, model, serial_number, device_type, status)
VALUES (?, ?, ?, ?, 'available')
''', ('Brand', 'Model', 'UNIQUE001', 'projector'))
db_conn.commit()
# Both should exist (different tables)
cursor.execute("SELECT COUNT(*) FROM tablets WHERE serial_number = 'UNIQUE001'")
tablet_count = cursor.fetchone()[0]
cursor.execute("SELECT COUNT(*) FROM non_loanable_devices WHERE serial_number = 'UNIQUE001'")
device_count = cursor.fetchone()[0]
assert tablet_count == 1
assert device_count == 1
# But duplicate within same table should fail
with pytest.raises(sqlite3.IntegrityError):
cursor.execute('''
INSERT INTO tablets (brand, model, serial_number, status)
VALUES (?, ?, ?, 'available')
''', ('Brand2', 'Model2', 'UNIQUE001'))
db_conn.commit()
class TestQueryOperations:
"""Tests for query and filtering operations"""
def test_query_available_tablets(self, populated_db):
"""Test querying available tablets"""
cursor = populated_db.cursor()
cursor.execute("SELECT * FROM tablets WHERE status = 'available'")
available = cursor.fetchall()
# Should have at least the sample available tablets
assert len(available) >= 3 # SN001, SN002, SN003 from sample
def test_query_loaned_tablets(self, populated_db):
"""Test querying loaned tablets"""
cursor = populated_db.cursor()
cursor.execute("SELECT * FROM tablets WHERE status = 'loaned'")
loaned = cursor.fetchall()
# Should have at least SN004 from sample data
assert len(loaned) >= 1
def test_query_active_loans(self, populated_db):
"""Test querying active loans"""
cursor = populated_db.cursor()
cursor.execute("SELECT * FROM loans WHERE status = 'active'")
active = cursor.fetchall()
# Should have at least 1 active loan from sample
assert len(active) >= 1
def test_query_returned_loans(self, populated_db):
"""Test querying returned loans"""
cursor = populated_db.cursor()
cursor.execute("SELECT * FROM loans WHERE status = 'returned'")
returned = cursor.fetchall()
# Should have at least 1 returned loan from sample
assert len(returned) >= 1
def test_query_loans_by_user(self, populated_db):
"""Test querying loans by user"""
cursor = populated_db.cursor()
cursor.execute("SELECT id FROM users LIMIT 1")
user = cursor.fetchone()
if user:
cursor.execute("SELECT * FROM loans WHERE user_id = ?", (user['id'],))
loans = cursor.fetchall()
# User should have at least 0 loans
assert isinstance(loans, list)
def test_query_tablets_by_brand(self, populated_db):
"""Test querying tablets by brand"""
cursor = populated_db.cursor()
cursor.execute("SELECT * FROM tablets WHERE brand = 'Samsung'")
samsung = cursor.fetchall()
# Should find Samsung tablet from sample data
assert len(samsung) >= 1
assert samsung[0]['brand'] == 'Samsung'

513
tests/test_edge_cases.py Normal file
View file

@ -0,0 +1,513 @@
"""
Edge case tests for Tablet Management System
Tests critical scenarios: already loaned devices, non-existent loans, duplicates, etc.
"""
import pytest
import sqlite3
from datetime import datetime
import sys
import os
sys.path.insert(0, os.path.dirname(os.path.dirname(os.path.abspath(__file__))))
class TestLoanEdgeCases:
"""Critical edge cases for loan operations"""
def test_loan_device_already_loaned(self, db_conn):
"""
CRITICAL: Test that a device already loaned cannot be loaned again
This prevents the same physical device from being loaned to multiple users
"""
cursor = db_conn.cursor()
# Add a tablet
cursor.execute('''
INSERT INTO tablets (brand, model, serial_number, status)
VALUES (?, ?, ?, 'available')
''', ('Brand', 'Model', 'ALREADY_LOANED'))
db_conn.commit()
cursor.execute("SELECT id FROM tablets WHERE serial_number = 'ALREADY_LOANED'")
tablet = cursor.fetchone()
# Add a user
cursor.execute('''
INSERT INTO users (name, identification)
VALUES (?, ?)
''', ('User1', 'USER1'))
db_conn.commit()
cursor.execute("SELECT id FROM users WHERE identification = 'USER1'")
user1 = cursor.fetchone()
# Add another user
cursor.execute('''
INSERT INTO users (name, identification)
VALUES (?, ?)
''', ('User2', 'USER2'))
db_conn.commit()
cursor.execute("SELECT id FROM users WHERE identification = 'USER2'")
user2 = cursor.fetchone()
# Loan to first user
loan_date = datetime.now().strftime('%Y-%m-%d %H:%M:%S')
cursor.execute("UPDATE tablets SET status = 'loaned' WHERE id = ?", (tablet['id'],))
cursor.execute('''
INSERT INTO loans (tablet_id, user_id, loan_date, status)
VALUES (?, ?, ?, 'active')
''', (tablet['id'], user1['id'], loan_date))
db_conn.commit()
# Verify tablet is loaned
cursor.execute("SELECT status FROM tablets WHERE id = ?", (tablet['id'],))
status = cursor.fetchone()['status']
assert status == 'loaned'
# Try to loan to second user - should check status first
# In the actual application, this would be prevented by checking status
cursor.execute("SELECT status FROM tablets WHERE id = ?", (tablet['id'],))
current_status = cursor.fetchone()['status']
# The application logic should prevent this
assert current_status == 'loaned'
# If we tried to loan it anyway (without checking), we'd get a constraint error
# because the tablet status is already 'loaned'
# The proper app logic checks status before allowing loan
def test_return_nonexistent_loan_id(self, db_conn):
"""
CRITICAL: Test returning a loan that doesn't exist
Should handle gracefully without crashing
"""
cursor = db_conn.cursor()
# Try to return a non-existent loan
loan_id = 99999
cursor.execute("SELECT tablet_id FROM loans WHERE id = ? AND status = 'active'", (loan_id,))
loan = cursor.fetchone()
# Should return None (no such loan)
assert loan is None
# The application should handle this by showing an error message
# rather than crashing
def test_return_already_returned_loan(self, db_conn):
"""
CRITICAL: Test returning a loan that's already been returned
Should handle gracefully
"""
cursor = db_conn.cursor()
# Add tablet and user
cursor.execute('''
INSERT INTO tablets (brand, model, serial_number, status)
VALUES (?, ?, ?, 'available')
''', ('Brand', 'Model', 'ALREADY_RETURNED'))
db_conn.commit()
cursor.execute("SELECT id FROM tablets WHERE serial_number = 'ALREADY_RETURNED'")
tablet = cursor.fetchone()
cursor.execute('''
INSERT INTO users (name, identification)
VALUES (?, ?)
''', ('User', 'RETURN_USER'))
db_conn.commit()
cursor.execute("SELECT id FROM users WHERE identification = 'RETURN_USER'")
user = cursor.fetchone()
# Create and return a loan
loan_date = datetime.now().strftime('%Y-%m-%d %H:%M:%S')
return_date = datetime.now().strftime('%Y-%m-%d %H:%M:%S')
cursor.execute("UPDATE tablets SET status = 'loaned' WHERE id = ?", (tablet['id'],))
cursor.execute('''
INSERT INTO loans (tablet_id, user_id, loan_date, return_date, status)
VALUES (?, ?, ?, ?, 'returned')
''', (tablet['id'], user['id'], loan_date, return_date))
cursor.execute("UPDATE tablets SET status = 'available' WHERE id = ?", (tablet['id'],))
db_conn.commit()
# Get the loan ID
cursor.execute("SELECT id FROM loans WHERE tablet_id = ?", (tablet['id'],))
loan = cursor.fetchone()
loan_id = loan['id']
# Try to return it again
cursor.execute("SELECT tablet_id FROM loans WHERE id = ? AND status = 'active'", (loan_id,))
active_loan = cursor.fetchone()
# Should be None because status is 'returned', not 'active'
assert active_loan is None
def test_loan_with_invalid_tablet_id(self, db_conn):
"""
CRITICAL: Test loaning with an invalid/non-existent tablet ID
Should fail gracefully
"""
cursor = db_conn.cursor()
# Add a user
cursor.execute('''
INSERT INTO users (name, identification)
VALUES (?, ?)
''', ('User', 'INVALID_USER'))
db_conn.commit()
cursor.execute("SELECT id FROM users WHERE identification = 'INVALID_USER'")
user = cursor.fetchone()
# Try to loan with invalid tablet ID
invalid_tablet_id = 99999
loan_date = datetime.now().strftime('%Y-%m-%d %H:%M:%S')
# Note: SQLite doesn't enforce foreign keys by default unless we enable it
# The application should validate this at the application level
# For now, we verify that the tablet doesn't exist
cursor.execute("SELECT id FROM tablets WHERE id = ?", (invalid_tablet_id,))
tablet_check = cursor.fetchone()
assert tablet_check is None # Tablet doesn't exist
# In a real app with FK enforcement, this would raise IntegrityError
# For SQLite without FK enforcement, we just verify the tablet check
def test_loan_with_invalid_user_id(self, db_conn):
"""
CRITICAL: Test loaning with an invalid/non-existent user ID
Should fail gracefully
"""
cursor = db_conn.cursor()
# Add a tablet
cursor.execute('''
INSERT INTO tablets (brand, model, serial_number, status)
VALUES (?, ?, ?, 'available')
''', ('Brand', 'Model', 'INVALID_LOAN'))
db_conn.commit()
cursor.execute("SELECT id FROM tablets WHERE serial_number = 'INVALID_LOAN'")
tablet = cursor.fetchone()
# Try to loan with invalid user ID
invalid_user_id = 99999
loan_date = datetime.now().strftime('%Y-%m-%d %H:%M:%S')
# Note: SQLite doesn't enforce foreign keys by default unless we enable it
# The application should validate this at the application level
# For now, we verify that the user doesn't exist
cursor.execute("SELECT id FROM users WHERE id = ?", (invalid_user_id,))
user_check = cursor.fetchone()
assert user_check is None # User doesn't exist
# In a real app with FK enforcement, this would raise IntegrityError
# For SQLite without FK enforcement, we just verify the user check
class TestDuplicatePrevention:
"""Tests for preventing duplicate entries"""
def test_duplicate_tablet_serial_number(self, db_conn):
"""
CRITICAL: Test that duplicate tablet serial numbers are prevented
Serial numbers must be unique for tracking
"""
cursor = db_conn.cursor()
# Add first tablet
cursor.execute('''
INSERT INTO tablets (brand, model, serial_number, status)
VALUES (?, ?, ?, 'available')
''', ('Brand1', 'Model1', 'DUP_SERIAL'))
db_conn.commit()
# Try to add duplicate
with pytest.raises(sqlite3.IntegrityError):
cursor.execute('''
INSERT INTO tablets (brand, model, serial_number, status)
VALUES (?, ?, ?, 'available')
''', ('Brand2', 'Model2', 'DUP_SERIAL'))
db_conn.commit()
def test_duplicate_user_identification(self, db_conn):
"""
CRITICAL: Test that duplicate user identifications are prevented
User identifications must be unique
"""
cursor = db_conn.cursor()
# Add first user
cursor.execute('''
INSERT INTO users (name, identification)
VALUES (?, ?)
''', ('User1', 'DUP_ID'))
db_conn.commit()
# Try to add duplicate
with pytest.raises(sqlite3.IntegrityError):
cursor.execute('''
INSERT INTO users (name, identification)
VALUES (?, ?)
''', ('User2', 'DUP_ID'))
db_conn.commit()
def test_duplicate_non_loanable_device_serial(self, db_conn):
"""
CRITICAL: Test that duplicate non-loanable device serials are prevented
"""
cursor = db_conn.cursor()
# Add first device
cursor.execute('''
INSERT INTO non_loanable_devices
(brand, model, serial_number, device_type, status)
VALUES (?, ?, ?, ?, 'available')
''', ('Brand', 'Model', 'DUP_DEVICE_SERIAL', 'projector'))
db_conn.commit()
# Try to add duplicate
with pytest.raises(sqlite3.IntegrityError):
cursor.execute('''
INSERT INTO non_loanable_devices
(brand, model, serial_number, device_type, status)
VALUES (?, ?, ?, ?, 'available')
''', ('Brand2', 'Model2', 'DUP_DEVICE_SERIAL', 'monitor'))
db_conn.commit()
class TestDataIntegrity:
"""Tests for data integrity constraints"""
def test_foreign_key_tablet_deletion(self, db_conn):
"""
Test that deleting a tablet with active loans is handled
SQLite defaults to allowing this, but we should be aware
"""
cursor = db_conn.cursor()
# Add tablet and user
cursor.execute('''
INSERT INTO tablets (brand, model, serial_number, status)
VALUES (?, ?, ?, 'available')
''', ('Brand', 'Model', 'FK_TEST'))
db_conn.commit()
cursor.execute("SELECT id FROM tablets WHERE serial_number = 'FK_TEST'")
tablet = cursor.fetchone()
cursor.execute('''
INSERT INTO users (name, identification)
VALUES (?, ?)
''', ('User', 'FK_USER'))
db_conn.commit()
cursor.execute("SELECT id FROM users WHERE identification = 'FK_USER'")
user = cursor.fetchone()
# Create active loan
loan_date = datetime.now().strftime('%Y-%m-%d %H:%M:%S')
cursor.execute("UPDATE tablets SET status = 'loaned' WHERE id = ?", (tablet['id'],))
cursor.execute('''
INSERT INTO loans (tablet_id, user_id, loan_date, status)
VALUES (?, ?, ?, 'active')
''', (tablet['id'], user['id'], loan_date))
db_conn.commit()
# SQLite allows this by default (no ON DELETE RESTRICT)
# In production, we might want to add CASCADE or RESTRICT
# For now, just verify the loan exists
cursor.execute("SELECT COUNT(*) FROM loans WHERE tablet_id = ?", (tablet['id'],))
loan_count = cursor.fetchone()[0]
assert loan_count == 1
def test_null_serial_number_prevention(self, db_conn):
"""
Test that NULL serial numbers are prevented
Serial numbers are required (NOT NULL constraint)
"""
cursor = db_conn.cursor()
# Try to add tablet with NULL serial number
with pytest.raises(sqlite3.IntegrityError):
cursor.execute('''
INSERT INTO tablets (brand, model, serial_number, status)
VALUES (?, ?, ?, 'available')
''', ('Brand', 'Model', None))
db_conn.commit()
def test_null_identification_prevention(self, db_conn):
"""
Test that NULL user identifications are prevented
Identifications are required (NOT NULL constraint)
"""
cursor = db_conn.cursor()
# Try to add user with NULL identification
# Note: identification is NOT marked as NOT NULL in the schema
# This test verifies the current behavior
cursor.execute('''
INSERT INTO users (name, identification)
VALUES (?, ?)
''', ('User', None))
db_conn.commit()
# This should work because identification is not NOT NULL
# But in practice, we should have this constraint
cursor.execute("SELECT COUNT(*) FROM users WHERE identification IS NULL")
count = cursor.fetchone()[0]
# This will be 1, showing that NULL is currently allowed
# In production, we should add NOT NULL constraint
def test_empty_string_serial_number(self, db_conn):
"""
Test handling of empty string serial numbers
Empty strings are different from NULL
"""
cursor = db_conn.cursor()
# Add tablet with empty string serial number
cursor.execute('''
INSERT INTO tablets (brand, model, serial_number, status)
VALUES (?, ?, ?, 'available')
''', ('Brand', 'Model', ''))
db_conn.commit()
# This should work (empty string is allowed unless we add CHECK constraint)
cursor.execute("SELECT COUNT(*) FROM tablets WHERE serial_number = ''")
count = cursor.fetchone()[0]
assert count == 1
# In production, we might want to prevent empty strings
# with a CHECK constraint: CHECK(serial_number <> '')
class TestConcurrentScenarioSimulations:
"""Simulate scenarios that could cause issues in concurrent environments"""
def test_loan_return_loan_sequence(self, db_conn):
"""
Test the sequence: loan -> return -> loan again
This simulates a device being loaned multiple times over its lifetime
"""
cursor = db_conn.cursor()
# Add tablet and user
cursor.execute('''
INSERT INTO tablets (brand, model, serial_number, status)
VALUES (?, ?, ?, 'available')
''', ('Brand', 'Model', 'SEQUENCE_TEST'))
db_conn.commit()
cursor.execute("SELECT id FROM tablets WHERE serial_number = 'SEQUENCE_TEST'")
tablet = cursor.fetchone()
cursor.execute('''
INSERT INTO users (name, identification)
VALUES (?, ?)
''', ('User', 'SEQUENCE_USER'))
db_conn.commit()
cursor.execute("SELECT id FROM users WHERE identification = 'SEQUENCE_USER'")
user = cursor.fetchone()
# First loan
loan_date1 = datetime.now().strftime('%Y-%m-%d %H:%M:%S')
cursor.execute("UPDATE tablets SET status = 'loaned' WHERE id = ?", (tablet['id'],))
cursor.execute('''
INSERT INTO loans (tablet_id, user_id, loan_date, status)
VALUES (?, ?, ?, 'active')
''', (tablet['id'], user['id'], loan_date1))
db_conn.commit()
# Return
return_date = datetime.now().strftime('%Y-%m-%d %H:%M:%S')
cursor.execute("SELECT id FROM loans WHERE tablet_id = ?", (tablet['id'],))
loan1 = cursor.fetchone()
cursor.execute('''
UPDATE loans SET status = 'returned', return_date = ? WHERE id = ?
''', (return_date, loan1['id']))
cursor.execute("UPDATE tablets SET status = 'available' WHERE id = ?", (tablet['id'],))
db_conn.commit()
# Second loan
loan_date2 = datetime.now().strftime('%Y-%m-%d %H:%M:%S')
cursor.execute("UPDATE tablets SET status = 'loaned' WHERE id = ?", (tablet['id'],))
cursor.execute('''
INSERT INTO loans (tablet_id, user_id, loan_date, status)
VALUES (?, ?, ?, 'active')
''', (tablet['id'], user['id'], loan_date2))
db_conn.commit()
# Verify we have 2 loans for this tablet
cursor.execute("SELECT COUNT(*) FROM loans WHERE tablet_id = ?", (tablet['id'],))
loan_count = cursor.fetchone()[0]
assert loan_count == 2
# Verify 1 active, 1 returned
cursor.execute("SELECT COUNT(*) FROM loans WHERE tablet_id = ? AND status = 'active'", (tablet['id'],))
active_count = cursor.fetchone()[0]
cursor.execute("SELECT COUNT(*) FROM loans WHERE tablet_id = ? AND status = 'returned'", (tablet['id'],))
returned_count = cursor.fetchone()[0]
assert active_count == 1
assert returned_count == 1
def test_multiple_users_multiple_tablets(self, db_conn):
"""
Test complex scenario with multiple users and tablets
Ensures the many-to-many relationship works correctly
"""
cursor = db_conn.cursor()
# Add 3 users
users = []
for i in range(3):
cursor.execute('''
INSERT INTO users (name, identification)
VALUES (?, ?)
''', (f'User{i}', f'MULTI_USER_{i}'))
db_conn.commit()
cursor.execute("SELECT id FROM users WHERE identification = ?", (f'MULTI_USER_{i}',))
users.append(cursor.fetchone())
# Add 5 tablets
tablets = []
for i in range(5):
cursor.execute('''
INSERT INTO tablets (brand, model, serial_number, status)
VALUES (?, ?, ?, 'available')
''', (f'Brand{i}', f'Model{i}', f'MULTI_TABLET_{i}'))
db_conn.commit()
cursor.execute("SELECT id FROM tablets WHERE serial_number = ?", (f'MULTI_TABLET_{i}',))
tablets.append(cursor.fetchone())
# Loan tablets to users in a pattern
# User 0: tablets 0, 1
# User 1: tablets 2, 3
# User 2: tablet 4
loan_date = datetime.now().strftime('%Y-%m-%d %H:%M:%S')
loans_map = {
users[0]['id']: [tablets[0]['id'], tablets[1]['id']],
users[1]['id']: [tablets[2]['id'], tablets[3]['id']],
users[2]['id']: [tablets[4]['id']],
}
for user_id, tablet_ids in loans_map.items():
for tablet_id in tablet_ids:
cursor.execute("UPDATE tablets SET status = 'loaned' WHERE id = ?", (tablet_id,))
cursor.execute('''
INSERT INTO loans (tablet_id, user_id, loan_date, status)
VALUES (?, ?, ?, 'active')
''', (tablet_id, user_id, loan_date))
db_conn.commit()
# Verify counts
cursor.execute("SELECT COUNT(*) FROM loans")
total_loans = cursor.fetchone()[0]
assert total_loans == 5 # 2 + 2 + 1
# Verify each user has correct number of loans
for user_id, expected_tablet_ids in loans_map.items():
cursor.execute("SELECT COUNT(*) FROM loans WHERE user_id = ?", (user_id,))
count = cursor.fetchone()[0]
assert count == len(expected_tablet_ids)
# Verify all loaned tablets have correct status
cursor.execute("SELECT COUNT(*) FROM tablets WHERE status = 'loaned'")
loaned_count = cursor.fetchone()[0]
assert loaned_count == 5

365
tests/test_minimal_app.py Normal file
View file

@ -0,0 +1,365 @@
"""
Unit tests for the actual application functions from minimal_app.py
Tests the real business logic with proper imports
"""
import pytest
import sqlite3
import sys
import os
from datetime import datetime
# Add project root to path
sys.path.insert(0, os.path.dirname(os.path.dirname(os.path.abspath(__file__))))
# Import functions from minimal_app
from minimal_app import (
add_tablet, add_user, loan_tablet, return_tablet,
show_available_tablets, show_active_loans, show_loan_history,
add_non_loanable_device, show_non_loanable_devices, delete_non_loanable_device
)
@pytest.fixture
def test_db_path():
"""Path to test database"""
return 'test_minimal_app.db'
@pytest.fixture
def init_test_db(test_db_path):
"""Initialize a fresh test database with schema (same as minimal_app)"""
# Remove existing test database if it exists
if os.path.exists(test_db_path):
os.remove(test_db_path)
# Use the same init_db function from minimal_app
from minimal_app import init_db
# Temporarily rename the database
original_db = 'tablets.db'
if os.path.exists(original_db):
os.rename(original_db, f'{original_db}.backup')
try:
# Create test database
os.environ['TEST_DB'] = test_db_path
init_db()
yield test_db_path
finally:
# Cleanup
if os.path.exists(test_db_path):
os.remove(test_db_path)
if os.path.exists(f'{original_db}.backup'):
os.rename(f'{original_db}.backup', original_db)
if 'TEST_DB' in os.environ:
del os.environ['TEST_DB']
@pytest.fixture
def clean_db():
"""Fixture that ensures we have a clean database for each test"""
# This is simpler - just create a temp database for each test
import tempfile
import shutil
# Create temp directory for database
temp_dir = tempfile.mkdtemp()
db_path = os.path.join(temp_dir, 'test.db')
# Initialize database
conn = sqlite3.connect(db_path)
cursor = conn.cursor()
cursor.execute('''
CREATE TABLE IF NOT EXISTS tablets (
id INTEGER PRIMARY KEY AUTOINCREMENT,
brand TEXT NOT NULL,
model TEXT NOT NULL,
serial_number TEXT UNIQUE NOT NULL,
status TEXT DEFAULT 'available'
)
''')
cursor.execute('''
CREATE TABLE IF NOT EXISTS users (
id INTEGER PRIMARY KEY AUTOINCREMENT,
name TEXT NOT NULL,
identification TEXT UNIQUE
)
''')
cursor.execute('''
CREATE TABLE IF NOT EXISTS loans (
id INTEGER PRIMARY KEY AUTOINCREMENT,
tablet_id INTEGER NOT NULL,
user_id INTEGER NOT NULL,
loan_date TEXT NOT NULL,
return_date TEXT,
status TEXT DEFAULT 'active',
FOREIGN KEY (tablet_id) REFERENCES tablets (id),
FOREIGN KEY (user_id) REFERENCES users (id)
)
''')
cursor.execute('''
CREATE TABLE IF NOT EXISTS non_loanable_devices (
id INTEGER PRIMARY KEY AUTOINCREMENT,
brand TEXT NOT NULL,
model TEXT NOT NULL,
serial_number TEXT UNIQUE NOT NULL,
device_type TEXT NOT NULL,
location TEXT,
status TEXT DEFAULT 'available',
notes TEXT,
purchase_date TEXT,
purchase_cost REAL
)
''')
conn.commit()
conn.close()
# Temporarily replace the database
original_db = 'tablets.db'
backup_path = f'{original_db}.test_backup'
# Backup original if exists
if os.path.exists(original_db):
if os.path.exists(backup_path):
os.remove(backup_path)
os.rename(original_db, backup_path)
# Copy temp db to tablets.db location
shutil.copy(db_path, original_db)
yield original_db
# Cleanup
if os.path.exists(original_db):
os.remove(original_db)
if os.path.exists(backup_path):
os.rename(backup_path, original_db)
shutil.rmtree(temp_dir, ignore_errors=True)
class TestMinimalAppFunctions:
"""Test the actual functions from minimal_app.py"""
def test_add_tablet_function(self, clean_db):
"""Test the add_tablet function"""
add_tablet('TestBrand', 'TestModel', 'TEST_SN_001')
# Verify it was added
conn = sqlite3.connect(clean_db)
cursor = conn.cursor()
cursor.execute("SELECT * FROM tablets WHERE serial_number = 'TEST_SN_001'")
tablet = cursor.fetchone()
conn.close()
assert tablet is not None
assert tablet[1] == 'TestBrand' # brand is index 1
assert tablet[2] == 'TestModel' # model is index 2
assert tablet[3] == 'TEST_SN_001' # serial_number is index 3
def test_add_tablet_duplicate(self, clean_db, capsys):
"""Test that duplicate serial numbers are rejected"""
add_tablet('Brand1', 'Model1', 'DUP_SN')
add_tablet('Brand2', 'Model2', 'DUP_SN')
captured = capsys.readouterr()
assert 'already exists' in captured.out
def test_add_user_function(self, clean_db):
"""Test the add_user function"""
add_user('Test User', 'TEST_ID_001')
conn = sqlite3.connect(clean_db)
cursor = conn.cursor()
cursor.execute("SELECT * FROM users WHERE identification = 'TEST_ID_001'")
user = cursor.fetchone()
conn.close()
assert user is not None
assert user[1] == 'Test User' # name is index 1
assert user[2] == 'TEST_ID_001' # identification is index 2
def test_add_user_duplicate(self, clean_db, capsys):
"""Test that duplicate user identifications are rejected"""
add_user('User1', 'DUP_ID')
add_user('User2', 'DUP_ID')
captured = capsys.readouterr()
assert 'already exists' in captured.out
def test_loan_tablet_function(self, clean_db):
"""Test the loan_tablet function"""
# Add tablet and user
add_tablet('LoanBrand', 'LoanModel', 'LOAN_SN_001')
add_user('LoanUser', 'LOAN_ID_001')
conn = sqlite3.connect(clean_db)
cursor = conn.cursor()
cursor.execute("SELECT id FROM tablets WHERE serial_number = 'LOAN_SN_001'")
tablet_id = cursor.fetchone()[0]
cursor.execute("SELECT id FROM users WHERE identification = 'LOAN_ID_001'")
user_id = cursor.fetchone()[0]
conn.close()
# Loan the tablet
loan_tablet(tablet_id, user_id)
# Verify loan was created and tablet status changed
conn = sqlite3.connect(clean_db)
cursor = conn.cursor()
cursor.execute("SELECT status FROM tablets WHERE id = ?", (tablet_id,))
status = cursor.fetchone()[0]
assert status == 'loaned'
cursor.execute("SELECT * FROM loans WHERE tablet_id = ? AND user_id = ?", (tablet_id, user_id))
loan = cursor.fetchone()
assert loan is not None
assert loan[5] == 'active' # status is index 5
conn.close()
def test_loan_already_loaned_tablet(self, clean_db, capsys):
"""Test loaning a tablet that's already loaned"""
add_tablet('Brand', 'Model', 'ALREADY_LOANED_SN')
add_user('User1', 'USER1_ID')
add_user('User2', 'USER2_ID')
conn = sqlite3.connect(clean_db)
cursor = conn.cursor()
cursor.execute("SELECT id FROM tablets WHERE serial_number = 'ALREADY_LOANED_SN'")
tablet_id = cursor.fetchone()[0]
cursor.execute("SELECT id FROM users WHERE identification = 'USER1_ID'")
user1_id = cursor.fetchone()[0]
cursor.execute("SELECT id FROM users WHERE identification = 'USER2_ID'")
user2_id = cursor.fetchone()[0]
conn.close()
# Loan to first user
loan_tablet(tablet_id, user1_id)
# Try to loan to second user - should fail
loan_tablet(tablet_id, user2_id)
captured = capsys.readouterr()
assert 'not available' in captured.out
def test_return_tablet_function(self, clean_db):
"""Test the return_tablet function"""
add_tablet('ReturnBrand', 'ReturnModel', 'RETURN_SN_001')
add_user('ReturnUser', 'RETURN_ID_001')
conn = sqlite3.connect(clean_db)
cursor = conn.cursor()
cursor.execute("SELECT id FROM tablets WHERE serial_number = 'RETURN_SN_001'")
tablet_id = cursor.fetchone()[0]
cursor.execute("SELECT id FROM users WHERE identification = 'RETURN_ID_001'")
user_id = cursor.fetchone()[0]
conn.close()
# Loan the tablet
loan_tablet(tablet_id, user_id)
# Get loan ID
conn = sqlite3.connect(clean_db)
cursor = conn.cursor()
cursor.execute("SELECT id FROM loans WHERE tablet_id = ?", (tablet_id,))
loan_id = cursor.fetchone()[0]
conn.close()
# Return the tablet
return_tablet(loan_id)
# Verify return
conn = sqlite3.connect(clean_db)
cursor = conn.cursor()
cursor.execute("SELECT status FROM tablets WHERE id = ?", (tablet_id,))
status = cursor.fetchone()[0]
assert status == 'available'
cursor.execute("SELECT status, return_date FROM loans WHERE id = ?", (loan_id,))
loan = cursor.fetchone()
assert loan[0] == 'returned'
assert loan[1] is not None # return_date should be set
conn.close()
def test_return_nonexistent_loan(self, clean_db, capsys):
"""Test returning a loan that doesn't exist"""
return_tablet(99999)
captured = capsys.readouterr()
assert 'not found' in captured.out or 'Loan' in captured.out
def test_show_available_tablets(self, clean_db, capsys):
"""Test showing available tablets"""
add_tablet('Avail1', 'Model1', 'AVAIL_SN_001')
add_tablet('Avail2', 'Model2', 'AVAIL_SN_002')
show_available_tablets()
captured = capsys.readouterr()
assert 'Available Tablets' in captured.out
assert 'AVAIL_SN_001' in captured.out
assert 'AVAIL_SN_002' in captured.out
def test_show_active_loans(self, clean_db, capsys):
"""Test showing active loans"""
add_tablet('LoanBrand', 'LoanModel', 'ACTIVE_LOAN_SN')
add_user('LoanUser', 'ACTIVE_LOAN_ID')
conn = sqlite3.connect(clean_db)
cursor = conn.cursor()
cursor.execute("SELECT id FROM tablets WHERE serial_number = 'ACTIVE_LOAN_SN'")
tablet_id = cursor.fetchone()[0]
cursor.execute("SELECT id FROM users WHERE identification = 'ACTIVE_LOAN_ID'")
user_id = cursor.fetchone()[0]
conn.close()
loan_tablet(tablet_id, user_id)
show_active_loans()
captured = capsys.readouterr()
assert 'Active Loans' in captured.out
assert 'ACTIVE_LOAN_SN' in captured.out
def test_non_loanable_device_crud(self, clean_db, capsys):
"""Test CRUD operations for non-loanable devices"""
# Add
add_non_loanable_device('Projector', 'P100', 'PROJ_001', 'projector', 'Room A')
captured = capsys.readouterr()
assert 'Added non-loanable device' in captured.out
# Show
show_non_loanable_devices()
captured = capsys.readouterr()
assert 'PROJ_001' in captured.out
# Get ID for delete
conn = sqlite3.connect(clean_db)
cursor = conn.cursor()
cursor.execute("SELECT id FROM non_loanable_devices WHERE serial_number = 'PROJ_001'")
device_id = cursor.fetchone()[0]
conn.close()
# Delete
delete_non_loanable_device(device_id)
captured = capsys.readouterr()
assert 'deleted' in captured.out
# Verify deletion
conn = sqlite3.connect(clean_db)
cursor = conn.cursor()
cursor.execute("SELECT * FROM non_loanable_devices WHERE serial_number = 'PROJ_001'")
device = cursor.fetchone()
conn.close()
assert device is None