| Commit message (Collapse) | Author | Age | Files | Lines |
|---|
| |
|
|
| |
This lets us skip fields which have the default values.
|
| | |
|
| |
|
|
|
|
| |
And wrap it up nicely in an assert_eq_or_diff macro.
I've tested the output by sabotaging one of the test2 tests.
|
| |
|
|
|
|
|
|
|
|
|
|
| |
This commit executes maint/add_warning with the just added change to
deny string slices except in tests.
I recommend auditing this by checking out the previous commit followed
by running the script yourself and then verifying that the diff is
identical to this commit.
This commit makes cargo clippy fail. We will add exceptions in the next
commit.
|
| |\
| |
| |
| |
| | |
Add debugging support to ItemValueParseable derive
See merge request tpo/core/arti!3881
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
With
cargo test --all-features -p tor-netdoc -- --no-capture test2
the output now contains, amongst many other things:
netdoc TestItem parse: item start, "test-item", "N", Some("TEST OBJECT")
netdoc TestItem parse: field needed, normal, args=1, "N"
netdoc TestItem parse: field optional, normal, args=Option<_>, ""
netdoc TestItem parse: field rest, normal, args=Vec<_>, ""
netdoc TestItem parse: field object, object, object=1, Some(("TEST OBJECT", Ok(5)))
netdoc TestItem parse: item complete Ok
|
| |/
|
|
| |
Ie for "arguments" that don't actually appear.
|
| | |
|
| |
|
|
|
| |
This lets us having document items that are "manually non exhaustive"
which is necessary for struct literal constructors.
|
| | |
|
| |
|
|
|
|
| |
* Make a fancy version of t_ok, t_ok_multi, that takes the expected boundary byte offsets.
* t_ok is now just for single-document files and is implemented in terms of t_ok_multi.
* Use t_ok_multi for the one test case with multiple documents.
|
| | |
|
| | |
|
| |
|
|
| |
This was missed, apparently.
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
The test case documents are often not in canonical form. We don't
want to change them to be in canonical form because we want to test
parsing edge cases. So, we provide a mangling scheme with in-document
notes, and annotate all the documents.
Apply suitable annotations to all the documents.
Test that the encoded forms are as expected.
This reverts commit 05e7b8b60061a7e8a8d004a822cc213d3ea22d47.
|
| |
|
|
|
|
|
| |
Make this whole module conditional on the encode feature.
Apply the derives. Provide the necessary manual implementations for
the types where we use `with`.
|
| |
|
|
|
|
|
| |
These cannot be encoded. So that is logically incoherent.
(Perhaps String ought not to be NormalItemArgument, but let's not
tackle that now.)
|
| |
|
|
|
|
|
|
|
|
| |
The path in `with` must always be a module, not a function, because we
need to use it for encoding too.
In the case of `rest`, it's probably sensible to require a different
function name, given that the semantics are subtly different to those
of `rest` without `with`, since the latter can only get a single word
but with `rest` you get the whole line.
|
| |
|
|
| |
Tidy up the imports.
|
| |
|
|
|
|
| |
Now that this module is not inside parse2 its identifier namespace
oughtn't to be parse-specific. Rename things so that every
imports/aliases related to parsing implies parsing in its name.
|
|
|
We're going to use this for the encoder testing too, so it needs to be
not buried inside parse2.
This commit is the file rename plus minimal changes to make it compile.
|
