DiamondLightSource/dodal

Move mock aperture scatterguard logic into the mock device

Open

#1.764 aberto em 8 de dez. de 2025

Ver no GitHub
 (0 comments) (0 reactions) (0 assignees)Python (12 forks)auto 404
good first issue

Métricas do repositório

Stars
 (4 stars)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

Currently there is quite a bit of logic in tests/devices.conftest that is used to give a vaguely usable aperture_scatterguard device. Similar logic also exists in mx-bluesky. Following https://github.com/bluesky/ophyd-async/issues/1039 we can now make this logic the default for the device and remove it from the tests of dodal/mx-bluesky.

Acceptance Criteria

  • When you create a mock ApertureScatterguard you get something like the one created in the conftest
  • The logic for this is no longer added in both the dodal and mx-bluesky tests

Guia do colaborador

Direção de pesquisa
Examine a lógica simulada de aperture scatterguard em tests/devices.conftest e as alterações relacionadas do problema #1039. Entenda como tornar essa lógica o padrão na classe de dispositivo ApertureScatterguard e remova a lógica duplicada das suítes de teste do dodal e mx bluesky.
Pilha de tecnologia
python
Domain
backend
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
PythonOphydpytest
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75