Remove unnecessary trips to the DB #1588
Triggered via pull request
September 20, 2024 15:01
RichDom2185
Status
Failure
Total duration
3m 30s
Artifacts
–
Annotations
6 errors and 17 warnings
|
Run CI:
test/cadet_web/controllers/assessments_controller_test.exs#L50
test GET /, all roles renders assessments overview (CadetWeb.AssessmentsControllerTest)
|
|
Run CI:
test/cadet_web/controllers/assessments_controller_test.exs#L241
test GET /, non-students renders unpublished assessments (CadetWeb.AssessmentsControllerTest)
|
|
Run CI:
test/cadet_web/controllers/assessments_controller_test.exs#L132
test GET /, student only does not render unpublished assessments (CadetWeb.AssessmentsControllerTest)
|
|
Run CI:
test/cadet_web/admin_controllers/admin_assessments_controller_test.exs#L110
test GET /:course_reg_id, staff only renders assessments overview of admin by staff (CadetWeb.AdminAssessmentsControllerTest)
|
|
Run CI:
test/cadet_web/admin_controllers/admin_assessments_controller_test.exs#L59
test GET /:course_reg_id, staff only renders assessments overview of student by staff (CadetWeb.AdminAssessmentsControllerTest)
|
|
Run CI
Process completed with exit code 2.
|
|
Run CI:
lib/cadet/assessments/assessments.ex#L943
variable "team" is unused (if the variable is not meant to be used, prefix it with an underscore)
|
|
Run CI:
lib/cadet/assessments/assessments.ex#L2682
variable "cr_id" is unused (if the variable is not meant to be used, prefix it with an underscore)
|
|
Run CI:
priv/repo/migrations/20210531155751_multitenant_upgrade.exs#L201
variable "configs" is unused (if the variable is not meant to be used, prefix it with an underscore)
|
|
Run CI
:included_environments option is deprecated. Use :dsn to control whether to send events to Sentry.
|
|
Run CI
CadetWeb.Endpoint.init/2 is deprecated, use config/runtime.exs instead or pass additional options when starting the endpoint in your supervision tree
|
|
Run CI:
priv/repo/seeds.exs#L87
variable "i" is unused (if the variable is not meant to be used, prefix it with an underscore)
|
|
Run CI:
priv/repo/seeds.exs#L121
variable "i" is unused (if the variable is not meant to be used, prefix it with an underscore)
|
|
Run CI
:included_environments option is deprecated. Use :dsn to control whether to send events to Sentry.
|
|
Run CI
CadetWeb.Endpoint.init/2 is deprecated, use config/runtime.exs instead or pass additional options when starting the endpoint in your supervision tree
|
|
Run CI:
test/cadet_web/admin_controllers/admin_assessments_controller_test.exs#L8
unused alias Assessments
|
|
Run CI:
test/cadet_web/plug/rate_limiter_test.exs#L25
variable "key" is unused (if the variable is not meant to be used, prefix it with an underscore)
|
|
Run CI:
test/cadet_web/plug/rate_limiter_test.exs#L3
unused import Plug.Conn
|
|
Run CI
attempting to determine the presence of embed_many field :choices with validate_required/3 or field_missing?/2 which has no effect. You can pass the :required option to Ecto.Changeset.cast_embed/3 to achieve this.
|
|
Run CI
attempting to determine the presence of embed_many field :choices with validate_required/3 or field_missing?/2 which has no effect. You can pass the :required option to Ecto.Changeset.cast_embed/3 to achieve this.
|
|
Run CI
attempting to determine the presence of embed_many field :choices with validate_required/3 or field_missing?/2 which has no effect. You can pass the :required option to Ecto.Changeset.cast_embed/3 to achieve this.
|
|
Run CI
attempting to determine the presence of embed_many field :choices with validate_required/3 or field_missing?/2 which has no effect. You can pass the :required option to Ecto.Changeset.cast_embed/3 to achieve this.
|
|
Run CI
attempting to determine the presence of embed_many field :choices with validate_required/3 or field_missing?/2 which has no effect. You can pass the :required option to Ecto.Changeset.cast_embed/3 to achieve this.
|