Changed DB to store node positions as Ordinals.

sql/connection_unittest.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "base/file_util.h"
 #include "base/scoped_temp_dir.h"
 #include "sql/connection.h"
 #include "sql/statement.h"
 #include "sql/meta_table.h"
 #include "testing/gtest/include/gtest/gtest.h"
@@ skipping 92 matching lines @@
   EXPECT_FALSE(db().DoesTableExist("FOO"));
 
   // Test DoesColumnExist.
   EXPECT_FALSE(db().DoesColumnExist("foo", "bar"));
   EXPECT_TRUE(db().DoesColumnExist("foo", "a"));
 
   // Testing for a column on a nonexistent table.
   EXPECT_FALSE(db().DoesColumnExist("bar", "b"));
 }
 
+TEST_F(SQLConnectionTest, GetColumnType) {
+  ASSERT_TRUE(db().Execute("CREATE TABLE foo (a bigint, b varchar)"));
+
+  const char *table_name = "foo";
+  EXPECT_EQ(db().GetColumnType(table_name, 0), "bigint");
+  EXPECT_EQ(db().GetColumnType(table_name, 1), "varchar");

[Scott Hess - ex-Googler, 2012/10/02 03:58:47]

"bigint" and "varchar" are meaningless to SQLite, these are INTEGER and TEXT. 
The fact that you need to depend on these means that you've been doing something
wrong.  I don't think it is appropriate to encourage people to expect
GetColumnType() to return anything other than the five native types.

+}
+
 TEST_F(SQLConnectionTest, GetLastInsertRowId) {
   ASSERT_TRUE(db().Execute("CREATE TABLE foo (id INTEGER PRIMARY KEY, value)"));
 
   ASSERT_TRUE(db().Execute("INSERT INTO foo (value) VALUES (12)"));
 
   // Last insert row ID should be valid.
   int64 row = db().GetLastInsertRowId();
   EXPECT_LT(0, row);
 
   // It should be the primary key of the row we just inserted.
@@ skipping 165 matching lines @@
   // Otherwise, sqlite3 doesn't find the correct directory to store
   // temporary files and will report the error 'unable to open
   // database file'.
   ASSERT_TRUE(meta_table.Init(&db(), 4, 4));
 }
 #endif
 
 // TODO(shess): Spin up a background thread to hold other_db, to more
 // closely match real life.  That would also allow testing
 // RazeWithTimeout().