You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tinkerpop.apache.org by GitBox <gi...@apache.org> on 2021/06/07 08:26:51 UTC
[GitHub] [tinkerpop] dependabot[bot] opened a new pull request #1431: Bump Moq from 4.7.99 to 4.16.1 in /gremlin-dotnet
dependabot[bot] opened a new pull request #1431:
URL: https://github.com/apache/tinkerpop/pull/1431
Bumps [Moq](https://github.com/moq/moq4) from 4.7.99 to 4.16.1.
<details>
<summary>Changelog</summary>
<p><em>Sourced from <a href="https://github.com/moq/moq4/blob/main/CHANGELOG.md">Moq's changelog</a>.</em></p>
<blockquote>
<h2>4.16.1 (2021-02-23)</h2>
<h4>Added</h4>
<ul>
<li><code>CallBase</code> can now be used with interface methods that have a default interface implementation. It will call <a href="https://docs.microsoft.com/en-us/dotnet/csharp/language-reference/proposals/csharp-8.0/default-interface-methods#the-most-specific-override-rule">the most specific override</a>. (<a href="https://github.com/stakx"><code>@stakx</code></a>, <a href="https://github-redirect.dependabot.com/moq/moq4/issues/1130">#1130</a>)</li>
</ul>
<h4>Changed</h4>
<ul>
<li>Improved error message formatting of <code>It.Is</code> lambda expressions that capture local variables. (<a href="https://github.com/bfriesen"><code>@bfriesen</code></a>, <a href="https://github-redirect.dependabot.com/moq/moq4/issues/1140">#1140</a>)</li>
</ul>
<h4>Fixed</h4>
<ul>
<li><code>AmbiguousMatchException</code> raised when interface has property indexer besides property in VB. (<a href="https://github.com/mujdatdinc"><code>@mujdatdinc</code></a>, <a href="https://github-redirect.dependabot.com/moq/moq4/issues/1129">#1129</a>)</li>
<li>Interface default methods are ignored (<a href="https://github.com/hahn-kev"><code>@hahn-kev</code></a>, <a href="https://github-redirect.dependabot.com/moq/moq4/issues/972">#972</a>)</li>
<li>Callback validation too strict when setting up a task's <code>.Result</code> property (<a href="https://github.com/stakx"><code>@stakx</code></a>, <a href="https://github-redirect.dependabot.com/moq/moq4/issues/1132">#1132</a>)</li>
<li><code>setup.Returns(InvocationFunc)</code> wraps thrown exceptions in <code>TargetInvocationException</code> (<a href="https://github.com/stakx"><code>@stakx</code></a>, <a href="https://github-redirect.dependabot.com/moq/moq4/issues/1141">#1141</a>)</li>
</ul>
<h2>4.16.0 (2021-01-16)</h2>
<h4>Added</h4>
<ul>
<li>
<p>Ability to directly set up the <code>.Result</code> of tasks and value tasks, which makes setup expressions more uniform by rendering dedicated async verbs like <code>.ReturnsAsync</code>, <code>.ThrowsAsync</code>, etc. unnecessary:</p>
<pre lang="diff"><code>-mock.Setup(x => x.GetFooAsync()).ReturnsAsync(foo)
+mock.Setup(x => x.GetFooAsync().Result).Returns(foo)
</code></pre>
<p>This is useful in places where there currently aren't any such async verbs at all:</p>
<pre lang="diff"><code>-Mock.Of<X>(x => x.GetFooAsync() == Task.FromResult(foo))
+Mock.Of<X>(x => x.GetFooAsync().Result == foo)
</code></pre>
<p>This also allows recursive setups / method chaining across async calls inside a single setup expression:</p>
<pre lang="diff"><code>-mock.Setup(x => x.GetFooAsync()).ReturnsAsync(Mock.Of<IFoo>(f => f.Bar == bar))
+mock.Setup(x => x.GetFooAsync().Result.Bar).Returns(bar)
</code></pre>
<p>or, with only <code>Mock.Of</code>:</p>
<pre lang="diff"><code>-Mock.Of<X>(x => x.GetFooAsync() == Task.FromResult(Mock.Of<IFoo>(f => f.Bar == bar)))
+Mock.Of<X>(x => x.GetFooAsync().Result.Bar == bar)
</code></pre>
</li>
</ul>
<!-- raw HTML omitted -->
</blockquote>
<p>... (truncated)</p>
</details>
<details>
<summary>Commits</summary>
<ul>
<li><a href="https://github.com/moq/moq4/commit/fc484fb85a28e8cd5d5c51c7e158df2ecc72621d"><code>fc484fb</code></a> Update version to 4.16.1</li>
<li><a href="https://github.com/moq/moq4/commit/0ddfdb89cd6d4641688cceaae0746ec67bbbc314"><code>0ddfdb8</code></a> <code>Returns(InvocationFunc)</code> shouldn't throw <code>TargetInvocationException</code></li>
<li><a href="https://github.com/moq/moq4/commit/f36d3e87e76692686f3d4a126955a915e724be24"><code>f36d3e8</code></a> Merge pull request <a href="https://github-redirect.dependabot.com/moq/moq4/issues/1140">#1140</a> from bfriesen/lambda_closure_support</li>
<li><a href="https://github.com/moq/moq4/commit/e96804f56d49fe8431d4f71fc61931ff3b24a5b3"><code>e96804f</code></a> Update the changelog</li>
<li><a href="https://github.com/moq/moq4/commit/5ae449c71a1bd2c168858c3a3ecfc57bb414bfd0"><code>5ae449c</code></a> Exclude name of the closure class</li>
<li><a href="https://github.com/moq/moq4/commit/8a2d2ed770e19c4df404cc4ade93b69282ff00b9"><code>8a2d2ed</code></a> Add test for closure access</li>
<li><a href="https://github.com/moq/moq4/commit/cf5af87dff83abe310e060c77563f5523baa9ef6"><code>cf5af87</code></a> Format lambda expression variables</li>
<li><a href="https://github.com/moq/moq4/commit/5b10a8ce3fc7c21e8b3c4e04a51423f9f9bfad0b"><code>5b10a8c</code></a> Add test for lamba matcher variables</li>
<li><a href="https://github.com/moq/moq4/commit/653db312e7b8fab824693f8bfa13755502e56199"><code>653db31</code></a> Some minor renames for consistency</li>
<li><a href="https://github.com/moq/moq4/commit/fc73131fe30f5c0be51437506da574a05ed383f1"><code>fc73131</code></a> Add missing copyright notices</li>
<li>Additional commits viewable in <a href="https://github.com/moq/moq4/compare/v4.7.99...v4.16.1">compare view</a></li>
</ul>
</details>
<br />
[](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)
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 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>
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [tinkerpop] FlorianHockmann merged pull request #1431: Bump Moq from 4.7.99 to 4.16.1 in /gremlin-dotnet
Posted by GitBox <gi...@apache.org>.
FlorianHockmann merged pull request #1431:
URL: https://github.com/apache/tinkerpop/pull/1431
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [tinkerpop] FlorianHockmann commented on pull request #1431: Bump Moq from 4.7.99 to 4.16.1 in /gremlin-dotnet
Posted by GitBox <gi...@apache.org>.
FlorianHockmann commented on pull request #1431:
URL: https://github.com/apache/tinkerpop/pull/1431#issuecomment-858610318
I'm going to merge this via CTR as it's only a test dependency.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org