8000 feat: Add ODP GraphQL Manager and Tests by mikechu-optimizely · Pull Request #310 · optimizely/csharp-sdk · GitHub
[go: up one dir, main page]

Skip to content

feat: Add ODP GraphQL Manager and Tests #310

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

Merged
merged 35 commits into from
Aug 24, 2022
Merged
Changes from 1 commit
Commits
Show all changes
35 commits
Select commit Hold shift + click to select a range
f7e0138
Initial GraphQLManager tests
mikechu-optimizely Aug 3, 2022
4536719
Correct GraphQLManagerTest.cs
mikechu-optimizely Aug 3, 2022
f2b4e79
Add GraphQL manager and supporting entities
mikechu-optimizely Aug 3, 2022
1e579ee
Fully cover json response
mikechu-optimizely Aug 3, 2022
d2df4a0
Adding GQLManager interface for testing
mikechu-optimizely Aug 3, 2022
368817f
Tests for common scenarios Swift ref
mikechu-optimizely Aug 3, 2022
728e1a8
Fix compiler error
mikechu-optimizely Aug 3, 2022
5e08827
Refactors
mikechu-optimizely Aug 3, 2022
5909f8e
WIP Handle versions of .NET
mikechu-optimizely Aug 3, 2022
7d35390
Merge branch 'master' into mike/ats-graphql
mikechu-optimizely Aug 4, 2022
5664b75
WebRequest and HttpClient ODP clients
mikechu-optimizely Aug 4, 2022
0e96ee1
Only support NET Standard; Inject & unit test
mikechu-optimizely Aug 4, 2022
3b267f3
Filled unit tests
mikechu-optimizely Aug 4, 2022
ed3f049
Copyright notices and ending line
mikechu-optimizely Aug 4, 2022
06a61a5
Corrections and simplifications
mikechu-optimizely Aug 5, 2022
705ce61
Corrections logged messages & supporting tests
mikechu-optimizely Aug 8, 2022
9a7ef8f
Remove excess validations
mikechu-optimizely Aug 8, 2022
dd91fce
Add unexpected node; Fix test
mikechu-optimizely Aug 8, 2022
96b84fa
Update OptimizelySDK.Tests/OdpTests/GraphQLManagerTest.cs
mikechu-optimizely Aug 8, 2022
0f444df
Update OptimizelySDK/Odp/Client/OdpClient.cs
mikechu-optimizely Aug 8, 2022
8c74e43
Update OptimizelySDK/Odp/Client/OdpClient.cs
mikechu-optimizely Aug 8, 2022
6866c82
Enhanced error handling and matching tests
mikechu-optimizely Aug 8, 2022
2b2b65d
Add end of file lines 8000
mikechu-optimizely Aug 8, 2022
cd4e8f1
Add class internal documentation
mikechu-optimizely Aug 16, 2022
d58e683
Add line at EOF
mikechu-optimizely Aug 16, 2022
0ef9849
Merge branch 'master' into mike/ats-graphql
mikechu-optimizely Aug 17, 2022
85b9fd3
Refactoring GraphQLManager
mikechu-optimizely Aug 18, 2022
f87d7f7
Refactor OdpClient
mikechu-optimizely Aug 18, 2022
c443b52
WIP Code review edits
mikechu-optimizely Aug 22, 2022
f57b438
Update doc
mikechu-optimizely Aug 22, 2022
98ad25d
QuerySegmentsParameters Builder
mikechu-optimizely Aug 23, 2022
2d76e3b
Documentation
mikechu-optimizely Aug 23, 2022
20016f9
Document constructors
mikechu-optimizely Aug 23, 2022
76e3e9e
Code review changes
mikechu-optimizely Aug 23, 2022
f5fc257
Adjust sync-to-async call methodology
mikechu-optimizely Aug 24, 2022
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fully cover json response
  • Loading branch information
mikechu-optimizely committed Aug 3, 2022
commit 1e579ee3d63d6678665ca90503745b1bb7713cee
44 changes: 36 additions & 8 deletions OptimizelySDK.Tests/OdpTests/GraphQLManagerTest.cs
Original file line number Diff line number Diff line change
@@ -1,15 +1,17 @@
using Moq;
using NUnit.Framework;
using OptimizelySDK.AudienceConditions;
using OptimizelySDK.Logger;
using OptimizelySDK.Odp;
using System.Security;

namespace OptimizelySDK.Tests.OdpTests
{
[TestFixture]
public class GraphQLManagerTest
{
private Mock<ILogger> MockLogger;

[SetUp]
public void Setup()
{
Expand All @@ -21,7 +23,8 @@ public void Setup()
public void ShouldParseSuccessfulResponse()
{
#region const string responseJson
const string responseJson = @"

var responseJson = @"
{
""data"": {
""customer"": {
Expand All @@ -44,19 +47,32 @@ public void ShouldParseSuccessfulResponse()
}
}
}";

#endregion

var manager = new GraphQLManager(MockLogger.Object);

var response = manager.ParseResponse(responseJson);
Assert.IsNull(response.Errors);

Assert.IsNull(response.Errors);
Assert.IsNotNull(response.Data);
Assert.IsNotNull(response.Data.Customer);
Assert.IsNotNull(response.Data.Customer.Audiences);
Assert.IsNotNull(response.Data.Customer.Audiences.Edges);
Assert.IsTrue(response.Data.Customer.Audiences.Edges.Length == 2);
var node = response.Data.Customer.Audiences.Edges[0].Node;
Assert.IsTrue(node.Name == "has_email");
Assert.IsTrue(node.State == "qualified");
node = response.Data.Customer.Audiences.Edges[1].Node;
Assert.IsTrue(node.Name == "has_email_opted_in");
Assert.IsTrue(node.State == "qualified");
}

[Test]
public void ShouldParseErrorResponse()
{
#region const string responseJson

const string responseJson = @"
{
""errors"": [
Expand All @@ -69,8 +85,8 @@ public void ShouldParseErrorResponse()
}
],
""path"": [
""customer""
],
""customer""
],
""extensions"": {
""classification"": ""InvalidIdentifierException""
}
Expand All @@ -80,13 +96,25 @@ public void ShouldParseErrorResponse()
""customer"": null
}
}";

#endregion

var manager = new GraphQLManager(MockLogger.Object);

var response = manager.ParseResponse(responseJson);

Assert.IsNull(response.Data.Customer);
Assert.IsNotNull(response.Errors);
Assert.IsTrue(response.Errors.Length == 1);
var error = response.Errors[0];
Assert.IsTrue(error.Message.Contains("asdsdaddddd"));
Assert.IsTrue(error.Locations.Length == 1);
var location = error.Locations[0];
Assert.IsTrue(location.Line == 2);
Assert.IsTrue(location.Column == 3);
Assert.IsTrue(error.Path.Length == 1);
Assert.IsTrue(error.Path[0] == "customer");
Assert.IsTrue(error.Extensions.Classification=="InvalidIdentifierException");
}
}
}
0