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

Fix/moved to xunit #19

Merged
merged 3 commits into from
Nov 7, 2023
Merged

Conversation

Mastermindzh
Copy link
Contributor

As discussed in #18 , fixes #17

Copy link
Contributor

@pascal89 pascal89 left a comment

Choose a reason for hiding this comment

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

I assume all tests still pass?


<ItemGroup>
<PackageReference Include="FluentAssertions" Version="6.5.1" />
<PackageReference Include="Microsoft.AspNetCore.Mvc.Testing" Version="5.0.10" />
Copy link
Contributor

Choose a reason for hiding this comment

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

While you're touching it. You might as well be updating the package references, because this package for example is already at version 7.x.x

@@ -2,7 +2,7 @@
using System.IO;
using System.Reflection;

namespace Growthbook.Tests.Json;
namespace GrowthBook.Tests.Json;
Copy link
Contributor

Choose a reason for hiding this comment

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

Looks like I wasn't the only one misspelling GrowthBook

@Mastermindzh
Copy link
Contributor Author

I assume all tests still pass?

Of course!

@darklordzw darklordzw merged commit e008a0d into growthbook:main Nov 7, 2023
@pascal89
Copy link
Contributor

pascal89 commented Nov 9, 2023

@darklordzw Would you be able to publish a new version of the package any time soon? Thanks in advance 😄

@darklordzw
Copy link
Collaborator

@pascal89 Sure thing! Latest version of the package is published now.

I've also been looking into setting up the secrets for automated deploy, but it looks like I don't have the permissions required to do so. @Auz, do you have permissions to take a look?

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.

Tests are not portable
4 participants