Maximilian Roos
Maximilian Roos
Similarly: - The plugin also prints stuff to stdout, even when deepeval isn't part of the test. `]Running teardown with pytest sessionfinish...`, and prints some more details about tests that...
One option is to write new snapshots with the shorter name while supporting older snapshots with longer names. It would be technically possible to have conflicts, but extremely unlikely (in...
yes that would be welcome! some more ambitious ideas in https://github.com/mitsuhiko/insta/issues/647, but just adding a config for `prepend_module_to_snapshot` would be good progress
One downside (but balanced against lots of upsides!) is that because we insert a newline at the start of a multiline string, a literal string can look a bit odd:...
I'm tempted to merge this as there's no downside to enforce starting newlines; and it simplifies the code overall lmk any objections...
merging but one click to revert if anyone disagrees
IIUC, upgrading toml has backward compatibility concerns: https://github.com/mitsuhiko/insta/issues/439. Is there a version that fixes the issues you were hitting without breaking backward-compat?
> I understand that it can break some existing snapshot, but on the other side, it makes the library unusable for valid types. Right, but breaking existing snapshots requires us...
Is it instead possible to do this with a feature?
Yes, we'd need to find some way of it using v8 when both features were enabled. To the extent we have both versions of toml on the `toml` feature in...