Agreed, this patch moves the failing tests to bugs instead.

For other testers: this was easy, just copy the test script to bugs, then remove fail-expected tests from one and pass-expected tests from the other.

There is still at least one expected failure in the tests, but it's platform-specific and I don't know what to do with that. Grep for TODO, patches welcome.
Wed Aug  6 12:13:36 PDT 2008  Simon Michael <[EMAIL PROTECTED]>
  * move (most) failing tests to bugs for clean test output

New patches:

[move (most) failing tests to bugs for clean test output
Simon Michael <[EMAIL PROTECTED]>**20080806191336] addfile ./bugs/add.pl
addfile ./bugs/dist.pl
addfile ./bugs/merging_newlines.pl
hunk ./bugs/add.pl 1
+#!/usr/bin/env perl
+
+# Some tests for 'darcs add'
+
+use lib 'lib/perl';
+use Test::More 'no_plan';
+use Test::Darcs;
+use Shell::Command;
+use strict;
+use warnings;
+
+init_tmp_repo();
+
+###
+
+TODO: {
+    local $TODO = 'waiting on coding';
+    my $test_name = "add should fail on files it can't read (because it would 
fail to record it later anyway).";
+    touch "no_perms.txt";
+    chmod(0000,"no_perms.txt");
+    like(darcs(qw(add no_perms.txt)), qr/permission denied/,$test_name);
+    rm_rf "no_perms.txt";
+
+}
+
+chdir '../';
hunk ./bugs/dist.pl 1
+#!/usr/bin/env perl
+
+# tests for "darcs dist"
+
+use lib 'lib/perl';
+use Test::More 'no_plan';
+use strict;
+use Test::Darcs;
+use Shell::Command;
+
+init_tmp_repo();
+
+TODO: {
+    local $TODO = "needs fixed on FreeBSD" if ($^O eq 'freebsd');
+    unlike( darcs("dist -v"), qr/error/i, "darcs dist -v avoids error message 
" );
+}
hunk ./bugs/merging_newlines.pl 1
+#!/usr/bin/env perl
+
+# A test for darcs resolve detecting a conflict, inspired by bug #152 in RT
+
+use lib 'lib/perl';
+use Test::More 'no_plan';
+use Test::Darcs;
+use Shell::Command;
+use strict;
+use warnings;
+use File::Slurp;
+use File::Temp 'tempdir';
+my $work_dir = tempdir( CLEANUP => 1 );
+chdir $work_dir;
+
+
+TODO: {
+    local $TODO = 'waiting on code to fix this';
+    unlike(darcs(q(resolve)),qr/no conflicts/i, "after a conflict, darcs 
resolve should report a conflict");
+}
+
hunk ./tests/add.pl 62
-TODO: {
-    local $TODO = 'waiting on coding';
-    my $test_name = "add should fail on files it can't read (because it would 
fail to record it later anyway).";
-    touch "no_perms.txt";
-    chmod(0000,"no_perms.txt");
-    like(darcs(qw(add no_perms.txt)), qr/permission denied/,$test_name);
-    rm_rf "no_perms.txt";
-
-}
-
-
hunk ./tests/dist.pl 17
-TODO: {
-    local $TODO = "needs fixed on FreeBSD" if ($^O eq 'freebsd');
-    unlike( darcs("dist -v"), qr/error/i, "darcs dist -v avoids error message 
" );
-}
hunk ./tests/merging_newlines.pl 46
-TODO: {
-    local $TODO = 'waiting on code to fix this';
-    unlike(darcs(q(resolve)),qr/no conflicts/i, "after a conflict, darcs 
resolve should report a conflict");
-}
-

Context:

[make order of running tests consistent
Simon Michael <[EMAIL PROTECTED]>**20080806172123] 
[fix an old spelling error
Simon Michael <[EMAIL PROTECTED]>**20080806170432] 
[cleanup and slight rewrite of the test docs
Simon Michael <[EMAIL PROTECTED]>**20080806165949] 
[make searching for "test:" in makefile work
Simon Michael <[EMAIL PROTECTED]>**20080805222241] 
[run only normal (expected to pass) tests by default
Simon Michael <[EMAIL PROTECTED]>**20080805222108] 
[small makefile refactoring: allow just the normal tests to be run, without 
bugs/*
Simon Michael <[EMAIL PROTECTED]>**20080805203242] 
[Rectify dist help
[EMAIL PROTECTED]
 Removed the "make dist" suggestion, the manual is a better place for that.
 Instead, make clear that it operates on a clean copy of the tree, and
 mention the "predist" functionality.
] 
[Canonize Gaetan Lehmann and Daniel Buenzli.
Eric Kow <[EMAIL PROTECTED]>**20080730104357
 (for Daniel B, avoid an accent in his name)
] 
[configure: check for packages needed with split base.
Eric Kow <[EMAIL PROTECTED]>**20080730103840
 Now that all packages must be used explicitly.
] 
[fix type witness compile errors specific to ghc 6.8
Jason Dagit <[EMAIL PROTECTED]>**20080722182729] 
[website: explain that darcs 2 is required to get the darcs source.
Simon Michael <[EMAIL PROTECTED]>**20080803181216] 
[avoid import of unused function fromMaybe.
David Roundy <[EMAIL PROTECTED]>**20080729172825] 
[configure: suggest regex-compat before text
Eric Kow <[EMAIL PROTECTED]>**20080725095336] 
[configure: mention Haskell in 'try installing' suggestion
Eric Kow <[EMAIL PROTECTED]>**20080725095015] 
[Typo (Text.Regex)
Eric Kow <[EMAIL PROTECTED]>**20080715121708] 
[Use haskeline to have a readline-like behavior when asking something to the 
user
[EMAIL PROTECTED]
 Unlike the implementations using readline or editline packages, this code
 code doesn't break the Ctrl-C behavior.
] 
[Improve generic rules for English plurals. 
Eric Kow <[EMAIL PROTECTED]>**20080604123728] 
[add configure check for Network.URI.
David Roundy <[EMAIL PROTECTED]>**20080711011914] 
[add -hide-all-packages to default GHCFLAGS.
David Roundy <[EMAIL PROTECTED]>**20080711010952] 
[add support for outputting patch numbers in darcs changes.
David Roundy <[EMAIL PROTECTED]>**20080710011211] 
[add support for matching single patches by index.
David Roundy <[EMAIL PROTECTED]>**20080710004512] 
[add support for matching ranges of patches (counting back from present).
David Roundy <[EMAIL PROTECTED]>**20080710003225] 
[Better avoid silly manpage error.
Trent W. Buck <[EMAIL PROTECTED]>**20080704024920
 
 It turned out only initialize's help string used 'quotes', so just
 remove them.  This makes init's docstring consistent with the others.
] 
[Missing period at end of sentence.
Trent W. Buck <[EMAIL PROTECTED]>**20080704024232] 
[darcs --overview no longer works, so don't document it.
Trent W. Buck <[EMAIL PROTECTED]>**20080704030804] 
[Avoid silly manpage error.
Trent W. Buck <[EMAIL PROTECTED]>**20080703010733
 man (nroff) treats an apostrophe in the first column specially,
 resulting in a syntax error without this patch.
 
 Ideally, all cases of 'foo' in the manpage (i.e. docstrings) should
 become `foo', since man -Tps turns ` and ' into left and right single
 quotes respectively.
] 
[obliterate whitespace in Darcs.Commands.Get
[EMAIL PROTECTED]
 'twas causing lhs/haddock difficulties where a \end{code} wasn't getting 
recognized.
] 
[rm haddock CPP business
[EMAIL PROTECTED]
 Try as I might, I can't see any reason to special-case some Haddock CPP logic 
to deal with some *commented-out guards*, unless CPP magically restores and 
uncomments the code if Haddock isn't being run.
] 
[make pull less verbose when --verbose flag is given.
David Roundy <[EMAIL PROTECTED]>**20080624170035] 
[fix makefile to remember to regenerate version information after running 
configure.
David Roundy <[EMAIL PROTECTED]>**20080624170001] 
[TAG 2.0.2
David Roundy <[EMAIL PROTECTED]>**20080624012041] 
Patch bundle hash:
3c98b632eba290396593e2c48d3705446fc93223
_______________________________________________
darcs-users mailing list
[email protected]
http://lists.osuosl.org/mailman/listinfo/darcs-users

Reply via email to