diff options
| author | David Robillard <d@drobilla.net> | 2021-10-21 15:38:10 -0400 |
|---|---|---|
| committer | David Robillard <d@drobilla.net> | 2022-01-28 21:57:07 -0500 |
| commit | b404312686874e539b617d1f27ccbaa5a82936af (patch) | |
| tree | c2fdb2cc046e6da53071629cd1750dcc327e6cd9 /test/test_write_error.py | |
| parent | d4aec28ba8ad24d5aef3ee12beeb1b805148eab1 (diff) | |
| download | serd-b404312686874e539b617d1f27ccbaa5a82936af.tar.gz serd-b404312686874e539b617d1f27ccbaa5a82936af.tar.bz2 serd-b404312686874e539b617d1f27ccbaa5a82936af.zip | |
Replace serdi with more fine-grained tools
Especially with the new functionality, the complexity of the command-line
interface alone was really becoming unmanageable. The serdi implementation
also had the highest cyclomatic complexity of the entire codebase by a huge
margin.
So, take a page from the Unix philosophy and split serdi into several more
finely-honed tools that can be freely composed. Though there is still
unfortunately quite a bit of option overlap between them due to the common
details of reading RDF, I think the resulting tools are a lot easier to
understand, both from a user and a developer perspective.
Diffstat (limited to 'test/test_write_error.py')
| -rwxr-xr-x | test/test_write_error.py | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/test/test_write_error.py b/test/test_write_error.py index 35b4693b..bc955ce9 100755 --- a/test/test_write_error.py +++ b/test/test_write_error.py @@ -1,6 +1,6 @@ #!/usr/bin/env python3 -"""Test errors writing to a file.""" +"""Test errors when writing to a file.""" import argparse import sys @@ -10,12 +10,12 @@ import os parser = argparse.ArgumentParser(description=__doc__) -parser.add_argument("--serdi", default="./serdi", help="path to serdi") +parser.add_argument("--tool", default="tools/serd-pipe", help="executable") parser.add_argument("--wrapper", default="", help="executable wrapper") parser.add_argument("input", help="valid input file") args = parser.parse_args(sys.argv[1:]) -command = shlex.split(args.wrapper) + [args.serdi, args.input] +command = shlex.split(args.wrapper) + [args.tool, args.input] if os.path.exists("/dev/full"): |