Bram Moolenaar | 46acad7 | 2023-06-11 19:04:18 +0100 | [diff] [blame] | 1 | Tests for syntax highlighting plugins |
| 2 | ===================================== |
| 3 | |
| 4 | Summary: Files in the "input" directory are edited by Vim with syntax |
| 5 | highlighting enabled. Screendumps are generated and compared with the |
| 6 | expected screendumps in the "dumps" directory. This will uncover any |
| 7 | character attributes that differ. |
| 8 | |
| 9 | Without any further setup a screendump is made at the top of the file (using |
| 10 | _00.dump) and another one at the end of the file (using _99.dump). The dumps |
| 11 | are normally 20 screen lines tall. |
| 12 | |
| 13 | When the screendumps are OK an empty "done/{name}" file is created. This |
| 14 | avoids running the test again until "make clean" is used. Thus you can run |
| 15 | "make test", see one test fail, try to fix the problem, then run "make test" |
| 16 | again to only repeat the failing test. |
| 17 | |
| 18 | When a screendump differs it is stored in the "failed" directory. This allows |
| 19 | for comparing it with the expected screendump, using a command like: |
| 20 | |
| 21 | let fname = '{name}_99.dump' |
| 22 | call term_dumpdiff('failed/' .. fname, 'dumps/' .. fname) |
| 23 | |
| 24 | |
| 25 | Creating a syntax plugin test |
| 26 | ----------------------------- |
| 27 | |
| 28 | Create a source file in the language you want to test in the "input" |
Aliaksei Budavei | 93edd25 | 2024-03-05 22:34:36 +0300 | [diff] [blame^] | 29 | directory. Use the filetype name as the base and a file name extension |
| 30 | matching the filetype. Let's use Java as an example. The file would then be |
Bram Moolenaar | 46acad7 | 2023-06-11 19:04:18 +0100 | [diff] [blame] | 31 | "input/java.java". |
| 32 | |
Aliaksei Budavei | 93edd25 | 2024-03-05 22:34:36 +0300 | [diff] [blame^] | 33 | Make sure to include some interesting constructs with plenty of complicated |
| 34 | highlighting. Optionally, pre-configure the testing environment by including |
| 35 | setup commands at the top of the input file. The format for these lines is: |
| 36 | |
| 37 | TEST_SETUP {command} |
| 38 | |
| 39 | where {command} is any valid Ex command, which extends to the end of the line. |
| 40 | The first 20 lines of the input file are ALWAYS scanned for setup commands and |
| 41 | these will be executed before the syntax highlighting is enabled. Typically, |
| 42 | these lines would be included as comments so as not to introduce any syntax |
| 43 | errors in the input file but this is not required. |
| 44 | |
| 45 | Continuing the Java example: |
| 46 | |
| 47 | // TEST_SETUP let g:java_space_errors = 1 |
| 48 | // TEST_SETUP let g:java_minlines = 5 |
| 49 | class Test { } |
| 50 | |
Bram Moolenaar | 46acad7 | 2023-06-11 19:04:18 +0100 | [diff] [blame] | 51 | If there is no further setup required, you can now run the tests: |
Aliaksei Budavei | 93edd25 | 2024-03-05 22:34:36 +0300 | [diff] [blame^] | 52 | |
Bram Moolenaar | 46acad7 | 2023-06-11 19:04:18 +0100 | [diff] [blame] | 53 | make test |
| 54 | |
| 55 | The first time this will fail with an error for a missing screendump. The |
| 56 | newly created screendumps will be "failed/java_00.dump", |
| 57 | "failed/java_01.dump", etc. You can inspect each with: |
| 58 | |
| 59 | call term_dumpload('failed/java_00.dump') |
| 60 | call term_dumpload('failed/java_01.dump') |
| 61 | ... |
| 62 | call term_dumpload('failed/java_99.dump') |
| 63 | |
| 64 | If they look OK, move them to the "dumps" directory: |
| 65 | |
| 66 | :!mv failed/java_00.dump dumps |
| 67 | :!mv failed/java_01.dump dumps |
| 68 | ... |
| 69 | :!mv failed/java_99.dump dumps |
| 70 | |
| 71 | If you now run the test again, it will succeed. |
| 72 | |
| 73 | |
| 74 | Adjusting a syntax plugin test |
| 75 | ------------------------------ |
| 76 | |
| 77 | If you make changes to the syntax plugin, you should add code to the input |
| 78 | file to see the effect of these changes. So that the effect of the changes |
| 79 | are covered by the test. You can follow these steps: |
| 80 | |
| 81 | 1. Edit the syntax plugin somewhere in your personal setup. Use a file |
| 82 | somewhere to try out the changes. |
| 83 | 2. Go to the directory where you have the Vim code checked out and replace the |
| 84 | syntax plugin. Run the tests: "make test". Usually the tests will still |
| 85 | pass, but if you fixed syntax highlighting that was already visible in the |
| 86 | input file, carefully check that the changes in the screendump are |
| 87 | intentional: |
Aliaksei Budavei | 93edd25 | 2024-03-05 22:34:36 +0300 | [diff] [blame^] | 88 | |
Bram Moolenaar | 46acad7 | 2023-06-11 19:04:18 +0100 | [diff] [blame] | 89 | let fname = '{name}_99.dump' |
| 90 | call term_dumpdiff('failed/' .. fname, 'dumps/' .. fname) |
Aliaksei Budavei | 93edd25 | 2024-03-05 22:34:36 +0300 | [diff] [blame^] | 91 | |
Bram Moolenaar | 46acad7 | 2023-06-11 19:04:18 +0100 | [diff] [blame] | 92 | Fix the syntax plugin until the result is good. |
| 93 | 2. Edit the input file for your language to add the items you have improved. |
| 94 | (TODO: how to add another screendump?). |
| 95 | Run the tests and you should get failures. Like with the previous step, |
| 96 | carefully check that the new screendumps in the "failed" directory are |
| 97 | good. Update the syntax plugin and the input file until the highlighting |
| 98 | is good and you can see the effect of the syntax plugin improvements. Then |
| 99 | move the screendumps from the "failed" to the "dumps" directory. Now "make |
| 100 | test" should succeed. |
| 101 | 3. Prepare a pull request with the modified files: |
| 102 | - syntax plugin: syntax/{name}.vim |
| 103 | - test input file: syntax/testdir/input/{name}.{ext} |
| 104 | - test dump files: syntax/testdir/dumps/{name}_99.dump |
| 105 | |
| 106 | As an extra check you can temporarily put back the old syntax plugin and |
| 107 | verify that the tests fail. Then you know your changes are covered by the |
| 108 | test. |
| 109 | |
| 110 | |
| 111 | |
Bram Moolenaar | 46acad7 | 2023-06-11 19:04:18 +0100 | [diff] [blame] | 112 | |
Aliaksei Budavei | 93edd25 | 2024-03-05 22:34:36 +0300 | [diff] [blame^] | 113 | TODO: run test for one specific filetype |
Bram Moolenaar | 46acad7 | 2023-06-11 19:04:18 +0100 | [diff] [blame] | 114 | TODO: test syncing by jumping around |