Elixir tests

test: integration test for stats on other nodes #630

Sign in to view logs

Triggered via pull request February 4, 2025 16:48

hauleth

synchronize #560

test/integration-test-for-stats-on-other-node

Status Failure

Total duration 16m 5s

Artifacts –

elixir.yml

on: pull_request

Formatting checks

Run integration tests

Annotations

2 errors and 14 warnings

Run tests: test/supavisor/client_handler/stats_test.exs#L104

test client network usage another instance do not send events here (Supavisor.ClientHandler.StatsTest)

Process completed with exit code 1.

Code style: test/support/e2e_case.ex#L23

Nested modules could be aliased at the top of the invoking module.

Code style: test/supavisor_web/controllers/metrics_controller_test.exs#L37

Nested modules could be aliased at the top of the invoking module.

Code style: test/supavisor/monitoring/prom_ex_test.exs#L115

Nested modules could be aliased at the top of the invoking module.

Code style: test/supavisor/monitoring/prom_ex_test.exs#L87

Nested modules could be aliased at the top of the invoking module.

Code style: test/supavisor/monitoring/prom_ex_test.exs#L59

Nested modules could be aliased at the top of the invoking module.

Code style: test/supavisor/client_handler/stats_test.exs#L105

Nested modules could be aliased at the top of the invoking module.

Code style: test/support/e2e_case.ex#L1

Modules should have a @moduledoc tag.

Code style: test/support/downloader.ex#L1

Modules should have a @moduledoc tag.

Code style: lib/supavisor/protocol/server.ex#L477

Do not use parentheses when defining a function which has no arguments.

Code style: lib/supavisor/db_handler.ex#L316

Avoid negated conditions in if-else blocks.

first..last inside match is deprecated, you must always match on the step: first..last//var or first..last//_ if you want to ignore it

first..last inside match is deprecated, you must always match on the step: first..last//var or first..last//_ if you want to ignore it

Run integration tests

first..last inside match is deprecated, you must always match on the step: first..last//var or first..last//_ if you want to ignore it

first..last inside match is deprecated, you must always match on the step: first..last//var or first..last//_ if you want to ignore it