test: improve test coverage to 93% (Issue #16)

.github/workflows/ci.yml

@@ -32,7 +32,7 @@ jobs:
 
       - name: Run tests
         run: |
-          pytest --tb=short -v --cov=email_processor --cov-report=term-missing --cov-report=xml --cov-fail-under=70
+          pytest --tb=short -v --cov=email_processor --cov-report=term-missing --cov-report=xml --cov-fail-under=95
 
       - name: Upload coverage to Codecov
         uses: codecov/codecov-action@v4

.pre-commit-config.yaml

@@ -25,6 +25,15 @@ repos:
         args: [--fix, --exit-non-zero-on-fix]
       - id: ruff-format
 
+  - repo: local
+    hooks:
+      - id: ruff-check
+        name: ruff check
+        entry: python -m ruff check .
+        language: system
+        pass_filenames: false
+        always_run: true
+
   - repo: https://github.com/pre-commit/mirrors-mypy
     rev: v1.8.0
     hooks:

CONTRIBUTING.md

@@ -58,21 +58,23 @@ MyPy is configured via `.mypy.ini` and `pyproject.toml`.
 mypy email_processor
 ```
 
-### 3. Pre-commit Hooks (Optional)
+### 3. Pre-commit Hooks (Required)
 
-Pre-commit hooks automatically run code quality checks before each commit.
+Pre-commit hooks automatically run code quality checks before each commit. **It is required to run pre-commit checks before committing and pushing.**
 
 ```bash
 # Install pre-commit
 pip install pre-commit
 
-# Install git hooks
+# Install git hooks (optional - hooks will run automatically on commit)
 pre-commit install
 
-# Run hooks manually
+# Run hooks manually (REQUIRED before commit and push)
 pre-commit run --all-files
 ```
 
+**Note:** Even if hooks are installed, always run `pre-commit run --all-files` manually before committing to ensure all checks pass.
+
 ### 4. Running Tests
 
 ```bash
@@ -83,33 +85,74 @@ pytest
 pytest --cov=email_processor --cov-report=html
 
 # Run specific test file
-pytest tests/unit/processor/test_email_processor.py
+pytest tests/unit/imap/test_fetcher_header.py
+
+# Run all fetcher tests
+pytest tests/unit/imap/test_fetcher_*.py
+
+# Run specific test category
+pytest tests/unit/imap/test_fetcher_attachment.py
 ```
 
 ### 5. Code Quality Workflow
 
-Before committing:
+**⚠️ IMPORTANT: Before committing and pushing, always run these checks:**
 
-1. **Format code**:
+1. **Check formatting** (required):
+   ```bash
+   ruff format --check .
+   ```
+   If files need formatting, run:
    ```bash
    ruff format .
    ```
 
-2. **Lint code**:
+2. **Run pre-commit checks** (required):
+   ```bash
+   pre-commit run --all-files
+   ```
+   This will run all hooks including:
+   - Trailing whitespace check
+   - End of file fixer
+   - Ruff linting and formatting
+   - Ruff check (full project check)
+   - MyPy type checking
+
+3. **Lint code** (if pre-commit didn't fix everything):
    ```bash
    ruff check --fix .
    ```
 
-3. **Type check** (optional):
+4. **Type check** (optional, also runs in pre-commit):
    ```bash
    mypy email_processor
    ```
 
-4. **Run tests**:
+5. **Run tests**:
    ```bash
    pytest
    ```
 
+**Recommended workflow:**
+```bash
+# 1. Check formatting first
+ruff format --check .
+
+# 2. If needed, format code
+ruff format .
+
+# 3. Run all pre-commit checks
+pre-commit run --all-files
+
+# 4. Run tests
+pytest
+
+# 5. Only then commit and push
+git add .
+git commit -m "your message"
+git push
+```
+
 ### 6. IDE Integration
 
 #### VS Code

PLAN.md

@@ -179,11 +179,11 @@ git push origin feature/issue-3-add-project-documentation
 ### Задачи:
 - [ ] Добавить бейджи в начало README.md
   - PyPI version badge: `[![PyPI](https://img.shields.io/pypi/v/email-processor)](https://pypi.org/project/email-processor/)`
-  - CI status badge: `[![CI](https://github.com/vkholodilin/python-email-automation-processor/actions/workflows/ci.yml/badge.svg)](https://github.com/vkholodilin/python-email-automation-processor/actions/workflows/ci.yml)`
-  - Test Coverage badge: `[![Test Coverage](https://codecov.io/gh/vkholodilin/python-email-automation-processor/branch/main/graph/badge.svg)](https://codecov.io/gh/vkholodilin/python-email-automation-processor)`
+  - CI status badge: `[![CI](https://github.com/KHolodilin/python-email-automation-processor/actions/workflows/ci.yml/badge.svg)](https://github.com/KHolodilin/python-email-automation-processor/actions/workflows/ci.yml)`
+  - Test Coverage badge: `[![Test Coverage](https://codecov.io/gh/KHolodilin/python-email-automation-processor/branch/main/graph/badge.svg)](https://codecov.io/gh/KHolodilin/python-email-automation-processor)`
   - Python version badge: `[![Python Version](https://img.shields.io/badge/python-3.9%2B-blue.svg)](https://www.python.org/downloads/)`
-  - License badge: `[![License](https://img.shields.io/github/license/vkholodilin/python-email-automation-processor)](LICENSE)`
-  - Stars badge: `[![Stars](https://img.shields.io/github/stars/vkholodilin/python-email-automation-processor)](https://github.com/vkholodilin/python-email-automation-processor/stargazers)`
+  - License badge: `[![License](https://img.shields.io/github/license/KHolodilin/python-email-automation-processor)](LICENSE)`
+  - Stars badge: `[![Stars](https://img.shields.io/github/stars/KHolodilin/python-email-automation-processor)](https://github.com/KHolodilin/python-email-automation-processor/stargazers)`
   - Code style badge (Ruff): `[![Code style: Ruff](https://img.shields.io/endpoint?url=https://raw.githubusercontent.com/astral-sh/ruff/main/assets/badge/v2.json)](https://github.com/astral-sh/ruff)`
 - [ ] Проверить корректность ссылок
   - Убедиться, что все ссылки указывают на правильные репозитории