Add test case for #3290

Author: fredericDelaporte

src/NHibernate.Test/NHSpecificTest/GH3290/Entity.cs

@@ -0,0 +1,13 @@
+using System;
+using System.Collections.Generic;
+
+namespace NHibernate.Test.NHSpecificTest.GH3290
+{
+	class Entity
+	{
+		public virtual Guid Id { get; set; }
+		public virtual string Name { get; set; }
+		public virtual ISet<Entity> Parents { get; set; }
+		public virtual ISet<Entity> Children { get; set; }
+	}
+}

src/NHibernate.Test/NHSpecificTest/GH3290/Fixture.cs

@@ -0,0 +1,131 @@
+using System.Collections.Generic;
+using NHibernate.Cfg;
+using NHibernate.Cfg.MappingSchema;
+using NHibernate.Mapping.ByCode;
+using NHibernate.Transform;
+using NUnit.Framework;
+
+namespace NHibernate.Test.NHSpecificTest.GH3290
+{
+	[TestFixture]
+	public class Fixture : TestCaseMappingByCode
+	{
+		protected override HbmMapping GetMappings()
+		{
+			var mapper = new ModelMapper();
+			mapper.Class<Entity>(rc =>
+			{
+				rc.Id(x => x.Id, map => map.Generator(Generators.GuidComb));
+
+				rc.Property(
+					x => x.Name
+				);
+
+				rc.Set(
+					x => x.Children,
+					v =>
+					{
+						v.Table("EntityToEntity");
+						v.Cascade(Mapping.ByCode.Cascade.None);
+						v.Inverse(true);
+						v.Key(x =>
+						{
+							x.Column("ParentId");
+							x.NotNullable(true);
+						});
+						v.Lazy(CollectionLazy.Lazy);
+						v.Fetch(CollectionFetchMode.Join);
+					},
+					h => h.ManyToMany(m => m.Column("ChildId"))
+				);
+
+				rc.Set(
+					x => x.Parents,
+					v =>
+					{
+						v.Table("EntityToEntity");
+						v.Cascade(Mapping.ByCode.Cascade.All);
+
+						v.Key(x =>
+						{
+							x.Column("ChildId");
+							x.NotNullable(true);
+						});
+						v.Lazy(CollectionLazy.Lazy);
+						v.Fetch(CollectionFetchMode.Join);
+					},
+					h => h.ManyToMany(m => m.Column("ParentId"))
+				);
+			});
+
+			return mapper.CompileMappingForAllExplicitlyAddedEntities();
+		}
+
+		protected override void Configure(Configuration configuration)
+		{
+			// Workaround fo the test case:
+			/*
+			configuration.SetProperty(Environment.DetectFetchLoops, "false");
+			configuration.SetProperty(Environment.MaxFetchDepth, "2");
+			*/
+		}
+
+		protected override void OnSetUp()
+		{
+			using var session = OpenSession();
+			using var transaction = session.BeginTransaction();
+
+			var person = new Entity
+			{
+				Name = "pers",
+				Parents = new HashSet<Entity>()
+			};
+			session.Save(person);
+			var job = new Entity
+			{
+				Name = "job",
+				Children = new HashSet<Entity>()
+			};
+			session.Save(job);
+
+			job.Children.Add(person);
+			person.Parents.Add(job);
+
+			transaction.Commit();
+		}
+
+		protected override void OnTearDown()
+		{
+			using var session = OpenSession();
+			using var transaction = session.BeginTransaction();
+
+			session.CreateSQLQuery("delete EntityToEntity").ExecuteUpdate();
+			session.CreateQuery("delete from System.Object").ExecuteUpdate();
+
+			transaction.Commit();
+		}
+
+		[Test]
+		public void QueryWithFetch()
+		{
+			using var session = OpenSession();
+			using var _ = session.BeginTransaction();
+
+			var all = session
+				.QueryOver<Entity>()
+				.Fetch(SelectMode.Fetch, x => x.Children)
+				.Fetch(SelectMode.Fetch, x => x.Parents)
+				.TransformUsing(Transformers.DistinctRootEntity)
+				.List();
+
+			foreach (var entity in all)
+			{
+				var isPerson = entity.Name == "pers";
+				if (isPerson)
+					Assert.That(entity.Parents, Has.Count.EqualTo(1), "Person's job not found or non-unique.");
+				else
+					Assert.That(entity.Children, Has.Count.EqualTo(1), "Job's employee not found or non-unique.");
+			}
+		}
+	}
+}