Thanks to visit codestin.com
Credit goes to github.com

Skip to content

feat(cp): support update flag when recursing #889

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.

Sign up for GitHub

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

Merged
merged 11 commits into from
Nov 13, 2018
Merged

feat(cp): support update flag when recursing #889

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
c40496a
Propogate the -u option when copying recursively
joshi-sh Sep 18, 2018
4ca5694
Added test for cp -Ru
joshi-sh Sep 18, 2018
dface6c
Fixed lint errors
joshi-sh Sep 18, 2018
8d5811d
Better description of test case
joshi-sh Sep 19, 2018
3f2d0fd
Fixed test to check contents. Added test for succesful updates
joshi-sh Sep 20, 2018
3283d5e
Fixed undefined reference error
joshi-sh Sep 20, 2018
ee59552
Fixed lint errors
joshi-sh Sep 20, 2018
cdb269f
Cleaned up comments and strings to make them more descriptive
joshi-sh Oct 28, 2018
d4c27fc
Created constant for magic number (two days in ms)
joshi-sh Oct 28, 2018
0b1579b
Merge branch 'master' into issue-808
joshi-sh Oct 28, 2018
f590747
Removed trailing spaces
joshi-sh Oct 28, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Cleaned up comments and strings to make them more descriptive
  • Loading branch information
@joshi-sh
joshi-sh committed Oct 28, 2018
commit cdb269f8c4111ca603152a752d75e99ce58d9c7c
15 changes: 7 additions & 8 deletions test/cp.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -458,16 +458,16 @@ test('-Ru respects the -u flag recursively (don\'t update newer file)', t => {
const destDir = `${dir}/new`;
const destFile = `${destDir}/file`;
[sourceDir, destDir].forEach(d => shell.mkdir('-p', d));
shell.ShellString('foo\n').to(sourceFile);
shell.ShellString('bar\n').to(destFile);
shell.ShellString('Source File Contents\n').to(sourceFile);
shell.ShellString('Destination File Contents\n').to(destFile);
// Get the old mtime for dest
const oldTime = fs.statSync(destFile).mtimeMs;
// Send the old file to two days ago
// Send the source file to be older than the destination file
shell.touch('-m', oldTime - (2 * 24 * 60 * 60 * 1000), sourceFile);
// Now, copy the old dir to the new one
shell.cp('-Ru', sourceDir, destDir);
// Check that dest has not been updated
t.is(shell.cat(destFile).stdout, 'bar\n');
t.is(shell.cat(destFile).stdout, 'Destination File Contents\n');
});

test('-Ru respects the -u flag recursively (update older file)', t => {
Expand All @@ -477,17 +477,16 @@ test('-Ru respects the -u flag recursively (update older file)', t => {
const destDir = `${dir}/new`;
const destFile = `${destDir}/file`;
[sourceDir, destDir].forEach(d => shell.mkdir('-p', d));
shell.ShellString('foo\n').to(sourceFile);
shell.ShellString('bar\n').to(destFile);
shell.ShellString('Source File Contents\n').to(sourceFile);
shell.ShellString('Destination File Contents\n').to(destFile);
// Get the old mtime for dest
const oldTime = fs.statSync(destFile).mtimeMs;
// Send the source file to two days ahead
shell.touch('-m', oldTime + (2 * 24 * 60 * 60 * 1000), sourceFile);
// Now, copy the old dir to the new one
shell.cp('-Ru', sourceDir, destDir);
// Get the new mtime for dest
// Check that dest has been updated
t.is(shell.cat(sourceFile).stdout, 'foo\n');
t.is(shell.cat(sourceFile).stdout, 'Source File Contents\n');
});

test('using -P explicitly works', t => {
Expand Down