Permalink
Browse files

[BUILD] Missing opcodes should be an error - (#48497) fix build test …

…failure, document existance of those tests...

git-svn-id: https://svn.parrot.org/parrot/trunk@24716 d31e2699-5ff4-0310-a27c-f18f2fbe73fe
  • Loading branch information...
simoncozens committed Jan 10, 2008
1 parent 9201978 commit c4608d5cf85e8ae87af65c7f53f508ba6fd54809
Showing with 17 additions and 10 deletions.
  1. +13 −0 docs/tests.pod
  2. +4 −10 t/tools/ops2pmutils/08-sort_ops.t
View
@@ -191,6 +191,19 @@ F<t/postconfigure/>, F<t/tools/>, and so on.
It is our objective to test all Perl 5 components of the Parrot distribution
using the methodology above.
+=head3 Build Tools Tests
+
+The files in F<t/postconfigure> are tests for build system. The build tools
+tests are intended to be run after someone has made changes in modules such as
+F<lib/Parrot/Pmc2cUtils/>, F<Ops2cUtils/> and F<Ops2pmutils/>. They're set up
+to be run after F<Configure.pl> has completed but before make has been invoked.
+(In fact, they will generate errors if make has completed.) You can run them
+with any of the following:
+
+ perl Configure.pl --test
+ perl Configure.pl --test=build
+ make buildtools_tests (following Configure.pl)
+
=head2 Testing language implementations
Language implementations are usually tested with
@@ -26,7 +26,7 @@ use File::Copy;
use File::Temp (qw| tempdir |);
my $cwd = cwd();
-plan -e "$cwd/DEVELOPING" ? ( tests => 87 ) :
+plan -e "$cwd/DEVELOPING" ? ( tests => 86 ) :
( skip_all => 'Requires DEVELOPING file' );
use_ok('Parrot::Ops2pm::Utils');
@@ -249,17 +249,11 @@ DUMMYOPS
ok( -f $skip, "ops.skip located after renumbering" );
my ($stdout, $stderr);
- my $ret = capture(
- sub { $self->sort_ops() },
- \$stdout,
- \$stderr
- );
- ok($ret, "sort_ops returned successfully" );
-
+ eval { $self->sort_ops() };
like(
- $stderr,
+ $@,
qr|not in ops\.num nor ops\.skip|,
- "Got expected warning about ops in neither ops.num or ops.skip"
+ "Got expected failure about ops in neither ops.num or ops.skip"
);
# To do: Test that the sorting was correct.

0 comments on commit c4608d5

Please sign in to comment.