Pandas: [좋은 첫 번째 이슈] TST: 베어 pytest.raises 허용 안 함

에 만든 2020년 01월 14일 · 51코멘트 · 출처: pandas-dev/pandas

최종 사용자는 디버깅 목적으로 오류 메시지에 의존합니다. 따라서 트리거된 오류에 따라 올바른 오류 메시지가 표시되는지 확인하는 것이 중요합니다.

핵심 아이디어는 이것을 변환하는 것입니다.

with pytest.raises(klass):
    # Some code that raise an error

이에:

with pytest.raises(klass, match=msg):
    # Some code that raise an error

pytest.raises 에 대한 자세한 내용은 여기 를 참조하십시오 .

참고 사항:

제기 오류 메시지 (즉, 특정 팬더 아니다 의미) 외부 오류 메시지가이 경우에, 당신은 사용해야 external_error_raised 대신 pytest.raises .

의 사용 external_error_raised 같은 __exactly__입니다 pytest.raises 의 유일한 차이점은 당신이 전달되지 않는다는 것입니다 match 인수.

예를 들어:

import pandas._testing as tm

def test_foo():
    with tm.external_error_raised(ValueError):
        raise ValueError("foo")

기조 연설:

이 문제를 PR에 연결하는 것을 잊지 말고 붙여넣으십시오.

https://github.com/pandas-dev/pandas/issues/30999

당신의 PR에서.

작업 계획에 대해 의견을 말하여 작업을 두 배로 하지 않도록 하십시오(저를 언급할 필요가 없습니다. 작업 계획을 선언할 수 있습니다. 이미 수행한 작업이 있는지 확인하는 것을 기억하십시오).
"완료"로 표시되어야 하는 파일(더 이상 할 일이 없는 것처럼)이 "완료"로 표시되지 않은 경우, 이에 대해 알려주는 댓글을 달아주세요(그리고 @MomIsBestFriend 넣어 저를 언급하세요.

전체 목록을 직접 생성하려면 다음을 실행할 수 있습니다.

python scripts/validate_unwanted_patterns.py -vt="bare_pytest_raises" pandas/tests/

다음과 같은 단일 파일에 대해 실행할 수도 있습니다.

python scripts/validate_unwanted_patterns.py -vt="bare_pytest_raises" pandas/tests/PATH/TO/SPECIFIC/FILE.py

파일에 베어 pytest.raises 가 포함되어 있으면 스크립트는 다음을 출력합니다.

pandas/tests/arithmetic/test_numeric.py:553:Bare pytests raise have been found. Please pass in the argument 'match' as well the exception

이는 553 라인의 pandas/tests/arithmetic/test_numeric.py 에 pytest.raises

현재 목록은 다음과 같습니다.

[x] 판다/테스트/io/pytables/test_timezones.py
[ ] 팬더/테스트/일반/메소드/test_pipe.py
[ ] pandas/tests/reshape/merge/test_merge_asof.py
[ ] 팬더/테스트/확장/베이스/reduce.py
[x] 팬더/테스트/배열/test_datetimelike.py
[ ] 팬더/테스트/확장/test_boolean.py
[ ] 팬더/테스트/확장/베이스/getitem.py
[ ] 팬더/테스트/배열/부울/test_arithmetic.py
[ ] 팬더/테스트/확장/베이스/setitem.py
[ ] 팬더/테스트/인덱스/간격/test_astype.py
[ ] 팬더/테스트/io/parser/test_network.py
[ ] 팬더/테스트/확장/test_integer.py
[ ] 팬더/테스트/인덱싱/다중 인덱스/test_partial.py
[ ] 팬더/테스트/io/parser/test_python_parser_only.py
[ ] 팬더/tests/io/test_html.py
[ ] 팬더/테스트/감소/test_stat_reductions.py
[ ] 팬더/테스트/dtypes/test_inference.py
[ ] 팬더/테스트/플로팅/test_hist_method.py
[ ] 팬더/테스트/시리즈/apply/test_series_apply.py
[ ] 팬더/테스트/io/excel/test_xlrd.py
[ ] 팬더/테스트/인덱스/test_common.py
[ ] 팬더/테스트/유틸/테스트_assert_series_equal.py
[ ] 팬더/테스트/확장/베이스/ops.py
[ ] 팬더/테스트/io/test_clipboard.py
[ ] 팬더/테스트/플로팅/프레임/테스트_프레임_색상.py
[ ] 팬더/테스트/창/모멘트/test_moments_ewm.py
[ ] 팬더/테스트/io/test_gbq.py
[ ] 판다/테스트/감소/test_reductions.py
[ ] 팬더/테스트/io/test_feather.py
[ ] 팬더/테스트/resample/test_resampler_grouper.py
[ ] 팬더/테스트/인덱스/멀티/테스트_인덱싱.py
[ ] 팬더/테스트/io/test_common.py
[ ] 팬더/테스트/io/test_sql.py
[ ] pandas/tests/plotting/test_series.py
[ ] 팬더/테스트/io/test_fsspec.py
[ ] 팬더/tests/extension/test_floating.py
[ ] 팬더/테스트/인덱스/멀티/테스트_setops.py
[ ] 팬더/테스트/reshape/test_get_dummies.py
[ ] 팬더/테스트/플로팅/프레임/테스트_프레임_서브플롯.py
[ ] 팬더/테스트/플로팅/test_backend.py
[ ] 팬더/테스트/일반/메소드/test_sample.py
[ ] 팬더/테스트/플로팅/test_boxplot_method.py
[ ] 팬더/테스트/io/test_parquet.py
[ ] 팬더/테스트/확장/test_string.py
[ ] 팬더/테스트/io/pytables/test_complex.py
[ ] 팬더/테스트/인덱스/test_numpy_compat.py
[ ] 팬더/테스트/io/test_gcs.py
[ ] 팬더/테스트/io/sas/test_sas7bdat.py
[ ] 팬더/테스트/창/테스트_적용.py
[ ] 팬더/테스트/시리즈/테스트_ufunc.py
[ ] 팬더/테스트/플로팅/프레임/테스트_프레임.py
[ ] pandas/tests/reshape/test_union_categoricals.py
[ ] 팬더/테스트/io/json/test_ujson.py
[ ] 팬더/테스트/인덱싱/test_coercion.py
[ ] 팬더/테스트/io/pytables/test_store.py
[ ] 팬더/tests/computation/test_compat.py
[ ] 판다/테스트/io/json/test_pandas.py
[ ] 팬더/테스트/io/json/test_json_table_schema.py
- [ ] 팬더/테스트/스칼라/test_nat.py

노트:

파일이 지속적으로 이동/이름 변경됨에 따라 목록이 변경될 수 있습니다.

@gfyoung이 처음 열었던 #23922에서 거의 모든 것을 가져왔습니다.

Style Testing good first issue

출처

ShaharNaveh

모든 51 댓글

내가 걸릴 것입니다:

[x] 판다/테스트/test_common.py
[x] 팬더/테스트/테스트_다운스트림.py
[x] pandas/tests/test_errors.py
[x] 팬더/테스트/test_lib.py
[x] 판다/테스트/test_take.py
[x] pandas/tests/internals/test_internals.py
[x] 팬더/테스트/창/테스트_롤링.py

ShaharNaveh 에 2020년 01월 14일

다음 작업을 시작할 것입니다.

팬더/테스트/산술/test_numeric.py
팬더/테스트/산술/test_object.py
팬더/테스트/산술/test_period.py
팬더/테스트/산술/test_timedelta64.py
팬더/테스트/배열/간격/test_interval.py

gdex1 에 2020년 01월 14일

@gdex1 도움이 되었으면 합니다 :)

(숫자는 줄 번호를 나타냅니다)

pandas/tests/arithmetic/test_numeric.py:138
pandas/tests/arithmetic/test_numeric.py:141
pandas/tests/arithmetic/test_numeric.py:190
pandas/tests/arithmetic/test_numeric.py:208
pandas/tests/arithmetic/test_numeric.py:210
pandas/tests/arithmetic/test_numeric.py:212
pandas/tests/arithmetic/test_numeric.py:214
pandas/tests/arithmetic/test_numeric.py:232
pandas/tests/arithmetic/test_numeric.py:234
pandas/tests/arithmetic/test_numeric.py:236
pandas/tests/arithmetic/test_numeric.py:238
pandas/tests/arithmetic/test_numeric.py:519
pandas/tests/arithmetic/test_numeric.py:610
pandas/tests/arithmetic/test_numeric.py:615
pandas/tests/arithmetic/test_numeric.py:617
pandas/tests/arithmetic/test_numeric.py:795
pandas/tests/arithmetic/test_numeric.py:798
pandas/tests/arithmetic/test_numeric.py:819

pandas/tests/arithmetic/test_object.py:140
pandas/tests/arithmetic/test_object.py:152
pandas/tests/arithmetic/test_object.py:154
pandas/tests/arithmetic/test_object.py:278
pandas/tests/arithmetic/test_object.py:280
pandas/tests/arithmetic/test_object.py:282
pandas/tests/arithmetic/test_object.py:284
pandas/tests/arithmetic/test_object.py:298
pandas/tests/arithmetic/test_object.py:301
pandas/tests/arithmetic/test_object.py:315
pandas/tests/arithmetic/test_object.py:318



md5-634e15eb80aa764171dbacd11a06b70b



pandas/tests/arithmetic/test_timedelta64.py:51
pandas/tests/arithmetic/test_timedelta64.py:445
pandas/tests/arithmetic/test_timedelta64.py:607
pandas/tests/arithmetic/test_timedelta64.py:609
pandas/tests/arithmetic/test_timedelta64.py:703
pandas/tests/arithmetic/test_timedelta64.py:705
pandas/tests/arithmetic/test_timedelta64.py:707
pandas/tests/arithmetic/test_timedelta64.py:709
pandas/tests/arithmetic/test_timedelta64.py:741
pandas/tests/arithmetic/test_timedelta64.py:743
pandas/tests/arithmetic/test_timedelta64.py:960
pandas/tests/arithmetic/test_timedelta64.py:972
pandas/tests/arithmetic/test_timedelta64.py:1028
pandas/tests/arithmetic/test_timedelta64.py:1037
pandas/tests/arithmetic/test_timedelta64.py:1039
pandas/tests/arithmetic/test_timedelta64.py:1502
pandas/tests/arithmetic/test_timedelta64.py:1505
pandas/tests/arithmetic/test_timedelta64.py:1508
pandas/tests/arithmetic/test_timedelta64.py:1511
pandas/tests/arithmetic/test_timedelta64.py:1536
pandas/tests/arithmetic/test_timedelta64.py:1591
pandas/tests/arithmetic/test_timedelta64.py:1783
pandas/tests/arithmetic/test_timedelta64.py:1785
pandas/tests/arithmetic/test_timedelta64.py:1911
pandas/tests/arithmetic/test_timedelta64.py:1960
pandas/tests/arithmetic/test_timedelta64.py:1962
pandas/tests/arithmetic/test_timedelta64.py:1968



md5-634e15eb80aa764171dbacd11a06b70b



pandas/tests/arrays/interval/test_interval.py:155

ShaharNaveh 에 2020년 01월 14일

👍1

@gfyoung 목록은 실제로 grep -r -e "pytest.raises([a-zA-Z]*)" pandas/tests -l 에 의해 생성된 것이 아니라 #30755의 스크립트에 의해 생성되었습니다( bare_pytest_raises 라는 유효성 검사 유형). 한 번 문제 본문에 지침을 입력하겠습니다. 병합됩니다 :스마일:

ShaharNaveh 에 2020년 01월 14일

👍1

@MomIsBestFriend 도와드리겠습니다:
팬더/테스트/베이스/test_constructors.py
팬더/테스트/베이스/test_ops.py

shubchat 에 2020년 01월 14일

👍1

나는 이것들을 돌볼 수 있다:
@MomIsBestFriend

팬더/tests/io/test_html.py
팬더/테스트/io/test_parquet.py
팬더/테스트/io/test_sql.py
팬더/테스트/io/test_stata.py
팬더/테스트/플로팅/test_backend.py
팬더/테스트/플로팅/test_boxplot_method.py
팬더/테스트/플로팅/test_frame.py
팬더/테스트/플로팅/test_hist_method.py
팬더/테스트/플로팅/test_series.py
팬더/tests/reductions/test_reductions.py

DylanBrdt 에 2020년 01월 15일

👍1

@MomIsBestFriend https://github.com/pandas-dev/pandas/issues/23922에서 이에 대한 논의가 꽤 있었습니다. 내가 거기에서 말했듯이 반복하기 때문에: 나는 우리가 모든 오류 메시지를 "맹목적으로" 주장해야 한다고 생각하지 않습니다.

그 스레드에서 말한 몇 가지: 내부 오류 메시지로 제한하고, 메시지의 몇 가지 핵심 단어로 일치를 제한하고, 복잡한 패턴을 피하십시오.

또한 오류 메시지를 주장하는 것은 실제로 그것이 훌륭하고 명확한 오류 메시지인지 확인하고 이를 잠재적으로 개선하는 것과 함께 이루어져야 한다고 생각합니다.

다른 문제의 토론에서 주의점 목록을 추출하여 여기에 넣는 것이 좋습니다.

jorisvandenbossche 에 2020년 01월 15일

@jorisvandenbossche

@MomIsBestFriend #23922에서 이에 대해 꽤 많은 토론이 있었습니다. 내가 거기에서 말했듯이 반복하기 때문에: 나는 우리가 모든 오류 메시지를 "맹목적으로" 주장해야 한다고 생각하지 않습니다.

나는 전적으로 동의하지만 문제는 우리가 주장할 오류 메시지와 주장하지 않을 것에 대한 규칙을 어떻게든 정의하고 동시에 이 문제는 "초보자 친화적"이며 훌륭할 것입니다(IMO).

또한 CI에서 이것을 시행할 계획이라면 bare pytest raises 가 의도적으로 "베어"인 것을 어떻게든 표시해야 합니다( isort: skip 스타일의 IMO 주석이면 충분합니다). 사람들은 특정 bare pytest raise 가 의도적으로 베어 있다는 것을 알게 될 것입니다.

그 스레드에서 말한 몇 가지: 내부 오류 메시지로 제한하고, 메시지의 몇 가지 핵심 단어로 일치를 제한하고, 복잡한 패턴을 피하십시오.

내부 오류 메시지를 테스트하고 싶지 않은 이유를 모르겠습니다. 더 자세히 설명해 주시겠습니까?

나는 당신이 https://github.com/pandas-dev/pandas/issues/23922#issuecomment -458551763에서 지적한 요점을 보았고 나는 그것에 대해 +1 이지만 나는 +2 https://github.com/pandas-dev/pandas/issues/23922#issuecomment -458733117 및 https://github.com/pandas-dev/pandas/issues/ 에서 +2 (이해가 되는 경우) 23922#issuecomment -458735169 IMO의 이익이 비용보다 크기 때문입니다.

또한 오류 메시지를 주장하는 것은 실제로 그것이 훌륭하고 명확한 오류 메시지인지 확인하고 이를 잠재적으로 개선하는 것과 함께 이루어져야 한다고 생각합니다.

절대적으로 동의합니다.

다른 문제의 토론에서 주의점 목록을 추출하여 여기에 넣는 것이 좋습니다.

#23922에서 대화를 읽었지만 IMO 가 문제 본문에 "메모"로 넣을 가치가 있는 것을 보지 못했습니다. 제가 놓친 부분을 지적해 주시겠습니까?

ShaharNaveh 에 2020년 01월 15일

#23922에서 대화를 읽었지만 IMO가 문제 본문에 "메모"로 넣을 가치가 있는 것을 보지 못했습니다. 제가 놓친 부분을 지적해 주시겠습니까?

그 문제에서 추가할 내용도 별로 없습니다.

나는 전적으로 동의하지만 문제는 우리가 주장할 오류 메시지와 주장하지 않을 것에 대한 규칙을 어떻게든 정의하고 동시에 이 문제는 "초보자 친화적"이며 훌륭할 것입니다(IMO).

또한, CI에서 이것을 시행할 계획이라면, 베어 pytest가 제기하는 것이 의도적으로 "베어"인 것을 어떻게든 표시해야 하며(isort 스타일의 IMO 주석: 건너뛰기가 충분함), 다른 사람들도 특정 베어 pytest 인상은 의도적으로 베어 있습니다.

테스트할 항목과 테스트하지 않을 항목을 선택하고 선택하는 것이 내가 선호하는 방향이 아닌 부분적인 이유입니다. 또한 때때로 except 블록에서 오류 메시지 문자열을 확인하므로 좋은 오류 메시지는 개발 중에도 도움이 된다고 덧붙였습니다.

또한 이러한 "내부" 메시지가 그다지 중요하지 않은 경우 처음에 오류 메시지가 표시되는 이유는 무엇입니까? 그런 다음 메시지가 비어 있다고 주장하는 도우미를 만듭니다.

gfyoung 에 2020년 01월 16일

내부 오류 메시지를 테스트하고 싶지 않은 이유를 모르겠습니다. 더 자세히 설명해 주시겠습니까?

그래서 저는 "내부 오류 메시지로 제한"이라고 말했지만 "내부"는 약간 모호할 수 있습니다... 제 말은: 팬더 자체에서 발생하는 오류 메시지이고 물론 테스트하려는 것입니다. 하지만 그래서 나는 우리(IMO)가 너무 많은 외부 오류 메시지를 테스트해서는 안 된다는 것을 의미했습니다. 즉, numpy 또는 다른 라이브러리에서 오는 메시지입니다. Numpy는 그것들을 변경할 수 있으며, 우리의 테스트는 numpy의 외관상의 변경으로 인해 실패하기 시작합니다(이것은 가설이 아니며 지난주에 일어난 일이라고 생각합니다).

이제 https://github.com/pandas-dev/pandas/pull/30998#discussion_r366726966 의 다른 컨텍스트에서 "내부"를 사용했습니다 절대 발생해서는 안 되는 내부, 개발자 지향 오류 메시지를 의미했습니다. IMO, 오류 메시지로 정확하게 테스트하는 데 중요하지 않습니다.

나는 당신이 #23922(댓글)에서 지적한 요점을 보았고 나는 그것에 대해 +1이지만 #23922(댓글)과 #23922(댓글)에 대해서는 +2(의미가 있다면)입니다. 왜냐하면 IMO 이익이 비용보다 큽니다.

여기에 링크하는 @simonjayhawkins 의 의견을 넣어 보겠습니다.

나는 그것이 다음과 같은 경우에 도움이 될 것이라는 가정을 하고 있습니다.
매개변수화할 수 있는 테스트 식별
분할해야 하는 테스트 식별
테스트된 고장 모드에 대한 더 나은 이해
테스트에 간접적으로 더 많은 문서 추가
오류 메시지의 일관성을 높일 수 있는 위치 식별
중복되는 테스트 식별
오류 메시지 개선에 도움
잘못된 이유로 현재 통과 중인 테스트를 식별합니다.

모두 유용한 내용입니다. 전적으로 동의합니다. 그러나 그것은 간단하지 않으며 우리가 이 문제에서 그런 것들을 얻고자 한다면 이 문제는 초보자를 위한 것이 아닙니다. 물론 초보자는 이 모든 작업을 한 번에 수행할 필요가 없지만 주장을 추가하는 PR이 아무 작업도 수행하지 않고 "현재 오류 메시지를 pytest.raises 호출에 맹목적으로 추가"하는 경우가 종종 있다는 느낌이 듭니다. 더 나아가(위의 사항).

또한, 위의 사항들이 이 연습을 유용하게 만든다면, 이 문제의 맨 위에 두는 것이 유용한 이에 대한 보다 구체적인 지침이라고 생각합니다.

분명히 하자면, 저는 더 나은 오류 메시지와 더 나은 테스트를 통해 이러한 오류 메시지를 잘
IMO, 대신 "오류 메시지 개선"에 초점을 맞추는 것보다 "베어 pytest 발생 수정"에 초점을 맞추는 것이 더 유용할 것입니다(이렇게 하는 동안 더 잘 테스트).

jorisvandenbossche 에 2020년 01월 16일

또한, 위의 사항들이 이 연습을 유용하게 만든다면, 이 문제의 맨 위에 두는 것이 유용한 이에 대한 보다 구체적인 지침이라고 생각합니다.

이를 추적하기 위해 더 큰 문제를 만드는 것이 합리적일 수 있습니다(이러한 문제에 포함할 가치가 있는 다른 문제는 https://github.com/pandas-dev/pandas/issues/19159 및 https://github.com/pandas-입니다. dev/pandas/issues/21575).

이 부분 자체가 독립적이며 초보자가 접근하기 매우 쉽습니다.

gfyoung 에 2020년 01월 16일

@gfyoung 이 토론과 관련하여 링크한 문제는 어떻습니까?

jorisvandenbossche 에 2020년 01월 16일

@simonjayhawkins 에서 명시적으로 소개한 댓글과 관련이 있습니다.

gfyoung 에 2020년 01월 16일

31072 `stata.py` 에 하나의 누락된 일치 항목이 있습니다.

bashtage 에 2020년 01월 16일

👍1

여기에서 말했듯이 https://github.com/pandas-dev/pandas/pull/31091#issuecomment -575422207 저는 @jorisvandenbossche 의 아이디어에

ShaharNaveh 에 2020년 01월 17일

특정 오류 메시지를 테스트하고 싶지 않다면(외부 메시지에 대해 어느 쪽이든 공정하게 진행할 수 있음) 다음과 같은 도우미 함수를 만들어야 한다고 생각합니다.

~~~파이썬def 외부 오류 발생(예상_예외):반환 pytest.raises(예상_예외, 일치=없음)~~~

이것은 이것이 팬더가 아닌 오류라는 것을 우리의 미래 자아에게 분명히 할 것이며 match=None 는 우리가 벌거벗은 pytest 인상을 위해 개발한 모든 린트 검사를 진정시키는 역할을 합니다.

gfyoung 에 2020년 01월 17일

특정 오류 메시지를 테스트하고 싶지 않다면(외부 메시지에 대해 어느 쪽이든 공정하게 진행할 수 있음) 다음과 같은 도우미 함수를 만들어야 한다고 생각합니다.
def external_error_raised(expected_exception):
   return pytest.raises(expected_exception, match=None)
이것은 이것이 팬더가 아닌 오류라는 것을 우리의 미래 자아에게 명백하게 할 것이며 match=None 는 우리가 벌거벗은 pytest 인상을 위해 개발한 모든 보풀 확인을 진정시키는 역할을 합니다.

+1 .

나는 그 아이디어가 정말 마음에 듭니다. 우리 테스트를 위한 관례로 만들 수 있습니까?

함수/메서드에서 오류가 발생하는지 테스트하는 중이고 오류가 외부 오류인 경우 "pytest.raises```에 match=None 를 넣으면 됩니다.

테스트를 위한 규칙으로 만들 수 있습니까?

그 말은 기여 가이드 에 섹션을 넣는 것을 의미합니다.

ShaharNaveh 에 2020년 01월 17일

함수/메서드에서 오류가 발생하는지 테스트하는 중이고 오류가 외부 오류인 경우 "pytest.raises```에 match=None을 넣으면 됩니다.

나는 당신이 그것을 추가하는 것에 대해 생각할 필요가 없기 때문에 도우미 기능을 선호합니다. 또한 도우미 이름은 우리가 왜 하는지에 대해 훨씬 더 명확합니다.

gfyoung 에 2020년 01월 17일

특정 오류 메시지를 테스트하고 싶지 않다면(외부 메시지에 대해 어느 쪽이든 공정하게 진행할 수 있음) 다음과 같은 도우미 함수를 만들어야 한다고 생각합니다.
def external_error_raised(expected_exception):
   return pytest.raises(expected_exception, match=None)
이것은 이것이 팬더가 아닌 오류라는 것을 우리의 미래 자아에게 명백하게 할 것이며 match=None 는 우리가 벌거벗은 pytest 인상을 위해 개발한 모든 보풀 확인을 진정시키는 역할을 합니다.

@gfyoung 이 도우미 기능을 어디에 두는 것이 좋습니까? (어떤 파일에 있는 것처럼?)

ShaharNaveh 에 2020년 01월 18일

pandas._testing

gfyoung 에 2020년 01월 18일

👍1

안녕하십니까,

다음 작업을 하고 싶습니다.

팬더/테스트/배열/간격/test_ops.py
팬더/테스트/배열/test_array.py
팬더/테스트/배열/test_boolean.py

ukarroum 에 2020년 01월 24일

안녕하세요 - 저는 다음과 같은 일을 하고 싶습니다.

팬더/테스트/산술/test_period.py
팬더/테스트/산술/test_timedelta64.py

GrugLife 에 2020년 02월 01일

안녕하세요 여러분, 다음과 같이 하겠습니다.

팬더/테스트/계산/test_compat.py
팬더/테스트/계산/test_eval.py
팬더/테스트/dtypes/cast/test_upcast.py
팬더/테스트/dtypes/test_dtypes.py

Vlek 에 2020년 02월 02일

@MomIsBestFriend 이것은 이미 완료되었지만 완료로 표시되지 않았습니다.
팬더/테스트/산술/test_numeric.py

업데이트

@MomIsBestFriend 이것들도:
팬더/테스트/산술/test_period.py
팬더/테스트/배열/test_integer.py
팬더/테스트/배열/test_period.py

MarcoGorelli 에 2020년 02월 09일

👍1

이들은 #31852에 포함되어 있습니다.

팬더/tests/extension/decimal/test_decimal.py
팬더/테스트/확장/json/test_json.py
팬더/테스트/확장/test_boolean.py
팬더/테스트/확장/test_categorical.py
팬더/테스트/프레임/인덱싱/test_categorical.py
팬더/테스트/프레임/인덱싱/test_indexing.py
팬더/테스트/프레임/인덱싱/test_where.py
팬더/테스트/프레임/메소드/test_explode.py
팬더/테스트/프레임/메소드/test_isin.py
팬더/테스트/프레임/메소드/test_quantile.py
팬더/테스트/프레임/메소드/test_round.py
팬더/테스트/프레임/메소드/test_sort_values.py
팬더/테스트/프레임/메소드/test_to_dict.py

RaisaDZ 에 2020년 02월 19일

내가 가져갈게

팬더/테스트/io/excel/test_readers.py
팬더/테스트/io/excel/test_writers.py
팬더/테스트/io/excel/test_xlwt.py
팬더/테스트/io/formats/test_format.py
팬더/tests/io/formats/test_style.py
팬더/tests/io/formats/test_to_latex.py

quangngd 에 2020년 03월 19일

@MomIsBestFriend
다음은 표시 없이 수행됩니다.

팬더/테스트/인덱스/datetimes/test_astype.py

pandas/tests/indexes/datetimes/test_tools.py가 존재하지 않습니다

내가 할거야:

pandas/tests/indexes/datetimes/test_constructors.py
팬더/테스트/인덱스/datetimes/test_date_range.py
팬더/tests/indexes/datetimes/test_indexing.py
팬더/테스트/인덱스/datetimes/test_shift.py
팬더/테스트/인덱스/datetimes/test_timezones.py

quangngd 에 2020년 03월 21일

pytest raises 를 감지하는 스크립트가 있으므로 원본 게시물을 업데이트했습니다. 사용 방법에 대한 지침이 포함되어 있습니다. 여전히 질문이 있는 사람이 있으면 언제든지 물어볼 수 있습니다. :)

ShaharNaveh 에 2020년 03월 23일

내가 가져갈게,

팬더/테스트/산술/test_timedelta64.py

팬더/테스트/스칼라/타임스탬프/test_arithmetic.py
팬더/테스트/스칼라/타임스탬프/test_comparisons.py
팬더/테스트/스칼라/타임스탬프/test_constructors.py
팬더/테스트/스칼라/타임스탬프/test_timezones.py
팬더/테스트/스칼라/타임스탬프/test_unary_ops.py

sathyz 에 2020년 03월 24일

pandas/tests/scalar/timestamp/의 모든 테스트가 이미 수정된 것 같습니다.

$ git checkout master
Already on 'master'
$ python scripts/validate_unwanted_patterns.py -vt="bare_pytest_raises"  pandas/tests/scalar/timestamp/
$

pandas/tests/arrays/test_boolean.py =>이(가) 없습니다.

내가 챙길거다
팬더/테스트/배열/간격/test_ops.py
팬더/테스트/배열/test_datetimelike.py

팬더/tests/groupby/test_categorical.py
팬더/tests/groupby/test_groupby.py
팬더/tests/groupby/test_timegrouper.py

sathyz 에 2020년 03월 25일

팬더/테스트/산술/test_timedelta64.py => #33010

pandas/tests/scalar/timestamp/test_arithmetic.py => 문제 없음
pandas/tests/scalar/timestamp/test_comparisons.py => 문제 없음
pandas/tests/scalar/timestamp/test_constructors.py => 문제 없음
pandas/tests/scalar/timestamp/test_timezones.py => 문제 없음
pandas/tests/scalar/timestamp/test_unary_ops.py => 문제 없음

pandas/tests/arrays/test_boolean.py =>이(가) 없습니다.

pandas/tests/arrays/interval/test_ops.py => #33010
팬더/테스트/배열/test_datetimelike.py => #33010

pandas/tests/groupby/test_categorical.py => #33144
pandas/tests/groupby/test_groupby.py => 문제 없음
pandas/tests/groupby/test_timegrouper.py => 문제 없음

pandas/tests/indexes/categorical/test_category.py => 문제 없음
팬더/테스트/인덱스/common.py #33144
팬더/테스트/인덱스/datetimelike.py #33144

pandas/tests/indexes/interval/test_astype.py => 영향을 받는 모든 테스트가 xfailed로 표시됩니다. 여전히 수정해야 합니까? 어떻게 합니까?

pandas/tests/indexes/multi/test_compat.py #33144
pandas/tests/indexes/multi/test_duplicates.py => 문제 없음
pandas/tests/indexes/multi/test_format.py => 파일을 찾을 수 없습니다.
pandas/tests/indexes/multi/test_reshape.py #33144
pandas/tests/indexes/multi/test_setops.py => 문제 없음
pandas/tests/indexes/multi/test_sorting.py #33144

sathyz 에 2020년 03월 28일

@sumanau7 사용 중인 파일을 나열했습니까? 병합된 파일 중 일부를 작업 중입니다.

sathyz 에 2020년 03월 30일

완료

pandas/tests/indexes/categorical/test_category.py
pandas/tests/indexes/period/test_constructors.py
pandas/tests/indexes/period/test_join.py
pandas/tests/indexes/period/test_partial_slicing.py
pandas/tests/indexes/period/test_setops.py
pandas/tests/indexes/timedeltas/test_delete.py

sumanau7 에 2020년 03월 30일

나는 함께 일하고있다

pandas/tests/indexes/ranges/test_constructors.py
팬더/테스트/인덱스/범위/test_range.py
팬더/테스트/인덱싱/다중 인덱스/test_chaining_and_caching.py
팬더/테스트/인덱싱/다중 인덱스/test_partial.py
팬더/테스트/시리즈/인덱싱/test_alter_index.py
팬더/테스트/배열/부울/test_function.py

proost 에 2020년 04월 19일

작업:

pandas/tests/reshape/merge/test_multi.py

DominicG88 에 2020년 04월 26일

내가 걸릴 것입니다:
팬더/테스트/창/순간/test_moments_ewm.py
팬더/tests/window/moments/test_moments_rolling.py
팬더/테스트/창/test_dtypes.py
팬더/테스트/창/test_ewm.py
팬더/테스트/창/test_expanding.py
팬더/tests/window/test_timeseries_window.py

boweyism 에 2020년 06월 20일

또한 다음을 취합니다.

팬더/테스트/프레임/메소드/test_assign.py
팬더/테스트/프레임/메소드/test_at_time.py
팬더/테스트/프레임/메소드/test_between_time.py
팬더/테스트/프레임/메소드/test_first_and_last.py
팬더/테스트/프레임/메소드/test_interpolate.py
팬더/테스트/프레임/메소드/test_replace.py
팬더/테스트/프레임/test_query_eval.py

boweyism 에 2020년 06월 21일

안녕하세요,
저는 이 프로젝트의 새로운 개발자이며 이 문제를 돕고 싶습니다. 나머지 테스트 중 초보자에게 가장 적합한 테스트는 무엇입니까?

감사 해요,
케빈

kevomaco 에 2020년 07월 14일

안녕하세요,
저는 이 프로젝트의 새로운 개발자이며 이 문제를 돕고 싶습니다. 나머지 테스트 중 초보자에게 가장 적합한 테스트는 무엇입니까?
감사 해요,
케빈

환영합니다 - 이 중 어느 것도 다른 것보다 쉽거나 어렵지 않다고 생각합니다. 시작하기에 좋은 곳이 될 것입니다.

MarcoGorelli 에 2020년 07월 14일

validate_unwanted_patterns.py 스크립트를 실행하는 동안 오류가 발생합니다.

Traceback (most recent call last):
  File "C:\Users\Kevom\git\pandas\scripts\validate_unwanted_patterns.py", line 397, in <module>
    main(
  File "C:\Users\Kevom\git\pandas\scripts\validate_unwanted_patterns.py", line 352, in main
    for line_number, msg in function(file_obj):
  File "C:\Users\Kevom\git\pandas\scripts\validate_unwanted_patterns.py", line 88, in bare_pytest_raises
    contents = file_obj.read()
  File "C:\Program Files (x86)\Python\lib\encodings\cp1252.py", line 23, in decode
    return codecs.charmap_decode(input,self.errors,decoding_table)[0]
UnicodeDecodeError: 'charmap' codec can't decode byte 0x8d in position 76843: character maps to <undefined>

'pandas/tests/test_strings.py'를 읽을 때 발생합니다. 모든 파일은 cp1252로 인코딩됩니다.

중복 작업을 피하기 위해 아직 완료되지 않은 테스트를 다시 확인하기 위해 스크립트를 실행하고 싶었습니다.

kevomaco 에 2020년 07월 17일

대부분의 NotImplementedError 에는 일치시킬 특정 메시지가 없습니다. 목적에 어긋난다는 것을 알고 있지만 린터를 침묵시키기 위해 pytest.raises(NotImplementedError, match=None) 로 변경하는 것이 좋습니다. ?

MJafarMashhadi 에 2020년 07월 28일

내가 걸릴 것입니다:

팬더/테스트/도구/test_to_datetime.py
팬더/테스트/tseries/offsets/test_offsets.py

시작한다.

- 케빈

kevomaco 에 2020년 08월 07일

나는 이것에 익숙하지 않으므로 다음과 같이 시작할 것입니다.

팬더/테스트/tseries/offsets/test_ticks.py

krajatcl 에 2020년 09월 27일

안녕하세요, 저는 기여에 처음입니다. 문제에 대해 명확하게 작성해 주셔서 감사합니다. pandas/tests/generic/test_duplicate_labels.py 부터 시작하고 문제가 해결되면 몇 가지 더 다룰 것입니다.

theMogget 에 2020년 10월 14일

나는 pandas/tests/arrays/test_datetimelike.py를 시작으로 사용할 것입니다.
또한 python scripts/validate_unwanted_patterns.py -vt="bare_pytest_raises" pandas/tests/ 성공적으로 실행할 수 없으면 다음을 시도하십시오.
대신 python scripts/validate_unwanted_patterns.py -vt="bare_pytest_raises" pandas/tests/**/*.py

liaoaoyuan97 에 2020년 11월 15일

지금 실행하는 가장 쉬운 방법은 다음을 추가하는 것입니다.

    -   id: unwanted-patterns-bare-pytest-raises
        name: Check for use of bare use of pytest raises
        language: python
        entry: python scripts/validate_unwanted_patterns.py --validation-type="bare_pytest_raises"
        types: [python]
        files: ^pandas/tests/

- repo: local 섹션에서 .pre-commit-config.yaml 로 변경한 다음 실행

pre-commit run unwanted-patterns-bare-pytest-raises --all-files.

나머지 미해결 파일로 문제를 업데이트했습니다.

MarcoGorelli 에 2020년 11월 15일

나는 이것을 가지고 갈 수 있다:

[x] 판다/테스트/io/pytables/test_timezones.py
[ ] 팬더/테스트/일반/메소드/test_pipe.py
[ ] pandas/tests/reshape/merge/test_merge_asof.py
[ ] 팬더/테스트/확장/베이스/reduce.py
[ ] 팬더/테스트/확장/베이스/getitem.py
~~팬더/테스트/배열/test_datetimelike.py~~

이들은 오늘 현재 목록에서 상위 5개입니다.

marktgraham 에 2020년 11월 18일

@marktgraham 아직 test_datetime.py 을 하지 않았다면. PR을 하려고 하니 그냥 두세요.

liaoaoyuan97 에 2020년 11월 19일

@liaoaoyuan97 걱정하지 마세요. 아직 test_datetimelike.py 건드리지 않았습니다.

대신 pandas/tests/extension/base/getitem.py를 사용하겠습니다.

marktgraham 에 2020년 11월 19일

validate_unwanted_patterns.py 내 쪽에서 오류 발생

$ python scripts/validate_unwanted_patterns.py -vt="bare_pytest_raises" pandas/tests/
Traceback (most recent call last):
  File "scripts/validate_unwanted_patterns.py", line 479, in <module>
    output_format=args.format,
  File "scripts/validate_unwanted_patterns.py", line 435, in main
    with open(file_path, encoding="utf-8") as file_obj:
IsADirectoryError: [Errno 21] Is a directory: 'pandas/tests/'

아마도 #37419와 관련이 있는 것 같은데요?

@MarcoGorelli가 제안한 접근 방식을 시도했고 완벽하게 작동했습니다.

지금 실행하는 가장 쉬운 방법은 다음을 추가하는 것입니다.

    -   id: unwanted-patterns-bare-pytest-raises
        name: Check for use of bare use of pytest raises
        language: python
        entry: python scripts/validate_unwanted_patterns.py --validation-type="bare_pytest_raises"
        types: [python]
        files: ^pandas/tests/

- repo: local 섹션에서 .pre-commit-config.yaml 로 변경한 다음 실행

pre-commit run unwanted-patterns-bare-pytest-raises --all-files.

이것을 .pre-commit-config.yaml 에 추가한 다음 이 스레드의 지침을 업데이트하는 것이 합리적입니까?

lucasrodes 에 2020년 11월 20일

이것을 .pre-commit-config.yaml에 추가한 다음 이 스레드의 지침을 업데이트하는 것이 의미가 있습니까?

발생하는 모든 오류가 수정되면 .pre-commit-config.yaml 추가합니다. 예

아마도 #37419와 관련이 있는 것 같은데요?

아니요, #37379와 관련이 있습니다(이 스크립트를 사전 커밋으로 옮겼으므로 더 이상 디렉터리에서 실행할 필요가 없었습니다)

MarcoGorelli 에 2020년 11월 20일

👍1

이 페이지가 도움이 되었나요?

0 / 5 - 0 등급

Pandas: [좋은 첫 번째 이슈] TST: 베어 pytest.raises 허용 안 함

참고 사항:

기조 연설:

현재 목록은 다음과 같습니다.

- [ ] 팬더/테스트/스칼라/test_nat.py

노트:

모든 51 댓글

31072 stata.py 에 하나의 누락된 일치 항목이 있습니다.

업데이트

관련 문제

31072 `stata.py` 에 하나의 누락된 일치 항목이 있습니다.