search

pyvec / naucse

A server for open course material
Other
2 stars 9 forks source link

Fix & test git_identifer #62

Closed encukou closed 2 years ago

encukou commented 2 years ago

Alas, what's not tested is broken. The function misbehaved with capital letters, and a few edge cases. Let's throw some property-based testing at it.

encukou commented 2 years ago

Thinking about it on the way to lunch, I decided to make the output more readable (for URLs), similarly to how punycode works. Need to stop yak-shaving now.

hroncok commented 2 years ago

Si there any way to see the tested strings in pytest output? It seems only 6 tests are listed in the output.

encukou commented 2 years ago

Si there any way to see the tested strings in pytest output? It seems only 6 tests are listed in the output.

Not really – Hypothesis tries a bunch of them at random. You could add a print and run with -s

encukou commented 2 years ago

Thank you!