llnl / llnl/CARE

ArrayUtils - reorganize and more tests

Open
#13 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
C++
Stars
32
Forks
7
Avg merge
19h 49m
Merged PRs (30d)
14

Description

Two issues
1) Most functions in array_utils.h are declared at the top of the file, but some are declared somewhere in the middle. Move all function declarations to the top for consistency.
2) Many of the functions declared in the middle of the file don't have tests (I missed them). Add tests to TestArrayUtils.cpp

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by comparing the declarations in array_utils.h with the existing cases in TestArrayUtils.cpp. Move the declarations to the file’s top and add coverage for the currently untested functions, then verify that the ArrayUtils tests pass.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
testing
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.