Phase 1 \u2014 \ubbf8\ub9ac \uc5f0\ucc28 \ub4f1\ub85d
- DB: get_leave_minutes_for(date) / has_full_day_leave(date) /
get_leave_records_by_date(date) / get_leave_records_by_range(start, end)
- TimeCalculator.effective_work_minutes(date_obj, db): \uc5f0\ucc28 \ubd84\ub9cc\ud07c \uc815\uaddc \uadfc\ubb34 \ucc28\uac10
- update_display() 1Hz hot-path:
\u2022 \uc885\uc77c \uc5f0\ucc28 \ub4f1\ub85d\uc77c + \ucd9c\uadfc \uc548 \ud55c \uc0c1\ud0dc \u2192 "\ud83c\udf34 \uc624\ub298\uc740 \ud734\uac00" \uce74\ub4dc \ud45c\uc2dc, \uce74\uc6b4\ud2b8\ub2e4\uc6b4 \uc81c\uac70
\u2022 \uc885\uc77c \uc5f0\ucc28 + \ucd9c\uadfc override \u2192 \ud734\uc77c\ucc98\ub7fc \uc804\uccb4 \uc801\ub9bd
\u2022 \ubd80\ubd84 \uc5f0\ucc28(\ubc18\ucc28/\uc2dc\uac04) \u2192 leave_used_today \uacbd\ub85c\ub85c \uae30\uc874 \ub2e8\ucd95 \uacc4\uc0b0 \uc720\uc9c0
- \uc790\ub3d9 \ucd9c\uadfc\uac10\uc9c0 \uac00\ub4dc: load_today_data\uc5d0\uc11c \uc885\uc77c \uc5f0\ucc28\uc77c\uc774\uba74 event_monitor \ud638\ucd9c \uc790\uccb4 \uc2a4\ud0b5
- \uc218\ub3d9 \ucd9c\uadfc \uac00\ub4dc: manual_clock_in\uc5d0\uc11c \uc885\uc77c \uc5f0\ucc28\uc77c \ud655\uc778 \ud504\ub86c\ud504\ud2b8
- AddLeaveDialog \uac80\uc99d \uac15\ud654:
\u2022 \ubbf8\ub798 1\ub144\uae4c\uc9c0 setMaximumDate
\u2022 \uc8fc\ub9d0/\uacf5\ud734\uc77c \ub4f1\ub85d \ucc28\ub2e8 (\uc774\ubbf8 \ube44\uadfc\ubb34\uc77c)
\u2022 \uac19\uc740 \ub0a0 1\uc77c \ucd08\uacfc \ub204\uc801 \ucc28\ub2e8
- leave_calendar_view: \uc608\uc815(\ud30c\ub791) / \uc0ac\uc6a9\uc644\ub8cc(\ub179/\ub178/\ubcf4) \uc0c9\uc0c1 \ubd84\ub9ac
Phase 2 \u2014 \ud1b5\ud569 \uc2a4\ucf00\uc904 + \ubc18\ubcf5 \uc5f0\ucc28
- recurring_leaves \ud14c\uc774\ube14 (pattern/leave_type/days/start/end/memo)
- core/recurring_leaves.py: weekly / biweekly / monthly \ud328\ud134 \ud30c\uc11c + expand_for_range/date
- get_leave_minutes_for() / has_full_day_leave()\uac00 \ubc18\ubcf5 \ud328\ud134\ub3c4 \ud568\uaed8 \ud569\uc0b0
- ui/recurring_leave_dialog.py: \ub9e4\uc8fc/\uaca9\uc8fc/\ub9e4\uc6d4 \uc785\ub825 + \uc785\ub825 \ub9ac\uc2a4\ud2b8 \uad00\ub9ac
- ui/schedule_view.py: \uc6d4\uac04 \uc2a4\ud50c\ub9ac\ud130 \ub808\uc774\uc544\uc6c3 (\uce98\ub9b0\ub354 + \uc0c1\uc138)
\u2022 \ud734\uc77c(\ube68\uac15) / \uc5f0\ucc28 \uc0ac\uc6a9(\ub179\u30fb\ub178\u30fb\ubcf4) / \uc608\uc815(\ud30c\ub791) / \ubc18\ubcf5(\ud68c\uc0c9) \uc0c9 \ucf54\ub4dc
\u2022 \ub0a0\uc9dc \ud074\ub9ad \u2192 \uc0c1\uc138 \ud328\ub110 (\ub3d9\uc77c\uc77c\uc790 \uad6c\uccb4 \uc5f0\ucc28 + \ubc18\ubcf5 \ub9e4\uce58)
\u2022 \ub9ac\uc2a4\ud2b8 \uc6b0\ud074\ub9ad \uc0ad\uc81c (\uad6c\uccb4 / \ubc18\ubcf5 \uad6c\ubd84)
\u2022 \uc6d4 \ubcc0\uacbd \uc2dc \uc790\ub3d9 reload
- \uc9c4\uc785\uc810: main_window.show_schedule(), tray menu '\ud83d\uddd3\ufe0f \uc2a4\ucf00\uc904', LeaveView '\ud83d\uddd3\ufe0f \uc2a4\ucf00\uc904' \ubc84\ud2bc
Tests
- tests/test_recurring_leaves.py 32\uac1c (\ud328\ud134 \ud30c\uc2f1 / \ub9e4\uce6d / expand / describe)
- tests/test_database.py +12 (TestLeaveQueriesByDate + TestRecurringLeavesDB)
- _integration_test.py +4 (S52B-S52E)
- pytest: 122 \u2192 175 \uc804\ubd80 green
- \ud1b5\ud569: 49 \u2192 53 \uc804\ubd80 green
- UI-5/UI-7 \uae30\uc874 \uace0\uc7a5 (v2.8.0 \ub514\uc790\uc778 \ub9ac\ub274\uc5bc \ub9c8\ub108)
205 lines
8.0 KiB
Python
205 lines
8.0 KiB
Python
"""
|
|
Database 단위 테스트 — 마이그레이션, 동기화, 헬퍼.
|
|
"""
|
|
import os
|
|
import sys
|
|
import tempfile
|
|
from datetime import date
|
|
|
|
import pytest
|
|
|
|
sys.path.insert(0, os.path.dirname(os.path.dirname(os.path.abspath(__file__))))
|
|
|
|
from core.database import Database
|
|
|
|
|
|
@pytest.fixture
|
|
def fresh_db(tmp_path):
|
|
"""매 테스트마다 빈 DB."""
|
|
return Database(str(tmp_path / "test.db"))
|
|
|
|
|
|
class TestSettingsHelpers:
|
|
def test_get_setting_int_valid(self, fresh_db):
|
|
fresh_db.set_setting('foo', '42')
|
|
assert fresh_db.get_setting_int('foo') == 42
|
|
|
|
def test_get_setting_int_invalid_returns_default(self, fresh_db):
|
|
fresh_db.set_setting('foo', 'abc')
|
|
assert fresh_db.get_setting_int('foo', 99) == 99
|
|
|
|
def test_get_setting_int_missing_returns_default(self, fresh_db):
|
|
assert fresh_db.get_setting_int('missing', 7) == 7
|
|
|
|
def test_get_setting_bool_truthy(self, fresh_db):
|
|
fresh_db.set_setting('flag', 'true')
|
|
assert fresh_db.get_setting_bool('flag') is True
|
|
|
|
def test_get_setting_bool_falsy(self, fresh_db):
|
|
fresh_db.set_setting('flag', 'no')
|
|
assert fresh_db.get_setting_bool('flag') is False
|
|
|
|
def test_get_setting_float(self, fresh_db):
|
|
fresh_db.set_setting('rate', '3.14')
|
|
assert fresh_db.get_setting_float('rate') == 3.14
|
|
|
|
|
|
class TestSettingsAutoSync:
|
|
def test_work_minutes_to_work_hours_floor(self, fresh_db):
|
|
"""work_minutes 저장 시 work_hours는 floor 동기화 (450 → 7)"""
|
|
fresh_db.save_settings({'work_minutes': 450})
|
|
assert fresh_db.get_setting('work_hours') == '7'
|
|
|
|
def test_work_hours_to_work_minutes(self, fresh_db):
|
|
fresh_db.save_settings({'work_hours': 8})
|
|
assert fresh_db.get_setting('work_minutes') == '480'
|
|
|
|
def test_annual_leave_bidirectional(self, fresh_db):
|
|
fresh_db.save_settings({'annual_leave_days': 12})
|
|
assert fresh_db.get_setting('annual_leave_total') == '12'
|
|
|
|
|
|
class TestWorkMinutes:
|
|
def test_get_work_minutes_default(self, fresh_db):
|
|
assert fresh_db.get_work_minutes() == 480
|
|
|
|
def test_get_work_minutes_after_save(self, fresh_db):
|
|
fresh_db.save_settings({'work_minutes': 450})
|
|
assert fresh_db.get_work_minutes() == 450
|
|
|
|
|
|
class TestLeaveCalculation:
|
|
def test_leave_minutes_for_short_worker(self, fresh_db):
|
|
"""단축근무자(7h30m) 1일 연차 = 450분"""
|
|
fresh_db.save_settings({'work_minutes': 450})
|
|
today = date.today().isoformat()
|
|
fresh_db.add_leave_record(today, 'annual', 1.0)
|
|
assert fresh_db.get_today_leave_minutes() == 450
|
|
|
|
def test_half_day_leave(self, fresh_db):
|
|
today = date.today().isoformat()
|
|
fresh_db.add_leave_record(today, 'half', 0.5)
|
|
assert fresh_db.get_today_leave_minutes() == 240 # 8h * 0.5
|
|
|
|
|
|
class TestMigrationIdempotency:
|
|
def test_annual_leave_keys_migrated_sentinel(self, fresh_db):
|
|
assert fresh_db.get_setting('annual_leave_keys_migrated') == 'true'
|
|
|
|
def test_re_init_does_not_break(self, tmp_path):
|
|
path = str(tmp_path / "test.db")
|
|
db1 = Database(path)
|
|
db1.save_settings({'work_minutes': 450})
|
|
# 두 번째 init
|
|
db2 = Database(path)
|
|
assert db2.get_work_minutes() == 450
|
|
|
|
|
|
class TestConsecutiveOvertimeDays:
|
|
def test_no_records(self, fresh_db):
|
|
assert fresh_db.get_consecutive_overtime_days() == 0
|
|
|
|
def test_three_consecutive(self, fresh_db):
|
|
from datetime import date, timedelta
|
|
today = date.today()
|
|
for i in range(3):
|
|
d = (today - timedelta(days=i)).isoformat()
|
|
fresh_db.add_work_record(d, '09:00:00')
|
|
fresh_db.update_clock_out(d, '20:00:00', total_hours=11.0,
|
|
overtime_minutes=120, overtime_earned=120)
|
|
assert fresh_db.get_consecutive_overtime_days() == 3
|
|
|
|
|
|
class TestLeaveQueriesByDate:
|
|
def test_get_leave_minutes_for_no_records(self, fresh_db):
|
|
assert fresh_db.get_leave_minutes_for('2026-05-01') == 0
|
|
|
|
def test_full_day_leave_detected(self, fresh_db):
|
|
fresh_db.add_leave_record('2026-05-15', '연차', 1.0, '여행')
|
|
assert fresh_db.has_full_day_leave('2026-05-15')
|
|
assert fresh_db.get_leave_minutes_for('2026-05-15') == 480
|
|
|
|
def test_half_day_not_full(self, fresh_db):
|
|
fresh_db.add_leave_record('2026-05-15', '반차', 0.5)
|
|
assert not fresh_db.has_full_day_leave('2026-05-15')
|
|
assert fresh_db.get_leave_minutes_for('2026-05-15') == 240
|
|
|
|
def test_two_halves_become_full(self, fresh_db):
|
|
fresh_db.add_leave_record('2026-05-15', '오전반차', 0.5)
|
|
fresh_db.add_leave_record('2026-05-15', '오후반차', 0.5)
|
|
assert fresh_db.has_full_day_leave('2026-05-15')
|
|
assert fresh_db.get_leave_minutes_for('2026-05-15') == 480
|
|
|
|
def test_records_by_date(self, fresh_db):
|
|
fresh_db.add_leave_record('2026-05-15', '연차', 1.0, '메모')
|
|
recs = fresh_db.get_leave_records_by_date('2026-05-15')
|
|
assert len(recs) == 1
|
|
assert recs[0]['leave_type'] == '연차'
|
|
assert recs[0]['memo'] == '메모'
|
|
|
|
def test_records_by_range(self, fresh_db):
|
|
fresh_db.add_leave_record('2026-05-01', '연차', 1.0)
|
|
fresh_db.add_leave_record('2026-05-10', '반차', 0.5)
|
|
fresh_db.add_leave_record('2026-06-01', '연차', 1.0)
|
|
recs = fresh_db.get_leave_records_by_range('2026-05-01', '2026-05-31')
|
|
assert len(recs) == 2
|
|
# 날짜 정렬
|
|
assert recs[0]['date'] == '2026-05-01'
|
|
assert recs[1]['date'] == '2026-05-10'
|
|
|
|
|
|
class TestRecurringLeavesDB:
|
|
def test_add_and_list(self, fresh_db):
|
|
rid = fresh_db.add_recurring_leave(
|
|
'weekly:friday', '반차', 0.5, '2026-05-01', '2026-12-31', '단축'
|
|
)
|
|
assert rid > 0
|
|
recs = fresh_db.get_recurring_leaves()
|
|
assert len(recs) == 1
|
|
assert recs[0]['pattern'] == 'weekly:friday'
|
|
assert recs[0]['memo'] == '단축'
|
|
|
|
def test_active_on_filter(self, fresh_db):
|
|
# 종료일이 지난 패턴
|
|
fresh_db.add_recurring_leave('weekly:fri', '반차', 0.5,
|
|
'2025-01-01', '2025-12-31')
|
|
# 아직 시작 안 한 패턴
|
|
fresh_db.add_recurring_leave('weekly:mon', '반차', 0.5,
|
|
'2027-01-01', None)
|
|
# 현재 활성 패턴
|
|
fresh_db.add_recurring_leave('monthly:15', '연차', 1.0,
|
|
'2026-01-01', None)
|
|
active = fresh_db.get_recurring_leaves(active_on='2026-05-15')
|
|
assert len(active) == 1
|
|
assert active[0]['pattern'] == 'monthly:15'
|
|
|
|
def test_delete(self, fresh_db):
|
|
rid = fresh_db.add_recurring_leave('weekly:fri', '반차', 0.5,
|
|
'2026-01-01')
|
|
assert len(fresh_db.get_recurring_leaves()) == 1
|
|
fresh_db.delete_recurring_leave(rid)
|
|
assert fresh_db.get_recurring_leaves() == []
|
|
|
|
def test_recurring_contributes_to_leave_minutes(self, fresh_db):
|
|
# 매주 금요일 반차
|
|
fresh_db.add_recurring_leave('weekly:friday', '반차', 0.5,
|
|
'2026-01-01')
|
|
# 2026-05-01 = Friday → 240분
|
|
assert fresh_db.get_leave_minutes_for('2026-05-01') == 240
|
|
# 2026-05-04 = Monday → 0분
|
|
assert fresh_db.get_leave_minutes_for('2026-05-04') == 0
|
|
|
|
def test_concrete_plus_recurring_sum(self, fresh_db):
|
|
# 매주 금요일 반차 + 그날 별도 반반차 추가
|
|
fresh_db.add_recurring_leave('weekly:friday', '반차', 0.5, '2026-01-01')
|
|
fresh_db.add_leave_record('2026-05-01', '반반차', 0.25)
|
|
# 0.5 + 0.25 = 0.75일 = 360분
|
|
assert fresh_db.get_leave_minutes_for('2026-05-01') == 360
|
|
assert not fresh_db.has_full_day_leave('2026-05-01')
|
|
|
|
def test_concrete_plus_recurring_full_day(self, fresh_db):
|
|
fresh_db.add_recurring_leave('weekly:friday', '반차', 0.5, '2026-01-01')
|
|
fresh_db.add_leave_record('2026-05-01', '오후반차', 0.5)
|
|
# 0.5 + 0.5 = 1.0일
|
|
assert fresh_db.has_full_day_leave('2026-05-01')
|