From e2a5bcb526fc75251859a7584553bbc56488cd3d Mon Sep 17 00:00:00 2001
From: kkmuffme <11071985+kkmuffme@users.noreply.github.com>
Date: Wed, 15 Nov 2023 21:04:58 +0100
Subject: [PATCH] Remove tests for partially stubbed classes introduced in
 https://github.com/vimeo/psalm/issues/849, as this didn't work correctly and
 lead to errors not being reported at all in many cases (since the error was
 reported for the stubs file and therefore suppressed) With the previous
 commit "Fix for classes what https://github.com/vimeo/psalm/pull/8503 fixed
 for functions", the stubs of classes where the actual file is in the analyzed
 files are ignored completely.

---
 tests/StubTest.php | 96 ++--------------------------------------------
 1 file changed, 4 insertions(+), 92 deletions(-)

diff --git a/tests/StubTest.php b/tests/StubTest.php
index 8d92421a1a2..6c835a65d8d 100644
--- a/tests/StubTest.php
+++ b/tests/StubTest.php
@@ -317,10 +317,10 @@ public function testPhpStormMetaParsingFile(): void
             '<?php
                 namespace Ns {
                     class MyClass {
-                    
+
                         public const OBJECT = "object";
                         private const EXCEPTION = "exception";
-                        
+
                         /**
                          * @return mixed
                          * @psalm-suppress InvalidReturnType
@@ -332,7 +332,7 @@ public function create(string $s) {}
                          * @psalm-suppress InvalidReturnType
                          */
                         public function create2(string $s) {}
-                        
+
                         /**
                          * @return mixed
                          * @psalm-suppress InvalidReturnType
@@ -384,7 +384,7 @@ function bar(array $a) {}
 
                     $y1 = (new \Ns\MyClass)->creAte2("object");
                     $y2 = (new \Ns\MyClass)->creaTe2("exception");
-                    
+
                     $const1 = (new \Ns\MyClass)->creAte3(\Ns\MyClass::OBJECT);
                     $const2 = (new \Ns\MyClass)->creaTe3("exception");
 
@@ -1194,94 +1194,6 @@ class Bar extends PartiallyStubbedClass  {}
         $this->analyzeFile($file_path, new Context());
     }
 
-    public function testStubFileWithPartialClassDefinitionWithCoercion(): void
-    {
-        $this->expectExceptionMessage('TypeCoercion');
-        $this->expectException(CodeException::class);
-        $this->project_analyzer = $this->getProjectAnalyzerWithConfig(
-            TestConfig::loadFromXML(
-                dirname(__DIR__),
-                '<?xml version="1.0"?>
-                <psalm
-                    errorLevel="1"
-                >
-                    <projectFiles>
-                        <directory name="src" />
-                    </projectFiles>
-
-                    <stubs>
-                        <file name="tests/fixtures/stubs/partial_class.phpstub" />
-                    </stubs>
-                </psalm>',
-            ),
-        );
-
-        $file_path = getcwd() . '/src/somefile.php';
-
-        $this->addFile(
-            $file_path,
-            '<?php
-                namespace Foo;
-
-                class PartiallyStubbedClass  {
-                    /**
-                     * @param string $a
-                     * @return object
-                     */
-                    public function foo(string $a) {
-                        return new self;
-                    }
-                }
-
-                (new PartiallyStubbedClass())->foo("dasda");',
-        );
-
-        $this->analyzeFile($file_path, new Context());
-    }
-
-    public function testStubFileWithPartialClassDefinitionGeneralReturnType(): void
-    {
-        $this->expectExceptionMessage('InvalidReturnStatement');
-        $this->expectException(CodeException::class);
-        $this->project_analyzer = $this->getProjectAnalyzerWithConfig(
-            TestConfig::loadFromXML(
-                dirname(__DIR__),
-                '<?xml version="1.0"?>
-                <psalm
-                    errorLevel="1"
-                >
-                    <projectFiles>
-                        <directory name="src" />
-                    </projectFiles>
-
-                    <stubs>
-                        <file name="tests/fixtures/stubs/partial_class.phpstub" />
-                    </stubs>
-                </psalm>',
-            ),
-        );
-
-        $file_path = getcwd() . '/src/somefile.php';
-
-        $this->addFile(
-            $file_path,
-            '<?php
-                namespace Foo;
-
-                class PartiallyStubbedClass  {
-                    /**
-                     * @param string $a
-                     * @return object
-                     */
-                    public function foo(string $a) {
-                        return new \stdClass;
-                    }
-                }',
-        );
-
-        $this->analyzeFile($file_path, new Context());
-    }
-
     public function testStubFileWithTemplatedClassDefinitionAndMagicMethodOverride(): void
     {
         $this->project_analyzer = $this->getProjectAnalyzerWithConfig(