tests

[dbsrgits/SQL-Translator.git] / t / 60roundtrip.t

diff --git a/t/60roundtrip.t b/t/60roundtrip.t
index f39c9c7..857226e 100644 (file)
--- a/t/60roundtrip.t
+++ b/t/60roundtrip.t
@@ -9,13 +9,19 @@ use FindBin qw/$Bin/;
 
 use SQL::Translator;
 
+
 ### Set $ENV{SQLTTEST_RT_DEBUG} = 1 for more output
 
 # What tests to run - parser/producer name, and optional args
 my $plan = [
   {
     engine => 'XML',
+    req => 'XML::LibXML 1.69',
+  },
+  {
+    engine => 'YAML',
   },
+
   {
     engine => 'SQLite',
     producer_args => {},
@@ -43,25 +49,29 @@ my $plan = [
     producer_args => {},
     parser_args => {},
   },
+  {
+    engine => 'SQLServer',
+    producer_args => {},
+    parser_args => {},
+  },
+
 #  {
 #    engine => 'Oracle',
 #    producer_args => {},
 #    parser_args => {},
-#  },
-#  {
-#    engine => 'SQLServer',
-#    producer_args => {},
-#    parser_args => {},
+#    todo => 'Needs volunteers',
 #  },
 #  {
 #    engine => 'Sybase',
 #    producer_args => {},
 #    parser_args => {},
+#    todo => 'Needs volunteers',
 #  },
 #  {
 #    engine => 'DB2',
 #    producer_args => {},
 #    parser_args => {},
+#    todo => 'Needs volunteers',
 #  },
 
 # There is no Access producer
@@ -75,26 +85,51 @@ my $plan = [
 
 # This data file has the right mix of table/view/procedure/trigger
 # definitions, and lists enough quirks to trip up most combos
-# I am not sure if augmenting it will break other tests - experiment
-my $base_file = "$Bin/data/xml/schema.xml";
+my $base_file = "$Bin/data/roundtrip_autogen.yaml";
+open (my $base_fh, '<', $base_file) or die "$base_file: $!";
 
 my $base_t = SQL::Translator->new;
 $base_t->$_ (1) for qw/add_drop_table no_comments/;
 
 my $base_schema = $base_t->translate (
-  parser => 'XML',
-  file => $base_file,
+  parser => 'YAML',
+  data => do { local $/; <$base_fh>; },
 ) or die $base_t->error;
 
 
+#assume there is at least one table
+my $string_re = {
+  XML => qr/<tables>\s*<table/,
+  YAML => qr/\A---\n.+tables\:/s,
+  SQL => qr/^\s*CREATE TABLE/m,
+};
+
 for my $args (@$plan) {
+  SKIP: {
+    $args->{name} ||= $args->{engine};
+
+    my @req = ref $args->{req} ? @{$args->{req}} : $args->{req}||();
+    my @missing;
+    for (@req) {
+      eval "use $_ ()";
+      push @missing, $_ if ($@);
+    }
+    if (@missing) {
+      skip sprintf ('Need %s for %s roundtrip test',
+        join (', ', @missing),
+        $args->{name},
+      );
+    }
 
-  $args->{name} ||= $args->{engine};
+    TODO: {
+      local $TODO = $args->{todo} if $args->{todo};
 
-  lives_ok (
-    sub { check_roundtrip ($args, $base_schema) },
-    "Round trip for $args->{name} did not throw an exception",
-  );
+      lives_ok (
+        sub { check_roundtrip ($args, $base_schema) },
+        "Round trip for $args->{name} did not throw an exception",
+      );
+    }
+  }
 }
 
 
@@ -111,10 +146,7 @@ sub check_roundtrip {
 
   like (
     $base_out,
-    $args->{engine} eq 'XML'  #assume there is at least one table
-      ? qr/<tables>\s*<table/m
-      : qr/^\s*CREATE TABLE/m
-    ,
+    $string_re->{$args->{engine}} || $string_re->{SQL},
     "Received some meaningful output from the first $args->{name} production",
   ) or do {
     diag ( _gen_diag ($base_t->error) );
@@ -164,10 +196,7 @@ sub check_roundtrip {
 
   like (
     $rt_out,
-    $args->{engine} eq 'XML'  #assume there is at least one table
-      ? qr/<tables>\s*<table/m
-      : qr/^\s*CREATE TABLE/m
-    ,
+    $string_re->{$args->{engine}} || $string_re->{SQL},
     "Received some meaningful output from the second $args->{name} production",
   ) or do {
     diag ( _gen_diag ( $parser_t->error ) );