Skip to content

Commit

Permalink
Add reorganized cli tests to the matrix (radius-project#7286)
Browse files Browse the repository at this point in the history
# Description

cli tests were moved out of the shared folder. The functional test
workflow was missing the cli tests in the matrix and it does not use
"make test-functional-all" unlike the long running tests. As a result,
these tests were skipped. This PR adds the cli tests to the matrix.

## Type of change

<!--

Please select **one** of the following options that describes your
change and delete the others. Clearly identifying the type of change you
are making will help us review your PR faster, and is used in authoring
release notes.

If you are making a bug fix or functionality change to Radius and do not
have an associated issue link please create one now.

-->

- This pull request fixes a bug in Radius and has an approved issue
(radius-project#7279 ).


<!--

Please update the following to link the associated issue. This is
required for some kinds of changes (see above).

-->

Fixes: radius-project#7279

Signed-off-by: vinayada1 <[email protected]>
  • Loading branch information
vinayada1 authored Mar 6, 2024
1 parent 3a7568b commit 5db34ab
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion .github/workflows/functional-test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -307,7 +307,7 @@ jobs:
fail-fast: true
matrix:
os: [ubuntu-latest]
name: [ucp,kubernetes,shared,msgrp,daprrp,samples]
name: [ucp,kubernetes,shared,msgrp,daprrp,samples,cli]
include:
# datastorerp functional tests need the larger VM.
- os: ubuntu-latest-m
Expand Down

0 comments on commit 5db34ab

Please sign in to comment.