Skip to content

Commit

Permalink
adding a test to make sure we run pio as pioreactor in update scripts
Browse files Browse the repository at this point in the history
  • Loading branch information
CamDavidsonPilon committed May 5, 2024
1 parent ef7a44b commit 8dd4f0d
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 2 deletions.
2 changes: 1 addition & 1 deletion pioreactor/tests/test_update_scripts.py
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ def test_pio_commands():
with open(script, "r") as file:
for line_number, line in enumerate(file, start=1):
# Checking for 'pio' not preceded by 'su -u pioreactor'
if " pio " in line and "su -u pioreactor" not in line:
if (" pio " in line or line.strip().startswith('pio')) and "su -u pioreactor" not in line:
error_msgs.append(
f"Error in {script} at line {line_number}: 'pio' command must be prefixed with 'su -u pioreactor'."
)
Expand Down
1 change: 0 additions & 1 deletion update_scripts/24.5.1/pre_update.sh
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@ set -xeu

export LC_ALL=C


# Lower bound version
min_version="24.4.11"

Expand Down

0 comments on commit 8dd4f0d

Please sign in to comment.