Add testing helper for DELETE requests

[teisenbe]

This introduces test_util.object_delete(), to go along with the existing test_util helpers. It's helpful for testing CRUD APIs.

[teisenbe]

Hmm, those tests seem to be failing for me on main. Not sure what's going on there

[smklein]

Hmm, those tests seem to be failing for me on main. Not sure what's going on there

What's the error message you're seeing? I just did git checkout main && cargo test, not seeing anything

(Edit: I see it - in actions, though not locally. More detail below)

[smklein]

Hmm, those tests seem to be failing for me on main. Not sure what's going on there

What's the error message you're seeing? I just did git checkout main && cargo test, not seeing anything

Ahhhh taking a closer look here - and at https://github.com/oxidecomputer/dropshot/actions - I think the reason tests (unrelated to your PR) tests are failing is because:

1. We have some tests that check for "compiler failures" looking like we want
2. I suspect the compiler changed, and the compiler output changed
3. This change - though probably innocent - caused "expected != actual", and the test failed.

Eyeballing it, it actually kinda looks like the only line that changed was actually part of the compiler message giving path hints:

Expected: ::: $WORKSPACE/dropshot/src/api_description.rs

vs

Actual: ::: src/api_description.rs:54:33

I'll follow-up with a separate issue.

[smklein]

Filed #138

[teisenbe]

Okay, should be ready for another review. Tests all pass locally after merging #139 in

[teisenbe]

Hmm, the windows build failed logging::test::test_config_file with this output:

---- logging::test::test_config_file stdout ----
config "file": Ok(File { level: Warn, path: "C:\\Users\\RUNNER~1\\AppData\\Local\\Temp\\dropshot-af77ac2c25c9d09a.exe.2092.file_dir\\log.out", if_exists: Fail })
note: configured to log to "C:\Users\RUNNER~1\AppData\Local\Temp\dropshot-af77ac2c25c9d09a.exe.2092.file_dir\log.out"
config "file": Ok(File { level: Warn, path: "C:\\Users\\RUNNER~1\\AppData\\Local\\Temp\\dropshot-af77ac2c25c9d09a.exe.2092.file_dir\\log.out", if_exists: Append })
note: configured to log to "C:\Users\RUNNER~1\AppData\Local\Temp\dropshot-af77ac2c25c9d09a.exe.2092.file_dir\log.out"
config "file": Ok(File { level: Trace, path: "C:\\Users\\RUNNER~1\\AppData\\Local\\Temp\\dropshot-af77ac2c25c9d09a.exe.2092.file_dir\\log.out", if_exists: Truncate })
note: configured to log to "C:\Users\RUNNER~1\AppData\Local\Temp\dropshot-af77ac2c25c9d09a.exe.2092.file_dir\log.out"
thread 'logging::test::test_config_file' panicked at 'assertion failed: should_be_before.timestamp() <= time_after.timestamp()', dropshot\src\test_util.rs:786:13

This seems unrelated to my change; I can open a ticket for this

[teisenbe]

It passed on a re-run. I opened #141 to track