forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commitca8217c
committed
Add a test module for the regular expression package.
This module provides a function test_regex() that is functionallyrather like regexp_matches(), but with additional debugging-orientedoptions and additional output. The debug options are somewhat obscure;they are chosen to match the API of the test harness that Henry Spencerwrote way-back-when for use in Tcl. With this, we can import all thetest cases that Spencer wrote originally, even for regex functionalitythat we don't currently expose in Postgres. This seems necessarybecause we can no longer rely on Tcl to act as upstream and verifyany fixes or improvements that we make.In addition to Spencer's tests, I added a few for lookbehindconstraints (which we added in 2015, and Tcl still hasn't absorbed)that are modeled on his tests for lookahead constraints. After lookingat code coverage reports, I also threw in a couple of tests to morefully exercise our "high colormap" logic.According to my testing, this brings the check-world coveragefor src/backend/regex/ from 71.1% to 86.7% of lines.(coverage.postgresql.org shows a slightly different number,which I think is because it measures a non-assert build.)Discussion:https://postgr.es/m/2873268.1609732164@sss.pgh.pa.us1 parent4656e3d commitca8217c
File tree
12 files changed
+7264
-0
lines changed- src/test/modules
- test_regex
- expected
- sql
12 files changed
+7264
-0
lines changedLines changed: 1 addition & 0 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
22 | 22 |
| |
23 | 23 |
| |
24 | 24 |
| |
| 25 | + | |
25 | 26 |
| |
26 | 27 |
| |
27 | 28 |
| |
|
Lines changed: 4 additions & 0 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + |
Lines changed: 23 additions & 0 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
| 22 | + | |
| 23 | + |
Lines changed: 78 additions & 0 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
| 32 | + | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
| 36 | + | |
| 37 | + | |
| 38 | + | |
| 39 | + | |
| 40 | + | |
| 41 | + | |
| 42 | + | |
| 43 | + | |
| 44 | + | |
| 45 | + | |
| 46 | + | |
| 47 | + | |
| 48 | + | |
| 49 | + | |
| 50 | + | |
| 51 | + | |
| 52 | + | |
| 53 | + | |
| 54 | + | |
| 55 | + | |
| 56 | + | |
| 57 | + | |
| 58 | + | |
| 59 | + | |
| 60 | + | |
| 61 | + | |
| 62 | + | |
| 63 | + | |
| 64 | + | |
| 65 | + | |
| 66 | + | |
| 67 | + | |
| 68 | + | |
| 69 | + | |
| 70 | + | |
| 71 | + | |
| 72 | + | |
| 73 | + | |
| 74 | + | |
| 75 | + | |
| 76 | + | |
| 77 | + | |
| 78 | + |
0 commit comments
Comments
(0)