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

fix: Alternative function clause for put_event with suite_finished #2074

Merged
merged 1 commit into from
Aug 25, 2021

Conversation

Thau
Copy link
Contributor

@Thau Thau commented Aug 24, 2021

Using Elixir 1.12 and ExUnit, a test run ends in this warning:

warning: TeamCityExUnitFormatting does not know how to process event ({:suite_finished, %{async: 64164, load: nil, run: 121385}}).  Please report this message to https://github.com/KronicDeth/intellij-elixir/issues/new.
  /tmp/intellij_elixir/exunit/1.6.0/team_city_ex_unit_formatting.ex:210: TeamCityExUnitFormatting.put_event/2
  /tmp/intellij_elixir/exunit/1.6.0/team_city_ex_unit_formatter.ex:11: TeamCityExUnitFormatter.handle_cast/2
  (stdlib 3.15.2) gen_server.erl:695: :gen_server.try_dispatch/4
  (stdlib 3.15.2) gen_server.erl:771: :gen_server.handle_msg/6
  (stdlib 3.15.2) proc_lib.erl:226: :proc_lib.init_p_do_apply/3

Looks to me like the function call might have changed, so I've added another clause to take that into account, while keeping the previous one for compatibility with previous versions.

Solves #2045, #2016, and #1949

Using Elixir 1.12 and ExUnit, a test run ends in this warning:

```
warning: TeamCityExUnitFormatting does not know how to process event ({:suite_finished, %{async: 64164, load: nil, run: 121385}}).  Please report this message to https://github.com/KronicDeth/intellij-elixir/issues/new.
  /tmp/intellij_elixir/exunit/1.6.0/team_city_ex_unit_formatting.ex:210: TeamCityExUnitFormatting.put_event/2
  /tmp/intellij_elixir/exunit/1.6.0/team_city_ex_unit_formatter.ex:11: TeamCityExUnitFormatter.handle_cast/2
  (stdlib 3.15.2) gen_server.erl:695: :gen_server.try_dispatch/4
  (stdlib 3.15.2) gen_server.erl:771: :gen_server.handle_msg/6
  (stdlib 3.15.2) proc_lib.erl:226: :proc_lib.init_p_do_apply/3
```

Looks to me like the function call might have changed, so I've added another clause to take that into account, while keeping the previous one for compatibility with previous versions.
@Thau Thau changed the title fix: Alternative call shape for put_event with suite_finished fix: Alternative function clause for put_event with suite_finished Aug 24, 2021
@KronicDeth KronicDeth merged commit 4019ce8 into KronicDeth:main Aug 25, 2021
@KronicDeth KronicDeth added this to the 11.13.1 milestone Nov 5, 2021
KronicDeth added a commit that referenced this pull request Nov 5, 2021
KronicDeth added a commit that referenced this pull request Nov 5, 2021
KronicDeth added a commit that referenced this pull request Nov 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants