MusicalNinjaDad / FizzBuzz

Combining rust & python with pyo3
https://musicalninjadad.github.io/FizzBuzz/
2 stars 1 forks source link

Update googletest requirement from 0.11.0 to 0.12.0 #63

Open dependabot[bot] opened 2 months ago

dependabot[bot] commented 2 months ago

Updates the requirements on googletest to permit the latest version.

Release notes

Sourced from googletest's releases.

v0.12.0

Breaking changes

Matcher trait definition

The Matcher trait definition has changed:

Pre 0.12.0

pub trait Matcher {
    type ActualT: Debug + ?Sized;
    fn matches(&self, actual: &Self::ActualT) -> MatcherResult;
}

0.12.0

pub trait Matcher<ActualT: Debug + Copy> {
    fn matches(&self, actual: ActualT) -> MatcherResult;
}

This makes the trait implementation more generic in three ways:

  • A single struct can implement Matcher for multiple types, since Actual is now a generic type parameter.
  • ActualT can be passed as value if it implements Copy, which solved #351.
  • When ActualT is a reference, the Matcher implementation can put constraint on its lifetime, which solved #323.

We tried to make sure this change had as limited an impact on users of the library as possible. Often, the library can be updated without changes on the tests. However, there are two areas requiring changes:

Matcher implementation

Obviously, libraries implementing their own matcher will need to update their implementation to match the new trait definition. For instance:

Pre 0.12.0

#[derive(Debug)]
enum MyEnum {
    ...
}

struct MyEnumMatcher { ... }

impl Matcher for MyEnumMatcher { type ActualT = MyEnumMatcher;

fn matches(&amp;self, actual: &amp;Self::ActualT) -&gt; MatcherResult {
    match actual {
        ...
    }
}

</tr></table>

... (truncated)

Commits
  • de68875 Merge pull request #442 from google:new_version
  • 70a6da1 Generalize into_test_result() extension method to all std::result::Result and...
  • 63421a6 Eliminate an unnecessary memory allocation in is_utf8_string
  • 876dbe2 Prepare for 0.12 release.
  • 89655d0 Handle non-copyable type in (expect|verify)_(ne|eq)!
  • c33408e Handle non-copyable type in (expect|verify)_(ne|eq)!
  • 07dffcf Handle non-copyable type in (expect|verify)_(ne|eq)!
  • cccc94a Add #[track_caller] since near can panic.
  • 896b468 Merge pull request #436 from google:cargo_lock
  • ac19539 Add new latest deps workflow.
  • Additional commits viewable in compare view


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 commands and options
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 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)
codecov[bot] commented 2 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Additional details and impacted files | [Flag](https://app.codecov.io/gh/MusicalNinjaDad/FizzBuzz/pull/63/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None) | Coverage Δ | | |---|---|---| | [fizzbuzz](https://app.codecov.io/gh/MusicalNinjaDad/FizzBuzz/pull/63/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None) | `?` | | | [fizzbuzzo3](https://app.codecov.io/gh/MusicalNinjaDad/FizzBuzz/pull/63/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None) | `?` | | | [python](https://app.codecov.io/gh/MusicalNinjaDad/FizzBuzz/pull/63/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None) | `100.00% <ø> (ø)` | | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None#carryforward-flags-in-the-pull-request-comment) to find out more.

:loudspeaker: Thoughts on this report? Let us know!