Page MenuHomePhabricator

Add unit tests for `ArcanistChmodLinter`
ClosedPublic

Authored by joshuaspence on Jan 12 2015, 10:06 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Dec 20, 10:50 PM
Unknown Object (File)
Sat, Dec 14, 4:28 PM
Unknown Object (File)
Dec 11 2024, 9:17 AM
Unknown Object (File)
Dec 8 2024, 12:36 PM
Unknown Object (File)
Dec 7 2024, 6:02 PM
Unknown Object (File)
Dec 7 2024, 10:59 AM
Unknown Object (File)
Dec 3 2024, 12:57 PM
Unknown Object (File)
Dec 2 2024, 4:29 PM
Subscribers

Details

Summary

Moar test coverage.

Test Plan

arc unit

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

joshuaspence retitled this revision from to Add unit tests for `ArcanistChmodLinter`.
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.
epriestley edited edge metadata.
This revision is now accepted and ready to land.Jan 14 2015, 2:59 PM
src/lint/linter/ArcanistChmodLinter.php
47

I was considering whether to add an isExecutable() method to ArcanistLintEngine, but decided against it for no particular reason. It could be reasonable to add this method though.

src/lint/linter/__tests__/ArcanistLinterTestCase.php
105

Oh, one other thing I noticed is that Filesystem::changePermissions doesn't do any validation on $mode, which seems wrong to me given that PHP will do unexpected things if you pass a string to $mode (see http://php.net/manual/en/function.chmod.php#96086, for example).

This revision was automatically updated to reflect the committed changes.