Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test organization proposal #35

Merged
merged 2 commits into from
May 21, 2024
Merged

test organization proposal #35

merged 2 commits into from
May 21, 2024

Conversation

vinayada1
Copy link
Contributor

Proposal to reorganize tests

@vinayada1 vinayada1 requested review from a team as code owners December 12, 2023 01:06
@vinayada1 vinayada1 force-pushed the test-organization branch 3 times, most recently from 1708177 to 720e57d Compare December 12, 2023 23:18
@vinayada1 vinayada1 force-pushed the test-organization branch 2 times, most recently from 3606516 to 6921824 Compare December 13, 2023 18:15
@vinayada1 vinayada1 force-pushed the test-organization branch 2 times, most recently from 160b7f3 to ee6392e Compare January 8, 2024 18:13
vinayada1 added a commit to radius-project/radius that referenced this pull request Mar 1, 2024
# Description

As a part of #6588, as proposed by
radius-project/design-notes#35, we will use
RequiredFeatures check to separate AWS and Azure specific tests. This is
Part1 of a multi-part PR to change the organization of the tests

## 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 adds or changes features of Radius and has an
approved issue (#6588 ).


<!--

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

-->

Fixes: A part of #6588 but multiple PRs to follow to fix completely

---------

Signed-off-by: vinayada1 <[email protected]>
Co-authored-by: Young Bu Park <[email protected]>
vinayada1 added a commit to radius-project/radius that referenced this pull request Mar 4, 2024
…ud tests (#7221)

# Description

- Separate cloud and non-cloud tests for messaging RP
- Organize the UCP tests as per
radius-project/design-notes#35. There will be
more PRs that will follow to re-org the other tests.

## 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 adds or changes features of Radius and has an
approved issue (#6588 ).

<!--

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

-->

Fixes: This is part of a multi-PR fix for #6588.

Signed-off-by: vinayada1 <[email protected]>
willdavsmith pushed a commit to willdavsmith/radius that referenced this pull request Mar 4, 2024
# Description

As a part of radius-project#6588, as proposed by
radius-project/design-notes#35, we will use
RequiredFeatures check to separate AWS and Azure specific tests. This is
Part1 of a multi-part PR to change the organization of the tests

## 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 adds or changes features of Radius and has an
approved issue (radius-project#6588 ).


<!--

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

-->

Fixes: A part of radius-project#6588 but multiple PRs to follow to fix completely

---------

Signed-off-by: vinayada1 <[email protected]>
Co-authored-by: Young Bu Park <[email protected]>
vinayada1 added a commit to radius-project/radius that referenced this pull request Mar 4, 2024
# Description

- Separate cloud/non-cloud tests
- Organize the cli tests as per
radius-project/design-notes#35. There will be
more PRs that will follow to re-org the other tests.

## 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 adds or changes features of Radius and has an
approved issue (#6588 ).

<!--

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

-->

Fixes: Fixes part of
#6588. This is a part of
a multi-PR fix

Signed-off-by: vinayada1 <[email protected]>
vinayada1 added a commit to radius-project/radius that referenced this pull request Mar 5, 2024
# Description

- Separate cloud/non-cloud tests
- Organize the daprrp tests as per
radius-project/design-notes#35. There will be
more PRs that will follow to re-org the other tests.

## 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 adds or changes features of Radius and has an
approved issue (#6588 ).

<!--

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

-->

Fixes: Fixes part of
#6588. This is a part of
a multi-PR fix

---------

Signed-off-by: vinayada1 <[email protected]>
vinayada1 added a commit to radius-project/radius that referenced this pull request Mar 5, 2024
…non-cloud tests (#104)

# Description

- Separate cloud/non-cloud tests
- Organize the UCP tests as per
radius-project/design-notes#35. There will be
more PRs that will follow to re-org the other tests.

## 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 adds or changes features of Radius and has an
approved issue #6588 .

<!--

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

-->

Fixes: Fixes part of
#6588. This is a part of
a multi-PR fix

---------

Signed-off-by: vinayada1 <[email protected]>
vinayada1 added a commit to radius-project/radius that referenced this pull request Mar 5, 2024
# Description

- Separate cloud/non-cloud tests
- Organize the daprrp tests as per
radius-project/design-notes#35. There will be
more PRs that will follow to re-org the other tests.

## 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 adds or changes features of Radius and has an
approved issue (#6588).

<!--

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

-->

Fixes: Fixes part of
#6588. This is a part of
a multi-PR fix

---------

Signed-off-by: vinayada1 <[email protected]>
vinayada1 added a commit to radius-project/radius that referenced this pull request Mar 5, 2024
# Description

- Separate cloud/non-cloud tests
- Organize the daprrp tests as per
radius-project/design-notes#35. There will be
more PRs that will follow to re-org the other tests.

## 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 adds or changes features of Radius and has an
approved issue (#6588 ).

<!--

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

-->

Fixes: Fixes part of
#6588. This is a part of
a multi-PR fix

---------

Signed-off-by: vinayada1 <[email protected]>
vinayada1 added a commit to radius-project/radius that referenced this pull request Mar 5, 2024
…7219)

# Description

- Separate cloud/non-cloud tests
- Organize the UCP tests as per
radius-project/design-notes#35. There will be
more PRs that will follow to re-org the other tests.

## 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 adds or changes features of Radius and has an
approved issue (#6588 ).


<!--

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

-->

Fixes: Fixes part of #6588. This is a part of a multi-PR fix

---------

Signed-off-by: vinayada1 <[email protected]>
vinayada1 added a commit to radius-project/radius that referenced this pull request Mar 26, 2024
# Description

- Change test organization for shared tests. These tests are now
organized under the corerp directory.
- Separate cloud/non-cloud tests
- Organize the corerp tests as per
radius-project/design-notes#35. There will be
more PRs that will follow to re-org the other tests.

## 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 adds or changes features of Radius and has an
approved issue (#6588 ).

<!--

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

-->

Fixes: Fixes part of
#6588. This is a part of
a multi-PR fix

---------

Signed-off-by: vinayada1 <[email protected]>
Copy link

github-actions bot commented May 6, 2024

This pull request is stale because it has been open 14 days with no activity. Remove stale label or comment or this will be closed in 7 days.

@github-actions github-actions bot added the Stale label May 6, 2024
vinayada1 added 2 commits May 21, 2024 15:32
Signed-off-by: vinayada1 <[email protected]>
@vinayada1 vinayada1 force-pushed the test-organization branch from ee6392e to f4f0b7b Compare May 21, 2024 22:32
@vinayada1
Copy link
Contributor Author

Outstanding implementation work items:
radius-project/radius#7624
radius-project/radius#7625

@vinayada1 vinayada1 closed this May 21, 2024
@vinayada1 vinayada1 reopened this May 21, 2024
@vinayada1 vinayada1 merged commit cdf2e06 into main May 21, 2024
3 checks passed
@vinayada1 vinayada1 deleted the test-organization branch May 21, 2024 22:42
Reshrahim pushed a commit to Reshrahim/design-notes that referenced this pull request Sep 20, 2024
Proposal to reorganize tests

---------

Signed-off-by: vinayada1 <[email protected]>
Signed-off-by: Reshma Abdul Rahim <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants