From d1769e2b963191b86325e74939659b2cdc1aae00 Mon Sep 17 00:00:00 2001
From: john-mcpeek <juan.mcpeek@gmail.com>
Date: Fri, 13 Dec 2013 23:04:25 -0500
Subject: [PATCH] Moved static methods out of TableOperations.java and fixed
 junit imports.

---
 .../keyspace/AlterTableSpecification.java     |  8 +++
 .../keyspace/CreateTableSpecification.java    |  8 +++
 .../core/keyspace/DropTableSpecification.java |  8 +++
 .../core/keyspace/TableOperations.java        | 49 -------------------
 .../unit/core/cql/CqlStringUtilsTest.java     |  3 +-
 .../AlterTableCqlGeneratorTests.java          |  5 +-
 .../CreateTableCqlGeneratorTests.java         |  7 ++-
 .../generator/DropTableCqlGeneratorTests.java |  5 +-
 .../test/unit/core/keyspace/OptionTest.java   |  4 +-
 9 files changed, 33 insertions(+), 64 deletions(-)
 delete mode 100644 spring-cassandra/src/main/java/org/springframework/cassandra/core/keyspace/TableOperations.java

diff --git a/spring-cassandra/src/main/java/org/springframework/cassandra/core/keyspace/AlterTableSpecification.java b/spring-cassandra/src/main/java/org/springframework/cassandra/core/keyspace/AlterTableSpecification.java
index 80db2ce22..5e7530887 100644
--- a/spring-cassandra/src/main/java/org/springframework/cassandra/core/keyspace/AlterTableSpecification.java
+++ b/spring-cassandra/src/main/java/org/springframework/cassandra/core/keyspace/AlterTableSpecification.java
@@ -63,4 +63,12 @@ public class AlterTableSpecification extends TableOptionsSpecification<AlterTabl
 	public List<ColumnChangeSpecification> getChanges() {
 		return Collections.unmodifiableList(changes);
 	}
+
+	/**
+	 * Entry point into the {@link AlterTableSpecification}'s fluent API to alter a table. Convenient if imported
+	 * statically.
+	 */
+	public static AlterTableSpecification alterTable() {
+		return new AlterTableSpecification();
+	}
 }
diff --git a/spring-cassandra/src/main/java/org/springframework/cassandra/core/keyspace/CreateTableSpecification.java b/spring-cassandra/src/main/java/org/springframework/cassandra/core/keyspace/CreateTableSpecification.java
index b1b912142..0d8924ee3 100644
--- a/spring-cassandra/src/main/java/org/springframework/cassandra/core/keyspace/CreateTableSpecification.java
+++ b/spring-cassandra/src/main/java/org/springframework/cassandra/core/keyspace/CreateTableSpecification.java
@@ -46,4 +46,12 @@ public class CreateTableSpecification extends TableSpecification<CreateTableSpec
 	public boolean getIfNotExists() {
 		return ifNotExists;
 	}
+
+	/**
+	 * Entry point into the {@link CreateTableSpecification}'s fluent API to create a table. Convenient if imported
+	 * statically.
+	 */
+	public static CreateTableSpecification createTable() {
+		return new CreateTableSpecification();
+	}
 }
diff --git a/spring-cassandra/src/main/java/org/springframework/cassandra/core/keyspace/DropTableSpecification.java b/spring-cassandra/src/main/java/org/springframework/cassandra/core/keyspace/DropTableSpecification.java
index 33986d767..f9f7afd52 100644
--- a/spring-cassandra/src/main/java/org/springframework/cassandra/core/keyspace/DropTableSpecification.java
+++ b/spring-cassandra/src/main/java/org/springframework/cassandra/core/keyspace/DropTableSpecification.java
@@ -36,4 +36,12 @@ public class DropTableSpecification extends TableNameSpecification<DropTableSpec
 	public boolean getIfExists() {
 		return ifExists;
 	}
+
+	/**
+	 * Entry point into the {@link DropTableSpecification}'s fluent API to drop a table. Convenient if imported
+	 * statically.
+	 */
+	public static DropTableSpecification dropTable() {
+		return new DropTableSpecification();
+	}
 }
diff --git a/spring-cassandra/src/main/java/org/springframework/cassandra/core/keyspace/TableOperations.java b/spring-cassandra/src/main/java/org/springframework/cassandra/core/keyspace/TableOperations.java
deleted file mode 100644
index 57543e49f..000000000
--- a/spring-cassandra/src/main/java/org/springframework/cassandra/core/keyspace/TableOperations.java
+++ /dev/null
@@ -1,49 +0,0 @@
-/*
- * Copyright 2011-2013 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.
- * You may obtain a copy of the License at
- * 
- *      http://www.apache.org/licenses/LICENSE-2.0
- * 
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-package org.springframework.cassandra.core.keyspace;
-
-/**
- * Class that offers static methods as entry points into the fluent API for building create, drop and alter table
- * specifications. These methods are most convenient when imported statically.
- * 
- * @author Matthew T. Adams
- */
-public class TableOperations {
-
-	/**
-	 * Entry point into the {@link CreateTableSpecification}'s fluent API to create a table. Convenient if imported
-	 * statically.
-	 */
-	public static CreateTableSpecification createTable() {
-		return new CreateTableSpecification();
-	}
-
-	/**
-	 * Entry point into the {@link DropTableSpecification}'s fluent API to drop a table. Convenient if imported
-	 * statically.
-	 */
-	public static DropTableSpecification dropTable() {
-		return new DropTableSpecification();
-	}
-
-	/**
-	 * Entry point into the {@link AlterTableSpecification}'s fluent API to alter a table. Convenient if imported
-	 * statically.
-	 */
-	public static AlterTableSpecification alterTable() {
-		return new AlterTableSpecification();
-	}
-}
\ No newline at end of file
diff --git a/spring-cassandra/src/test/java/org/springframework/cassandra/test/unit/core/cql/CqlStringUtilsTest.java b/spring-cassandra/src/test/java/org/springframework/cassandra/test/unit/core/cql/CqlStringUtilsTest.java
index a8c97c3e1..e54d7b29b 100644
--- a/spring-cassandra/src/test/java/org/springframework/cassandra/test/unit/core/cql/CqlStringUtilsTest.java
+++ b/spring-cassandra/src/test/java/org/springframework/cassandra/test/unit/core/cql/CqlStringUtilsTest.java
@@ -1,7 +1,6 @@
 package org.springframework.cassandra.test.unit.core.cql;
 
-import static junit.framework.Assert.assertFalse;
-import static junit.framework.Assert.assertTrue;
+import static org.junit.Assert.*;
 import static org.springframework.cassandra.core.cql.CqlStringUtils.isQuotedIdentifier;
 import static org.springframework.cassandra.core.cql.CqlStringUtils.isUnquotedIdentifier;
 
diff --git a/spring-cassandra/src/test/java/org/springframework/cassandra/test/unit/core/cql/generator/AlterTableCqlGeneratorTests.java b/spring-cassandra/src/test/java/org/springframework/cassandra/test/unit/core/cql/generator/AlterTableCqlGeneratorTests.java
index 420c7b35d..da0167667 100644
--- a/spring-cassandra/src/test/java/org/springframework/cassandra/test/unit/core/cql/generator/AlterTableCqlGeneratorTests.java
+++ b/spring-cassandra/src/test/java/org/springframework/cassandra/test/unit/core/cql/generator/AlterTableCqlGeneratorTests.java
@@ -1,7 +1,6 @@
 package org.springframework.cassandra.test.unit.core.cql.generator;
 
-import static junit.framework.Assert.assertTrue;
-import static org.springframework.cassandra.core.keyspace.TableOperations.alterTable;
+import static org.junit.Assert.*;
 
 import org.junit.Test;
 import org.springframework.cassandra.core.cql.generator.AlterTableCqlGenerator;
@@ -46,7 +45,7 @@ public class AlterTableCqlGeneratorTests {
 		public String dropped = "dropped";
 
 		public AlterTableSpecification specification() {
-			return alterTable().name(name).alter(altered, alteredType).add(added, addedType).drop(dropped);
+			return AlterTableSpecification.alterTable().name(name).alter(altered, alteredType).add(added, addedType).drop(dropped);
 		}
 
 		public AlterTableCqlGenerator generator() {
diff --git a/spring-cassandra/src/test/java/org/springframework/cassandra/test/unit/core/cql/generator/CreateTableCqlGeneratorTests.java b/spring-cassandra/src/test/java/org/springframework/cassandra/test/unit/core/cql/generator/CreateTableCqlGeneratorTests.java
index 7ebd71580..0f3b37d1b 100644
--- a/spring-cassandra/src/test/java/org/springframework/cassandra/test/unit/core/cql/generator/CreateTableCqlGeneratorTests.java
+++ b/spring-cassandra/src/test/java/org/springframework/cassandra/test/unit/core/cql/generator/CreateTableCqlGeneratorTests.java
@@ -1,7 +1,6 @@
 package org.springframework.cassandra.test.unit.core.cql.generator;
 
-import static junit.framework.Assert.assertTrue;
-import static org.springframework.cassandra.core.keyspace.TableOperations.createTable;
+import static org.junit.Assert.*;
 
 import org.junit.Test;
 import org.springframework.cassandra.core.cql.generator.CreateTableCqlGenerator;
@@ -57,7 +56,7 @@ public class CreateTableCqlGeneratorTests {
 		public String column1 = "column1";
 
 		public CreateTableSpecification specification() {
-			return createTable().name(name).partitionKeyColumn(partitionKey0, partitionKeyType0).column(column1, columnType1);
+			return CreateTableSpecification.createTable().name(name).partitionKeyColumn(partitionKey0, partitionKeyType0).column(column1, columnType1);
 		}
 
 		@Test
@@ -82,7 +81,7 @@ public class CreateTableCqlGeneratorTests {
 
 		@Override
 		public CreateTableSpecification specification() {
-			return createTable().name(name).partitionKeyColumn(partKey0, partKeyType0)
+			return CreateTableSpecification.createTable().name(name).partitionKeyColumn(partKey0, partKeyType0)
 					.partitionKeyColumn(partKey1, partKeyType1).column(column0, columnType0);
 		}
 
diff --git a/spring-cassandra/src/test/java/org/springframework/cassandra/test/unit/core/cql/generator/DropTableCqlGeneratorTests.java b/spring-cassandra/src/test/java/org/springframework/cassandra/test/unit/core/cql/generator/DropTableCqlGeneratorTests.java
index 330fe7382..87d63a1bd 100644
--- a/spring-cassandra/src/test/java/org/springframework/cassandra/test/unit/core/cql/generator/DropTableCqlGeneratorTests.java
+++ b/spring-cassandra/src/test/java/org/springframework/cassandra/test/unit/core/cql/generator/DropTableCqlGeneratorTests.java
@@ -1,7 +1,6 @@
 package org.springframework.cassandra.test.unit.core.cql.generator;
 
-import static junit.framework.Assert.assertTrue;
-import static org.springframework.cassandra.core.keyspace.TableOperations.dropTable;
+import static org.junit.Assert.*;
 
 import org.junit.Test;
 import org.springframework.cassandra.core.cql.generator.DropTableCqlGenerator;
@@ -37,7 +36,7 @@ public class DropTableCqlGeneratorTests {
 		public String name = "mytable";
 
 		public DropTableSpecification specification() {
-			return dropTable().name(name);
+			return DropTableSpecification.dropTable().name(name);
 		}
 
 		public DropTableCqlGenerator generator() {
diff --git a/spring-cassandra/src/test/java/org/springframework/cassandra/test/unit/core/keyspace/OptionTest.java b/spring-cassandra/src/test/java/org/springframework/cassandra/test/unit/core/keyspace/OptionTest.java
index 1d2eff43e..e5a8a3345 100644
--- a/spring-cassandra/src/test/java/org/springframework/cassandra/test/unit/core/keyspace/OptionTest.java
+++ b/spring-cassandra/src/test/java/org/springframework/cassandra/test/unit/core/keyspace/OptionTest.java
@@ -1,8 +1,6 @@
 package org.springframework.cassandra.test.unit.core.keyspace;
 
-import static junit.framework.Assert.assertEquals;
-import static junit.framework.Assert.assertFalse;
-import static junit.framework.Assert.assertTrue;
+import static org.junit.Assert.*;
 
 import java.lang.annotation.RetentionPolicy;