Bug 136577
| Summary: | import-w3c-tests should handle paths in ref files that originate above the test file's parent dir. | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Rebecca Hauck <rhauck> |
| Component: | Tools / Tests | Assignee: | Nobody <webkit-unassigned> |
| Status: | NEW | ||
| Severity: | Normal | CC: | bjonesbe, clopez, svillar, webkit |
| Priority: | P2 | ||
| Version: | 528+ (Nightly build) | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
Rebecca Hauck
Currently, the script assumes that the reference file originates in a subdirectory of the test file's parent directory and it is imported up N levels to be the same directory as the test. In this (most common) case, the script removes relpath from its support file paths upon import. This won't work if the reference file originated in a directory above the test file and is imported down N levels. This scenario occurs many times in the CSS test repo, but it happens to be one single shared reference file that many of the css21 tests share from a common ancestor directory. In all of these cases, the original ref file and the test file's parent dir both have the same relative path to the ref file's support file, so the new location of the ref file is a lateral move and everything 'just works'. Ideally, the import script should do the right thing wherever the the ref fileorginates, but this is an edge case and highly unlikely.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Sergio Villar Senin
This is a real issue with flexbox tests. For that we have
css/support/
css/css-flexbox/support
The import script is converting every "support/" path to "../support" forcing us to manually edit the imported tests.
Sergio Villar Senin
Carlos do you know if there is an "easy" fix for this or would it require a fair amount of effort?
Sergio Villar Senin
I guess the problem comes from the fix from bug 135929?