Commit 4d2c2bdc authored by Per Cederqvist's avatar Per Cederqvist
Browse files

Increase coverage of the aux-item definitions parser. (Bug 64). Get

rid of a spurious error message found in the process.  Use less
preprocessor macro magic.  Test suite framework improvements:
simple_expect can now expect lines that don't contain the line leader,
and lyskomd_start and lyskomd_fail_start can now handle output sent
before the database lock is taken.  Fixed a comment.
parent 33920779
2004-07-21 Per Cederqvist <ceder@ingate.com>
Increase coverage of the aux-item definitions parser. (Bug 64).
Get rid of a spurious error message found in the process. Use
less preprocessor macro magic.
* src/server/testsuite/lyskomd.0/bug-64.exp: New file. Increase
test coverage of aux-item-def-parse.y.
* src/server/testsuite/lyskomd.0/bug-64.aux: New file.
* src/server/testsuite/lyskomd.0/bug-64-2.aux: New file.
* src/server/simple-cache.c (pre_sync): Use VBUG instead of BUG to
report sync start and stop.
* src/server/aux-item-def-parse.y (CHK_ASSIGN): Macro rewritten as
the new chk_assign() function, with a slightly different API.
This was mainly done to get rid of the error message "invalid
field name: xyzzy" that occured if the value of xyzzy was bogus,
even if xyzzy itself was a valid name. All callers updated.
(CHK_FLAG_A): Similarly rewritten as chk_flag_a().
(assign): Check for valid field name and valid field content
separately, so that the "invalid field name: xyzzy" message isn't
printed just because the content is bad.
(chk_assign): New name for former aux_item_def_check_assign. New
arguments: found and good; return status via them.
(chk_flag_a): New function.
(aux_item_def_check_trigger): New arguments: found and good;
return status via them.
(aux_item_def_check_validate): Ditto.
Test suite framework improvements: simple_expect can now expect
lines that don't contain the line leader, and lyskomd_start and
lyskomd_fail_start can now handle output sent before the database
lock is taken.
* src/server/testsuite/config/unix.exp (simple_expect): If the
regexp starts with "<<<", remove it, and don't expect a line
leader. This is useful for catching messages from lyskomd that
are created by libraries that we have no control over, and that
don't emit the expected line leader.
(set_debug_calls): New proc, extracted from lyskomd_start and
lyskomd_fail_start. Expect the new message that is emitted when
the debug calls are disabled.
(lyskomd_start): New optional argument: pre_lock_messages. Moved
code to set_debug_calls().
(lyskomd_fail_start): Ditto.
* src/server/ramkomd.c (main): Write a message when debug calls
are disabled, to help the test suite.
Fixed a comment.
* src/server/testsuite/timeval-overflow.c: Fixed a comment.
2004-07-18 Per Cederqvist <ceder@ingate.com> 2004-07-18 Per Cederqvist <ceder@ingate.com>
   
Implement async-text-read and async-invalidate-text-read. (Bug Implement async-text-read and async-invalidate-text-read. (Bug
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment