WIP Feature/handle users in multiple experiments

Gibe.AbTest.Tests/AbTestRepositoryTests.cs

@@ -11,11 +11,53 @@ namespace Gibe.AbTest.Tests
 	[TestFixture]
 	public class AbTestRepositoryTests
 	{
+		private const string ValidExperimentId = "vapBwUPvTEuGcEVEKThGCA";
+		private const string InvalidExperimentId = "NotRealId";
+
 		[Test]
-		public void Test()
+		public void GetExperiments_Returns_All_Experiments()
 		{
-			var repo = new AbTestRepository(new DefaultDatabaseProvider("GibeCommerce"));
-			var experiments = repo.GetExperiments().ToArray();
+			var experiments = Repo().GetExperiments().ToArray();
+
+			Assert.That(experiments.Length, Is.EqualTo(2));
+		}
+
+		[Test]
+		public void GetExperiment_Returns_Experiment_When_Id_Exists()
+		{
+			var experiment = Repo().GetExperiment(ValidExperimentId);
+
+			Assert.That(experiment, Is.Not.Null);
+			Assert.That(experiment.Id, Is.EqualTo(ValidExperimentId));
+		}
+
+		[Test]
+		public void GetExperiment_Returns_Null_When_Id_Does_Not_Exist()
+		{
+			var experiment = Repo().GetExperiment(InvalidExperimentId);
+
+			Assert.That(experiment, Is.Null);
+		}
+
+		[Test]
+		public void GetVariations_Returns_All_Variations_For_Experiment_When_Id_Exists()
+		{
+			var variations = Repo().GetVariations(ValidExperimentId).ToArray();
+
+			Assert.That(variations.Count(), Is.EqualTo(2));
+			Assert.That(variations.All(v => v.ExperimentId == ValidExperimentId), Is.True);
+		}
+
+		[Test]
+		public void GetVariations_Returns_Empty_Enumerable_For_Experiment_When_Id_Does_Not_Exist()
+		{
+			var variations = Repo().GetVariations(InvalidExperimentId);
+
+			Assert.That(variations.Count(), Is.EqualTo(0));
 		}
+
+		private IAbTestRepository Repo() => new AbTestRepository(new DefaultDatabaseProvider("GibeCommerce"));
+
+
 	}
 }

Gibe.AbTest.Tests/AbTestTests.cs

@@ -1,51 +1,102 @@
-using System;
+using NUnit.Framework;
+using System;
+using System.Collections.Generic;
 using System.Linq;
-using NUnit.Framework;
 
 namespace Gibe.AbTest.Tests
 {
 	[TestFixture]
 	public class AbTestTests
 	{
+		private IAbTestingService _abTestingService;
+
+		[SetUp]
+		public void Setup()
+		{
+			_abTestingService = new FakeAbTestingService(new List<Experiment>
+			{
+				new Experiment("Ex1", "Exp1", "Experiment 1", 1, true, DateTime.Now, null,
+					new []{
+						new Variation(1, 0, 1, true, "{Exp1:'Variant 1'}", "Exp1"),
+						new Variation(2, 1, 1, true, "{Exp1:'Variant 2'}", "Exp1")
+
+					}),
+				new Experiment("Ex2", "Exp2", "Experiment 2", 1, true, DateTime.Now, null,
+					new []{
+						new Variation(3, 0, 1, true, "{Exp2:'Variant 1'}", "Exp2"),
+						new Variation(4, 1, 1, true, "{Exp2:'Variant 2'}", "Exp2")
+					}),
+				new Experiment("Ex3", "Exp3", "Experiment 3", 1, false, DateTime.Now, null,
+					new []{
+						new Variation(5, 0, 1, true, "{Exp3:'Variant 1'}", "Exp3"),
+						new Variation(6, 1, 1, true, "{Exp2:'Variant 2'}", "Exp3")
+					})
+			});
+		}
+
 		[Test]
-		public void AssignVariation_assigns_first_experiment_variation_when_random_number_is_0()
+		public void AssignVariation_Assigns_First_Experiment_Variation_When_Random_Number_Is_0()
 		{
-			var fakeAbTestingService = new FakeAbTestingService();
+			var abTest = new AbTest(_abTestingService, new FakeRandomNumber(new [] { 0, 0 }));
+
+			var variation = abTest.AssignRandomVariation();
 
-			var abTest = new AbTest(fakeAbTestingService, new FakeRandomNumber(new [] { 0, 0 }));
+			Assert.AreEqual(_abTestingService.GetExperiments().First().Variations.First().Id, variation.Id);
+		}
+
+		[Test]
+		public void AssignVariation_Assigns_Second_Experiment_Variation_When_Random_Number_Is_1()
+		{
+			var abTest = new AbTest(_abTestingService, new FakeRandomNumber(new[] { 1, 1 }));
 
-			var variation = abTest.AssignVariation();
+			var variation = abTest.AssignRandomVariation();
 
-			Assert.AreEqual(fakeAbTestingService.GetExperiments().First().Variations.First().Id, variation.Id);
+			Assert.AreEqual(_abTestingService.GetExperiments().ElementAt(1).Variations.ElementAt(1).Id, variation.Id);
 		}
 
 		[Test]
-		public void AssignVariation_assigns_second_experiment_variation_when_random_number_is_1()
+		public void AssignVariation_Assigns_Second_Experiment_Variation_When_Random_Number_Is_2()
 		{
-			var fakeAbTestingService = new FakeAbTestingService();
+			var abTest = new AbTest(_abTestingService, new FakeRandomNumber(new[] { 2, 2 }));
 
-			var abTest = new AbTest(fakeAbTestingService, new FakeRandomNumber(new[] { 1, 1 }));
+			var variation = abTest.AssignRandomVariation();
 
-			var variation = abTest.AssignVariation();
+			Assert.AreEqual(_abTestingService.GetExperiments().ElementAt(1).Variations.ElementAt(1).Id, variation.Id);
+		}
 
-			Assert.AreEqual(fakeAbTestingService.GetExperiments().ElementAt(1).Variations.ElementAt(1).Id, variation.Id);
+		[Test]
+		public void AssignVariation_Assigns_Given_Experiment_First_Variation()
+		{
+			var abTest = new AbTest(_abTestingService, new FakeRandomNumber(new[] { 0, 0 }));
+
+			var variation = abTest.AssignVariation("Exp1");
+
+			Assert.That(_abTestingService.GetExperiments().First(x => x.Key == "Exp1").Variations.First().Id, Is.EqualTo(variation.Id));
+		}
+
+		[Test]
+		public void AssignVariations_Assigns_First_Experiment_Variation_From_Each_Enabled_Experiment()
+		{
+			var abTest = new AbTest(_abTestingService, new FakeRandomNumber(new[] { 0, 0 }));
+
+			var variations = abTest.AllCurrentVariations().ToList();
+
+			Assert.That(_abTestingService.GetExperiments().Where(e => e.Enabled).Select(e => e.Variations.First().Id), Is.EqualTo(variations.Select(v => v.Id)));
+			Assert.That(variations.Count, Is.EqualTo(2));
 		}
 
 		[Test]
 		public void GetAssignedVariation_returns_variation_from_AbTestingService()
 		{
-			const string experimentId = "ABC";
+			const string experimentId = "Exp4";
 			const int variationNo = 1;
 
-			var fakeAbTestingService = new FakeAbTestingService();
+			var abTest = new AbTest(_abTestingService, new FakeRandomNumber(new int[] {}));
 
-			var abTest = new AbTest(fakeAbTestingService, new FakeRandomNumber(new int[] {}));
-
-			var variation = abTest.GetAssignedVariation(experimentId, variationNo);
+			var variation = abTest.Variation(experimentId, variationNo);
 
 			Assert.AreEqual(experimentId, variation.ExperimentId);
 			Assert.AreEqual(variationNo, variation.VariationNumber);
-
 		}
 	}
 }

Gibe.AbTest.Tests/App.config

@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="utf-8" ?>
 <configuration>
 	<connectionStrings>
-		<add name="GibeCommerce" connectionString="Data Source=cartman2.gibe.local;Initial Catalog=TapWarehouse;Integrated Security=true" providerName="System.Data.SqlClient" />
+		<add name="GibeCommerce" connectionString="Data Source=cartman2.office.gibe.digital;Initial Catalog=TapWarehouse;Integrated Security=true" providerName="System.Data.SqlClient" />
 	</connectionStrings>
 </configuration>

Gibe.AbTest.Tests/ExperimentServiceTests.cs

@@ -0,0 +1,149 @@
+using System.Collections.Generic;
+using System.Linq;
+using Gibe.Cookies;
+using Moq;
+using NUnit.Framework;
+
+namespace Gibe.AbTest.Tests
+{
+	[TestFixture]
+	public class ExperimentServiceTests
+	{
+		private const string CookieKey = "GCEXP";
+
+		private Mock<ICookieService> _cookieService;
+		private IAbTest _abTest;
+
+		[SetUp]
+		public void Setup()
+		{
+			_cookieService = new Mock<ICookieService>();
+			_abTest = new FakeAbTest(
+				new List<Variation>{
+					new Variation(1, 0, 1,true,"{Test:'test1'}", "vapBwUPvTEuGcEVEKThGCA"),
+					new Variation(2, 1, 1,true,"{Test:'test1'}", "vapBwUPvTEuGcEVEKThGCA"),
+					new Variation(3, 0, 1,true,"{Test:'test2'}", "vapBwUPvTEuGcEVEKThGCB"),
+					new Variation(4, 1, 1,true,"{Test:'test2'}", "vapBwUPvTEuGcEVEKThGCB"),
+					new Variation(5, 0, 1,true,"{Test:'test3'}", "vapBwUPvTEuGcEVEKThGCC"),
+					new Variation(6, 1, 1,true,"{Test:'test3'}", "vapBwUPvTEuGcEVEKThGCC")
+				});
+		}
+
+		private IExperimentService ExperimentService()
+		{
+			return new DefaultExperimentService(_cookieService.Object, _abTest, new ExperimentCookieValueFactory(_abTest));
+		}
+
+		[Test]
+		public void IsCurrentUserInExperiment_Returns_True_When_Cookie_Contains_Experiment()
+		{
+			_cookieService.Setup(s => s.GetValue<string>(It.IsAny<string>())).Returns("vapBwUPvTEuGcEVEKThGCA~0-vapBwUPvTEuGcEVEKThGCB~1");
+
+			var result = ExperimentService().IsCurrentUserInExperiment();
+
+			Assert.That(result, Is.True);
+		}
+
+		[Test]
+		public void IsCurrentUserInExperiment_Returns_False_When_No_Experiment_Cookie_Found()
+		{
+			_cookieService.Setup(s => s.GetValue<string>(It.IsAny<string>())).Returns("");
+
+			var result = ExperimentService().IsCurrentUserInExperiment();
+
+			Assert.That(result, Is.False);
+		}
+
+		[Test]
+		public void CurrentUserVariations_Returns_Enumerable_Of_Variations_Based_On_The_Cookie_Value()
+		{
+			_cookieService.Setup(s => s.GetValue<string>(It.IsAny<string>())).Returns("vapBwUPvTEuGcEVEKThGCA~0-vapBwUPvTEuGcEVEKThGCB~1");
+
+			var results = ExperimentService().CurrentUserVariations();
+
+			AssertVariations(results, "vapBwUPvTEuGcEVEKThGCA~0-vapBwUPvTEuGcEVEKThGCB~1");
+		}
+
+		[Test]
+		public void CurrentUserVariation_Returns_Variation_For_ExperiementId_Based_On_The_Cookie_Value()
+		{
+			_cookieService.Setup(s => s.GetValue<string>(It.IsAny<string>())).Returns("vapBwUPvTEuGcEVEKThGCA~0-vapBwUPvTEuGcEVEKThGCB~1");
+
+			var results = ExperimentService().CurrentUserVariation("vapBwUPvTEuGcEVEKThGCA");
+
+			AssertVariations(new []{results}, "vapBwUPvTEuGcEVEKThGCA~0");
+		}
+
+		[Test]
+		public void AssignUserVariations_Assigns_User_To_Random_Variations_For_Each_Experiment()
+		{
+
+			var results = ExperimentService().AssignUserVariations();
+
+			Assert.That(results.Count(), Is.EqualTo(3));
+		}
+
+		[Test]
+		public void AssignUserVariations_Updates_The_Cookie_Value_For_The_Requested_Variation_Only()
+		{
+			_cookieService.Setup(s => s.GetValue<string>(It.IsAny<string>())).Returns("vapBwUPvTEuGcEVEKThGCA~0-vapBwUPvTEuGcEVEKThGCB~1-vapBwUPvTEuGcEVEKThGCC~0");
+
+			var results = ExperimentService().AssignUserVariations("vapBwUPvTEuGcEVEKThGCA~1");
+
+			AssertVariations(results, "vapBwUPvTEuGcEVEKThGCA~1-vapBwUPvTEuGcEVEKThGCB~1-vapBwUPvTEuGcEVEKThGCC~0");
+		}
+
+		[Test]
+		public void AssignUserVariations_Does_Not_Change_The_Cookie_Value_For_The_Requested_Variation_When_The_Same()
+		{
+			_cookieService.Setup(s => s.GetValue<string>(It.IsAny<string>())).Returns("vapBwUPvTEuGcEVEKThGCA~0-vapBwUPvTEuGcEVEKThGCB~1-vapBwUPvTEuGcEVEKThGCC~0");
+
+			var results = ExperimentService().AssignUserVariations("vapBwUPvTEuGcEVEKThGCA~0");
+
+			AssertVariations(results, "vapBwUPvTEuGcEVEKThGCA~0-vapBwUPvTEuGcEVEKThGCB~1-vapBwUPvTEuGcEVEKThGCC~0");
+		}
+
+		[Test]
+		public void AssignUserVariations_Does_Not_Change_The_Cookie_Value_When_The_Requested_Experiment_Does_Not_Exist()
+		{
+			_cookieService.Setup(s => s.GetValue<string>(It.IsAny<string>())).Returns("vapBwUPvTEuGcEVEKThGCA~0-vapBwUPvTEuGcEVEKThGCB~1-vapBwUPvTEuGcEVEKThGCC~0");
+
+			var results = ExperimentService().AssignUserVariations("vapBwUPvTEuGcEVEKThGCD~0");
+
+			AssertVariations(results, "vapBwUPvTEuGcEVEKThGCA~0-vapBwUPvTEuGcEVEKThGCB~1-vapBwUPvTEuGcEVEKThGCC~0");
+		}
+
+		[Test]
+		public void AssignCurrentUserToVariation_Does_Not_Change_The_Cookie_Value_When_The_Requested_Variation_Does_Not_Exist()
+		{
+			_cookieService.Setup(s => s.GetValue<string>(It.IsAny<string>())).Returns("vapBwUPvTEuGcEVEKThGCA~0-vapBwUPvTEuGcEVEKThGCB~1-vapBwUPvTEuGcEVEKThGCC~0");
+
+			var results = ExperimentService().AssignUserVariations("vapBwUPvTEuGcEVEKThGCA~3");
+
+			AssertVariations(results, "vapBwUPvTEuGcEVEKThGCA~0-vapBwUPvTEuGcEVEKThGCB~1-vapBwUPvTEuGcEVEKThGCC~0");
+		}
+
+		[Test]
+		public void AssignUserVariations_Assigns_User_To_Experiments_Not_Currently_In_When_Updating_Variation()
+		{
+			_cookieService.Setup(s => s.GetValue<string>(It.IsAny<string>())).Returns("vapBwUPvTEuGcEVEKThGCA~0");
+
+			var results = ExperimentService().AssignUserVariations("vapBwUPvTEuGcEVEKThGCA~1");
+
+			AssertVariations(results, "vapBwUPvTEuGcEVEKThGCA~1-vapBwUPvTEuGcEVEKThGCB~0-vapBwUPvTEuGcEVEKThGCC~0");
+		}
+
+		private static void AssertVariations(IEnumerable<Variation> results, string variationsString)
+		{
+			var variations = variationsString.Split('-')
+				.Select(e => new Variation(0, int.Parse(e.Split('~')[1]), 1, true, "", e.Split('~')[0]));
+
+			foreach (var variation in variations)
+			{
+				Assert.That(results.Any(r => r.ExperimentId == variation.ExperimentId && r.VariationNumber == variation.VariationNumber));
+			}
+			Assert.That(results.Count(), Is.EqualTo(variations.Count()));
+		}
+	}
+}
+

Gibe.AbTest.Tests/Gibe.AbTest.Tests.csproj

@@ -35,10 +35,16 @@
     <WarningLevel>4</WarningLevel>
   </PropertyGroup>
   <ItemGroup>
+    <Reference Include="Gibe.Cookies, Version=1.0.0.0, Culture=neutral, processorArchitecture=MSIL">
+      <HintPath>..\packages\Gibe.Cookies.1.0.356\lib\Gibe.Cookies.dll</HintPath>
+    </Reference>
     <Reference Include="Gibe.NPoco, Version=1.0.0.0, Culture=neutral, processorArchitecture=MSIL">
       <HintPath>..\packages\Gibe.NPoco.1.0.159\lib\Gibe.NPoco.dll</HintPath>
       <Private>True</Private>
     </Reference>
+    <Reference Include="Moq, Version=4.2.1502.911, Culture=neutral, PublicKeyToken=69f491c39445e920, processorArchitecture=MSIL">
+      <HintPath>..\packages\Moq.4.2.1502.0911\lib\net40\Moq.dll</HintPath>
+    </Reference>
     <Reference Include="NPoco, Version=2.0.0.0, Culture=neutral, processorArchitecture=MSIL">
       <HintPath>..\packages\NPoco.2.5.77\lib\net40\NPoco.dll</HintPath>
       <Private>True</Private>
@@ -61,6 +67,7 @@
   <ItemGroup>
     <Compile Include="AbTestRepositoryTests.cs" />
     <Compile Include="AbTestTests.cs" />
+    <Compile Include="ExperimentServiceTests.cs" />
     <Compile Include="Properties\AssemblyInfo.cs" />
   </ItemGroup>
   <ItemGroup>

Gibe.AbTest.Tests/packages.config

@@ -1,6 +1,8 @@
 <?xml version="1.0" encoding="utf-8"?>
 <packages>
+  <package id="Gibe.Cookies" version="1.0.356" targetFramework="net452" />
   <package id="Gibe.NPoco" version="1.0.159" targetFramework="net452" />
+  <package id="Moq" version="4.2.1502.0911" targetFramework="net452" />
   <package id="NPoco" version="2.5.77" targetFramework="net452" />
   <package id="NUnit" version="3.4.1" targetFramework="net452" />
 </packages>