-
Notifications
You must be signed in to change notification settings - Fork 151
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
Ele 2626 support singular tests in test results runs #1449
Ele 2626 support singular tests in test results runs #1449
Conversation
👋 @NoyaArie |
@@ -20,8 +20,8 @@ | |||
class FiltersAPI(APIClient): | |||
def get_filters( | |||
self, | |||
test_results_totals: Dict[Optional[str], TotalsSchema], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
keys of test_results, test_runs, test_results_totals, test_runs_totals are no longer nullable
@@ -67,14 +67,14 @@ def get_report_data( | |||
models = models_api.get_models(exclude_elementary_models) | |||
sources = models_api.get_sources() | |||
exposures = models_api.get_exposures() | |||
tests = tests_api.get_singular_tests() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
rename
@@ -140,9 +140,11 @@ def get_report_data( | |||
|
|||
invocations_job_identification = defaultdict(list) | |||
for invocation in invocations: | |||
key = invocation.job_name or invocation.job_id |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
rename key to invocation_key
13ec388
to
e52b704
Compare
No description provided.