Merge pull request #230 from jeffgbutler/master

Change AllColumns to a basic column

Author: jeffgbutler

src/main/java/org/mybatis/dynamic/sql/SqlTable.java

@@ -1,5 +1,5 @@
 /**
- *    Copyright 2016-2019 the original author or authors.
+ *    Copyright 2016-2020 the original author or authors.
  *
  *    Licensed under the Apache License, Version 2.0 (the "License");
  *    you may not use this file except in compliance with the License.
@@ -82,7 +82,7 @@ public String tableNameAtRuntime() {
         return nameSupplier.get();
     }
 
-    public <T> SqlColumn<T> allColumns() {
+    public BasicColumn allColumns() {
         return SqlColumn.of("*", this); //$NON-NLS-1$
     }
 

src/test/kotlin/examples/kotlin/spring/canonical/CanonicalSpringKotlinTemplateDirectTest.kt

@@ -314,6 +314,17 @@ class CanonicalSpringKotlinTemplateDirectTest {
         assertThat(rows).hasSize(6)
     }
 
+    @Test
+    fun testSelectAllWithSelectStar() {
+        val rows = template.select(Person.allColumns())
+            .from(Person) {
+                allRows()
+                orderBy(id)
+            }.withRowMapper(personRowMapper)
+
+        assertThat(rows).hasSize(6)
+    }
+
     @Test
     fun testSelect() {
         val rows = template.select(