document that @psalm-internal works for namespace + class too

as used in #9584
vimeo · Mar 27, 2024 · f8724a2 · f8724a2
1 parent 4b01704
commit f8724a2
Show file tree

Hide file tree

Showing 2 changed files with 85 additions and 3 deletions.
diff --git a/docs/annotating_code/supported_annotations.md b/docs/annotating_code/supported_annotations.md
@@ -252,9 +252,9 @@ $b = $a->bar(); // this call fails
 
 ### `@psalm-internal`
 
-Used to mark a class, property or function as internal to a given namespace. Psalm treats this slightly differently to
-the PHPDoc `@internal` tag. For `@internal`, an issue is raised if the calling code is in a namespace completely
-unrelated to the namespace of the calling code, i.e. not sharing the first element of the namespace.
+Used to mark a class, property or function as internal to a given namespace or class. Psalm treats this slightly
+differently to the PHPDoc `@internal` tag. For `@internal`, an issue is raised if the calling code is in a namespace
+completely unrelated to the namespace of the calling code, i.e. not sharing the first element of the namespace.
 
 In contrast for `@psalm-internal`, the docblock line must specify a namespace. An issue is raised if the calling code
 is not within the given namespace.

diff --git a/tests/InternalAnnotationTest.php b/tests/InternalAnnotationTest.php
@@ -606,6 +606,46 @@ public function baz(): void
                     }
                 ',
             ],
+            'psalmInternalClassWithCallClass' => [
+                'code' => '<?php
+                    namespace A {
+                        /**
+                         * @psalm-internal B\Bat
+                         */
+                        class Foo {
+                            public static function barBar(): void {
+                            }
+                        }
+                    }
+
+                    namespace B {
+                        class Bat {
+                            public function batBat() : void {
+                                \A\Foo::barBar();
+                            }
+                        }
+                    }',
+            ],
+            'psalmInternalMethodWithCallClass' => [
+                'code' => '<?php
+                    namespace A {
+                        class Foo {
+                            /**
+                             * @psalm-internal B\Bat
+                             */
+                            public static function barBar(): void {
+                            }
+                        }
+                    }
+
+                    namespace B {
+                        class Bat {
+                            public function batBat() : void {
+                                \A\Foo::barBar();
+                            }
+                        }
+                    }',
+            ],
             'callToInternalMethodFromAnonymousClass' => [
                 'code' => <<<'PHP'
                 <?php
@@ -1118,6 +1158,48 @@ public function __construct() {}
                 ',
                 'error_message' => 'InternalMethod',
             ],
+            'psalmInternalClassWithCallClass' => [
+                'code' => '<?php
+                    namespace A {
+                        /**
+                         * @psalm-internal B\Bar
+                         */
+                        class Foo {
+                            public static function barBar(): void {
+                            }
+                        }
+                    }
+
+                    namespace B {
+                        class Bat {
+                            public function batBat() : void {
+                                \A\Foo::barBar();
+                            }
+                        }
+                    }',
+                'error_message' => 'InternalMethod',
+            ],
+            'psalmInternalMethodWithCallClass' => [
+                'code' => '<?php
+                    namespace A {
+                        class Foo {
+                            /**
+                             * @psalm-internal B\Bar
+                             */
+                            public static function barBar(): void {
+                            }
+                        }
+                    }
+
+                    namespace B {
+                        class Bat {
+                            public function batBat() : void {
+                                \A\Foo::barBar();
+                            }
+                        }
+                    }',
+                'error_message' => 'InternalMethod',
+            ],
         ];
     }
 }