-
-
Notifications
You must be signed in to change notification settings - Fork 194
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
Either fix ert failures or mark them as expected #1201
Comments
The failing test cases need to be corrected, they should not be failing. I am slowly fixing some but it's a long process. |
Oh, yes, I absolutely get that. Sorry, if that sounded pressuring. I was just reading the contributing guide, which advised to run a test-suite, which I ran and it showed a lot of errors. I haven't found any documentation about that in the repo, so I decided to make an issue for others who might be confused. |
No worries :) |
fix some of the tests; partially addresses #1201
I'll make a new issue with all the failing tests so I can track progress individually. |
Expected behavior
Running
cask exec ert-runner
results in no failing tests.Actual behavior
First,
ert-runner
complains about duplicated test names. However, even after we fix those, we get 42 failures. The best and simplest way to deal with those is to mark them as «expected failures». It is slightly complicated due to some tests being wrapped in macros.Steps to reproduce the problem
Run
cask build
followed bycask exec ert-runner
.Backtraces if necessary (
M-x toggle-debug-on-error
)Environment & version information
smartparens
version: 20231024.1804major-mode
:fundamental-mode
M-x emacs-version
): GNU Emacs 29.2 (build 1, x86_64-pc-linux-gnu, GTK+ Version 3.24.40, cairo version 1.18.0)The text was updated successfully, but these errors were encountered: