activerecord-postgresql-extensions 0.0.10 → 0.0.11

data/test/{functions_test.rb → functions_tests.rb}

@@ -28,7 +28,7 @@ class FunctionsTests < Test::Unit::TestCase
 %{CREATE FUNCTION "test"(integer) RETURNS integer AS $$
 select 10;
 $$
-LANGUAGE "sql"},
+LANGUAGE "sql";},
 
 %{CREATE OR REPLACE FUNCTION "test"(integer) RETURNS integer AS $__$
 return 10;
@@ -38,7 +38,7 @@ LANGUAGE "sql"
     STRICT
     COST 1
     ROWS 10
-    SET TIME ZONE "America/Halifax"}
+    SET TIME ZONE "America/Halifax";}
     ], statements)
   end
 
@@ -47,8 +47,8 @@ LANGUAGE "sql"
     ARBC.drop_function(:test, :integer, :if_exists => true, :cascade => true)
 
     assert_equal([
-      "DROP FUNCTION \"test\"(integer)",
-      "DROP FUNCTION IF EXISTS \"test\"(integer) CASCADE"
+      "DROP FUNCTION \"test\"(integer);",
+      "DROP FUNCTION IF EXISTS \"test\"(integer) CASCADE;"
     ], statements)
   end
 
@@ -56,7 +56,7 @@ LANGUAGE "sql"
     ARBC.rename_function(:test, 'integer, text', :foo)
 
     assert_equal([
-      "ALTER FUNCTION \"test\"(integer, text) RENAME TO \"foo\""
+      "ALTER FUNCTION \"test\"(integer, text) RENAME TO \"foo\";"
     ], statements)
   end
 
@@ -64,7 +64,7 @@ LANGUAGE "sql"
     ARBC.alter_function_owner(:test, 'integer, text', :admin)
 
     assert_equal([
-      "ALTER FUNCTION \"test\"(integer, text) OWNER TO \"admin\""
+      "ALTER FUNCTION \"test\"(integer, text) OWNER TO \"admin\";"
     ], statements)
   end
 
@@ -72,7 +72,7 @@ LANGUAGE "sql"
     ARBC.alter_function_schema(:test, 'integer, text', :geospatial)
 
     assert_equal([
-      "ALTER FUNCTION \"test\"(integer, text) SET SCHEMA \"geospatial\""
+      "ALTER FUNCTION \"test\"(integer, text) SET SCHEMA \"geospatial\";"
     ], statements)
   end
 
@@ -95,8 +95,8 @@ LANGUAGE "sql"
     end
 
     assert_equal([
-%{ALTER FUNCTION "my_function"(integer) RENAME TO "another_function"},
-%{ALTER FUNCTION "another_function"(integer) OWNER TO "jdoe"},
+%{ALTER FUNCTION "my_function"(integer) RENAME TO "another_function";},
+%{ALTER FUNCTION "another_function"(integer) OWNER TO "jdoe";},
 %{ALTER FUNCTION "my_function"(integer) RENAME TO "another_function";
 ALTER FUNCTION "another_function"(integer) OWNER TO "jdoe";
 ALTER FUNCTION "another_function"(integer) SET SCHEMA "foo";
@@ -106,7 +106,7 @@ ALTER FUNCTION "another_function"(integer) COST 10;
 ALTER FUNCTION "another_function"(integer) ROWS 10;
 ALTER FUNCTION "another_function"(integer) SET "log_duration" TO "0.4";
 ALTER FUNCTION "another_function"(integer) RESET ALL;
-ALTER FUNCTION "another_function"(integer) RESET "debug_assertions" RESET "trace_notify"}
+ALTER FUNCTION "another_function"(integer) RESET "debug_assertions" RESET "trace_notify";}
     ], statements)
   end
 end

data/test/{geometry_test.rb → geometry_tests.rb}

@@ -16,10 +16,10 @@ class GeometryTests < Test::Unit::TestCase
   "the_geom" geometry,
   CONSTRAINT "enforce_srid_the_geom" CHECK (srid("the_geom") = (4326)),
   CONSTRAINT "enforce_dims_the_geom" CHECK (ndims("the_geom") = 2)
-)},
-  %{DELETE FROM "geometry_columns" WHERE f_table_catalog = '' AND f_table_schema = 'public' AND f_table_name = 'foo' AND f_geometry_column = 'the_geom'},
-  %{INSERT INTO "geometry_columns" VALUES ('', 'public', 'foo', 'the_geom', 2, 4326, 'GEOMETRY')},
-  %{CREATE INDEX "foo_the_geom_gist_index" ON "foo" USING "gist"("the_geom")}
+);},
+  %{DELETE FROM "geometry_columns" WHERE f_table_catalog = '' AND f_table_schema = 'public' AND f_table_name = 'foo' AND f_geometry_column = 'the_geom';},
+  %{INSERT INTO "geometry_columns" VALUES ('', 'public', 'foo', 'the_geom', 2, 4326, 'GEOMETRY');},
+  %{CREATE INDEX "foo_the_geom_gist_index" ON "foo" USING "gist"("the_geom");}
     ], statements)
   end
 
@@ -34,10 +34,10 @@ class GeometryTests < Test::Unit::TestCase
   "the_geom" geometry,
   CONSTRAINT "enforce_srid_the_geom" CHECK (srid("the_geom") = (4326)),
   CONSTRAINT "enforce_dims_the_geom" CHECK (ndims("the_geom") = 2)
-)},
-  %{DELETE FROM "geometry_columns" WHERE f_table_catalog = '' AND f_table_schema = 'public' AND f_table_name = 'foo' AND f_geometry_column = 'the_geom'},
-  %{INSERT INTO "geometry_columns" VALUES ('', 'public', 'foo', 'the_geom', 2, 4326, 'GEOMETRY')},
-  %{CREATE INDEX "foo_the_geom_gist_index" ON "foo" USING "gist"("the_geom")}
+);},
+  %{DELETE FROM "geometry_columns" WHERE f_table_catalog = '' AND f_table_schema = 'public' AND f_table_name = 'foo' AND f_geometry_column = 'the_geom';},
+  %{INSERT INTO "geometry_columns" VALUES ('', 'public', 'foo', 'the_geom', 2, 4326, 'GEOMETRY');},
+  %{CREATE INDEX "foo_the_geom_gist_index" ON "foo" USING "gist"("the_geom");}
     ], statements)
   end
 
@@ -52,10 +52,10 @@ class GeometryTests < Test::Unit::TestCase
   "the_geom" geometry NOT NULL,
   CONSTRAINT "enforce_srid_the_geom" CHECK (srid("the_geom") = (4326)),
   CONSTRAINT "enforce_dims_the_geom" CHECK (ndims("the_geom") = 2)
-)},
-  %{DELETE FROM "geometry_columns" WHERE f_table_catalog = '' AND f_table_schema = 'public' AND f_table_name = 'foo' AND f_geometry_column = 'the_geom'},
-  %{INSERT INTO "geometry_columns" VALUES ('', 'public', 'foo', 'the_geom', 2, 4326, 'GEOMETRY')},
-  %{CREATE INDEX "foo_the_geom_gist_index" ON "foo" USING "gist"("the_geom")}
+);},
+  %{DELETE FROM "geometry_columns" WHERE f_table_catalog = '' AND f_table_schema = 'public' AND f_table_name = 'foo' AND f_geometry_column = 'the_geom';},
+  %{INSERT INTO "geometry_columns" VALUES ('', 'public', 'foo', 'the_geom', 2, 4326, 'GEOMETRY');},
+  %{CREATE INDEX "foo_the_geom_gist_index" ON "foo" USING "gist"("the_geom");}
     ], statements)
   end
 
@@ -71,10 +71,10 @@ class GeometryTests < Test::Unit::TestCase
   CONSTRAINT "enforce_srid_the_geom" CHECK (srid("the_geom") = (4326)),
   CONSTRAINT "enforce_dims_the_geom" CHECK (ndims("the_geom") = 2),
   CONSTRAINT "enforce_geotype_the_geom" CHECK (geometrytype("the_geom") = 'POLYGON'::text OR "the_geom" IS NULL)
-)},
-  %{DELETE FROM "geometry_columns" WHERE f_table_catalog = '' AND f_table_schema = 'public' AND f_table_name = 'foo' AND f_geometry_column = 'the_geom'},
-  %{INSERT INTO "geometry_columns" VALUES ('', 'public', 'foo', 'the_geom', 2, 4326, 'POLYGON')},
-  %{CREATE INDEX "foo_the_geom_gist_index" ON "foo" USING "gist"("the_geom")}
+);},
+  %{DELETE FROM "geometry_columns" WHERE f_table_catalog = '' AND f_table_schema = 'public' AND f_table_name = 'foo' AND f_geometry_column = 'the_geom';},
+  %{INSERT INTO "geometry_columns" VALUES ('', 'public', 'foo', 'the_geom', 2, 4326, 'POLYGON');},
+  %{CREATE INDEX "foo_the_geom_gist_index" ON "foo" USING "gist"("the_geom");}
     ], statements)
   end
 end

data/test/{index_test.rb → index_tests.rb}

@@ -29,12 +29,12 @@ class IndexTests < Test::Unit::TestCase
     })
 
     assert_equal([
-      "CREATE INDEX \"foo_names_idx\" ON \"foo\"(\"first_name\", \"last_name\")",
-      "CREATE INDEX \"foo_bar_id_idx\" ON \"foo\"(\"bar_id\")",
-      "CREATE INDEX \"foo_coalesce_bar_id_idx\" ON \"foo\"((COALESCE(bar_id, 0)))",
-      "CREATE INDEX \"foo_search_idx\" ON \"foo\" USING \"gin\"(\"search\")",
-      "CREATE INDEX \"foo_names_idx\" ON \"foo\"(\"name\" \"text_pattern_ops\")",
-      "CREATE UNIQUE INDEX CONCURRENTLY \"foo_bar_id_idx\" ON \"foo\"(\"bar_id\" ASC NULLS LAST) WITH (FILLFACTOR = 10) TABLESPACE \"fubar\" WHERE bar_id IS NOT NULL"
+      "CREATE INDEX \"foo_names_idx\" ON \"foo\"(\"first_name\", \"last_name\");",
+      "CREATE INDEX \"foo_bar_id_idx\" ON \"foo\"(\"bar_id\");",
+      "CREATE INDEX \"foo_coalesce_bar_id_idx\" ON \"foo\"((COALESCE(bar_id, 0)));",
+      "CREATE INDEX \"foo_search_idx\" ON \"foo\" USING \"gin\"(\"search\");",
+      "CREATE INDEX \"foo_names_idx\" ON \"foo\"(\"name\" \"text_pattern_ops\");",
+      "CREATE UNIQUE INDEX CONCURRENTLY \"foo_bar_id_idx\" ON \"foo\"(\"bar_id\" ASC NULLS LAST) WITH (FILLFACTOR = 10) TABLESPACE \"fubar\" WHERE bar_id IS NOT NULL;"
     ], statements)
   end
 
@@ -44,9 +44,9 @@ class IndexTests < Test::Unit::TestCase
     Mig.drop_index(:foo_names_idx, :cascade => true)
 
     assert_equal([
-      "DROP INDEX \"foo_names_idx\"",
-      "DROP INDEX IF EXISTS \"foo_names_idx\"",
-      "DROP INDEX \"foo_names_idx\" CASCADE"
+      "DROP INDEX \"foo_names_idx\";",
+      "DROP INDEX IF EXISTS \"foo_names_idx\";",
+      "DROP INDEX \"foo_names_idx\" CASCADE;"
     ], statements)
   end
 
@@ -54,7 +54,7 @@ class IndexTests < Test::Unit::TestCase
     Mig.rename_index(:foo_names_idx, :foo_renamed_idx)
 
     assert_equal([
-      "ALTER INDEX \"foo_names_idx\" RENAME TO \"foo_renamed_idx\""
+      "ALTER INDEX \"foo_names_idx\" RENAME TO \"foo_renamed_idx\";"
     ], statements)
   end
 
@@ -62,7 +62,7 @@ class IndexTests < Test::Unit::TestCase
     Mig.alter_index_tablespace(:foo_names_idx, :fubar)
 
     assert_equal([
-      "ALTER INDEX \"foo_names_idx\" SET TABLESPACE \"fubar\""
+      "ALTER INDEX \"foo_names_idx\" SET TABLESPACE \"fubar\";"
     ], statements)
   end
 end

data/test/{languages_test.rb → languages_tests.rb}

@@ -15,8 +15,8 @@ class LanguagesTests < Test::Unit::TestCase
     )
 
     assert_equal([
-      "CREATE PROCEDURAL LANGUAGE \"foo\"",
-      "CREATE TRUSTED PROCEDURAL LANGUAGE \"foo\" HANDLER \"plpgsql\" VALIDATOR test()"
+      "CREATE PROCEDURAL LANGUAGE \"foo\";",
+      "CREATE TRUSTED PROCEDURAL LANGUAGE \"foo\" HANDLER \"plpgsql\" VALIDATOR test();"
     ], statements)
   end
 
@@ -25,8 +25,8 @@ class LanguagesTests < Test::Unit::TestCase
     ARBC.drop_language(:foo, :if_exists => true, :cascade => true)
 
     assert_equal([
-      "DROP PROCEDURAL LANGUAGE \"foo\"",
-      "DROP PROCEDURAL LANGUAGE IF EXISTS \"foo\" CASCADE"
+      "DROP PROCEDURAL LANGUAGE \"foo\";",
+      "DROP PROCEDURAL LANGUAGE IF EXISTS \"foo\" CASCADE;"
     ], statements)
   end
 
@@ -34,7 +34,7 @@ class LanguagesTests < Test::Unit::TestCase
     ARBC.alter_language_name(:foo, :bar)
 
     assert_equal([
-      "ALTER PROCEDURAL LANGUAGE \"foo\" RENAME TO \"bar\""
+      "ALTER PROCEDURAL LANGUAGE \"foo\" RENAME TO \"bar\";"
     ], statements)
   end
 
@@ -42,7 +42,7 @@ class LanguagesTests < Test::Unit::TestCase
     ARBC.alter_language_owner(:foo, :bar)
 
     assert_equal([
-      "ALTER PROCEDURAL LANGUAGE \"foo\" OWNER TO \"bar\""
+      "ALTER PROCEDURAL LANGUAGE \"foo\" OWNER TO \"bar\";"
     ], statements)
   end
 end

data/test/{permissions_test.rb → permissions_tests.rb}

@@ -13,11 +13,11 @@ class PermissionsTests < Test::Unit::TestCase
     Mig.grant_table_privileges(:foo, :select, :public, :cascade => true)
 
     assert_equal([
-      "GRANT SELECT ON TABLE \"foo\" TO \"nobody\"",
-      "GRANT SELECT, UPDATE, DELETE, INSERT ON TABLE \"foo\" TO \"nobody\", \"somebody\"",
-      "GRANT SELECT ON TABLE \"foo\" TO \"nobody\" WITH GRANT OPTION",
-      "GRANT SELECT ON TABLE \"foo\" TO \"nobody\"",
-      "GRANT SELECT ON TABLE \"foo\" TO PUBLIC"
+      "GRANT SELECT ON TABLE \"foo\" TO \"nobody\";",
+      "GRANT SELECT, UPDATE, DELETE, INSERT ON TABLE \"foo\" TO \"nobody\", \"somebody\";",
+      "GRANT SELECT ON TABLE \"foo\" TO \"nobody\" WITH GRANT OPTION;",
+      "GRANT SELECT ON TABLE \"foo\" TO \"nobody\";",
+      "GRANT SELECT ON TABLE \"foo\" TO PUBLIC;"
     ], statements)
   end
 
@@ -29,11 +29,11 @@ class PermissionsTests < Test::Unit::TestCase
     Mig.revoke_table_privileges(:foo, :select, :public, :cascade => true)
 
     assert_equal([
-      "REVOKE SELECT ON TABLE \"foo\" FROM \"nobody\"",
-      "REVOKE SELECT, UPDATE, DELETE, INSERT ON TABLE \"foo\" FROM \"nobody\", \"somebody\"",
-      "REVOKE SELECT ON TABLE \"foo\" FROM \"nobody\"",
-      "REVOKE SELECT ON TABLE \"foo\" FROM \"nobody\" CASCADE",
-      "REVOKE SELECT ON TABLE \"foo\" FROM PUBLIC CASCADE"
+      "REVOKE SELECT ON TABLE \"foo\" FROM \"nobody\";",
+      "REVOKE SELECT, UPDATE, DELETE, INSERT ON TABLE \"foo\" FROM \"nobody\", \"somebody\";",
+      "REVOKE SELECT ON TABLE \"foo\" FROM \"nobody\";",
+      "REVOKE SELECT ON TABLE \"foo\" FROM \"nobody\" CASCADE;",
+      "REVOKE SELECT ON TABLE \"foo\" FROM PUBLIC CASCADE;"
     ], statements)
   end
 
@@ -42,8 +42,8 @@ class PermissionsTests < Test::Unit::TestCase
     Mig.grant_sequence_privileges(:foo, [ :select, :update ], [ :nobody, :somebody ])
 
     assert_equal([
-      "GRANT SELECT ON SEQUENCE \"foo\" TO \"nobody\"",
-      "GRANT SELECT, UPDATE ON SEQUENCE \"foo\" TO \"nobody\", \"somebody\""
+      "GRANT SELECT ON SEQUENCE \"foo\" TO \"nobody\";",
+      "GRANT SELECT, UPDATE ON SEQUENCE \"foo\" TO \"nobody\", \"somebody\";"
     ], statements)
   end
 
@@ -52,8 +52,8 @@ class PermissionsTests < Test::Unit::TestCase
     Mig.revoke_sequence_privileges(:foo, [ :select, :update ], [ :nobody, :somebody ])
 
     assert_equal([
-      "REVOKE SELECT ON SEQUENCE \"foo\" FROM \"nobody\"",
-      "REVOKE SELECT, UPDATE ON SEQUENCE \"foo\" FROM \"nobody\", \"somebody\""
+      "REVOKE SELECT ON SEQUENCE \"foo\" FROM \"nobody\";",
+      "REVOKE SELECT, UPDATE ON SEQUENCE \"foo\" FROM \"nobody\", \"somebody\";"
     ], statements)
   end
 
@@ -62,8 +62,8 @@ class PermissionsTests < Test::Unit::TestCase
     Mig.grant_function_privileges('test(text, integer)', :all, [ :nobody, :somebody ])
 
     assert_equal([
-      "GRANT EXECUTE ON FUNCTION test(text, integer) TO \"nobody\"",
-      "GRANT ALL ON FUNCTION test(text, integer) TO \"nobody\", \"somebody\""
+      "GRANT EXECUTE ON FUNCTION test(text, integer) TO \"nobody\";",
+      "GRANT ALL ON FUNCTION test(text, integer) TO \"nobody\", \"somebody\";"
     ], statements)
   end
 
@@ -72,8 +72,8 @@ class PermissionsTests < Test::Unit::TestCase
     Mig.revoke_function_privileges('test(text, integer)', :all, [ :nobody, :somebody ])
 
     assert_equal([
-      "REVOKE EXECUTE ON FUNCTION test(text, integer) FROM \"nobody\"",
-      "REVOKE ALL ON FUNCTION test(text, integer) FROM \"nobody\", \"somebody\""
+      "REVOKE EXECUTE ON FUNCTION test(text, integer) FROM \"nobody\";",
+      "REVOKE ALL ON FUNCTION test(text, integer) FROM \"nobody\", \"somebody\";"
     ], statements)
   end
 
@@ -82,8 +82,8 @@ class PermissionsTests < Test::Unit::TestCase
     Mig.grant_language_privileges('plpgsql', :all, [ :nobody, :somebody ])
 
     assert_equal([
-      "GRANT USAGE ON LANGUAGE \"plpgsql\" TO \"nobody\"",
-      "GRANT ALL ON LANGUAGE \"plpgsql\" TO \"nobody\", \"somebody\""
+      "GRANT USAGE ON LANGUAGE \"plpgsql\" TO \"nobody\";",
+      "GRANT ALL ON LANGUAGE \"plpgsql\" TO \"nobody\", \"somebody\";"
     ], statements)
   end
 
@@ -92,8 +92,8 @@ class PermissionsTests < Test::Unit::TestCase
     Mig.revoke_language_privileges('plpgsql', :all, [ :nobody, :somebody ])
 
     assert_equal([
-      "REVOKE USAGE ON LANGUAGE \"plpgsql\" FROM \"nobody\"",
-      "REVOKE ALL ON LANGUAGE \"plpgsql\" FROM \"nobody\", \"somebody\""
+      "REVOKE USAGE ON LANGUAGE \"plpgsql\" FROM \"nobody\";",
+      "REVOKE ALL ON LANGUAGE \"plpgsql\" FROM \"nobody\", \"somebody\";"
     ], statements)
   end
 
@@ -102,8 +102,8 @@ class PermissionsTests < Test::Unit::TestCase
     Mig.grant_schema_privileges(:foo, :all, [ :nobody, :somebody ])
 
     assert_equal([
-      "GRANT USAGE ON SCHEMA \"foo\" TO \"nobody\"",
-      "GRANT ALL ON SCHEMA \"foo\" TO \"nobody\", \"somebody\""
+      "GRANT USAGE ON SCHEMA \"foo\" TO \"nobody\";",
+      "GRANT ALL ON SCHEMA \"foo\" TO \"nobody\", \"somebody\";"
     ], statements)
   end
 
@@ -112,8 +112,8 @@ class PermissionsTests < Test::Unit::TestCase
     Mig.revoke_schema_privileges(:foo, :all, [ :nobody, :somebody ])
 
     assert_equal([
-      "REVOKE USAGE ON SCHEMA \"foo\" FROM \"nobody\"",
-      "REVOKE ALL ON SCHEMA \"foo\" FROM \"nobody\", \"somebody\""
+      "REVOKE USAGE ON SCHEMA \"foo\" FROM \"nobody\";",
+      "REVOKE ALL ON SCHEMA \"foo\" FROM \"nobody\", \"somebody\";"
     ], statements)
   end
 
@@ -122,8 +122,8 @@ class PermissionsTests < Test::Unit::TestCase
     Mig.grant_tablespace_privileges(:foo, :all, [ :nobody, :somebody ])
 
     assert_equal([
-      "GRANT CREATE ON TABLESPACE \"foo\" TO \"nobody\"",
-      "GRANT ALL ON TABLESPACE \"foo\" TO \"nobody\", \"somebody\""
+      "GRANT CREATE ON TABLESPACE \"foo\" TO \"nobody\";",
+      "GRANT ALL ON TABLESPACE \"foo\" TO \"nobody\", \"somebody\";"
     ], statements)
   end
 
@@ -132,8 +132,8 @@ class PermissionsTests < Test::Unit::TestCase
     Mig.revoke_tablespace_privileges(:foo, :all, [ :nobody, :somebody ])
 
     assert_equal([
-      "REVOKE CREATE ON TABLESPACE \"foo\" FROM \"nobody\"",
-      "REVOKE ALL ON TABLESPACE \"foo\" FROM \"nobody\", \"somebody\""
+      "REVOKE CREATE ON TABLESPACE \"foo\" FROM \"nobody\";",
+      "REVOKE ALL ON TABLESPACE \"foo\" FROM \"nobody\", \"somebody\";"
     ], statements)
   end
 
@@ -143,9 +143,9 @@ class PermissionsTests < Test::Unit::TestCase
     Mig.grant_role_membership(:foo, [ :nobody, :somebody ], :with_admin_option => true)
 
     assert_equal([
-      "GRANT \"foo\" TO \"nobody\"",
-      "GRANT \"foo\" TO \"nobody\", \"somebody\"",
-      "GRANT \"foo\" TO \"nobody\", \"somebody\" WITH ADMIN OPTION"
+      "GRANT \"foo\" TO \"nobody\";",
+      "GRANT \"foo\" TO \"nobody\", \"somebody\";",
+      "GRANT \"foo\" TO \"nobody\", \"somebody\" WITH ADMIN OPTION;"
     ], statements)
   end
 
@@ -155,9 +155,9 @@ class PermissionsTests < Test::Unit::TestCase
     Mig.revoke_role_membership(:foo, [ :nobody, :somebody ], :with_admin_option => true)
 
     assert_equal([
-      "REVOKE \"foo\" FROM \"nobody\"",
-      "REVOKE \"foo\" FROM \"nobody\", \"somebody\"",
-      "REVOKE \"foo\" FROM \"nobody\", \"somebody\""
+      "REVOKE \"foo\" FROM \"nobody\";",
+      "REVOKE \"foo\" FROM \"nobody\", \"somebody\";",
+      "REVOKE \"foo\" FROM \"nobody\", \"somebody\";"
     ], statements)
   end
 end

data/test/{roles_test.rb → roles_tests.rb}

@@ -23,8 +23,8 @@ class RolesTests < Test::Unit::TestCase
     })
 
     assert_equal([
-      'CREATE ROLE "foo"',
-      %{CREATE ROLE "foo" SUPERUSER CREATEDB CREATEROLE NOINHERIT LOGIN CONNECTION LIMIT 10 ENCRYPTED PASSWORD 'testing' VALID UNTIL '2011-10-12' IN ROLE "bar" ROLE "baz" ADMIN "blort"}
+      'CREATE ROLE "foo";',
+      %{CREATE ROLE "foo" SUPERUSER CREATEDB CREATEROLE NOINHERIT LOGIN CONNECTION LIMIT 10 ENCRYPTED PASSWORD 'testing' VALID UNTIL '2011-10-12' IN ROLE "bar" ROLE "baz" ADMIN "blort";}
     ], statements)
   end
 
@@ -45,7 +45,7 @@ class RolesTests < Test::Unit::TestCase
     })
 
     assert_equal([
-      %{ALTER ROLE "foo" SUPERUSER CREATEDB CREATEROLE NOINHERIT LOGIN CONNECTION LIMIT 10 ENCRYPTED PASSWORD 'testing' VALID UNTIL '2011-10-12' IN ROLE "bar" ROLE "baz" ADMIN "blort"}
+      %{ALTER ROLE "foo" SUPERUSER CREATEDB CREATEROLE NOINHERIT LOGIN CONNECTION LIMIT 10 ENCRYPTED PASSWORD 'testing' VALID UNTIL '2011-10-12' IN ROLE "bar" ROLE "baz" ADMIN "blort";}
     ], statements)
   end
 
@@ -55,9 +55,9 @@ class RolesTests < Test::Unit::TestCase
     ARBC.drop_role(%w{ foo bar baz }, :if_exists => true)
 
     assert_equal([
-      'DROP ROLE "foo"',
-      'DROP ROLE "foo", "bar", "baz"',
-      'DROP ROLE IF EXISTS "foo", "bar", "baz"',
+      'DROP ROLE "foo";',
+      'DROP ROLE "foo", "bar", "baz";',
+      'DROP ROLE IF EXISTS "foo", "bar", "baz";',
     ], statements)
   end
 end

data/test/{rules_test.rb → rules_tests.rb}

@@ -17,8 +17,8 @@ class RulesTests < Test::Unit::TestCase
     )
 
     assert_equal([
-      "CREATE RULE \"ignore_root\" AS ON UPDATE TO \"users\" WHERE user_id = 0 DO INSTEAD NOTHING",
-      "CREATE  OR REPLACE RULE \"ignore_root\" AS ON UPDATE TO \"users\" WHERE user_id > 0 DO INSTEAD SELECT * FROM non_admins"
+      "CREATE RULE \"ignore_root\" AS ON UPDATE TO \"users\" WHERE user_id = 0 DO INSTEAD NOTHING;",
+      "CREATE  OR REPLACE RULE \"ignore_root\" AS ON UPDATE TO \"users\" WHERE user_id > 0 DO INSTEAD SELECT * FROM non_admins;"
     ], statements)
   end
 
@@ -26,7 +26,7 @@ class RulesTests < Test::Unit::TestCase
     ARBC.drop_rule(:foo, :bar)
 
     assert_equal([
-      "DROP RULE \"foo\" ON \"bar\"",
+      "DROP RULE \"foo\" ON \"bar\";",
     ], statements)
   end
 end

data/test/{schemas_test.rb → schemas_tests.rb}

@@ -10,8 +10,8 @@ class SchemasTests < Test::Unit::TestCase
     Mig.create_schema(:foo, :authorization => 'bar')
 
     assert_equal([
-      "CREATE SCHEMA \"foo\"",
-      "CREATE SCHEMA \"foo\" AUTHORIZATION \"bar\""
+      "CREATE SCHEMA \"foo\";",
+      "CREATE SCHEMA \"foo\" AUTHORIZATION \"bar\";"
     ], statements)
   end
 
@@ -20,8 +20,8 @@ class SchemasTests < Test::Unit::TestCase
     Mig.drop_schema(:foo, :if_exists => true, :cascade => true)
 
     assert_equal([
-      "DROP SCHEMA \"foo\"",
-      "DROP SCHEMA IF EXISTS \"foo\" CASCADE"
+      "DROP SCHEMA \"foo\";",
+      "DROP SCHEMA IF EXISTS \"foo\" CASCADE;"
     ], statements)
   end
 
@@ -29,7 +29,7 @@ class SchemasTests < Test::Unit::TestCase
     Mig.alter_schema_name(:foo, :bar)
 
     assert_equal([
-      "ALTER SCHEMA \"foo\" RENAME TO \"bar\""
+      "ALTER SCHEMA \"foo\" RENAME TO \"bar\";"
     ], statements)
   end
 
@@ -37,7 +37,7 @@ class SchemasTests < Test::Unit::TestCase
     Mig.alter_schema_owner(:foo, :bar)
 
     assert_equal([
-      "ALTER SCHEMA \"foo\" OWNER TO \"bar\""
+      "ALTER SCHEMA \"foo\" OWNER TO \"bar\";"
     ], statements)
   end
 end