prophecy | Highly opinionated mocking framework for PHP | Unit Testing library

Deleted vendor directory. Ran composer install. Noticed message after installation

Although the Test passes but I'm not really convinced that's the way to do it.

If you've written that test and this is your judgement, I suggest you temporarily rewrite the test (e.g. in another test-method) where you test for your expectations in testing to verify they are addressed.

Otherwise it seems the test is not of your benefit as you don't understand what it tests for and therefore is superfluous code and waste (in the agile sense) and you can cleanly just remove it and not let it lurk there open to lying around.

Who needs a test that is unclear in what it tests? Especially in unit tests there should be only one reason why a test fails. Not possible with an unclear test.

Is it already cleanup time and then back to drawing board? Maybe. I'd suggest incremental improvement and some sandboxing personally first. Like adding a much reduced test-case-method for verifying your own expectation of the test-suite-framework and the (two?) mocking library/ies in use.

This will also help you get going with the framework in use and gain a deeper understanding - this normally immediately pays off.

I've another one ViewHandler::class which also uses a service for data for listing, which I'm sure I can implement if I get a clue for this one.

Your code your tests. Only you can say whether or not your tests full-fill your requirements.

And if you allow me a personal comment, I hate to do mocking in tests. Even for code mocking technically works it becomes cumbersome pretty soon and has the tendency that the tests are only testing the mocks that have been written for the test only so entirely needless work.

Instead I try to either have the code under test straight forward and if a certain abstraction requires a lot of set-up upfront, create a factory for it and then that factory can be used in tests, too, reducing the overhead to a minimum.

Then some specialization of the factory can be done for testing automatically inject the testing configuration (e.g. in form of mocks if it must be to set blank other systems the test should not reach into) and then just let it pass. But this is just exemplary.

In a system where you like to test system($request, $response)->assert(diverse on $response afterwards) where system is * of concrete classes you write (your implementation), you may want to have a tester for * so that your test-procedure remains clear on all the interfacing that system offers and * implements and you don't need to set-up internals of all of system for * only to test any *, e.g. a HandlerTester.

Also check if Mezzio itself is not offering a tester if there is a higher level abstraction implementation necessary for handlers. A good library normally ships with good testing utilities (and even in this case not, you can fork it anytime).

Testing should be before development, this is oh so true for libraries, so actually I would personally expect the stuff is already there in 0.0.1. But this can vary.

Enable code coverage also for your tests so you can more easily review if your tests do run the way it's intended and put also all collaborators under test and coverage. This can help to gain more understanding what a test does and maybe already clarifies if it is of use or not.

Your error message seems to be an error generated Composer. The documentation for composer seems to imply that any version >= 5.3.2 is sufficient src. However, with PHP 8.1 not even being in Alpha until June, I should imagine it's safe to say there will not be official support for this for some time.

I did do a quick search for your error and got this article which states that one of the breaking changes that 8.1 introduces is passing null to a function that is not nullable. Official PHP Docs for strpos() indicate that none of the 3 arguments in strpos() are nullable, and thus what was 'allowed' in previous versions due to scalar types is no longer allowed for internal functions in 8.1, so you'll likely need to wait for Composer to officially support 8.1.

Even if you do fix Composer you're still using an unsupported version of PHP for Laravel. From their git the composer.json file specifies

Try and set the encoding to UTF-8

For example:

file = open(filename, encoding="utf8")

For reference check this post:

UnicodeDecodeError: 'charmap' codec can't decode byte X in position Y: character maps to

always chose Target_name and fallback to Name.

This could be accomplished by a single XPath expression:

Well, I did some research and found a way, which works for me. PHPSpec is so much more different to PHPUnit.

I got the solution. I used Object.keys(res.body) to get all the keys from the object. I got this output when I printed Object.keys(res.body):

Actually you should make use of dependency injection in the class which exposes the getRootline() API, then you can have the PageService injected this way:

I have cloned private git repository, the problem is I have a lot of git conflict markers which are causing my code to fail, for example:

Well then whoever put the original code there included them and broke the code, and then pushed it in a non-functional state. Write them an email and tell them to fix that, if it's still an option. Not your fault, shouldn't be your problem.

It's often easiest to find the commit where they did that (git blame). It's probably a merge commit, which they didn't do (probably because they don't understand what a git merge is). Then, git reset to the commit before that, do the merge, this time properly. (Here's where you can do "automatically right", but I don't know your merge situation. Git merging isn't hard, read up on it!)

After you've done the merge without leaving all these conflicts in the source code, you then just cherry-pick whatever came after the git merge.

Again, someone checked in code that clearly and obviously was broken, and if it wasn't you, you should be very careful with the code - whoever commanded git didn't care to make sure the things they check in works, even on a basic level.