Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG] Broken BusyBox compatibility #617

Open
lens0021 opened this issue Nov 25, 2024 · 3 comments
Open

[BUG] Broken BusyBox compatibility #617

lens0021 opened this issue Nov 25, 2024 · 3 comments
Labels
bug Something isn't working

Comments

@lens0021
Copy link
Contributor

Describe the bug
Run cargo test while BusyBox sed falis

failures:

---- tests::stdlib::test_stdlib_src_tests_stdlib_capitalize_ab stdout ----
thread 'tests::stdlib::test_stdlib_src_tests_stdlib_capitalize_ab' panicked at src/tests/mod.rs:20:28:
assertion `left == right` failed
  left: "Uhello world"
error: test failed, to rerun pass `--bin amber`
 right: "Hello world"

---- tests::stdlib::test_stdlib_src_tests_stdlib_replace_regex_ext_ab stdout ----
thread 'tests::stdlib::test_stdlib_src_tests_stdlib_replace_regex_ext_ab' panicked at src/tests/mod.rs:20:28:
assertion `left == right` failed
  left: "abc123def\naeon aeons ### eons\n#path#to#file.txt"
 right: "abc[123]def\naeon aeons ### eons\n#path#to#file.txt"

---- tests::validity::test_validity_src_tests_validity_import_with_trailing_comma_ab stdout ----
thread 'tests::validity::test_validity_src_tests_validity_import_with_trailing_comma_ab' panicked at src/tests/mod.rs:20:28:
assertion `left == right` failed
  left: "bb\n3"
 right: "c\n3"


failures:
    tests::stdlib::test_stdlib_src_tests_stdlib_capitalize_ab
    tests::stdlib::test_stdlib_src_tests_stdlib_replace_regex_ext_ab
    tests::validity::test_validity_src_tests_validity_import_with_trailing_comma_ab

To Reproduce
A workflow is ready: #616
Or you can do that directly:

cargo run -- build src/tests/stdlib/capitalize.ab
cargo run -- build src/tests/stdlib/replace_regex_ext.ab
cargo run -- build src/tests/validity/import_with_trailing_comma.ab
docker run --rm -v $PWD:/root --workdir /root bash bash src/tests/stdlib/capitalize.sh
docker run --rm -v $PWD:/root --workdir /root bash bash src/tests/stdlib/replace_regex_ext.sh
docker run --rm -v $PWD:/root --workdir /root bash bash src/tests/validity/import_with_trailing_comma.sh
@lens0021 lens0021 added the bug Something isn't working label Nov 25, 2024
@github-project-automation github-project-automation bot moved this to 🆕 New in Amber Project Nov 25, 2024
@lens0021
Copy link
Contributor Author

lens0021 commented Nov 26, 2024

The failure of tests::stdlib::test_stdlib_src_tests_stdlib_replace_regex_ext_ab is because of #619.

@lens0021
Copy link
Contributor Author

import_with_trailing_comma.ab also includes replace_regex(), so the cause is the same as replace_regex_ext.ab

@lens0021 lens0021 changed the title [BUG] Broken BusyBox sed compatibility [BUG] Broken BusyBox compatibility Nov 28, 2024
@Ph0enixKM
Copy link
Member

Ph0enixKM commented Dec 4, 2024

Related #616

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants