Creates target/doc in the wrong directory

[dpc]

When used in in a sub-crate of a larger workspace, when running cargo test a file is created:

./<sub-crate-dir>/target/doc/mermaid.min.js

instead of expected ./target/doc/mermaid.min.js for the whole workspace.

Seems to be caused by:

aquamarine/src/attrs.rs

Line 140 in 636675b

let docs_dir = Path::new("./target/doc");

Possibly CARGO_TARGET_DIR should be used instead as per https://doc.rust-lang.org/cargo/reference/environment-variables.html ?

[frehberg]

thanks for filing the issue. You are right "cargo test" should not generate the file target/doc/mermaid.min.js.

In fact the file should be created only and only if invoking "cargo doc"

No file should be created invoking "cargo test" as the tests do not require the JS package.

[frehberg]

Fixing this issue https://github.com/mersinvald/aquamarine/pull/36/commits

• extract into target/doc/MERMAID_FILE only if target/doc exists (created when calling "cargo doc")
• do not extract if target/doc/MERMAID_FILE already exists

[frehberg]

has been fixed in newly released version 0.3.1
https://crates.io/crates/aquamarine/0.3.1

[frehberg]

as with release aquamarine 0.3.1 this issue has been solved