Skip to content

Commit

Permalink
Case RE-851: Only run OpenStack GitHub Action when TestSuite Passes
Browse files Browse the repository at this point in the history
Changelog:
  • Loading branch information
davelcpanelnet committed Sep 26, 2024
1 parent 2cd9f9f commit 3e8239c
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/openstack.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ on:
workflow_dispatch:

concurrency:
group: ${{ github.workflow }}-${{ github.ref }}
group: ${{ github.workflow }}-${{ github.ref }}-openstack
cancel-in-progress: true

env:
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/testsuite.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ on:
workflow_call:

concurrency:
group: ${{ github.workflow }}-${{ github.ref }}
group: ${{ github.workflow }}-${{ github.ref }}-testsuite
cancel-in-progress: true

jobs:
Expand Down
2 changes: 1 addition & 1 deletion t/ssystem.t
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ my $mock_log_file = Test::MockFile->file('/var/log/elevate-cpanel.log');

my $cpev = cpev->new->_init;

is( cpev->ssystem('nope'), undef, q[ssystem( 'nope' ) is disallowed] );
is( cpev->ssystem('nope'), 42, q[ssystem( 'nope' ) is disallowed] );
is( cpev->ssystem('grep'), 42, 'Commands that are not absolute paths are not allowed' );

is( cpev->ssystem('/etc/apache2/conf/httpd.conf'), 42, 'Commands that are not executable are not allowed' );
Expand Down

0 comments on commit 3e8239c

Please sign in to comment.