Skip to content

nexthopbmc: add CIT busybox syntax test - #294

Open
nate-nexthop wants to merge 1 commit into
facebook:heliumfrom
nexthop-ai:nate.busybox-syntax
Open

nate-nexthop wants to merge 1 commit into
facebook:heliumfrom
nexthop-ai:nate.busybox-syntax

Conversation

@nate-nexthop

Copy link
Copy Markdown
Contributor

Description

Add the common busybox shell syntax check for nexthopbmc.

Motivation

Ensure busybox syntax is correct on nexthopbmc.

Test Plan

Run on a nexthopbmc with tests2 copied to /run/tests2:

$ python3 cit_runner.py --run-test tests.nexthopbmc.test_busybox_syntax
test_busybox_usage_strings (tests.nexthopbmc.test_busybox_syntax.BusyBoxSyntaxTest.test_busybox_usage_strings) ... ok

----------------------------------------------------------------------
Ran 1 test in 1.995s

OK

Summary:
Add the common busybox shell syntax check for nexthopbmc.

# Test Plan

Run on a nexthopbmc with tests2 copied to /run/tests2:

    $ python3 cit_runner.py --run-test tests.nexthopbmc.test_busybox_syntax
    test_busybox_usage_strings (tests.nexthopbmc.test_busybox_syntax.BusyBoxSyntaxTest.test_busybox_usage_strings) ... ok

    ----------------------------------------------------------------------
    Ran 1 test in 1.995s

    OK
@meta-cla meta-cla Bot added the CLA Signed label Sep 22, 2026
@nate-nexthop

Copy link
Copy Markdown
Contributor Author

@ezeob002 FYI, more nexthopbmc tests.

@meta-codesync

meta-codesync Bot commented Sep 22, 2026

Copy link
Copy Markdown

This pull request has been imported. If you are a Meta employee, you can view this in D121233870. (Because this pull request was imported automatically, there will not be any future comments.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant