This is in regards to https://github.com/appwrite/console/pull/553 @Steven Basically there are 2 problems:
- Unit tests are validating Incorrectly (Investiging it)
- A regex pattern to match the cron expression is incresingly complex, Instead i have broken it down and tested the regex of indiviidual parts of the cron, while combing it however there are edges cases that are popping up
Paths I can Take:
- I can continue on trying to debug this regex expression (need some help)
- A different route would be, us relying on this function instead of a single regex expresssion for client side validation, as far as my tests went this function is working correctly (not sure if this is a good idea, but it would work well)
can we simplify the cron syntax? covering the usual expressions should be good enough
as for testing...i would want to make sure we're testing with the input because it seems like it may not accept all regex patterns
i was trying to cover all the test cases to be accepted, they are extensive. i can simplify it to cover a subset of the tests
like i said, we could switch to manual validation if that is okay
Recommended threads
- Domain is already used. Please try again...
I have a website with where the www.domain.me This website works just fine But if I try to visit domain.me. I get this error. I keep getting sent to some app ri...
- Realtime api and labels as permission
in my tables i set labels as permission and real-time capabilities stopped working. Before when i was having "any" role everything was working. Note: user have...
- "Restore project" button fails: "Invalid...
In the dashboard, it clicking "Restore project" fails. The request sent to `PATCH https://cloud.appwrite.io/v1/projects/:project_id` with payload `{status: "act...