DiamondLightSource/dodal

Move mock aperture scatterguard logic into the mock device

Open

#1 764 ouverte le 8 déc. 2025

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)Python (12 forks)auto 404
good first issue

Métriques du dépôt

Stars
 (4 stars)
Métriques de merge PR
 (Métriques PR en attente)

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

Guide contributeur

Direction de recherche
Examinez la logique simulée d'aperture scatterguard dans tests/devices.conftest et les modifications associées du problème #1039. Comprenez comment faire de cette logique la valeur par défaut dans la classe de périphérique ApertureScatterguard, puis supprimez la logique dupliquée des suites de tests de dodal et mx bluesky.
Stack technique
python
Domaine
backend
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
PythonOphydpytest
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75