chore(deps): update rstest requirement from 0.22.0 to 0.23.0 (#149)

Updates the requirements on [rstest](https://github.com/la10736/rstest)
to permit the latest version.
<details>
<summary>Release notes</summary>
<p><em>Sourced from <a
href="https://github.com/la10736/rstest/releases">rstest's
releases</a>.</em></p>
<blockquote>
<h2>0.23.0</h2>
<h2>What's Changed</h2>
<ul>
<li>Upgrade to async-std v1.13 by <a
href="https://github.com/jayvdb"><code>@​jayvdb</code></a> in <a
href="https://redirect.github.com/la10736/rstest/pull/274">la10736/rstest#274</a></li>
<li>Allow environment variables in #[files] attributes by <a
href="https://github.com/hansl"><code>@​hansl</code></a> in <a
href="https://redirect.github.com/la10736/rstest/pull/277">la10736/rstest#277</a></li>
</ul>
<h2>New Contributors</h2>
<ul>
<li><a href="https://github.com/jayvdb"><code>@​jayvdb</code></a> made
their first contribution in <a
href="https://redirect.github.com/la10736/rstest/pull/274">la10736/rstest#274</a></li>
<li><a href="https://github.com/hansl"><code>@​hansl</code></a> made
their first contribution in <a
href="https://redirect.github.com/la10736/rstest/pull/277">la10736/rstest#277</a></li>
</ul>
<p><strong>Full Changelog</strong>: <a
href="https://github.com/la10736/rstest/compare/v0.22.0...v0.23.0">https://github.com/la10736/rstest/compare/v0.22.0...v0.23.0</a></p>
</blockquote>
</details>
<details>
<summary>Changelog</summary>
<p><em>Sourced from <a
href="https://github.com/la10736/rstest/blob/master/CHANGELOG.md">rstest's
changelog</a>.</em></p>
<blockquote>
<h2>[0.23.0] 2024/9/29</h2>
<h3>Add</h3>
<ul>
<li>You can now use environment variables in <code>#[files]</code> with
an optional default value (see <a
href="https://redirect.github.com/la10736/rstest/pull/277">#277</a>).</li>
<li>You can now set a base_dir for <code>#[files]</code> with the
<code>$[base_dir = &quot;...&quot;]</code> attribute (see <a
href="https://redirect.github.com/la10736/rstest/pull/277">#277</a>).</li>
</ul>
<h2>[0.22.0] 2024/8/4</h2>
<h3>Changed</h3>
<ul>
<li>Now it's possible destructuring input values both for cases, values
and fixtures. See <a
href="https://redirect.github.com/la10736/rstest/issues/231">#231</a>
for details</li>
</ul>
<h3>Add</h3>
<ul>
<li>Implemented <code>#[ignore]</code> attribute to ignore test
parameters during fixtures resolution/injection. See <a
href="https://redirect.github.com/la10736/rstest/issues/228">#228</a>
for details</li>
</ul>
<h3>Fixed</h3>
<ul>
<li>Lot of typo in code</li>
</ul>
<h2>[0.21.0] 2024/6/1</h2>
<h3>Changed</h3>
<ul>
<li>Add feature <code>crate-name</code> enabled by default to opt-in
crate rename
support. See <a
href="https://redirect.github.com/la10736/rstest/issues/258">#258</a></li>
</ul>
<h2>[0.20.0] 2024/5/30</h2>
<h3>Add</h3>
<ul>
<li>Implemented <code>#[by_ref]</code> attribute to take get a local
lifetime for test arguments.
See <a
href="https://redirect.github.com/la10736/rstest/issues/241">#241</a>
for more details. Thanks to
<a href="https://github.com/narpfel"><code>@​narpfel</code></a> for
suggesting it and useful discussions.</li>
<li>Support for import <code>rstest</code> with another name. See <a
href="https://redirect.github.com/la10736/rstest/issues/221">#221</a></li>
</ul>
<h3>Fixed</h3>
<ul>
<li>Don't remove Lifetimes from test function if any. See <a
href="https://redirect.github.com/la10736/rstest/issues/230">#230</a>
<a href="https://redirect.github.com/la10736/rstest/issues/241">#241</a>
for more details.</li>
<li><a
href="https://doc.rust-lang.org/std/path/struct.PathBuf.html"><code>PathBuf</code></a>
does no longer need to be
in scope when using <code>#[files]</code> (see <a
href="https://redirect.github.com/la10736/rstest/pull/242">#242</a>)</li>
<li><code>#[from(now::accept::also::path::for::fixture)]</code> See <a
href="https://redirect.github.com/la10736/rstest/issues/246">#246</a>
for more details</li>
</ul>
<h2>[0.19.0] 2024/4/9</h2>
<h3>Changed</h3>
<!-- raw HTML omitted -->
</blockquote>
<p>... (truncated)</p>
</details>
<details>
<summary>Commits</summary>
<ul>
<li><a
href="https://github.com/la10736/rstest/commit/da11d4e48a80a91c275104db6297df0d7f77cf7f"><code>da11d4e</code></a>
Update dependency and prepare the release</li>
<li><a
href="https://github.com/la10736/rstest/commit/0c6e203b5356ef8bd208823a84aa1f9252eba270"><code>0c6e203</code></a>
Update checkout list</li>
<li><a
href="https://github.com/la10736/rstest/commit/20e88586114be43faa6aec0b75cea1c6deace2cf"><code>20e8858</code></a>
Make clippy happy</li>
<li><a
href="https://github.com/la10736/rstest/commit/57a93425a5864fee6b497b1ba2fc0643dc750ae4"><code>57a9342</code></a>
Playground should use dev package</li>
<li><a
href="https://github.com/la10736/rstest/commit/8a0480358071c2024aca6e2506d9a1ac43eb0211"><code>8a04803</code></a>
Removed the useless build rerun variable</li>
<li><a
href="https://github.com/la10736/rstest/commit/8c232bce88264c43757bd355890618e2a7729bb0"><code>8c232bc</code></a>
Add a test for invalid base_dir value</li>
<li><a
href="https://github.com/la10736/rstest/commit/3520861ca5d56e74e1419b1305a032739743ffdc"><code>3520861</code></a>
Add tests for replace_env_vars</li>
<li><a
href="https://github.com/la10736/rstest/commit/465a4012af6d0cf5f4b5848718b8bbf067fdcfa3"><code>465a401</code></a>
Add unit tests for parsing attributes, and add base_dir</li>
<li><a
href="https://github.com/la10736/rstest/commit/a970613f432c7ebe574572ab10eb8e83807a2a28"><code>a970613</code></a>
Add a test for declared environment variables</li>
<li><a
href="https://github.com/la10736/rstest/commit/c65d1c002cc4e0915a9885a56ac4220e067c8571"><code>c65d1c0</code></a>
Add tests and #[ignore_missing_env_vars]</li>
<li>Additional commits viewable in <a
href="https://github.com/la10736/rstest/compare/v0.22.0...v0.23.0">compare
view</a></li>
</ul>
</details>
<br />


Dependabot will resolve any conflicts with this PR as long as you don't
alter it yourself. You can also trigger a rebase manually by commenting
`@dependabot rebase`.

[//]: # (dependabot-automerge-start)
[//]: # (dependabot-automerge-end)

---

<details>
<summary>Dependabot commands and options</summary>
<br />

You can trigger Dependabot actions by commenting on this PR:
- `@dependabot rebase` will rebase this PR
- `@dependabot recreate` will recreate this PR, overwriting any edits
that have been made to it
- `@dependabot merge` will merge this PR after your CI passes on it
- `@dependabot squash and merge` will squash and merge this PR after
your CI passes on it
- `@dependabot cancel merge` will cancel a previously requested merge
and block automerging
- `@dependabot reopen` will reopen this PR if it is closed
- `@dependabot close` will close this PR and stop Dependabot recreating
it. You can achieve the same result by closing it manually
- `@dependabot show <dependency name> ignore conditions` will show all
of the ignore conditions of the specified dependency
- `@dependabot ignore this major version` will close this PR and stop
Dependabot creating any more for this major version (unless you reopen
the PR or upgrade to it yourself)
- `@dependabot ignore this minor version` will close this PR and stop
Dependabot creating any more for this minor version (unless you reopen
the PR or upgrade to it yourself)
- `@dependabot ignore this dependency` will close this PR and stop
Dependabot creating any more for this dependency (unless you reopen the
PR or upgrade to it yourself)


</details>

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Agustin Borgna <[email protected]>

.github/pre-commit

@@ -22,7 +22,7 @@ then
     exit 1
 fi
 
-if ! cargo test --all-features --workspace
+if ! cargo test --features "proptest,serde,petgraph"
 then
     echo "There are some test issues."
     exit 1

Cargo.toml

@@ -40,7 +40,7 @@ petgraph = ["dep:petgraph"]
 [dev-dependencies]
 criterion = { version = "0.5.1", features = ["html_reports"] }
 rmp-serde = "1.1.1"
-rstest = "0.22.0"
+rstest = "0.23.0"
 itertools = "0.13.0"
 insta = "1.39.0"
 

src/view/subgraph.rs

@@ -74,12 +74,11 @@ where
         let mut inputs = Vec::new();
         let mut outputs = Vec::new();
 
-        let boundary = boundary.into_iter().map(|p| {
+        let boundary = boundary.into_iter().inspect(|&p| {
             match graph.port_direction(p).unwrap() {
                 Direction::Incoming => inputs.push(p),
                 Direction::Outgoing => outputs.push(p),
             };
-            p
         });
 
         let (nodes, ports) = traverse_subgraph(graph.clone(), boundary);