move-language / move

Apache License 2.0
2.25k stars 684 forks source link

[Bug] cargo test on M2(Macbook pro) fails in docgen #1095

Open ksolana opened 8 months ago

ksolana commented 8 months ago

🐛 Bug

Fresh clone of move repository (the main branch) after installing all the tools. cargo build passes.

To reproduce

cargo test

Stack trace/error message

FETCHING GIT DEPENDENCY https://github.com/move-language/move.git
UPDATING GIT DEPENDENCY https://github.com/move-language/move.git
UPDATING GIT DEPENDENCY https://github.com/move-language/move.git
INCLUDING DEPENDENCY MoveStdlib
BUILDING Foo
Command `docgen --template template.md --exclude-impl --exclude-private-fun --exclude-specs --include-call-diagrams --include-dep-diagrams --independent-specs --no-collapsed-sections --output-directory doc --references-file template.md --section-level-start 3 --toc-depth 3`:
Generated "doc/template.md"
Generated "doc/Foo.md"

Documentation generation successful!
External Command `grep documentation doc/Foo.md`:
Test documentation comment
External Command `cat doc/template.md`:
This is a test template

This is a test template

Run with `env UB=1` (or `env UPDATE_BASELINE=1`) to save the current output as the new expected output
test run_all::include_exclude_stdlib/args.txt ... ok

Expected Behavior

All tests should pass

System information

ksolana commented 8 months ago

This fixes the error but i'm not sure if this is okay. Can someone please verify on their machine?

diff --git a/language/tools/move-cli/tests/build_tests/simple_build_with_docs/args.exp b/language/tools/move-cli/tests/build_tests/simple_build_with_docs/args.exp
index 85b896215..5cad016e3 100644
--- a/language/tools/move-cli/tests/build_tests/simple_build_with_docs/args.exp
+++ b/language/tools/move-cli/tests/build_tests/simple_build_with_docs/args.exp
@@ -1,6 +1,6 @@
 Command `new --path . Foo`:
 Command `build`:
-FETCHING GIT DEPENDENCY https://github.com/move-language/move.git
+UPDATING GIT DEPENDENCY https://github.com/move-language/move.git
 UPDATING GIT DEPENDENCY https://github.com/move-language/move.git
 INCLUDING DEPENDENCY MoveStdlib
 BUILDING Foo