Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CI test mismatch on local runner #12404

Closed
s-puig opened this issue Mar 10, 2024 · 0 comments
Closed

CI test mismatch on local runner #12404

s-puig opened this issue Mar 10, 2024 · 0 comments
Labels
A-Build-System Related to build systems or continuous integration C-Bug An unexpected or incorrect behavior

Comments

@s-puig
Copy link
Contributor

s-puig commented Mar 10, 2024

Bevy version

Main branch bc4d8bb

What you did

cargo run -p ci -- compile

What went wrong

test tests/reflect_derive/generics.fail.rs [should fail to compile] ... mismatch

ACTUAL OUTPUT:
┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈
error[E0277]: the trait bound `NoReflect: Reflect` is not satisfied
   --> tests/reflect_derive/generics.fail.rs:16:21
    |
16  |     foo.get_field::<NoReflect>("a").unwrap();
    |         ---------   ^^^^^^^^^ the trait `Reflect` is not implemented for `NoReflect`
    |         |
    |         required by a bound introduced by this call
    |
    = help: the following other types implement trait `Reflect`:
              bool
              char
              isize
              i8
              i16
              i32
              i64
              i128
            and $N others
note: required by a bound in `bevy_reflect::GetField::get_field`
   --> /home/foo/bevy/crates/bevy_reflect/src/struct_trait.rs:242:21
    |
242 |     fn get_field<T: Reflect>(&self, name: &str) -> Option<&T>;
    |                     ^^^^^^^ required by this bound in `GetField::get_field`

error[E0277]: the trait bound `NoReflect: GetTypeRegistration` is not satisfied
  --> tests/reflect_derive/generics.fail.rs:14:36
   |
14 |     let mut foo: Box<dyn Struct> = Box::new(Foo::<NoReflect> { a: NoReflect(42.0) });
   |                                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ the trait `GetTypeRegistration` is not implemented for `NoReflect`
   |
   = help: the following other types implement trait `GetTypeRegistration`:
             bool
             char
             isize
             i8
             i16
             i32
             i64
             i128
           and $N others
   = note: required for `NoReflect` to implement `RegisterForReflection`
note: required for `Foo<NoReflect>` to implement `bevy_reflect::Struct`
  --> tests/reflect_derive/generics.fail.rs:3:10
   |
3  | #[derive(Reflect)]
   |          ^^^^^^^ unsatisfied trait bound introduced in this `derive` macro
4  | #[reflect(from_reflect = false)]
5  | struct Foo<T> {
   |        ^^^^^^
   = note: required for the cast from `Box<Foo<NoReflect>>` to `Box<(dyn bevy_reflect::Struct + 'static)>`
   = note: this error originates in the derive macro `Reflect` (in Nightly builds, run with -Z macro-backtrace for more info)
┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈

Additional information

I believe this is caused by #5781 due to a test trying to match the home directory.

@s-puig s-puig added C-Bug An unexpected or incorrect behavior S-Needs-Triage This issue needs to be labelled labels Mar 10, 2024
@james7132 james7132 added A-Build-System Related to build systems or continuous integration and removed S-Needs-Triage This issue needs to be labelled labels Mar 10, 2024
@s-puig s-puig closed this as completed May 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Build-System Related to build systems or continuous integration C-Bug An unexpected or incorrect behavior
Projects
None yet
Development

No branches or pull requests

2 participants