Skip to content
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.

Sign up for GitHub

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

Jump to bottom

relabel: move tests to separate package #14077

Closed
wants to merge 3 commits into from
Closed

relabel: move tests to separate package #14077

wants to merge 3 commits into from

Commits on May 10, 2024

  1. relabel_test: rename TestRelabel to TestProcess and BenchmarkRelabel … 

    …to BenchmarkProcess

We should be naming the test functions based on the function they are testing.
TestRelabel was testing the Process function, so it should be named TestProcess.
Additionally, relabel is an unexported function so it should have its test function as Test_relabel instead.

Signed-off-by: tesla59 <[email protected]>
    @tesla59
    tesla59 committed May 10, 2024
    Configuration menu
    Copy the full SHA
    5c6d23e View commit details
    Browse the repository at this point in the history

  2. relabel: move tests to rebel_test package 

    Signed-off-by: tesla59 <[email protected]>
    @tesla59
    tesla59 committed May 10, 2024
    Configuration menu
    Copy the full SHA
    02b79bc View commit details
    Browse the repository at this point in the history

  3. relabel: rename RelabelTarget to Target 

    Relabel is the name of package so can be removed from the variable name.

Signed-off-by: tesla59 <[email protected]>
    @tesla59
    tesla59 committed May 10, 2024
    Configuration menu
    Copy the full SHA
    4d95aa1 View commit details
    Browse the repository at this point in the history