2022-06-14 23:14:08 +01:00
|
|
|
I care a lot about being able to automatically check _any_ property about my
|
2022-06-15 06:47:49 +01:00
|
|
|
program before it ever runs. However, some things don't have tests yet, either
|
|
|
|
because I don't know how to test them or because I've been lazy. I'll at least
|
|
|
|
record those here.
|
2022-06-14 23:14:08 +01:00
|
|
|
|
2022-10-30 00:43:13 +01:00
|
|
|
* Initializing settings:
|
2023-03-31 07:17:14 +01:00
|
|
|
- from previous session
|
|
|
|
- from defaults
|
2023-03-31 07:18:02 +01:00
|
|
|
- default_map absent/present
|
2022-07-20 16:53:31 +01:00
|
|
|
|
2022-07-20 17:11:29 +01:00
|
|
|
* How the screen looks. Our tests use a level of indirection to check text and
|
|
|
|
graphics printed to screen, but not the precise pixels they translate to.
|
|
|
|
- where exactly the cursor is drawn to highlight a given character
|
2022-08-04 02:40:07 +01:00
|
|
|
- analogously, how a shape precisely looks as you draw it
|
2022-07-20 17:11:29 +01:00
|
|
|
|
2023-04-09 18:34:24 +01:00
|
|
|
### Protocol with driver; error-handling
|
|
|
|
|
|
|
|
* clone this repo to a new client app, clear its save dir[1], run it, run the
|
|
|
|
driver, add a definition:
|
|
|
|
```
|
|
|
|
on.draw = function()
|
|
|
|
end
|
|
|
|
```
|
|
|
|
Hit F4. No error.
|
|
|
|
Quit driver, quit client app. Restart client app. No error.
|
|
|
|
|
|
|
|
* clone this repo to a new client app, clear its save dir, run it, run the
|
|
|
|
driver, add a definition that draws to screen:
|
|
|
|
```
|
|
|
|
on.draw = function()
|
|
|
|
love.graphics.print('hello!', 50,50)
|
|
|
|
end
|
|
|
|
```
|
|
|
|
Hit F4. Client app shows 'hello!'
|
|
|
|
Quit driver, quit client app. Restart client app. No error. Client app shows 'hello!'
|
|
|
|
|
|
|
|
* clone this repo to a new client app, clear its save dir, run it, run the
|
|
|
|
driver, add a definition containing invalid Lua:
|
|
|
|
```
|
|
|
|
on.draw = function(
|
|
|
|
```
|
|
|
|
Hit F4. Driver shows an error under the definition.
|
2023-04-23 01:48:54 +01:00
|
|
|
Fix the definition:
|
|
|
|
```
|
|
|
|
on.draw = function()
|
|
|
|
end
|
|
|
|
```
|
|
|
|
Hit F4. The error disappears.
|
2023-04-09 18:34:24 +01:00
|
|
|
|
|
|
|
* clone this repo to a new client app, clear its save dir, run it, run the
|
|
|
|
driver, add a definition containing invalid Lua:
|
|
|
|
```
|
|
|
|
on.draw = function(
|
|
|
|
```
|
|
|
|
Hit F4. Driver shows an error under the definition as before.
|
2023-04-23 02:06:38 +01:00
|
|
|
Quit the client app.
|
2023-04-09 18:34:24 +01:00
|
|
|
Restart the client app. It loads up without error.
|
|
|
|
Switch back to the driver. Fix the definition.
|
|
|
|
```
|
|
|
|
on.draw = function()
|
|
|
|
end
|
|
|
|
```
|
|
|
|
Hit F4. The error disappears.
|
|
|
|
|
|
|
|
* clone this repo to a new client app, clear its save dir, run it, run the
|
|
|
|
driver, add a definition containing invalid Lua:
|
|
|
|
```
|
|
|
|
on.draw = function(
|
|
|
|
```
|
|
|
|
Hit F4. Driver shows an error under the definition as before.
|
2023-04-21 16:18:00 +01:00
|
|
|
Quit the client app.
|
2023-04-09 18:34:24 +01:00
|
|
|
Switch back to the driver. Hit F4. The driver hangs and needs to be
|
|
|
|
force-quit. [This is not ideal, but how things are today.]
|
|
|
|
|
|
|
|
[1] We never clear the app from the driver's config. driver.love needs to be
|
|
|
|
robust to apps changing out from under it.
|
|
|
|
|
2022-07-20 16:53:31 +01:00
|
|
|
### Other compromises
|
2022-06-14 23:14:08 +01:00
|
|
|
|
|
|
|
Lua is dynamically typed. Tests can't patch over lack of type-checking.
|
|
|
|
|
|
|
|
* All strings are UTF-8. Bytes within them are not characters. I try to label
|
2022-06-20 16:24:56 +01:00
|
|
|
byte offsets with the suffix `_offset`, and character positions as `_pos`.
|
|
|
|
For example, `string.sub` should never use a `_pos` to substring, only an
|
|
|
|
`_offset`.
|
2022-06-21 15:20:25 +01:00
|
|
|
|
|
|
|
* Some ADT/interface support would be helpful in keeping per-line state in
|
|
|
|
sync. Any change to line data should clear line `fragments` and
|
|
|
|
`screen_line_starting_pos`.
|
2022-06-23 19:41:29 +01:00
|
|
|
|
|
|
|
* Some inputs get processed in love.textinput and some in love.keypressed.
|
|
|
|
Several bugs have arisen due to destructive interference between the two for
|
|
|
|
some key chord. I wish I could guarantee that the two sets are disjoint. But
|
|
|
|
perhaps I'm not thinking about this right.
|
2022-07-01 18:30:52 +01:00
|
|
|
|
|
|
|
* Like any high-level language, it's easy to accidentally alias two non-scalar
|
|
|
|
variables. I wish there was a way to require copy when assigning.
|
2022-07-01 21:55:43 +01:00
|
|
|
|
2023-03-29 06:00:43 +01:00
|
|
|
* I wish I could require pixel coordinates to be integers. The editor
|
|
|
|
defensively converts input margins to integers.
|
2022-12-24 00:57:04 +00:00
|
|
|
|
2022-07-12 06:14:59 +01:00
|
|
|
* My test harness automatically runs `test_*` methods -- but only at the
|
|
|
|
top-level. I wish there was a way to raise warnings if someone defines such
|
|
|
|
a function inside a dict somewhere.
|