Page MenuHomePhabricator
Files F15483026

D11052.id26981.diff
No OneTemporary
Actions

D11052.id26981.diff
View Options

diff --git a/src/__tests__/PhutilLibraryTestCase.php b/src/__tests__/PhutilLibraryTestCase.php
--- a/src/__tests__/PhutilLibraryTestCase.php
+++ b/src/__tests__/PhutilLibraryTestCase.php
@@ -39,6 +39,59 @@
'arc liberate'));
}
+ public function testMethodVisibility() {
+ $symbols = id(new PhutilSymbolLoader())
+ ->selectSymbolsWithoutLoading();
+
+ $classes = array();
+ foreach ($symbols as $symbol) {
+ if ($symbol['type'] == 'class') {
+ $classes[$symbol['name']] = new ReflectionClass($symbol['name']);
+ }
+ }
+
+ foreach ($classes as $class_name => $class) {
+ $parents = array();
+ $parent = $class;
+ while ($parent = $parent->getParentClass()) {
+ $parents[] = $parent;
+ }
+
+ $interfaces = $class->getInterfaces();
+
+ foreach ($class->getMethods() as $method) {
+ $visibility = $this->getVisibility($method);
+ $method_name = $method->getName();
+
+ foreach (array_merge($parents, $interfaces) as $extends) {
+ if ($extends->hasMethod($method_name)) {
+ $xmethod = $extends->getMethod($method_name);
+ $xvisibility = $this->getVisibility($xmethod);
+ if ($visibility != $xvisibility) {
+ $this->assertEqual(
+ $xvisibility,
+ $visibility,
+ pht(
+ 'Class "%s" implements method "%s" with the wrong '.
+ 'visibility. The method has visibility "%s", but it is '.
+ 'defined in parent "%s" with visibility "%s". In '.
+ 'Phabricator, a method which overrides another must '.
+ 'always have the same visibility.',
+ $class_name,
+ $method_name,
+ $visibility,
+ $extends->getName(),
+ $xvisibility));
+ }
+
+ // We found a declaration somewhere, so stop looking.
+ break;
+ }
+ }
+ }
+ }
+ }
+
/**
* Get the root directory for the library currently being tested.
*/
@@ -47,4 +100,14 @@
return phutil_get_library_root_for_path($caller['file']);
}
+ private function getVisibility(ReflectionMethod $method) {
+ if ($method->isPrivate()) {
+ return 'private';
+ } else if ($method->isProtected()) {
+ return 'protected';
+ } else {
+ return 'public';
+ }
+ }
+
}

File Metadata

Mime Type
text/plain
Expires
Thu, Apr 10, 9:23 AM (5 d, 5 h ago)
Storage Engine
blob
Storage Format
Encrypted (AES-256-CBC)
Storage Handle
7698273
Default Alt Text
D11052.id26981.diff (2 KB)

Event Timeline

Phabricator · Built by Phacility