ash-project / ash-project/ash

Likely bugs detected by mistyped calls to Info.calculation/relationship

Open
#1,426 0 comments 0 reactions 0 assignees View on GitHub
bug
Dominant language
Elixir
Stars
2.5k
Forks
426
Avg merge
23h 26m
Merged PRs (30d)
46

Description

If you remove:
def calculation(_, _), do: nil
def relationship(_, _), do: nil

These cause around 60 tests to fail, because there's various paths that result in non string, non atoms being passed to these functions. Mostly this is passing attribute structs rather than names, but there are also cases of passing other structs: querys, calculations etc.

Contributor guide

Open the contributing guide

Research direction

Start by locating Info.calculation/2 and Info.relationship/2 and temporarily removing the clauses described in the issue. Run the test suite, then trace the roughly 60 failures to find calls receiving attribute structs, queries, calculations, or other non-string/non-atom values. Done means the invalid call paths are corrected and the suite passes without those fallback clauses.

Written by the indexing model from the issue text.

Assessment

Tech stack
elixir
Domain
backend
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
42/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.