[PHP] Add cp method to Universal PHP

[mho22]

Motivation for the change, related issues

Based on this issue

While playing with PHP.wasm, I found out it has a mv method, but no cp method.

Implementation details

• Using directly FSHelpers.copyRecursive inside the php.cp() method
• Adding tests
• Keep consistency with the mv() when throwing errors
• Throws a new error when copyRecursive is recursively copying inside himself.

Testing Instructions

CI

or manually :

cp.ts :

import { PHP } from '@php-wasm/universal';
import { loadNodeRuntime } from "@php-wasm/node";

const php = new PHP(await loadNodeRuntime('8.5'));

php.mkdir('test');

php.chdir('test');

php.writeFile('foo', `<?php echo "File copied successfully";`);

php.cp('foo', 'bar');

const response = await php.runStream({scriptPath: 'bar'});

console.log(await response.stdoutText, php.listFiles('/test'));

Output :

> node cp.ts

File copied successfully [ 'foo', 'bar' ]

[brandonpayton]

It would be good to add some tests like we have for mv() here:

wordpress-playground/packages/php-wasm/node/src/test/php-part-1.spec.ts

Line 1496 in 7b722e7

it('mv() should move a file', () => {

[mho22]

@brandonpayton I added tests and found out that if you copy a directory recursively inside himself, it freezes, so I also throw an error when it happens. Let me know if something looks wrong. And thank you for your review.

[brandonpayton]

Thanks, @mho22! This looks good to go except there appears to be one duplicate test. I left a note about that. Once that is fixed (if it is indeed an issue), let's merge!

[brandonpayton]

This looks like a duplicate of the previous test. Was this meant to be a MEMFS-to-NODEFS variation?

[mho22]

You're right, I made a mistake here, I probably copy pasted it to work on the MEMFS to NODEFS later and forgot to implement it correctly. Done now.