Drop cross-project references from ORM test comments

Author: mikasoukhov

Tests/Data/ExpressionQueryTranslatorTests.cs

@@ -770,13 +770,13 @@ public void StringConstant_WithMultipleSingleQuotes_ShouldBeParameterized()
 
 	#region Closure-capture Chain Parameterization Tests
 
-	// Reproduces the EXACT closure shape generated by an async method with two levels
-	// of nested if-blocks that capture an outer-method local. Mirrors
-	// <c>BrokerPortfolioRepository.SearchAsync</c>: outer state-machine field
-	// (tenantId), first if-block DC (referencing the state-machine), and a nested
-	// else-block DC (referencing the first DC). The lambda inside the inner block
-	// then reads tenantId through a 3-link chain: innerDC → outerDC → methodDC →
-	// tenantId. The compiler-generated tree shape is:
+	// Reproduces the closure shape generated by an async method with two levels
+	// of nested if-blocks that capture an outer-method local: outer
+	// state-machine field (tenantId), first if-block DC (referencing the
+	// state-machine), and a nested else-block DC (referencing the first DC).
+	// The lambda inside the inner block then reads tenantId through a 3-link
+	// chain: innerDC → outerDC → methodDC → tenantId. The compiler-generated
+	// tree shape is:
 	//   MemberAccess(tenantId)
 	//     MemberAccess(CS$&lt;&gt;8__locals1)
 	//       MemberAccess(CS$&lt;&gt;8__locals2)
@@ -962,11 +962,11 @@ public void Where_NavigationProperty_OrContainsFk_EmitsJoin()
 	}
 
 	/// <summary>
-	/// Reproduction of the downstream Broker failure: an entity with
-	/// inheritance (TestBrokerBase) carrying [RelationSingle] audit FKs
-	/// (CreatedBy/ModifiedBy → TestBrokerUser) plus another [RelationSingle]
-	/// User on the derived entity. Where-predicate on User.Email must emit
-	/// the INNER JOIN to TestBrokerUser, otherwise SQL Server reports
+	/// Inheritance + RelationSingle interaction: an entity with
+	/// <see cref="TestBrokerBase"/> audit FKs (CreatedBy/ModifiedBy →
+	/// <see cref="TestBrokerUser"/>) plus another [RelationSingle] User on
+	/// the derived entity. Where-predicate on User.Email must emit the INNER
+	/// JOIN, otherwise SQL Server reports
 	/// "multi-part identifier 'User.Email' could not be bound".
 	/// </summary>
 	[TestMethod]
@@ -990,12 +990,12 @@ public void Where_BrokerPortfolio_NavigationToUserEmail_EmitsJoin()
 	}
 
 	/// <summary>
-	/// Reproduction of the original Broker bug: SearchAsync built a single
-	/// Where with three OR'ed Like predicates over p.User.Email / FirstName /
-	/// LastName. ORM 1.0.29 translated this without the implicit JOIN, so
-	/// SQL Server failed with "multi-part identifier 'User.Email' could not
-	/// be bound". A single INNER JOIN must be emitted, and all three OR
-	/// branches must reference the joined alias.
+	/// A single Where with three OR'ed Like predicates over the same
+	/// navigation (p.User.Email / FirstName / LastName) must emit a single
+	/// INNER JOIN, and all three OR branches must reference the joined
+	/// alias. Earlier the translator skipped the implicit JOIN and SQL
+	/// Server failed with "multi-part identifier 'User.Email' could not be
+	/// bound".
 	/// </summary>
 	[TestMethod]
 	public void Where_BrokerPortfolio_OrAcrossUserColumns_EmitsSingleJoin()
@@ -1017,8 +1017,7 @@ public void Where_BrokerPortfolio_OrAcrossUserColumns_EmitsSingleJoin()
 		sql.Contains("[Ecng_TestBrokerUser]").AssertTrue(
 			$"Expected join target [Ecng_TestBrokerUser], got: {sql}");
 
-		// All three branches must qualify columns with the joined alias —
-		// the buggy translator emitted bare [Email]/[FirstName]/[LastName].
+		// All three branches must qualify columns with the joined alias.
 		sql.Contains("[User].[Email]").AssertTrue(
 			$"Expected qualified [User].[Email], got: {sql}");
 		sql.Contains("[User].[FirstName]").AssertTrue(

Tests/Data/OrmTestEntities.cs

@@ -364,8 +364,8 @@ public async ValueTask LoadAsync(SettingsStorage storage, IStorage db, Cancellat
 }
 
 /// <summary>
-/// Mirrors BaseBrokerEntity from the Broker project: abstract base with
-/// audit FKs as RelationSingle navigation properties.
+/// Abstract base with two RelationSingle audit FKs (Created/Modified) for
+/// regression tests around FK-cycle resolution and inheritance-driven joins.
 /// </summary>
 public abstract class TestBrokerBase : IDbPersistable
 {

Tests/Data/QueryTests.cs

@@ -93,12 +93,10 @@ public void Select_Top()
 	}
 
 	/// <summary>
-	/// Repro for downstream Broker bug: when an entity has [RelationSingle] User,
-	/// ExpressionQueryTranslator emits an INNER JOIN with alias = nav member name
-	/// ("User"). Query.Table(name, alias) currently appends the alias *unquoted*,
-	/// so PostgreSQL receives <c>"User" User</c> and fails with
-	/// <c>42601: syntax error at or near "User"</c>. Aliases must be quoted just
-	/// like table names so reserved words don't break the parser.
+	/// Aliases that collide with reserved words (e.g. <c>User</c> on PostgreSQL)
+	/// must be quoted just like table names; otherwise the joined SQL ends up
+	/// with bare <c>"User" User</c> and PostgreSQL reports
+	/// <c>42601: syntax error at or near "User"</c>.
 	/// </summary>
 	[TestMethod]
 	[DataRow("SqlServer")]