prettier / prettier/plugin-php
Unexpected Indentation Level in Tests
Chưa có ai nhận issue này.
- Ngôn ngữ chính
- PHP
- Star
- 1.9k
- Fork
- 139
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Mô tả
As mentioned in #950, the tests produce a strange behavior with indentation of PHP code blocks in markdown.
Even when no tabWidth is configured in the markdown test (currently it is set to 4 to work around this issue), PHP code is indented by 2 spaces with the standalone build:
[...]
```php
function test(){$a=1;}
```
[...]
becomes
[...]
```php
function test()
{
$a = 1;
}
```
[...]
Note: Be careful when copy-pasting the code above. The code fences contain zero-width spaces to retain syntax highlighting.
This incorrect behavior is pretty unique:
-
It does not occur in the same test for the Node.js build, which correctly indents by 4 spaces in the tests.
-
It does not occur in the standalone build when used in the browser.
-
It does not even occur in the test for the standalone build, when the
jest.config.jsis changed to not using projects (which it currently does), to something like this:module.exports = { setupFiles: ["<rootDir>/tests_config/run_spec.js"], testRegex: "jsfmt\\.spec\\.js$|__tests__/.*\\.js$", snapshotSerializers: ["jest-snapshot-serializer-raw"], testEnvironment: "jsdom", globals: { STANDALONE: true } };
So especially because of the last point, I'm assuming this has something to do with the architecture of the tests. I wasn't able to narrow it down any further.
As mentioned above, currently there's a workaround (introduced in #950) to not break the tests with something that doesn't occur "in the wild", but it should probably be taken a deeper look on.
Hướng dẫn đóng góp
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 bài kiểm thử markdown trong tests/markdown và so sánh kết quả của Node.js và standalone. Kiểm tra jest.config.js, đặc biệt là cấu hình projects, và tái hiện sự khác biệt bằng tests_config/run_spec.js. Hoàn tất khi bài kiểm thử standalone không còn cần workaround tabWidth: 4 và khớp với thụt lề mong đợ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ệ
- javascript, node.js, php
- Lĩnh vực
- testing-qa
- Loại issue
- Lỗi
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức độ hoạt động
- Đình trệ
- Độ rõ ràng
- Khá rõ ràng
- Mức phù hợp với người mới
- 35/100