[clang] [clang][ASTImporter] Link FunctionDecl into its DeclContext before importing its body (PR #214009)

[Balázs Kéri via cfe-commits]

================
@@ -6504,10 +6504,9 @@ TEST_P(ErrorHandlingTest, ErrorHappensBeforeCreatingANewNode) {
   EXPECT_EQ(OptErr->Error, ASTImportError::NameConflict);
 }
 
-// Check a case when a new AST node is created but not linked to the AST before
----------------
balazske wrote:

Purpose of this test is exactly what the original title and comment says. Instead of changing this, another case should be found where a node is created and fails but is not linked into the AST.

https://github.com/llvm/llvm-project/pull/214009