Skip to content

Commit

Permalink
Test
Browse files Browse the repository at this point in the history
  • Loading branch information
mosteo committed Sep 27, 2024
1 parent 1b1df86 commit 68a1db2
Show file tree
Hide file tree
Showing 2 changed files with 52 additions and 0 deletions.
48 changes: 48 additions & 0 deletions testsuite/tests/action/environment/test.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,48 @@
"""
Verify the proper environment is set up for the action. We create an executable
within a crate that will be run during pre-build. For it to be found, the
proper path must be set in the crate environment. This should work for a
regular build (as it did before) and for a manually triggered action (the bug
being fixed).
"""

import os
from shutil import which
from drivers.alr import run_alr, init_local_crate, add_action, alr_manifest
# from drivers.asserts import assert_eq, assert_match

tool_name = "myalrtestingtool"

# Tool crate to be used during build
init_local_crate(tool_name, enter=False)

# Ensure the tool is not found in path by pure chance!
assert not which(tool_name)

# Build the tool
run_alr("-C", tool_name, "build")

# Create the new crate that uses this tool during pre-build step
init_local_crate()

# Edit its manifest to require it during pre-build
add_action("pre-build", [tool_name])

# Build should fail because the tool is not found in the path
run_alr("build", complain_on_error=False)

# Likewise, stand-alone running of the action should fail
run_alr("action", "pre-build", complain_on_error=False)

# Add the tool location to the crate environment
with open(alr_manifest(), "at") as f:
f.write(f"""
[environment]
PATH.append = "..{os.path.sep}{tool_name}{os.path.sep}bin"
""")

# Now both build and action should succeed
run_alr("build")
run_alr("action", "pre-build")

print("SUCCESS")
4 changes: 4 additions & 0 deletions testsuite/tests/action/environment/test.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
driver: python-script
build_mode: both
indexes:
compiler_only_index: {}

0 comments on commit 68a1db2

Please sign in to comment.