Code Monkey home page Code Monkey logo

Comments (14)

ghostwriter avatar ghostwriter commented on May 26, 2024 7

Sorry about that… checking now.

from mockery.

ghostwriter avatar ghostwriter commented on May 26, 2024 5

Pushed a new tag 1.6.9, pointing to the previous tag/commit 1.6.7.

Please let me know if it resolves this issue and if there is anything else i need to do from my end.

from mockery.

driesvints avatar driesvints commented on May 26, 2024 4

1.6.9 resolved the issues for us at Laravel btw. Thanks!

from mockery.

mfn avatar mfn commented on May 26, 2024 2

Can confirm, tested the latest version on a private project, lots of failed tests, similar as shown in https://github.com/laravel/framework/actions/runs/8249377959/job/22561635869#step:8:178

from mockery.

taylorotwell avatar taylorotwell commented on May 26, 2024 2

Thanks!

from mockery.

taylorotwell avatar taylorotwell commented on May 26, 2024 2

Thanks - we'll take a look a bit later today.

from mockery.

seyfahni avatar seyfahni commented on May 26, 2024 2

Just as an additional note, I have a class Deliverable that defines the __set method. The test fails during the \Mockery::mock(...) call because __set of the mocked class calls a method, causing it to fail with

Mockery\Exception\BadMethodCallException: Received Mockery_5_App_Models_MyModel::setAttribute(), but no expectations were specified

/var/www/service/vendor/illuminate/database/Eloquent/Model.php:1539
/var/www/service/vendor/mockery/mockery/library/Mockery/Container.php:398
/var/www/service/vendor/mockery/mockery/library/Mockery.php:477
/var/www/service/tests/Services/Delivery/Dependency/ActionDependencyErrorTest.php:21

This might be the main cause of the breakages...

Edit: I can also confirm that 1.6.9 works again. Thank you very much for the fast response!

from mockery.

mfn avatar mfn commented on May 26, 2024 2

@ghostwriter can confirm, works for me, thank you 🫡

from mockery.

mfn avatar mfn commented on May 26, 2024 2

Let me know if you need testers for a new release with the changes you intended.

I mean, testing Laravel is easy, you can probably fork it, switch to your dev version in composer.json and see if it works.

I can also offer running it on a private code base (20k+ tests, lots-o-mockery) and see how it goes, if you let me know which version to test.

from mockery.

driesvints avatar driesvints commented on May 26, 2024 2

Hi @ghostwriter. Looks like all tests pass for us! laravel/framework#50643

from mockery.

seyfahni avatar seyfahni commented on May 26, 2024 2

I can confirm as well that all my tests work with that version.

from mockery.

icemouton avatar icemouton commented on May 26, 2024 2

Not to be the guy that re-opens a seemingly fixed issue, but since my issue is so similar in context I believe this is the place to report this.

Ever since 1.6.8 all my test suite fails. If I revert to 1.6.7 everything is fine.

On 1.6.8-1.6.11 I get multiple times this error :

Mockery\Exception\BadMethodCallException: Received Mockery_32_VMU_FileUpload::__construct(), but no expectations were specified

For each of my mocks used.

Is this an intended change ? Do I now need to set an expectation for __construct ? That seems a bit excessive and counter-productive.

from mockery.

sakarikl avatar sakarikl commented on May 26, 2024

Expectation check became more strict. For us it was expected array keys order and string / int mismatches.

from mockery.

ghostwriter avatar ghostwriter commented on May 26, 2024

Hey everyone,

we were able to test a fix for this issue in #1396 and it is now merged into 1.6.x branch.

To test this branch,

you can use composer require mockery/mockery:1.6.x-dev --dev

or you can manually include the following in your composer.json file:

"require-dev": {
    "mockery/mockery":"1.6.x-dev"
}

And then run composer update to fetch the latest changes from the 1.6.x branch.

Please let me know if there are any issues when you run your tests.

I will wait for your feedback before creating the next 1.6.10 tag.

Thanks

from mockery.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.