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

C#: Make IT to manage redis. #116

Conversation

Yury-Fridlyand
Copy link

@Yury-Fridlyand Yury-Fridlyand commented Feb 27, 2024

Run tests (IT & UT)

cd <repo root>/csharp
dotnet test -- NUnit.WorkDirectory=$PWD

Tests with report:

dotnet test "-l:html;LogFileName=TestReport.html" --results-directory . -- NUnit.WorkDirectory=$PWD

The approach is the same as for java client IT.
IntegrationTestBase class provides API to get redis ports, redis version and start/stop new instance.


using Glide;

// John Travolta is looking for tests

Choose a reason for hiding this comment

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

what-huh

Signed-off-by: Yury-Fridlyand <[email protected]>
@Yury-Fridlyand Yury-Fridlyand force-pushed the csharp/dev_yuryf_it_use_script branch from b7be683 to b077eae Compare February 29, 2024 23:19
Signed-off-by: Yury-Fridlyand <[email protected]>
Copy link

@aaron-congo aaron-congo left a comment

Choose a reason for hiding this comment

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

I'm not familiar with C# but LGTM

csharp/tests/Integration/IntegrationTestBase.cs Outdated Show resolved Hide resolved
Signed-off-by: Yury-Fridlyand <[email protected]>
@Yury-Fridlyand Yury-Fridlyand merged commit b382a3c into csharp/integ_yuryf_it_use_script Mar 4, 2024
9 checks passed
@Yury-Fridlyand Yury-Fridlyand deleted the csharp/dev_yuryf_it_use_script branch March 4, 2024 19:47
Yury-Fridlyand added a commit that referenced this pull request Mar 6, 2024
* C#: Make IT to manage redis. (#116)

Signed-off-by: Yury-Fridlyand <[email protected]>
cyip10 pushed a commit that referenced this pull request Jun 24, 2024
* C#: Make IT to manage redis. (#116)

Signed-off-by: Yury-Fridlyand <[email protected]>
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.

3 participants