Skip to content

Conversation

radik
Copy link

@radik radik commented Jul 9, 2025

Re-write of authzed-dotnet tests.
Solves #225

Existing tests are left for now, until review.

Copy link
Contributor

github-actions bot commented Jul 9, 2025

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@radik radik force-pushed the tests-refactoring branch from 403ea72 to 37df4d7 Compare July 10, 2025 17:41
@radik
Copy link
Author

radik commented Jul 10, 2025

I have read the CLA Document and I hereby sign the CLA

authzedbot added a commit to authzed/cla that referenced this pull request Jul 10, 2025
@radik radik changed the title [WIP] chore: Re-implementing dotnet tests [WIP] chore: Re-implementing tests (authzed-dotent tests are taken as template) Jul 10, 2025
@radik radik marked this pull request as ready for review July 10, 2025 20:24
@radik radik requested a review from a team as a code owner July 10, 2025 20:24
@radik radik changed the title [WIP] chore: Re-implementing tests (authzed-dotent tests are taken as template) chore: Re-implementing tests (authzed-dotent tests are taken as template) Jul 10, 2025
Copy link
Contributor

@tstirrat15 tstirrat15 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like it. Thank you!

@radik
Copy link
Author

radik commented Sep 6, 2025

@tstirrat15 great! How can I merge it?

@tstirrat15
Copy link
Contributor

tstirrat15 commented Sep 8, 2025

Existing tests are left for now, until review.

Was your plan to remove the other tests? I didn't want to merge if there was still something pending.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants