mirror of
https://github.com/storytold/cadcraft.git
synced 2026-10-28 21:06:56 -06:00
* test(cli): pin today's command-line behaviour
The later commits change these tests on purpose, so each behaviour change shows in their diff.
* refactor(cli): split main.rs into modules, move only
- output: outln!, stdout state, atomic write, export helpers, view/window helpers
- args: USAGE, option validation, framing options, MCP argument parsing and its tests
- info, convert, sample, mcp: one subcommand each
- perf: synthetic drawing, Rng, perf subcommands
- run: run subcommand and {last} handle chaining
- main: crate docs, mod lines, install_io, open, main, cli, unit tests
No behaviour change.
* refactor(cli): typed errors and run(args, out, err)
cadcraft-cli is now a library plus a thin binary. `cadcraft_cli::run(args, out, err)
-> i32` is the whole program; main.rs only passes `args_os()` and the process
streams. Handlers write through a `Streams` wrapper (output.rs) instead of the
process streams; it keeps the closed-stdout semantics (drop later output, carry
on, a closed pipe is not an error, other write errors are reported once and make
the exit code 1). `mcp` still speaks MCP on the real stdin/stdout.
Errors are typed (thiserror): CliError (error.rs) wraps EngineError, DocError,
IoError, io::Error and the per-module WindowError, ArgsError, RunError,
OutputError and PerfError. `run` prints `error: {e}` and exits 1.
The usage text and the "unknown option/argument" messages stay as one
transitional CliError::Usage(String), printed without the `error: ` prefix.
USAGE stays a const; step 4 (clap) removes both. The engine's IoHooks take
`Result<_, String>`, so the three `.map_err(|e| e.to_string())` in install_io
remain. The hooks are set-once (OnceLock), so installing them on every `run`
call is harmless.
tests/it/usage.rs calls `run` in-process with Vec<u8> writers; the headless mcp
case runs through the binary (it reads the real stdin) and a non-Unicode
argument test is new.
Behaviour changes (old -> new):
- Every error line other than the usage-class ones gains the prefix `error: `
(e.g. `--view needs ...` -> `error: --view needs ...`, `unknown command
`x`` -> `error: unknown command `x``, `invalid parameters for `open`: ...`
-> `error: invalid parameters ...`, `unsupported file type` likewise).
- `cadcraft-cli: cannot connect to HOST:PORT: e` -> `error: cannot connect to
HOST:PORT: e` (the old prefix is replaced).
- A non-Unicode argument: panic, exit 101 -> `error: argument "..." is not
valid Unicode`, exit 1.
- Unchanged: usage text, `X: unknown option --y`, `unknown option --y`,
`mcp: unknown argument ...` (no prefix); warnings and `wrote ...` lines;
the `cadcraft-cli: can't write to stdout: e` report; exit codes.
* refactor(cli): parse arguments with clap
Replace the hand-written parser (USAGE, validate_options, OptionSpec,
framing_option, parse_mcp_args, CliError::Usage) with clap derive types
in args.rs: a Cli parser and one Args struct per subcommand, with the
framing flags (--window, --view) in a FramingArgs struct flattened into
convert and run. USAGE's framing and --cmd chaining text moves into the
subcommands' and flags' doc comments and run's long help, so --help
carries it. `commands` moves from an inline closure to commands.rs.
`command()` exposes the clap definition for man pages and completions.
clap reads the arguments as OsStrings. Help and --version go to stdout
with exit 0; usage errors are clap's, on stderr with exit 2. A bare
--help or -h before `--` shows the subcommand's help even after a
mistake on the line (`run --script --help`). run opens its drawings
(FILEs, --sample, --metric) and runs its steps (--script, --script-file,
--cmd) in command-line order, from ArgMatches::indices_of. Values stay
strings checked by the handlers, with their messages and exit 1
(--window, --view, --cmd JSON, sample names, perf counts). The last of a
repeated flag wins (args_override_self on every subcommand but mcp).
Behaviour changes:
Help and version
- --help, -h, help: the USAGE page -> clap's help, first line
"cadcraft-cli: headless CADCraft".
- bare cadcraft-cli: USAGE on stderr, exit 1 -> clap's help on stderr,
exit 2.
- <subcommand> --help / -h, help <subcommand>: "unknown option --help"
(exit 1), the USAGE page (convert, help <subcommand>) or a file named
-h (info -h, run -h) -> that subcommand's help, stdout, exit 0.
- <subcommand> --version: "unknown option --version", exit 1 -> clap's
unexpected argument, exit 2.
Usage errors (exit 1 -> exit 2, clap's messages)
- Unknown subcommand, empty subcommand, unknown top-level flag: USAGE ->
"error: unrecognized subcommand 'foo'" / "error: unexpected argument
'--bogus' found".
- Unknown flag on any subcommand: "info: unknown option --bogus",
"unknown option --bogus", "mcp: unknown argument --bogus" or USAGE
(convert) -> "error: unexpected argument '--bogus' found".
- A short flag where a file is expected (info -x): opened as a file ->
unexpected argument; `--` before it keeps it a file name.
- A value flag at the end or followed by a flag (--window, --view,
--script, --script-file, --cmd, --save, --export, --connect): "error:
--script needs text" and the like, or the next flag taken as the
value (--window --view, --script --sample, --cmd --sample,
--script-file --sample) -> "error: a value is required for '--script
<TEXT>' but none was supplied". --connect followed by an unknown flag
(--connect --foo, --connect -v) names that flag instead (clap reports
the unknown argument, not the missing value).
- mcp --connect with an empty value: "error: --connect needs HOST:PORT
..." -> "error: invalid value ' ' for '--connect <HOST:PORT>':
expected HOST:PORT (...)"; --connect=-v likewise.
- mcp --connect twice: "error: --connect given more than once" ->
"error: the argument '--connect <HOST:PORT>' cannot be used multiple
times".
- mcp --lazy / --launch without --connect: "error: --lazy and --launch
need --connect HOST:PORT" -> "error: the following required arguments
were not provided: --connect <HOST:PORT>".
- Missing positionals (info, convert with 0 or 1 files, sample with 0
or 1 arguments): USAGE -> "error: the following required arguments
were not provided".
- Extra positionals: convert with three files (USAGE), and info,
commands, perf, sample, which ignored the extra argument -> "error:
unexpected argument 'extra' found".
- perf -5: opened as a file -> unexpected argument '-5'.
- A non-Unicode argument where text is expected: "error: argument ... is
not valid Unicode", exit 1 -> "error: invalid UTF-8 was detected in
one or more arguments", exit 2.
Accepted where they were errors
- --x=v is a spelling of --x v for every value flag (--window, --view,
--script, --script-file, --cmd, --save, --export, --connect); before
it was an unknown option or USAGE. A value starting with - needs the
= form (--cmd=-x).
- commands --: "unknown option --" -> `--` ends the options (all
commands listed).
- run --view extents --view current: "use only one of --window and
--view" -> the last wins, like every other repeat.
Other
- run --sample --sample opens one sample drawing (was two): repeated
flags are last-wins.
- run checks --window/--view values before opening any drawing, so a
bad framing value is reported ahead of a missing file.
- `-h` after a mistake shows the long help (`--help`): the retry asks
for --help.
Unchanged: --window with --view, framing without an image output,
--view's case-insensitivity, all value messages (--window numbers,
--view values, --cmd JSON, {last}, sample names, --script-file read
errors), multiple FILEs, --sample with --metric or a FILE, last-wins
for --save, --export, --window, --script/--cmd order, and `\n` in
--script.
clap forced: exit 2 for usage errors, its message wording, and the
unknown-flag-over-missing-value precedence above.
* refactor(cli): reject repeated flags, long version (photocraft#7071 decisions)
Behaviour changes, old -> new:
- --save, --export, --window, --view, --sample, --metric, --lazy, --launch given twice:
last one won -> usage error, exit 2 ("the argument ... cannot be used multiple times").
--connect was already rejected. --script, --script-file, --cmd and run's FILEs still repeat.
- --version / -V: "cadcraft-cli <version>" -> "cadcraft-cli <version> (<sha>)", sha from
CADCRAFT_BUILD_SHA or "dev", as the desktop app prints it.
- "cadcraft-cli version" is an unrecognized-subcommand usage error (as before; now tested).
- A value starting with "-" needs the = form for every value flag (as before, tested).
--window -10,-10,10,10 stays a usage error ("unexpected argument -1"); write
--window=-10,-10,10,10. clap's allow_negative_numbers does not accept a comma list, and
allow_hyphen_values would let --window swallow the next flag, so neither is set. The
--window help text names the = form.
* refactor(cli): typed values and flag relations
Step 6 of #2228: values are parsed into their types and flag relations
are checked at parse time, so a bad value or combination is a usage
error (exit 2) naming the flag, before any work starts. The handlers
receive typed values only. Runtime failures (files, the engine, a
`{last}` with no earlier handle) stay exit 1.
- --window parses to a Bounds2 (parse_window plus cadcraft_io's
check_window); its WindowError is the value parser's error.
- --view is a ValueEnum (extents, current), case-insensitive; the long
help lists what each value does.
- --window conflicts_with --view. In run, --window/--view require
--export (an ArgGroup "framing" with requires). A framing flag with
an output that isn't an image (convert's OUT, run's --export) is a
UsageError turned into a clap error by into_clap (ArgumentConflict).
- sample's name is a ValueEnum (bracket, floorplan); `floor` stays a
hidden alias.
- perf's N|FILE is one typed positional: a value that parses as a
count is one, clamped to 1..=20000000 as before (so `perf 0` still
times one entity); anything else is a drawing.
- --cmd 'id {json}' parses to an id and a serde_json::Value (default
{}); `{last}` is still filled in when the command runs.
- Paths are PathBufs (info FILE, convert IN/OUT, run FILEs,
--script-file, --save, --export, sample OUT, perf FILE). The engine
and io crates take file names as text, so a path that isn't valid
Unicode is refused by the handler, naming the file (exit 1).
- Deleted: check_framed_output, the --view match, ArgsError,
CliError::Window/Args/UnknownSample, RunError::CmdJson,
PerfError::BadCount.
- run's FILEs, --sample and --metric still combine (each opens another
drawing, the last is current): neither photocraft#7071 nor RFC-0012 requires
otherwise.
Behaviour changes (old -> new):
- --window with a bad value (wrong count, not a number, NaN/inf,
reversed corners, beyond 1e12): exit 1 `error: --window ...` ->
exit 2 `error: invalid value 'V' for '--window <x0,y0,x1,y1>':
<reason>`. The reasons no longer repeat the flag or the value.
- --view with a bad value: exit 1 `error: --view needs `extents` or
`current`` -> exit 2 `error: invalid value 'V' for '--view
<extents|current>'` plus clap's possible-values line.
- --window with --view: exit 1 `error: use only one of --window and
--view` -> exit 2 `error: the argument '--window <x0,y0,x1,y1>'
cannot be used with '--view <extents|current>'`.
- run --window/--view without --export: exit 1 `error: --window/--view
need an image output ...` -> exit 2 `error: the following required
arguments were not provided: --export <OUT.png>`.
- A framing flag with a non-image output: exit 1 `error:
--window/--view only apply to image output (png, svg, pdf), not
`OUT`` -> exit 2 `error: --window only applies to image output (png,
svg, pdf), not `OUT`` (or --view: the message names the flag given).
- sample with an unknown name: exit 1 `error: unknown sample `X`
(bracket, floorplan)` -> exit 2 `error: invalid value 'X' for
'<bracket|floorplan>'` plus the possible values. sample's help lists
the possible values.
- --cmd with JSON that doesn't parse: exit 1 `error: --cmd ID:
<serde error>`, after the earlier steps ran -> exit 2 `error: invalid
value 'V' for '--cmd <id {json}>': <serde error>`, before anything
runs.
- --cmd with an empty id (`--cmd=` or `--cmd ' {}'`): exit 1 `error:
unknown command `` ` -> exit 2 `... expected a command id, then
optionally its JSON parameters`.
- An empty path argument (info "", sample bracket "", perf "", ...):
exit 1 from the engine or io -> exit 2 `error: a value is required
for '<FILE>' but none was supplied` (clap's PathBuf parser).
- A path that isn't valid Unicode: exit 2 `error: invalid UTF-8 was
detected ...` -> exit 1 `error: PATH: the file name is not valid
Unicode` (text values that aren't Unicode stay exit 2). A non-Unicode
--save or --export is refused before any drawing opens.
- perf's help gives the count range.
Tests: usage.rs rows above moved to exit 2 with clap's messages; new
rows for --view with a non-image output, run's framing with a dxf
--export, an empty --cmd id, bad --cmd JSON after a good step, a
`{last}` with no earlier handle, --window beyond 1e12, case-sensitive
sample names and empty paths. The non-Unicode test covers both a path
(exit 1) and text (exit 2). Unit tests: the --window parser tests moved
to args.rs; new ones for --cmd values and perf targets; framing_options
checks the clap error kinds.
* refactor(cadcraft): move launch-argument parsing into launch_args
Moves the hand-written parser (the argument loop, control_port_from, control_flag and their tests) from main.rs into launch_args.rs, which returns LaunchArgs. No behaviour change.
* refactor(cadcraft): parse launch flags with clap (RFC-0012)
Replaces the hand-written loop in launch_args.rs with clap derive types, as photocraft's launch_args.rs does. --help is generated from the doc comments and lists the other CADCRAFT_* variables.
Behaviour changes (old -> new):
- An unknown flag (--sampel) was opened as a file; it is now a usage error on stderr, exit 2, before the window opens. A file starting with "-" goes after "--".
- A repeated --control or --sample was accepted (last wins); it is now exit 2.
- A bad port (--control abc, 70000, -1) was logged and the control channel left off; it is now exit 2. The same for CADCRAFT_CONTROL_PORT, and the error names the variable when the value came from it.
- A bare --control (no value) kept CADCRAFT_CONTROL_PORT's port with a warning; it is now exit 2 ("a value is required"). A following flag is never taken as the value.
- An empty or blank CADCRAFT_CONTROL_PORT was logged as a bad port; it is now unset.
- --control=7979 is accepted (the equals form was opened as a file before).
- --version / -V and --help stop parsing at once and exit 0; version text is unchanged ("cadcraft <version> (<sha>)").
- -psn_* (Finder process serial number) is dropped instead of opened as a file.
- A file name that is not valid Unicode used to panic in std::env::args(); it is now read from args_os and reported with a log warning, not opened, because CadApp::open_path takes text.
Copied from photocraft's crates/automation/src/args.rs: blank_env_is_unset and name_env_vars unchanged (name_env_vars gets a local use for ContextKind/ContextValue); control_port and NotAPort unchanged. allow_negative_numbers is set on --control as in photocraft, so -1 is a bad value rather than a missing one.
* feat(cli): shell completions and man pages
Adds examples/man-and-completions.rs, which writes cadcraft-cli's bash, zsh, fish, elvish and
PowerShell completions (clap_complete) and man pages (clap_mangen: cadcraft-cli.1 and one per
subcommand) into a directory. clap_complete and clap_mangen are dev-dependencies, so the binary
is unchanged.
packaging/linux/package.sh builds the example in its own cargo run, runs it (through $EMULATOR
for cross builds; warns and leaves the files out without one) and installs the man pages and the
bash, zsh and fish completions into the staged tree. nfpm.yaml already ships the whole
usr/share tree, so it needs no change.
CADCraft has no FreeBSD packaging, so there is no FreeBSD script to update. macOS and Windows
packaging are untouched.
tests/man_and_completions.rs checks the generated file names.
* ci(cli): check the command-line code with ast-grep (RFC-0012)
Adds ast-grep rules from RFC-0012, run by craft-common-actions'
reusable ast-grep workflow (Linux, no Rust build, path-filtered):
- clap-error-message: a string literal, format!, .to_string() or
SCREAMING_CASE const passed to Command::error / Error::raw
- clap-error-message-variable: the same message built into a variable
first, in the same function
- string-error: Result<_, String>, Err(format!(..)), Err(".."),
Err(x.to_string())
- args-os: std::env::args(), which panics on non-Unicode arguments
Scan paths: apps/cadcraft-cli/src, apps/cadcraft/src/launch_args.rs,
apps/cadcraft/src/main.rs. Rule tests run first in CI.
Source: photocraft#9750 at d16686cd. Only the scan paths and the
workflow path filters differ from it.
Fixes: none needed. The rule tests pass (4 passed) and the scan finds
nothing in the scanned paths.
* docs(cli): cadcraft-cli page, following RFC-0012
docs/cli.md: --help is the reference; exit codes, streams, usage
errors, values starting with -, paths, completions and man pages; the
environment variables (none for cadcraft-cli, CADCRAFT_CONTROL_PORT for
the desktop app's --control); and the rules not followed yet, with the
reason. The README links it.
* feat(cli): refuse an output that replaces an input
RFC-0012 rule 1.6: nothing destructive by default.
Behaviour change: `convert a.dxf a.dxf`, `run a.dxf … --save a.dxf` and
`run a.dxf … --export a.dxf` used to overwrite the input. They are now a
usage error (exit 2, `error: …` on stderr, nothing opened or written)
unless `--in-place` is given. "Same file" is file identity (same-file
crate), so `dir/./a.dxf` and a symlink to the input count. An output
that does not exist, or a failed identity check, is not refused; the
open or write reports the real error. Replacing an existing file that
is not an input is still allowed.
Scope: `--save` and `--export` only. A save that a --cmd or script runs
(for example `saveas`) is the engine's and is not checked.
Every write the CLI makes (convert OUT, --save, --export, sample OUT) resolves
an existing output first, so writing through a symlink updates the file it
points to and the link stays a link; the atomic rename used to replace the
link with a regular file, leaving the target (with --in-place, the input)
unchanged.
* feat(cli): --compact for JSON output
RFC-0012 rule 1.3: info and commands pretty-print their JSON by
default; --compact prints it on one line. run prints one JSON line per
--cmd result (JSON lines, one result per step), as its help and the
README document, so it takes no --compact. mcp's stdout is the MCP
protocol and is unchanged.
Behaviour change: info and commands accept --compact (it was an
unknown flag, exit 2). Their default output is unchanged.
docs/cli.md says which subcommand prints which form, and lists perf's
text table (no --json) and saves a --cmd or script runs (not checked
by --in-place) under "Not followed yet".
* fix(cadcraft): one Switch parser for boolean environment switches
CADCRAFT_VSYNC, CADCRAFT_SYSTEM_CURSOR, CADCRAFT_WAYLAND and
CADCRAFT_NO_NATIVE_MENU each had their own idea of "on". They now share
cadcraft_ui_egui::switch::Switch: 1/on/true/yes is On, 0/off/false/no is
Off (case- and space-insensitive), unset or blank is Default, and any
other value is Default and logged with the variable name. The ENVIRONMENT
help text says the same for all four.
Behaviour changes (old -> new):
- CADCRAFT_WAYLAND=0 counted as on (any value did); now only On keeps
native Wayland.
- CADCRAFT_NO_NATIVE_MENU=0 counted as on (any value did); now only On
keeps the menus inside the window.
- CADCRAFT_SYSTEM_CURSOR treated every value except ""/0/false/off as on;
now only the On spellings do, and an unknown value is logged.
- CADCRAFT_VSYNC is unchanged except that an unknown value is logged.
* fix(cli): keep paths as paths until a file is opened
A path is now text only where text is needed.
- cadcraft: FileNames.unreadable is Vec<PathBuf>, logged with {:?}
(escaped bytes) instead of a lossy rendering.
- cadcraft-cli: unicode() is called only inside open(), the one place
that sends a path to the engine as JSON. convert, sample and run
--save/--export pass the output as &Path. write_atomic, export,
export_framed and OutputError::Write take or hold a Path/PathBuf;
errors use .display(), and the `wrote ...` line uses .display().
- Dropped the dead find_subcommand lookup in launch_args (LaunchArgs has
no subcommands).
- docs/cli.md: non-Unicode names are refused only where a drawing is
opened.
Behaviour change: an output name that is not valid Unicode used to be
refused (exit 1); it is now written under that exact name. Opening such
a drawing is still refused.
Lossy strings that remain (all feed APIs that take text and only look at
the extension, never at the path the bytes are written to):
- cadcraft_io::write / write_framed (name picks the format) in sample.rs
and output.rs::export_framed;
- cadcraft_io::is_image in convert.rs, run.rs and args.rs (framing flag
check);
- cadcraft_engine::cmd::file::drops_unknown in export_framed.
These go away when cadcraft_io and the engine take impl AsRef<Path>
(the A1 follow-up). A non-Unicode extension is not a known format either
way, so it is rejected as an unsupported type, as before.
* refactor(cadcraft): read the environment once into Env
The desktop app reads its CADCRAFT_* variables once at startup into
apps/cadcraft/src/env.rs `Env` and passes the values on; no library crate
reads a CADCRAFT_* variable at run time any more.
- One `VARS` table (name, help line) feeds both the reader and the
Environment section of `cadcraft --help`. A test checks that every
variable the reader asks for is in the table, that every table entry
sets a field, and that the help lists every entry.
- vsync goes to `gpu::surface_config`, the system cursor to
`CadApp::set_system_cursor_default` (Options reset and Reset Palettes go
back to it), FONTALT/FONTFALLBACK to the `cadcraft_fonts::ttf` setters
(re-exported as `cadcraft_engine::fonts`), the settings directory to the
new `cadcraft_engine::settings_dir::set`, and to `Workspace::load`.
- The three copies of the settings-directory rule (main.rs, workspace.rs,
the engine's plotter folder) are one: `settings_dir::platform`.
- `Switch::from_env` is gone; `Switch::parse` stays.
- The child-process tests for CADCRAFT_CONTROL_PORT are replaced by tests
that inject the variable lookup (`blank_env_is_unset`) or point
`--control`'s fallback at PATH.
cadcraft-cli: plotter configurations matter to its output (`run --cmd plot`
or a script's PLOT with a user device, and the same through `mcp`), and the
font variables change how convert/run/sample draw text. So, following
RFC-0012 rule 1.7, the variables become fallbacks of flags declared with
`env =`: --config-dir (CADCRAFT_CONFIG_DIR), --fontalt (CADCRAFT_FONTALT)
and --fontfallback (CADCRAFT_FONTFALLBACK) on convert, run, mcp and sample.
Behaviour changes:
- CADCRAFT_CONFIG_DIR set but empty counts as unset everywhere: the
workspace file and the plotter folder use the platform folder (was:
workspace persistence and the plotter folder off). Logs and autosave
already treated it so.
- The workspace file and the plotter folder ignore a relative
XDG_CONFIG_HOME, as logs and autosave did (was: used it).
- A blank CADCRAFT_LOCALE means the system's languages (was: [""]).
- cadcraft-cli: the three variables apply only to convert, run, mcp and
sample, as flag fallbacks (was: read by the engine and fonts in every
subcommand, perf included). A font variable that isn't valid Unicode is
a usage error naming the flag (was: ignored).
* refactor(io): one Format enum for file extensions
`cadcraft_doc::Format` (re-exported as `cadcraft_io::Format`) is the one
place that maps a file name's extension to a format: Dxf, Dwg, Dwt, Svg,
Svgz, Png, Pdf. `from_path` is ASCII case-insensitive; no extension (or a
trailing dot) is Dxf; an unknown extension is None. `READ` and `WRITE`
list the formats `read` and `write` accept. It lives in cadcraft-doc
because the engine depends on cadcraft-io only for tests.
Each place used to match on its own lower-cased extension string and now
matches on Format, with the same results:
- io::read: svg/svgz -> SVG reader, dxf/none -> DXF reader, other ->
try DXF, else Unsupported(ext).
- io::write_with: dxf/none -> DXF, dwg -> DWG bridge, other with binary
-> error, rest -> write_framed.
- io::write_framed: dwt -> DWG; dxf/none/dwg -> write_with; svg, png,
pdf -> image writers; other -> Unsupported(ext).
- io::is_image: svg, png, pdf (not svgz, which is read only).
- engine file.rs is_dxf, drops_unknown (dwg, dwt), is_drawing_file
(dxf, dwg, dwt, none).
Behaviour change (help text only): the CLI help lists the formats from
Format::READ / Format::WRITE instead of hand-written lists. `info` and
`convert IN` now name svg and svgz as readable, and `convert`, `sample`,
`run --save` and `run --export` name dwt (and `sample` pdf) as writable.
A test checks each file argument's help names every format of its list.
* refactor: one version string for the app and the CLI
cadcraft_engine::build_version() builds "<version> (<commit>)" once; the
desktop app's --version, its start-up log line and cadcraft-cli --version
all use it. No change in output.
* refactor(cli): share the environment helpers with the CLI
Move blank_env_is_unset and name_env_vars from the desktop app into
cadcraft_mcp::args and apply them in the CLI parser.
Behaviour changes (old -> new):
- Empty or blank CADCRAFT_FONTALT / CADCRAFT_FONTFALLBACK: reached clap as
Some("") -> counts as unset.
- A bad value from an environment variable: the error named the flag ->
names the variable.
- CADCRAFT_CONFIG_DIR is unchanged: empty still means no settings folder.
- Desktop app: unchanged; it uses the shared helpers.
* refactor: pass paths, not text, through io, engine and open_path
cadcraft_io::{read, write, write_with, write_framed, is_image}, the engine's
drops_unknown/is_drawing_file/read_text_file and CadApp::open_path take
paths (impl AsRef<Path> / &Path). The desktop launch arguments and the CLI
no longer convert paths to text to call them.
Behaviour changes (old -> new):
- A dropped file whose name is not valid Unicode opened with a "can't
read" error naming a mangled path; it now opens natively. (Its path is
not remembered for QSAVE, which then asks where to save.)
- A launch argument that is not valid Unicode was logged and skipped; it
now opens like any other.
- A .dwt export builds the .dwg name as a path, not formatted text.
- On the web, a non-Unicode path passed to open_path reports "the path
is not valid Unicode" instead of a lossy name.
- Opening errors print path.display().
FileNames and LaunchArgs::file_names are removed; the launch tests assert
the parsed PathBufs. New io test: non-Unicode names pick the format by
extension.
* docs(cli): CADCRAFT_CONFIG_DIR is the one flag fallback where empty isn't unset
* fix(engine): refuse a drawing path that isn't valid Unicode
read_file opened such a path with no saved path, so QSAVE could not save back to it.
It now returns an open error naming the file before reading it. The ui-egui opener
already reports read_file errors like any other failed open.
* fix(cli): report a failed write of help or version
A write error on help or version output (other than a closed pipe) now exits 1 and
tries to say so on stderr; it was ignored and exited with the clap code. A closed pipe
keeps the clap exit code.
* fix(cli): --connect needs HOST:PORT
--connect example.com was accepted and failed later. It is now a usage error (exit 2)
unless the value is a non-empty host (or a bracketed IPv6 address) and a u16 port.
The host is not resolved.
* docs(cli): run prints no line for a command with no result
* refactor(cli): explicit imports and smaller functions for the stricter lints
Lints: clippy::wildcard_imports, too_many_arguments, cognitive_complexity (30).
Replaced the non-test `use super::*;` in args, commands, convert, info,
output, perf and run with explicit imports; lib.rs keeps only the imports
its non-test code uses, and its test module imports the rest itself. The
`super::*` in the test modules stays.
A clippy run with all three lints and threshold 30 reports nothing in
apps/cadcraft-cli or in code this branch added, so no function needed
splitting.
507 lines
22 KiB
Rust
507 lines
22 KiB
Rust
//! External references (#525): XATTACH and -XREF on real DXF files in a temporary folder —
|
|
//! loading with `xref|name` symbols, saving only the reference, reopening, Reload/Unload/Detach/
|
|
//! Path, Bind and XBIND, overlays, missing files, circular and deeply nested references.
|
|
|
|
use std::path::{Path, PathBuf};
|
|
use std::sync::Arc;
|
|
|
|
use cadcraft_engine::Session;
|
|
use cadcraft_engine::cmd::file::{IoHooks, set_io};
|
|
use cadcraft_engine::doc::color::Color;
|
|
use cadcraft_engine::doc::geom::Vec3;
|
|
use cadcraft_engine::doc::{Block, Circle, Drawing, Entity, EntityKind, Handle, Insert, Layer, Line, Linetype, MAX_BLOCK_DEPTH, XrefStatus};
|
|
use serde_json::json;
|
|
|
|
fn hooks() {
|
|
set_io(IoHooks {
|
|
read: |b, name| cadcraft_io::read(b, name).map_err(|e| e.to_string()),
|
|
write: |d, name| cadcraft_io::write(d, name).map_err(|e| e.to_string()),
|
|
plot: None,
|
|
});
|
|
}
|
|
|
|
/// A fresh folder for one test.
|
|
fn folder(tag: &str) -> PathBuf {
|
|
let dir = std::env::temp_dir().join(format!("cadcraft-xref-{}-{tag}", std::process::id()));
|
|
let _ = std::fs::remove_dir_all(&dir);
|
|
std::fs::create_dir_all(&dir).unwrap();
|
|
dir
|
|
}
|
|
|
|
fn save(d: &Drawing, path: &Path) {
|
|
std::fs::write(path, cadcraft_io::write(d, path).unwrap()).unwrap();
|
|
}
|
|
|
|
fn line(d: &mut Drawing, layer: &str, a: (f64, f64), b: (f64, f64)) {
|
|
let h = d.new_handle();
|
|
let mut e = Entity::new(h, EntityKind::Line(Line { a: Vec3::new(a.0, a.1, 0.0), b: Vec3::new(b.0, b.1, 0.0) }));
|
|
e.common.layer = layer.into();
|
|
d.model.push(e);
|
|
}
|
|
|
|
fn insert(block: &str, x: f64) -> EntityKind {
|
|
EntityKind::Insert(Insert {
|
|
block: block.into(),
|
|
insert: Vec3::new(x, 0.0, 0.0),
|
|
scale: Vec3::new(1.0, 1.0, 1.0),
|
|
rotation: 0.0,
|
|
attribs: Vec::new(),
|
|
cols: 1,
|
|
rows: 1,
|
|
col_spacing: 0.0,
|
|
row_spacing: 0.0,
|
|
})
|
|
}
|
|
|
|
/// A site plan: a red line on layer Walls and a reference to its block Tree (a circle).
|
|
fn site() -> Drawing {
|
|
let mut d = Drawing::new_metric();
|
|
d.layers.push(Layer { name: "Walls".into(), color: Color::Index(1), ..Layer::default() });
|
|
line(&mut d, "Walls", (0.0, 0.0), (10.0, 0.0));
|
|
let mut tree = Block::new("Tree");
|
|
tree.entities.push(Entity::new(Handle(0x50), EntityKind::Circle(Circle { center: Vec3::ZERO, radius: 1.0 })));
|
|
d.blocks.insert("Tree".into(), Arc::new(tree));
|
|
let h = d.new_handle();
|
|
d.model.push(Entity::new(h, insert("Tree", 5.0)));
|
|
d
|
|
}
|
|
|
|
/// A drawing with an xref named after `path`'s stem (as attach or overlay) placed at 0,0.
|
|
fn host_with(xrefs: &[(&str, bool)]) -> Drawing {
|
|
let mut d = Drawing::new_metric();
|
|
for (path, overlay) in xrefs {
|
|
let name = Path::new(path).file_stem().unwrap().to_string_lossy().into_owned();
|
|
let mut b = Block::new(&name);
|
|
b.xref_path = Some(path.to_string());
|
|
b.xref_overlay = *overlay;
|
|
d.blocks.insert(name.clone(), Arc::new(b));
|
|
let h = d.new_handle();
|
|
d.model.push(Entity::new(h, insert(&name, 0.0)));
|
|
}
|
|
d
|
|
}
|
|
|
|
fn open(s: &mut Session, path: &Path) {
|
|
s.execute("open", &json!({ "path": path.to_string_lossy() })).unwrap();
|
|
}
|
|
|
|
fn status(s: &Session, name: &str) -> Option<XrefStatus> {
|
|
s.doc().unwrap().block(name).map(|b| b.xref_status)
|
|
}
|
|
|
|
fn objects(s: &Session, name: &str) -> usize {
|
|
s.doc().unwrap().block(name).map_or(0, |b| b.entities.len())
|
|
}
|
|
|
|
fn has_layer(s: &Session, name: &str) -> bool {
|
|
s.doc().unwrap().layer(name).is_some()
|
|
}
|
|
|
|
#[test]
|
|
fn attach_loads_the_file_and_saving_keeps_only_the_reference() {
|
|
hooks();
|
|
let dir = folder("attach");
|
|
save(&site(), &dir.join("site.dxf"));
|
|
let host = dir.join("host.dxf");
|
|
save(&Drawing::new_metric(), &host);
|
|
let mut s = Session::new();
|
|
open(&mut s, &host);
|
|
|
|
// A path relative to the host drawing's folder, placed at 100,0.
|
|
let r = s.execute("xattach", &json!({ "path": "site.dxf", "at": [100, 0] })).unwrap();
|
|
assert_eq!(r["name"], "site");
|
|
assert_eq!(status(&s, "site"), Some(XrefStatus::Loaded));
|
|
assert_eq!(objects(&s, "site"), 2, "the site's model space");
|
|
let d = s.doc().unwrap();
|
|
assert_eq!(d.layer("site|Walls").map(|l| l.color), Some(Color::Index(1)), "layers come prefixed");
|
|
assert!(d.block("site|Tree").is_some(), "its blocks too");
|
|
let ext = d.extents(&cadcraft_engine::doc::Space::Model);
|
|
assert!(ext.max.x >= 110.0 - 1e-9, "the xref is drawn where it was placed: {ext:?}");
|
|
|
|
// Saved: the xref block with its path and no objects, and none of its symbols but its layers
|
|
// (their settings in the host are the host's, #886).
|
|
s.execute("qsave", &json!({})).unwrap();
|
|
let saved = cadcraft_io::read(&std::fs::read(&host).unwrap(), "host.dxf").unwrap();
|
|
let b = saved.block("site").unwrap();
|
|
assert_eq!((b.xref_path.as_deref(), b.entities.len()), (Some("site.dxf"), 0));
|
|
assert_eq!(saved.layer("site|Walls").map(|l| l.color), Some(Color::Index(1)));
|
|
assert!(saved.linetypes.iter().all(|t| !t.name.contains('|')), "only the ones its layers use");
|
|
assert!(saved.blocks.keys().all(|n| !n.contains('|')));
|
|
|
|
// Reopened: loaded again from the file.
|
|
let mut t = Session::new();
|
|
open(&mut t, &host);
|
|
assert_eq!(status(&t, "site"), Some(XrefStatus::Loaded));
|
|
assert_eq!(objects(&t, "site"), 2);
|
|
assert!(has_layer(&t, "site|Walls"));
|
|
}
|
|
|
|
#[test]
|
|
fn xref_layer_overrides_survive_save_and_reopen_in_dxf_and_dwg() {
|
|
// #886: a colour and a freeze set in the host on the xref's layers are the host's (AutoCAD,
|
|
// VISRETAIN = 1), so they come back after saving and reopening instead of the xref's own.
|
|
hooks();
|
|
let dir = folder("overrides");
|
|
let mut x = site();
|
|
x.linetypes.push(Linetype::simple("Dashed", "Dashed", &[0.5, -0.25]));
|
|
x.layers.push(Layer { name: "Doors".into(), color: Color::Index(3), linetype: "Dashed".into(), ..Layer::default() });
|
|
save(&x, &dir.join("site.dxf"));
|
|
for ext in ["dxf", "dwg"] {
|
|
let host = dir.join(format!("host.{ext}"));
|
|
save(&host_with(&[("site.dxf", false)]), &host);
|
|
let mut s = Session::new();
|
|
open(&mut s, &host);
|
|
let d = s.doc_mut().unwrap();
|
|
d.layer_mut("site|Walls").unwrap().color = Color::Index(252);
|
|
d.layer_mut("site|Doors").unwrap().frozen = true;
|
|
s.execute("qsave", &json!({})).unwrap();
|
|
|
|
let mut t = Session::new();
|
|
open(&mut t, &host);
|
|
assert_eq!(status(&t, "site"), Some(XrefStatus::Loaded), "{ext}");
|
|
let d = t.doc().unwrap();
|
|
assert_eq!(d.layer("site|Walls").map(|l| l.color), Some(Color::Index(252)), "{ext}: the host's colour, not the xref's red");
|
|
assert_eq!(d.layer("site|Doors").map(|l| (l.frozen, l.color)), Some((true, Color::Index(3))), "{ext}: frozen in the host");
|
|
assert_eq!(d.layer("site|Doors").map(|l| l.linetype.as_str()), Some("site|Dashed"), "{ext}: its linetype is still the xref's");
|
|
assert!(d.linetype("site|Dashed").is_some(), "{ext}");
|
|
}
|
|
}
|
|
|
|
#[test]
|
|
fn reload_unload_path_and_detach() {
|
|
hooks();
|
|
let dir = folder("manage");
|
|
save(&site(), &dir.join("site.dxf"));
|
|
let host = dir.join("host.dxf");
|
|
save(&host_with(&[("site.dxf", false)]), &host);
|
|
let mut s = Session::new();
|
|
open(&mut s, &host);
|
|
assert_eq!(objects(&s, "site"), 2);
|
|
|
|
// A frozen xref layer stays frozen when the xref reloads.
|
|
s.doc_mut().unwrap().layer_mut("site|Walls").unwrap().frozen = true;
|
|
let mut changed = site();
|
|
line(&mut changed, "Walls", (0.0, 5.0), (10.0, 5.0));
|
|
save(&changed, &dir.join("site.dxf"));
|
|
s.execute("-xref", &json!({ "option": "reload", "names": "SITE" })).unwrap();
|
|
assert_eq!(objects(&s, "site"), 3, "reload reads the file again");
|
|
assert!(s.doc().unwrap().layer("site|Walls").unwrap().frozen);
|
|
|
|
s.execute("-xref", &json!({ "option": "unload", "names": "site" })).unwrap();
|
|
assert_eq!((status(&s, "site"), objects(&s, "site")), (Some(XrefStatus::Unloaded), 0));
|
|
assert!(s.doc().unwrap().block("site|Tree").is_none());
|
|
s.execute("-xref", &json!({ "option": "reload" })).unwrap();
|
|
assert_eq!(status(&s, "site"), Some(XrefStatus::Loaded));
|
|
|
|
// Path: point it at another file.
|
|
std::fs::create_dir_all(dir.join("old")).unwrap();
|
|
save(&site(), &dir.join("old/site2.dxf"));
|
|
s.execute("-xref", &json!({ "option": "path", "names": "site", "path": "old/site2.dxf" })).unwrap();
|
|
assert_eq!(s.doc().unwrap().block("site").unwrap().xref_path.as_deref(), Some("old/site2.dxf"));
|
|
assert_eq!(objects(&s, "site"), 2);
|
|
|
|
// ?: the list the palette shows.
|
|
let r = s.execute("-xref", &json!({ "option": "?" })).unwrap();
|
|
assert_eq!(r["xrefs"][0]["status"], "Loaded");
|
|
assert_eq!(r["xrefs"][0]["references"], 1);
|
|
|
|
// Detach: the definition, its references and its symbols go.
|
|
s.execute("-xref", &json!({ "option": "detach", "names": "site" })).unwrap();
|
|
let d = s.doc().unwrap();
|
|
assert!(d.block("site").is_none());
|
|
assert!(d.model.iter().all(|e| !matches!(&e.kind, EntityKind::Insert(_))));
|
|
assert!(d.layers.iter().all(|l| !l.name.contains('|')));
|
|
assert!(s.execute("-xref", &json!({ "option": "reload", "names": "site" })).is_err(), "nothing left to reload");
|
|
}
|
|
|
|
#[test]
|
|
fn missing_files_leave_the_host_intact() {
|
|
hooks();
|
|
let dir = folder("missing");
|
|
let host = dir.join("host.dxf");
|
|
save(&host_with(&[("gone.dxf", false)]), &host);
|
|
let mut s = Session::new();
|
|
open(&mut s, &host);
|
|
assert_eq!((status(&s, "gone"), objects(&s, "gone")), (Some(XrefStatus::NotFound), 0));
|
|
assert!(s.log.iter().any(|m| m.contains("gone") && m.contains("not loaded")), "{:?}", s.log);
|
|
// The reference survives a save.
|
|
s.execute("qsave", &json!({})).unwrap();
|
|
let saved = cadcraft_io::read(&std::fs::read(&host).unwrap(), "host.dxf").unwrap();
|
|
assert_eq!(saved.block("gone").and_then(|b| b.xref_path.clone()).as_deref(), Some("gone.dxf"));
|
|
|
|
// Attaching a file that isn't there changes nothing.
|
|
let before = s.doc().unwrap().blocks.len();
|
|
assert!(s.execute("xattach", &json!({ "path": "nowhere/x.dxf" })).is_err());
|
|
assert_eq!(s.doc().unwrap().blocks.len(), before);
|
|
|
|
// Found later by its file name next to the host, wherever it was saved from.
|
|
save(&site(), &dir.join("gone.dxf"));
|
|
s.execute("-xref", &json!({ "option": "path", "names": "gone", "path": "C:\\old\\job\\gone.dxf" })).unwrap();
|
|
assert_eq!(status(&s, "gone"), Some(XrefStatus::Loaded));
|
|
}
|
|
|
|
#[test]
|
|
fn overlays_do_not_load_in_the_drawings_that_reference_their_host() {
|
|
hooks();
|
|
let dir = folder("overlay");
|
|
save(&site(), &dir.join("site.dxf"));
|
|
let plan = dir.join("plan.dxf");
|
|
save(&host_with(&[("site.dxf", true)]), &plan);
|
|
let mut s = Session::new();
|
|
open(&mut s, &plan);
|
|
assert_eq!(status(&s, "site"), Some(XrefStatus::Loaded), "an overlay loads in its own host");
|
|
s.execute("qsave", &json!({})).unwrap();
|
|
let saved = cadcraft_io::read(&std::fs::read(&plan).unwrap(), "plan.dxf").unwrap();
|
|
assert!(saved.block("site").unwrap().xref_overlay, "the overlay flag is saved");
|
|
|
|
// Attached plan: its overlaid site isn't loaded; an attached one would be.
|
|
let top = dir.join("top.dxf");
|
|
save(&host_with(&[("plan.dxf", false)]), &top);
|
|
let mut t = Session::new();
|
|
open(&mut t, &top);
|
|
assert_eq!(status(&t, "plan"), Some(XrefStatus::Loaded));
|
|
assert_eq!(objects(&t, "plan|site"), 0);
|
|
s.execute("-xref", &json!({ "option": "type", "names": "site", "overlay": false })).unwrap();
|
|
s.execute("qsave", &json!({})).unwrap();
|
|
t.execute("-xref", &json!({ "option": "reload", "names": "plan" })).unwrap();
|
|
assert_eq!(objects(&t, "plan|site"), 2, "nested attached xrefs load");
|
|
assert!(t.doc().unwrap().layer("plan|site|Walls").is_some());
|
|
}
|
|
|
|
#[test]
|
|
fn circular_and_deeply_nested_references_are_bounded() {
|
|
hooks();
|
|
let dir = folder("cycle");
|
|
// a → b → a.
|
|
let mut a = host_with(&[("b.dxf", false)]);
|
|
line(&mut a, "0", (0.0, 0.0), (1.0, 1.0));
|
|
save(&a, &dir.join("a.dxf"));
|
|
let mut b = host_with(&[("a.dxf", false)]);
|
|
line(&mut b, "0", (0.0, 0.0), (2.0, 2.0));
|
|
save(&b, &dir.join("b.dxf"));
|
|
let mut s = Session::new();
|
|
open(&mut s, &dir.join("a.dxf"));
|
|
assert_eq!(status(&s, "b"), Some(XrefStatus::Loaded));
|
|
assert_eq!(objects(&s, "b|a"), 0, "the circular reference back to a isn't loaded");
|
|
// A drawing can't reference itself.
|
|
assert!(s.execute("xattach", &json!({ "path": "a.dxf", "name": "self" })).is_err());
|
|
|
|
// A chain deeper than the nesting limit stops at the limit.
|
|
let n = MAX_BLOCK_DEPTH + 4;
|
|
for i in 0..n {
|
|
let mut d = host_with(&[(&format!("f{}.dxf", i + 1), false)]);
|
|
line(&mut d, "0", (0.0, 0.0), (i as f64, 1.0));
|
|
save(&d, &dir.join(format!("f{i}.dxf")));
|
|
}
|
|
let mut t = Session::new();
|
|
open(&mut t, &dir.join("f0.dxf"));
|
|
assert_eq!(status(&t, "f1"), Some(XrefStatus::Loaded));
|
|
let depth = t.doc().unwrap().blocks.keys().map(|k| k.matches('|').count()).max().unwrap_or(0);
|
|
assert!(depth <= MAX_BLOCK_DEPTH, "nesting is bounded: {depth}");
|
|
}
|
|
|
|
#[test]
|
|
fn bind_makes_the_xref_part_of_the_drawing() {
|
|
hooks();
|
|
let dir = folder("bind");
|
|
save(&site(), &dir.join("site.dxf"));
|
|
let host = dir.join("host.dxf");
|
|
save(&host_with(&[("site.dxf", false)]), &host);
|
|
let mut s = Session::new();
|
|
open(&mut s, &host);
|
|
|
|
// XBIND copies one layer; the xref stays attached.
|
|
let r = s.execute("xbind", &json!({ "type": "layer", "names": "site|walls" })).unwrap();
|
|
assert_eq!(r["bound"][0], "site$0$Walls");
|
|
assert!(s.doc().unwrap().block("site").unwrap().xref_path.is_some());
|
|
|
|
// Bind: a normal block, its symbols renamed site$n$name.
|
|
s.execute("-xref", &json!({ "option": "bind", "names": "site" })).unwrap();
|
|
let d = s.doc().unwrap();
|
|
let b = d.block("site").unwrap();
|
|
assert!(b.xref_path.is_none());
|
|
assert_eq!(b.entities.len(), 2);
|
|
assert!(d.layer("site$1$Walls").is_some(), "the next free number after the XBIND copy");
|
|
assert!(d.block("site$0$Tree").is_some());
|
|
assert!(d.layers.iter().all(|l| !l.name.contains('|')));
|
|
assert!(b.entities.iter().any(|e| matches!(&e.kind, EntityKind::Insert(i) if i.block == "site$0$Tree")));
|
|
|
|
// It is saved as a normal block now.
|
|
s.execute("qsave", &json!({})).unwrap();
|
|
let saved = cadcraft_io::read(&std::fs::read(&host).unwrap(), "host.dxf").unwrap();
|
|
assert_eq!(saved.block("site").map(|b| b.entities.len()), Some(2));
|
|
assert!(saved.layer("site$1$Walls").is_some());
|
|
|
|
// Insert type: the symbols keep their own names (an existing one is used instead).
|
|
let mut t = Session::new();
|
|
let host2 = dir.join("host2.dxf");
|
|
let mut h2 = host_with(&[("site.dxf", false)]);
|
|
h2.layers.push(Layer { name: "Walls".into(), color: Color::Index(5), ..Layer::default() });
|
|
save(&h2, &host2);
|
|
open(&mut t, &host2);
|
|
t.execute("-xref", &json!({ "option": "bind", "insert": true })).unwrap();
|
|
let d = t.doc().unwrap();
|
|
assert_eq!(d.layer("Walls").map(|l| l.color), Some(Color::Index(5)));
|
|
assert!(d.block("Tree").is_some());
|
|
assert!(d.layers.iter().all(|l| !l.name.contains('|')));
|
|
|
|
// Unloaded xrefs can't be bound.
|
|
let mut u = Session::new();
|
|
open(&mut u, &host2);
|
|
u.execute("-xref", &json!({ "option": "unload" })).unwrap();
|
|
assert!(u.execute("-xref", &json!({ "option": "bind" })).is_err());
|
|
}
|
|
|
|
#[test]
|
|
fn command_line_xref_and_xattach() {
|
|
hooks();
|
|
let dir = folder("cmdline");
|
|
save(&site(), &dir.join("site.dxf"));
|
|
let host = dir.join("host.dxf");
|
|
save(&Drawing::new_metric(), &host);
|
|
let mut s = Session::new();
|
|
open(&mut s, &host);
|
|
|
|
s.script("-XREF\nA\nsite.dxf\n50,0\n2\n\n0\n").unwrap();
|
|
assert!(s.running.is_none(), "finished: {}", s.prompt_text());
|
|
let ins: Vec<(String, f64, f64)> = s
|
|
.doc()
|
|
.unwrap()
|
|
.model
|
|
.iter()
|
|
.filter_map(|e| match &e.kind {
|
|
EntityKind::Insert(i) => Some((i.block.clone(), i.insert.x, i.scale.y)),
|
|
_ => None,
|
|
})
|
|
.collect();
|
|
assert_eq!(ins, vec![("site".to_string(), 50.0, 2.0)]);
|
|
|
|
s.script("-XREF\n?\n\n").unwrap();
|
|
assert!(s.log.iter().any(|m| m.starts_with("site") && m.contains("Loaded")), "{:?}", s.log);
|
|
s.script("-XREF\nU\nsite\n").unwrap();
|
|
assert_eq!(status(&s, "site"), Some(XrefStatus::Unloaded));
|
|
s.script("-XREF\nR\n*\n").unwrap();
|
|
assert_eq!(status(&s, "site"), Some(XrefStatus::Loaded));
|
|
s.script("XBIND\nLA\nsite|Walls\n").unwrap();
|
|
assert!(has_layer(&s, "site$0$Walls"));
|
|
s.script("XATTACH\nsite.dxf\n0,0\n1\n\n0\n").unwrap();
|
|
assert!(s.running.is_none(), "finished: {}", s.prompt_text());
|
|
assert_eq!(s.doc().unwrap().blocks.values().filter(|b| b.xref_path.is_some()).count(), 1, "the same xref is used again");
|
|
s.script("-XREF\nD\nsite\n").unwrap();
|
|
assert!(s.doc().unwrap().block("site").is_none());
|
|
assert!(s.undo().unwrap().is_some());
|
|
assert!(s.doc().unwrap().block("site").is_some(), "detach undoes");
|
|
}
|
|
|
|
#[test]
|
|
fn hostile_xref_input_never_panics() {
|
|
hooks();
|
|
let dir = folder("hostile");
|
|
std::fs::write(dir.join("junk.dxf"), b"0\nSECTION\n2\nENTITIES\n0\nLINE\n10\nnan\n").unwrap();
|
|
std::fs::write(dir.join("empty.dxf"), b"").unwrap();
|
|
let host = dir.join("host.dxf");
|
|
let mut d = host_with(&[("junk.dxf", false), ("empty.dxf", false), ("../../../../../../dev/null", false)]);
|
|
let mut blank = Block::new("blank");
|
|
blank.xref_path = Some(String::new());
|
|
d.blocks.insert("blank".into(), Arc::new(blank));
|
|
save(&d, &host);
|
|
let mut s = Session::new();
|
|
open(&mut s, &host);
|
|
let dirpath = dir.to_string_lossy().into_owned();
|
|
for p in [
|
|
json!({}),
|
|
json!({ "path": "" }),
|
|
json!({ "path": dirpath }),
|
|
json!({ "path": "junk.dxf", "scale": 0 }),
|
|
json!({ "path": "junk.dxf", "scale": [1, "x"] }),
|
|
json!({ "path": "junk.dxf", "rotation": 1e308, "name": "a|b" }),
|
|
json!({ "path": "\u{0}" }),
|
|
json!({ "path": 5 }),
|
|
] {
|
|
let _ = s.execute("xattach", &p);
|
|
}
|
|
for p in [
|
|
json!({ "option": "nonsense" }),
|
|
json!({ "option": "path" }),
|
|
json!({ "option": "bind", "names": [1, 2] }),
|
|
json!({ "option": "detach", "names": "*" }),
|
|
json!({ "option": "reload", "names": "*" }),
|
|
] {
|
|
let _ = s.execute("-xref", &p);
|
|
}
|
|
for p in [json!({}), json!({ "type": "layer" }), json!({ "type": "block", "names": "|" }), json!({ "type": "x", "names": "*" })] {
|
|
let _ = s.execute("xbind", &p);
|
|
}
|
|
let _ = s.execute("xopen", &json!({ "name": "nothing" }));
|
|
let _ = s.execute("xopen", &json!({}));
|
|
}
|
|
|
|
/// A drawing with xrefs `(name, saved path)`, each placed at 0,0.
|
|
fn host_with_paths(xrefs: &[(&str, &str)]) -> Drawing {
|
|
let mut d = Drawing::new_metric();
|
|
for (name, path) in xrefs {
|
|
let mut b = Block::new(name);
|
|
b.xref_path = Some(path.to_string());
|
|
d.blocks.insert(name.to_string(), Arc::new(b));
|
|
let h = d.new_handle();
|
|
d.model.push(Entity::new(h, insert(name, 0.0)));
|
|
}
|
|
d
|
|
}
|
|
|
|
#[test]
|
|
fn saved_xref_paths_are_followed_beside_the_host_or_on_this_computer_only() {
|
|
hooks();
|
|
let base = folder("saved-paths");
|
|
let (dir, elsewhere) = (base.join("plans"), base.join("elsewhere"));
|
|
std::fs::create_dir_all(dir.join("parts")).unwrap();
|
|
std::fs::create_dir_all(&elsewhere).unwrap();
|
|
save(&site(), &elsewhere.join("far.dxf"));
|
|
// A part beside the host that references drawings the same ways.
|
|
let mut nested = vec![("unc", r"\\192.0.2.1\share\far.dxf".to_string())];
|
|
// Read as a local path by Linux, and as a network path by the resolver.
|
|
let far_slashes = format!("/{}", elsewhere.join("far.dxf").display());
|
|
if cfg!(unix) {
|
|
nested.push(("slashes", far_slashes.clone()));
|
|
}
|
|
let mut part = host_with_paths(&nested.iter().map(|(n, p)| (*n, p.as_str())).collect::<Vec<_>>());
|
|
line(&mut part, "0", (0.0, 0.0), (1.0, 1.0));
|
|
save(&part, &dir.join("parts").join("part.dxf"));
|
|
|
|
let mut xrefs = vec![
|
|
("unc", r"\\192.0.2.1\share\far.dxf".to_string()),
|
|
("url", "file://192.0.2.1/s/far.dxf".to_string()),
|
|
("part", "parts/part.dxf".to_string()),
|
|
];
|
|
if cfg!(unix) {
|
|
xrefs.push(("slashes", far_slashes.clone()));
|
|
}
|
|
let host = dir.join("host.dxf");
|
|
save(&host_with_paths(&xrefs.iter().map(|(n, p)| (*n, p.as_str())).collect::<Vec<_>>()), &host);
|
|
let mut s = Session::new();
|
|
open(&mut s, &host);
|
|
|
|
assert_eq!(status(&s, "part"), Some(XrefStatus::Loaded), "a relative path beside the host: {:?}", s.log);
|
|
assert_eq!(objects(&s, "part"), 1 + nested.len(), "the part's line and references, with nothing loaded into them");
|
|
for name in ["unc", "url", "slashes"].into_iter().take(xrefs.len() - 1) {
|
|
assert_eq!((status(&s, name), objects(&s, name)), (Some(XrefStatus::Unresolved), 0), "{name}: {:?}", s.log);
|
|
assert!(s.log.iter().any(|m| m.contains(&format!("\"{name}\" not loaded")) && m.contains("another computer")), "{name}: {:?}", s.log);
|
|
}
|
|
assert!(s.log.iter().any(|m| m.contains("nested xref \"unc\" not loaded") && m.contains("another computer")), "{:?}", s.log);
|
|
if cfg!(unix) {
|
|
assert!(s.log.iter().any(|m| m.contains("nested xref \"slashes\" not loaded")), "{:?}", s.log);
|
|
}
|
|
let r = s.execute("-xref", &json!({ "option": "?" })).unwrap();
|
|
let found: Vec<(&str, bool)> = r["xrefs"].as_array().unwrap().iter().map(|x| (x["name"].as_str().unwrap(), x["foundAt"].is_string())).collect();
|
|
assert!(found.iter().all(|(n, f)| *f == (*n == "part")), "{found:?}");
|
|
assert!(s.execute("xopen", &json!({ "name": "unc" })).unwrap_err().to_string().contains("another computer"));
|
|
// Reloading follows the saved path the same way.
|
|
s.execute("-xref", &json!({ "option": "reload", "names": "*" })).unwrap();
|
|
assert_eq!(status(&s, "unc"), Some(XrefStatus::Unresolved));
|
|
|
|
// A path the user gives is used as given.
|
|
let far = elsewhere.join("far.dxf");
|
|
s.execute("-xref", &json!({ "option": "path", "names": "unc", "path": far.to_str().unwrap() })).unwrap();
|
|
assert_eq!(status(&s, "unc"), Some(XrefStatus::Loaded));
|
|
}
|