-
- {% if not error %}Кандидаты в диссертационный совет
-Сотрудники с докторской степенью в профиле и публикациями с отметками Q1 или Q2 в имеющейся выгрузке. Список помогает начать проверку состава комиссии.
+Диссертационный совет
+Публикации сотрудников для проверки состава совета. В списке все сотрудники с хотя бы одной публикацией в файле CSV или базе miem-workers, независимо от степени и количества работ.
{{ total }} предварительно подходят
{% endif %}
-
- 10 и болееразличных публикаций
-{{ start.strftime('%d.%m.%Y') }} — {{ end.strftime('%d.%m.%Y') }}скользящие пять лет
-К1 · RSCI · Q1/Q2итоговые основания отбора
-
- Предварительный список. Сейчас учитываются только отметки Q1/Q2 в CSV для Scopus и Web of Science. В нём нет названия журнала, точной даты статьи, типа материала и подтверждения К1 или RSCI. Год на границе периода и принадлежность к журналу нужно проверить. Сотрудники с публикациями только в К1 или RSCI могут отсутствовать.
-
+
+
+
Как читать счётчики и критерии
+«CSV по критериям» — уникальные публикации с отметкой Scopus или Web of Science Q1/Q2 за {{ start.strftime('%d.%m.%Y') }} — {{ end.strftime('%d.%m.%Y') }}. «В БД» — все уникальные публикации сотрудника из miem-workers, включая импортированные из CSV. Общий перечень объединяет оба источника; совпадения не увеличивают итоговое количество.
+В CSV нет типа материала, точной даты и подтверждения К1 или RSCI. Счётчик CSV предварительный: для пограничных лет нужно проверить дату, для материалов без типа — подтвердить, что это статьи. Материалы, для которых в БД указан другой тип, в этот счётчик не входят. Отсутствие отметки означает недостаток данных. Наличие публикаций не подтверждает соответствие всем требованиям совета.
+Сотрудники
-Откройте строку, чтобы увидеть публикации и основания.
+{% if matched %}Показаны {{ offset + 1 }}–{{ [offset + limit, matched]|min }} из {{ matched }}{% else %}Найдено: 0{% endif %}{% if q %} · Всего в списке: {{ total }}{% endif %}
Откройте сотрудника, чтобы увидеть публикации, источники и основания.
{% for candidate in candidates %}
- {{ candidate.name }}{{ candidate.degree }} - {{ candidate.count }}публикаций по выгрузке + {{ candidate.name }}{{ candidate.degree }}{% if candidate.dismissed %} · Уволен{% endif %} + + {{ candidate.csv_count }}CSV по критериям + {{ candidate.db_count }}В БД + {{ candidate.count }}Всего без дублей +
- Открыть профиль сотрудника
+ {% if candidate.profile_url %}Профиль на сайте ВШЭ{% endif %}
+ {% if candidate.ambiguous %}
+ {% if matched > limit %}
+
+ {% endif %}
{% else %}
{% endif %}
{% endif %}
diff --git a/app/version.py b/app/version.py
index b5d6a81..a0a4892 100644
--- a/app/version.py
+++ b/app/version.py
@@ -1,3 +1,3 @@
-APP_VERSION = "0.10.6"
-FRONTEND_VERSION = "0.10.6"
-BACKEND_VERSION = "0.10.6"
+APP_VERSION = "0.11.0"
+FRONTEND_VERSION = "0.11.0"
+BACKEND_VERSION = "0.11.0"
diff --git a/pyproject.toml b/pyproject.toml
index e454c9f..3073864 100644
--- a/pyproject.toml
+++ b/pyproject.toml
@@ -1,6 +1,6 @@
[project]
name = "miem-workers"
-version = "0.10.6"
+version = "0.11.0"
description = "MIEM employees parser, admin API, and web admin"
requires-python = ">=3.11"
dependencies = [
diff --git a/tests/browser_council.py b/tests/browser_council.py
new file mode 100644
index 0000000..b4defee
--- /dev/null
+++ b/tests/browser_council.py
@@ -0,0 +1,104 @@
+"""Offline public council UX check: python tests/browser_council.py.
+
+Requires Playwright/Chromium; BROWSER_EXECUTABLE may select an installed browser.
+Synthetic employees and intercepted requests; no production access.
+"""
+import os
+from pathlib import Path
+import sys
+from urllib.parse import urlsplit
+
+sys.path.insert(0, str(Path(__file__).resolve().parents[1]))
+
+from fastapi.testclient import TestClient
+from playwright.sync_api import expect, sync_playwright
+from sqlalchemy import create_engine
+from sqlalchemy.orm import sessionmaker
+from sqlalchemy.pool import StaticPool
+
+from app.db import Base, get_db
+from app.main import app
+from app.models import Employee, EmployeePublication
+from app.services import council
+
+
+def main():
+ engine = create_engine("sqlite://", connect_args={"check_same_thread": False}, poolclass=StaticPool)
+ Base.metadata.create_all(engine)
+ factory = sessionmaker(bind=engine)
+ with factory() as db:
+ for number in range(28):
+ person = Employee(profile_key=f"test:{number}", full_name=f"Сотрудник {number:02d}",
+ canonical_url="https://www.hse.ru/staff/test",
+ current_data={"sections": [{"title": "Учёные степени",
+ "items": ["Кандидат технических наук"]}]})
+ db.add(person)
+ db.flush()
+ db.add(EmployeePublication(employee_id=person.id, title="Исследование методов передачи данных в фотонных интегральных схемах",
+ year=2024, publication_type="ARTICLE", publication_id=f"{number}",
+ url=f"https://publications.hse.ru/view/{number}", source_hash=f"test:{number}"))
+ db.commit()
+
+ def database():
+ with factory() as db:
+ yield db
+
+ original = council._articles_by_author
+ council._articles_by_author = lambda: {}
+ app.dependency_overrides[get_db] = database
+ client = TestClient(app)
+ output = Path(".impeccable/review")
+ output.mkdir(parents=True, exist_ok=True)
+
+ def serve(route):
+ url = urlsplit(route.request.url)
+ response = client.get(url.path + ("?" + url.query if url.query else ""))
+ route.fulfill(status=response.status_code, body=response.content,
+ content_type=response.headers.get("content-type", "text/plain"))
+
+ try:
+ with sync_playwright() as playwright:
+ browser = playwright.chromium.launch(executable_path=os.getenv("BROWSER_EXECUTABLE"))
+ page = browser.new_page(viewport={"width": 1440, "height": 1000})
+ page.route("**/*", serve)
+ errors = []
+ page.on("pageerror", lambda error: errors.append(str(error)))
+ page.goto("http://miem.test/admin/council")
+ expect(page.get_by_role("status")).to_have_text("Показаны 1–25 из 28")
+ expect(page.get_by_role("link", name="Вход в админку")).to_be_visible()
+ page.get_by_role("link", name="Далее").click()
+ expect(page.get_by_role("status")).to_have_text("Показаны 26–28 из 28")
+ page.get_by_role("link", name="Назад").click()
+ summary = page.locator(".council__candidate-summary").first
+ summary.focus()
+ page.keyboard.press("Enter")
+ expect(page.get_by_text("БД miem-workers · сайт ВШЭ").first).to_be_visible()
+ expect(page.get_by_text("Q1/Q2, К1 и RSCI не подтверждены доступными данными").first).to_be_visible()
+ assert page.locator(".council__candidate").first.evaluate("e => e.open")
+ page.screenshot(path=str(output / "desktop.png"), full_page=True)
+ for width in (1024, 768, 700, 390, 320):
+ page.set_viewport_size({"width": width, "height": 844})
+ assert page.evaluate("document.documentElement.scrollWidth <= innerWidth"), width
+ assert summary.evaluate("e => e.scrollWidth <= e.clientWidth"), width
+ page.set_viewport_size({"width": 390, "height": 844})
+ page.screenshot(path=str(output / "mobile.png"), full_page=True)
+ page.get_by_label("Поиск по ФИО").fill("Сотрудник 27")
+ page.get_by_role("button", name="Найти", exact=True).click()
+ expect(page.get_by_role("status")).to_have_text("Показаны 1–1 из 1 · Всего в списке: 28")
+ page.get_by_label("Поиск по ФИО").fill("Нет совпадений")
+ page.get_by_role("button", name="Найти", exact=True).click()
+ expect(page.get_by_text("Сотрудники не найдены", exact=True)).to_be_visible()
+ page.get_by_role("link", name="Сбросить поиск").first.click()
+ expect(page.get_by_role("status")).to_have_text("Показаны 1–25 из 28")
+ assert not errors, errors
+ browser.close()
+ finally:
+ council._articles_by_author = original
+ app.dependency_overrides.clear()
+ client.close()
+ engine.dispose()
+ print(f"Council browser checks passed; screenshots: {output}")
+
+
+if __name__ == "__main__":
+ main()
diff --git a/tests/test_admin_templates.py b/tests/test_admin_templates.py
index c5c1672..fad421d 100644
--- a/tests/test_admin_templates.py
+++ b/tests/test_admin_templates.py
@@ -1,8 +1,13 @@
-from pathlib import Path
+from pathlib import Path
+from types import SimpleNamespace
+
+from jinja2 import Environment, FileSystemLoader
def test_base_navigation_is_russian_and_has_no_legacy_employees_link():
- template = Path("app/templates/base.html").read_text(encoding="utf-8")
+ template = Environment(loader=FileSystemLoader("app/templates")).get_template("base.html").render(
+ request=SimpleNamespace(url=SimpleNamespace(path="/admin"))
+ )
assert "Обзор" in template
assert "Сотрудники" in template
diff --git a/tests/test_api.py b/tests/test_api.py
index 7a3f977..42b16d7 100644
--- a/tests/test_api.py
+++ b/tests/test_api.py
@@ -18,7 +18,7 @@ def test_health_returns_versions():
response = TestClient(app).get("/api/health")
assert response.status_code == 200
- assert response.json()["backend_version"] == "0.10.6"
+ assert response.json()["backend_version"] == "0.11.0"
def test_api_employees_and_stats_require_admin_session():
diff --git a/tests/test_council.py b/tests/test_council.py
index a2bf816..9839dc3 100644
--- a/tests/test_council.py
+++ b/tests/test_council.py
@@ -1,76 +1,182 @@
import csv
from datetime import date
+import pytest
from fastapi.testclient import TestClient
+from sqlalchemy.exc import OperationalError
-from app.config import Settings, get_settings
from app.db import get_db
from app.main import app
-from app.models import Employee
-from app.security import SESSION_COOKIE, sign_session
+from app.models import Employee, EmployeePublication
from app.services import council
-def test_council_counts_unique_articles_and_only_active_doctors(db_session, monkeypatch, tmp_path):
- fields = ["Название публикации", "Год", "DOI", "EID (Scopus)", "UT (WoS)",
- "Авторы НИУ ВШЭ", "Квартиль scopus", "Квартиль wos"]
- export = tmp_path / "articles.csv"
- with export.open("w", encoding="utf-8-sig", newline="") as output:
- writer = csv.DictWriter(output, fieldnames=fields, delimiter=";")
- writer.writeheader()
- for number in range(10):
- writer.writerow({"Название публикации": f"Article {number}", "Год": 2024,
- "DOI": f"10.1234/{number}", "EID (Scopus)": f"eid-{number}",
- "Авторы НИУ ВШЭ": "Иванов Иван Иванович; Петров Пётр Петрович",
- "Квартиль scopus": "Q1"})
- writer.writerow({"Название публикации": "Article 0", "Год": 2024, "DOI": "10.1234/0",
- "UT (WoS)": "ut-0", "Авторы НИУ ВШЭ": "Иванов Иван Иванович",
- "Квартиль wos": "Q2"})
- monkeypatch.setattr(council, "EXPORT", export)
+@pytest.fixture()
+def export(monkeypatch, tmp_path):
+ path = tmp_path / "articles.csv"
+ def write(rows):
+ fields = ["Название публикации", "Год", "DOI", "EID (Scopus)", "UT (WoS)",
+ "Авторы НИУ ВШЭ", "Квартиль scopus", "Квартиль wos"]
+ with path.open("w", encoding="utf-8-sig", newline="") as output:
+ writer = csv.DictWriter(output, fieldnames=fields, delimiter=";")
+ writer.writeheader()
+ writer.writerows(rows)
+ council._articles_by_author.cache_clear()
+ monkeypatch.setattr(council, "EXPORT", path)
+ write([])
+ yield write
council._articles_by_author.cache_clear()
- db_session.add_all([
- Employee(profile_key="doctor", canonical_url="https://example.org/1", full_name="Иванов Иван Иванович",
- status="active", current_data={"sections": [{"title": "Учёные степени",
- "items": ["Доктор технических наук"]}]}),
- Employee(profile_key="candidate", canonical_url="https://example.org/2", full_name="Петров Пётр Петрович",
- status="active", current_data={"sections": [{"title": "Учёные степени",
- "items": ["Кандидат технических наук"]}]}),
- ])
+
+
+def employee(db, name="Иванов Иван Иванович", degree=None, status="active", key="doctor"):
+ person = Employee(profile_key=key, canonical_url="https://www.hse.ru/staff/test", full_name=name,
+ status=status, current_data={"contacts": {"emails": ["private@example.org"]},
+ "sections": [{"title": "Учёные степени", "items": [degree] if degree else []}]})
+ db.add(person)
+ db.flush()
+ return person
+
+
+def publication(db, person, title="Shared work", doi=None, year=2024, kind=None, **extra):
+ item = EmployeePublication(employee_id=person.id, title=title, year=year, doi_url=doi,
+ publication_type=kind, source_hash=f"hash-{len(db.new)}-{title}-{doi}", **extra)
+ db.add(item)
+ return item
+
+
+def row(title="Shared work", doi="10.1234/shared", year="2024", author="Иванов Иван Иванович", **extra):
+ return {"Название публикации": title, "Год": year, "DOI": doi, "Авторы НИУ ВШЭ": author,
+ "Квартиль scopus": "Q1", **extra}
+
+
+def test_council_union_counts_provenance_and_types(db_session, export):
+ export([row(), row(**{"Квартиль scopus": "", "Квартиль wos": "Q2"}),
+ row(title="CSV only", doi="", year="2022"),
+ row(title="Not eligible", doi="10.1234/old", year="2019")])
+ person = employee(db_session, degree="Доктор технических наук")
+ publication(db_session, person, doi="https://dx.doi.org/10.1234/SHARED", kind="ARTICLE",
+ publication_id="100", url="https://publications.hse.ru/view/100")
+ publication(db_session, person, title=" shared WORK ", kind="ARTICLE")
+ publication(db_session, person, title="Book only", kind="BOOK", year=2020)
+ publication(db_session, person, title="Imported", raw_data={"csv_enrichment": {"Квартиль wos": "Q2"}})
db_session.commit()
result = council.council_candidates(db_session, date(2026, 9, 30))
assert result["total"] == 1
- assert result["candidates"][0]["count"] == 10
- assert len(result["candidates"][0]["articles"][0]["sources"]) == 2
+ candidate = result["candidates"][0]
+ assert (candidate["csv_count"], candidate["db_count"], candidate["count"]) == (2, 3, 5)
+ shared = next(a for a in candidate["articles"] if a["doi"] == "10.1234/shared")
+ assert shared["type_label"] == "Статья"
+ assert shared["quartiles"] == ["Scopus Q1", "Web of Science Q2"]
+ assert shared["rows"] == [2, 3]
+ assert "Файл CSV" in shared["origins"] and "БД miem-workers · сайт ВШЭ" in shared["origins"]
+ imported = next(a for a in candidate["articles"] if a["title"] == "Imported")
+ assert imported["origins"] == ["БД miem-workers · импорт CSV"]
+ assert not imported["csv_qualifies"]
+ assert "Тип не указан" in imported["type_label"]
assert council.period(date(2024, 2, 29))[0] == date(2019, 2, 28)
- settings = Settings(_env_file=None, session_secret="test-session-secret")
- monkeypatch.setattr("app.admin.council_candidates", lambda db: council.council_candidates(db, date(2026, 9, 30)))
- app.dependency_overrides[get_db] = lambda: db_session
- app.dependency_overrides[get_settings] = lambda: settings
- try:
- client = TestClient(app)
- client.cookies.set(SESSION_COOKIE, sign_session(settings.admin_username, settings))
- response = client.get("/admin/council")
- assert response.status_code == 200
- assert "Иванов Иван Иванович" in response.text
- assert "Article 0" in response.text
- assert "публикаций по выгрузке" in response.text
- finally:
- app.dependency_overrides.clear()
- council._articles_by_author.cache_clear()
-def test_council_page_requires_admin_and_explains_preliminary_result(db_session, monkeypatch):
- monkeypatch.setattr(council, "_articles_by_author", lambda: {})
- settings = Settings(_env_file=None, session_secret="test-session-secret")
+def test_every_employee_with_a_publication_no_degree_or_count_threshold(db_session, export):
+ export([row(author="Кандидат", title="Single qualifying"),
+ row(author="Без степени", title="Not Q1/Q2", **{"Квартиль scopus": "Q3"}),
+ row(author="Уволен", title="Old", year="2010")])
+ employee(db_session, "Кандидат", "Кандидат технических наук", key="candidate")
+ employee(db_session, "Без степени", key="none")
+ employee(db_session, "Уволен", status="dismissed", key="dismissed")
+ only_db = employee(db_session, "Только БД", key="database")
+ publication(db_session, only_db, kind="PATENT")
+ employee(db_session, "Без публикаций", key="empty")
+ db_session.commit()
+ items = council.council_candidates(db_session, date(2026, 9, 30))["candidates"]
+ assert {c["name"] for c in items} == {"Кандидат", "Без степени", "Уволен", "Только БД"}
+ assert next(c for c in items if c["name"] == "Без степени")["csv_count"] == 0
+ assert next(c for c in items if c["name"] == "Уволен")["dismissed"]
+
+
+def test_no_false_merge_when_title_has_conflicting_dois(db_session, export):
+ export([row(doi="10.1234/one"), row(doi="10.1234/two"), row(doi="")])
+ person = employee(db_session)
+ publication(db_session, person, title="Renamed work", doi="https://doi.org/10.1234/one")
+ db_session.commit()
+ item = council.council_candidates(db_session, date(2026, 9, 30))["candidates"][0]
+ assert item["count"] == 3
+ assert item["csv_count"] == 3 and item["db_count"] == 1
+
+
+def test_csv_same_name_does_not_attribute_to_wrong_employee(db_session, export):
+ export([row()])
+ for key in ["one", "two"]:
+ person = employee(db_session, key=key)
+ publication(db_session, person, title=key)
+ db_session.commit()
+ items = council.council_candidates(db_session, date(2026, 9, 30))["candidates"]
+ assert len(items) == 2
+ assert all(c["ambiguous"] and c["csv_count"] == 0 and c["count"] == 1 for c in items)
+
+
+def test_missing_database_year_merges_only_unambiguous_title_and_non_article_excluded(db_session, export):
+ export([row(title="Known book"), row(title="Article", doi="10.1234/article")])
+ person = employee(db_session)
+ publication(db_session, person, title="Known book", year=None, kind="BOOK")
+ publication(db_session, person, title="Article", year=None, kind="ARTICLE")
+ db_session.commit()
+ candidate = council.council_candidates(db_session, date(2026, 9, 30))["candidates"][0]
+ assert (candidate["count"], candidate["csv_count"], candidate["db_count"]) == (2, 1, 2)
+ book = next(a for a in candidate["articles"] if a["type_label"] == "Книга")
+ assert not book["csv_qualifies"]
+ assert "не включён в счётчик CSV" in book["criteria"][-1]
+
+
+def test_search_and_pagination_include_all_workers(db_session, export):
+ for number in range(28):
+ person = employee(db_session, f"Петров {number:02d}", key=f"person-{number}")
+ publication(db_session, person, title=f"Work {number}")
+ db_session.commit()
+ first = council.council_candidates(db_session)
+ last = council.council_candidates(db_session, offset=25)
+ assert first["total"] == 28 and len(first["candidates"]) == 25
+ assert len(last["candidates"]) == 3
+ assert {c["name"] for c in first["candidates"]}.isdisjoint(c["name"] for c in last["candidates"])
+ assert council.council_candidates(db_session, q="петров 27")["matched"] == 1
+ assert council.council_candidates(db_session, offset=-1)["offset"] == 0
+ assert council.council_candidates(db_session, offset=999)["offset"] == 25
+
+
+def test_public_page_data_minimization_and_safe_links(db_session, export):
+ export([row(title='', doi="javascript:bad")])
+ person = employee(db_session, degree="Кандидат технических наук")
+ person.canonical_url = "javascript:bad"
+ publication(db_session, person, url="javascript:alert(1)", doi="javascript:bad", kind="BOOK")
+ db_session.commit()
app.dependency_overrides[get_db] = lambda: db_session
- app.dependency_overrides[get_settings] = lambda: settings
try:
client = TestClient(app)
- assert client.get("/admin/council", follow_redirects=False).status_code in (302, 303, 401, 403)
- client.cookies.set(SESSION_COOKIE, sign_session(settings.admin_username, settings))
- response = client.get("/admin/council")
+ response = client.get("/admin/council", follow_redirects=False)
assert response.status_code == 200
- assert "Предварительный список" in response.text
- assert "К1 или RSCI могут отсутствовать" in response.text
+ assert "Иванов Иван Иванович" in response.text and "Кандидат технических наук" in response.text
+ assert "CSV по критериям" in response.text and "В БД" in response.text and "Всего без дублей" in response.text
+ assert "Книга" in response.text and "Тип не указан" in response.text
+ assert "<script>" in response.text and "В базе есть сотрудники с одинаковым ФИО. Публикации из файла не присвоены автоматически; показаны записи, связанные с этим сотрудником в БД.
{% endif %}-
{% for article in candidate.articles %}
-
{{ article.title }} - {{ article.year }} + {{ article.year or 'Год не указан' }}+
-
+ {% for criterion in article.criteria %}
- {{ criterion }} {% endfor %} +
{% endfor %}