From d149585f29e69be7b49dca3217389bbad1d3f24c Mon Sep 17 00:00:00 2001
From: Lukas Eder <lukas.eder@gmail.com>
Date: Sat, 7 Dec 2013 12:57:33 +0100
Subject: [PATCH] [#2836] Bad values are stored in record when projecting
 "ambiguous" column names into a generated record - Added failing integration
 test

---
 .../org/jooq/test/_/testcases/FetchTests.java | 23 ++++++++++++++++---
 .../src/org/jooq/test/jOOQAbstractTest.java   |  5 ++++
 2 files changed, 25 insertions(+), 3 deletions(-)

diff --git a/jOOQ-test/src/org/jooq/test/_/testcases/FetchTests.java b/jOOQ-test/src/org/jooq/test/_/testcases/FetchTests.java
index 33e7be40f3..cb4a1f2972 100644
--- a/jOOQ-test/src/org/jooq/test/_/testcases/FetchTests.java
+++ b/jOOQ-test/src/org/jooq/test/_/testcases/FetchTests.java
@@ -1220,6 +1220,23 @@ extends BaseTest<A, AP, B, S, B2S, BS, L, X, DATE, BOOL, D, T, U, UU, I, IPK, T7
         }
     }
 
+    @Test
+    public void testFetchIntoTableRecordsWithColumnAmbiguities() throws Exception {
+
+        // [#2836] When fetching into TableRecord types, the "expected" behaviour would
+        // be to map "obvious" columns onto their corresponding values
+        // Fixing this might be a "dangerous" change.
+        B book =
+        create().select()
+                .from(TBook())
+                .join(TAuthor())
+                .on(TBook_AUTHOR_ID().eq(TAuthor_ID()))
+                .where(TBook_ID().eq(4))
+                .fetchOneInto(TBook().getRecordType());
+
+        assertEquals(4, (int) book.getValue(TBook_ID()));
+    }
+
     @Test
     public void testFetchAttachables() throws Exception {
         // [#2869] DefaultRecordMapper should recognise Attachable types and attach them
@@ -1227,14 +1244,14 @@ extends BaseTest<A, AP, B, S, B2S, BS, L, X, DATE, BOOL, D, T, U, UU, I, IPK, T7
 
         B b1 = create().selectFrom(TBook()).where(TBook_ID().eq(1)).fetchOne();
         B b2 = b1.into(TBook().getRecordType());
-        
+
         assertNotNull(((AttachableInternal) b1).configuration());
         assertNotNull(((AttachableInternal) b2).configuration());
-        
+
         B b3 = create(new Settings().withAttachRecords(false))
                     .selectFrom(TBook()).where(TBook_ID().eq(1)).fetchOne();
         B b4 = b3.into(TBook().getRecordType());
-        
+
         assertNull(((AttachableInternal) b3).configuration());
         assertNull(((AttachableInternal) b4).configuration());
     }
diff --git a/jOOQ-test/src/org/jooq/test/jOOQAbstractTest.java b/jOOQ-test/src/org/jooq/test/jOOQAbstractTest.java
index 4785d1cc8b..623162fa51 100644
--- a/jOOQ-test/src/org/jooq/test/jOOQAbstractTest.java
+++ b/jOOQ-test/src/org/jooq/test/jOOQAbstractTest.java
@@ -1496,6 +1496,11 @@ public abstract class jOOQAbstractTest<
         new FetchTests(this).testFetchIntoTableRecords();
     }
 
+    @Test
+    public void testFetchIntoTableRecordsWithColumnAmbiguities() throws Exception {
+        new FetchTests(this).testFetchIntoTableRecordsWithColumnAmbiguities();
+    }
+
     @Test
     public void testFetchAttachables() throws Exception {
         new FetchTests(this).testFetchAttachables();