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

Test Suite Assumes Default Config #4237

Open
nekodjin opened this issue Oct 12, 2022 · 0 comments
Open

Test Suite Assumes Default Config #4237

nekodjin opened this issue Oct 12, 2022 · 0 comments
Labels
C-bug Category: This is a bug

Comments

@nekodjin
Copy link

Summary

The test suite for Helix assumes that Helix is using the default config. If Helix - correctly - loads a custom user config which causes the test suite's assumptions about the config to fail, the tests will - incorrectly - fail. This affects platforms on which the helix package performs cargo test in order to verify the installation (e.g. the AUR package helix-git).

Reproduction Steps

Create a custom config file (~/.config/helix/languages.toml) that changes the indentation for C from two spaces to four. Run cargo test.

Helix log

No response

Platform

Arch Linux

Terminal Emulator

Windows Terminal

Helix Version

helix-git-r3147.eef9eff2-1 (AUR)

@nekodjin nekodjin added the C-bug Category: This is a bug label Oct 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-bug Category: This is a bug
Projects
None yet
Development

No branches or pull requests

1 participant