Skip to content

Fix settings paths for tests - #674

Open
FreddieAkeroyd wants to merge 1 commit into
masterfrom
new_galil_fixes
Open

Fix settings paths for tests#674
FreddieAkeroyd wants to merge 1 commit into
masterfrom
new_galil_fixes

Conversation

@FreddieAkeroyd

@FreddieAkeroyd FreddieAkeroyd commented Jul 18, 2026

Copy link
Copy Markdown
Member

See ISISComputingGroup/IBEX#8979

@FreddieAkeroyd FreddieAkeroyd changed the title Set record to sim mode Fix settings paths for tests Jul 22, 2026
Comment thread utils/testing.py Fixed
Comment thread utils/free_ports.py Fixed
Comment thread tests/heliox.py Fixed
@FreddieAkeroyd
FreddieAkeroyd force-pushed the new_galil_fixes branch 2 times, most recently from d6a72e1 to 905f182 Compare July 29, 2026 00:36
@FreddieAkeroyd

FreddieAkeroyd commented Jul 29, 2026

Copy link
Copy Markdown
Member Author

could of cases i have ignored BLE001 but maybe could catch AssertionError | UnableToConnectToPVException if that covered things e.g. in cryosms?

@FreddieAkeroyd

FreddieAkeroyd commented Jul 29, 2026

Copy link
Copy Markdown
Member Author

just rerunning a jenkins job as seen some AttributeError: type object 'datetime.datetime' has no attribute 'timezone'. Did you mean: 'astimezone'? in an earlier system test log. Feels like you might get this if you had done a from datetime import datetime rather than import datetime but i'd have thought ruff would have picked this up. Re-running as it was looking at a slightly older version of code, but I don't recall changing any datetime today.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants