Skip to content

Add get_agg() and resolve the issue with one of the test unit function#85

Merged
randrescastaneda merged 18 commits intomainfrom
check_error
Jan 23, 2026
Merged

Add get_agg() and resolve the issue with one of the test unit function#85
randrescastaneda merged 18 commits intomainfrom
check_error

Conversation

@Tefera19
Copy link
Copy Markdown
Contributor

@Tefera19 Tefera19 commented Dec 4, 2025

Hi @randrescastaneda , I have created get_agg() and fix the issue in one of the test file.

Copy link
Copy Markdown
Collaborator

@randrescastaneda randrescastaneda left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @Tefera19 ,

Make sure that get_agg() is also tested.

R/get_stats.R Outdated
format <- match.arg(format)

# Extract varibale name that don't have "_code" and "_name" suffixes from countries auxiliary table
df <- get_aux("countries")
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not the right aux file. You need to use one called country_list or something like that. Thanks

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The aux file has been replaced with "country_list"

@Tefera19
Copy link
Copy Markdown
Contributor Author

Test units added for get_agg()

@randrescastaneda randrescastaneda merged commit 7fd27f5 into main Jan 23, 2026
12 of 13 checks passed
@randrescastaneda randrescastaneda deleted the check_error branch January 23, 2026 16:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants