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
Branch
chmod-test
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 3781
Build 3792: [Placeholder Plan] Wait for 30 Seconds

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.