-
Notifications
You must be signed in to change notification settings - Fork 12
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 is marked as failed when it succeded #44
Comments
Does the describe name have any special characters or a dynamic value (string interpolation)? |
describe "Condition 1.
payor override = HHSC
TMHP Managed Care not active with line of business = STAR STAR+PLUS STAR Kids or MMP
TMHP Service Authorization NPI & Provider matches Roster Payor" do
test "no alert if payor override != HHSC", %{resident_0: context} do This is the example |
Btw, do you know how to run the test but not open the floating window? I'm using the output_panel |
It seems I completely forgot about describes when I implemented multiline test names. I don't have many of those in my work project, so I hadn't noticed. Dynamic test names aren't supported either.
See |
I have another similar case for this test: https://github.com/rudiejd/exercism-exercises/blob/f68783e18579f361e63ce875e4b197c55b3f51be/elixir/captains-log/test/captains_log_test.exs#L33 There is no dynamic test name, but the Strangely, the output file shows this test as succeeding: {
"id": "/home/jd/exercism/elixir/captains-log/test/captains_log_test.exs::random_ship_registry_number::start with \"NCC-\"",
"output": "/tmp/nvim.jd/zMpzmd/28/test_output_122641753",
"status": "passed",
"seed": 0,
"errors": []
} And the output is just the generic |
Probably related to the quotes in the test name. |
Good call! That fixes it. I'm not sure whether the quote issue is an issue with the |
Probably |
I'll open a separate issue for this one then, and maybe take a crack at it later 👍 |
Not sure why. I have a multiline describe do.. block. I attach a screenshot.
It also succeeds in the console
Here's my setup:
The text was updated successfully, but these errors were encountered: