pyvec / pyvec/naucse.python.cz
Arca backend used in tests
Chưa có ai nhận issue này.
- Ngôn ngữ chính
- Không có dữ liệu ngôn ngữ
- Star
- 335
- Fork
- 119
- Merge trung bình
- 3 giờ 6 phút
- Pull request đã merge (30 ngày)
- 1
Mô tả
So I was looking at the failing test in test_arca_render and at first, I also had no clue what the problem is... But then after crawling through the logs and some experiments, I found the issue: the test is being run with arca.backend.CurrentEnvironmentBackend and the current environment has got naucse-render installed.
I really struggled to find why the hell CurrentEnvironmentBackend is used even though VenvBackend is configured in the tests (and even when I set the environ variable), but I finally found it in pytest.ini, the environ there overrides the VenvBackend in tests and even the backend provided in CLI.
I'm not exactly sure what backend you actually want to run it with so I'm not making a PR.
Also, just as a heads up, VenvBackend does not work on Travis (https://github.com/travis-ci/travis-ci/issues/8589, issue closed because it was stale, but AFAIK still not resolved)
Hướng dẫn đóng góp
Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Hướng nghiên cứu
Bắt đầu với pytest.ini và test_arca_render đang bị lỗi, sau đó theo dõi cách backend được cấu hình ghi đè VenvBackend và thiết lập CLI. Xác nhận với các maintainer backend mà các bài kiểm thử nên sử dụng; hoàn thành khi backend dự định được chọn một cách nhất quán và test_arca_render không còn chạy trên một backend ngoài dự định của môi trường hiện tại.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- python
- Lĩnh vực
- backend, testing
- Loại issue
- Lỗi
- Độ khó
- 3/5
- Thời gian dự kiến
- 1-2 ngày
- Mức độ hoạt động
- Đình trệ
- Độ rõ ràng
- Cần làm rõ
- Mức phù hợp với người mới
- 35/100