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

NEST-282: Adding extension method #200

Merged
merged 6 commits into from
Sep 28, 2022
Merged

NEST-282: Adding extension method #200

merged 6 commits into from
Sep 28, 2022

Conversation

porgabi
Copy link
Member

@porgabi porgabi commented Sep 12, 2022


public static class TenantsUITestContextExtensions
{
public static async Task CreateNewTenantAsync(
Copy link
Member

Choose a reason for hiding this comment

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

There is a test controller here that already implements tenant creation.

And you can see how it is used here.

Use that in your tests.

Copy link
Member Author

Choose a reason for hiding this comment

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

I'm running into some trouble when using this method in the OSOCE.NuGet UI tests. When it runs, it produces an exception:
image

This is especially puzzling as it runs just fine in the normal OSOCE solution's UI tests.

I found that when manually creating and setting up the tenant during a UI test, it works fine in the NuGet solution too. I'm still looking into this, but I'm also considering simply re-adding the method that this thread is attached to, perhaps with the name CreateNewTenantManuallyAsync().

Is this exception familiar to you by any chance? 😃

Copy link
Member

Choose a reason for hiding this comment

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

I didn't meet with this error before. So it might be better to ask it in the general channel on Teams. Dávid might know it as he was the one who wrote this UI test function.

@wAsnk wAsnk merged commit f183777 into dev Sep 28, 2022
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