Skip to content

Tests: Add unit tests#19

Open
natalialuzuriaga wants to merge 10 commits intomainfrom
DSACMS/nat/tests
Open

Tests: Add unit tests#19
natalialuzuriaga wants to merge 10 commits intomainfrom
DSACMS/nat/tests

Conversation

@natalialuzuriaga
Copy link
Contributor

@natalialuzuriaga natalialuzuriaga commented Feb 24, 2026

Tests: Add unit tests

Problem

Currently, repo-sunsetter does not have tests. We would like to add tests that run as a PR check for development going forward.

Solution

  • Created tests/units containing unit tests for:

    • update-readme.sh
    • download-checklist.sh
    • fetch-tier.sh
  • Created a tests.yml, a workflow file that runs tests upon every PR and push to main, uses bats for tests and shellcheck for linting

  • Resolves linting errors identified by shellcheck

Result

repo-sunsetter now has tests!

Test Plan

Tests are passing as shown below!

Signed-off-by: Natalia Luzuriaga <natalia.luzuriaga@cms.hhs.gov>
Signed-off-by: Natalia Luzuriaga <natalia.luzuriaga@cms.hhs.gov>
Signed-off-by: Natalia Luzuriaga <natalia.luzuriaga@cms.hhs.gov>
Signed-off-by: Natalia Luzuriaga <natalia.luzuriaga@cms.hhs.gov>
@natalialuzuriaga natalialuzuriaga self-assigned this Feb 24, 2026
@natalialuzuriaga natalialuzuriaga added documentation Improvements or additions to documentation enhancement New feature or request labels Feb 24, 2026
Signed-off-by: Natalia Luzuriaga <natalia.luzuriaga@cms.hhs.gov>
Signed-off-by: Natalia Luzuriaga <natalia.luzuriaga@cms.hhs.gov>
Signed-off-by: Natalia Luzuriaga <natalia.luzuriaga@cms.hhs.gov>
Signed-off-by: Natalia Luzuriaga <natalia.luzuriaga@cms.hhs.gov>
Signed-off-by: Natalia Luzuriaga <natalia.luzuriaga@cms.hhs.gov>
Signed-off-by: Natalia Luzuriaga <natalia.luzuriaga@cms.hhs.gov>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant