Scopelint 오류: 범위 범위에서 변수 사용

테스트 함수 TestGetUID에서 코드는 다음과 같은 경고를 표시하는scopelint에서 보고한 오류에 직면합니다. 함수 리터럴 내의 범위 범위에서 변수 x를 사용합니다.

다음 코드를 고려하세요. 라인:

for _, x := range tests {
    t.Run(x.description, func(t *testing.T) {
        client := fake.NewSimpleClientset(x.objs...)
        actual := getUID(client, x.namespace)
        assert.Equal(t, x.expected, actual)
    })
}

x는 테스트 슬라이스를 반복하는 범위 루프 내의 루프 변수이기 때문에 이 줄과 관련된 오류입니다. Scopelint는 t.Run()에 전달된 함수 리터럴에서 x가 사용되고 있음을 감지합니다. 이로 인해 t.Run()이 반환된 후 함수 리터럴이 호출되면 잠재적인 문제가 발생할 수 있습니다.

원인 및 모범 사례

이 문제는 컴파일러가 생성되어 t.Run()에 전달된 함수 리터럴이 이후에 호출되지 않을 것이라고 보장할 수 없기 때문에 발생합니다. t.Run()이 종료됩니다. t.Run()이 반환된 후에 함수 리터럴이 호출되면 x 변수를 참조하게 되며, 이 변수는 루프의 후속 반복에서 얻은 값으로 덮어쓰여졌을 수 있습니다.

Go vet는 이 경고를 다음과 같이 발생시킵니다. 함수 리터럴이 다른 고루틴에서 동시에 실행될 경우 버그나 데이터 경합이 발생할 수 있는 의도하지 않은 동작을 방지합니다.

이러한 경우 권장되는 모범 사례는 다음 중 하나입니다. 루프 변수의 값을 함수 리터럴에 인수로 전달하거나 루프 변수의 사본을 생성하고 함수 리터럴 내에서 복사본을 참조합니다. 함수 리터럴의 서명은 변경할 수 없으므로 권장되는 해결 방법은 복사본을 만드는 것입니다. 예:

x2 := x

이 복사본을 선언한 후, 함수 리터럴은 루프 변수가 아닌 로컬 복사본을 참조합니다. 복사본에 동일한 이름을 지정하는 것은 혼란스러워 보일 수 있지만 복사본을 사용하려는 의도를 분명히 나타냅니다.