Fix profile test teardown

This should make #20420 pass
for this class

Civi/Test/ContactTestTrait.php

@@ -18,12 +18,14 @@ trait ContactTestTrait {
   /**
    * Emulate a logged in user since certain functions use that.
    * value to store a record in the DB (like activity)
+   *
    * @see https://issues.civicrm.org/jira/browse/CRM-8180
    *
    * @return int
    *   Contact ID of the created user.
+   * @throws \CiviCRM_API3_Exception
    */
-  public function createLoggedInUser() {
+  public function createLoggedInUser(): int {
     $params = [
       'first_name' => 'Logged In',
       'last_name' => 'User ' . rand(),

tests/phpunit/CiviTest/CiviUnitTestCase.php

@@ -30,6 +30,7 @@
 use Civi\Api4\RelationshipType;
 use Civi\Payment\System;
 use Civi\Api4\OptionValue;
+use Civi\Test\Api3DocTrait;
 use League\Csv\Reader;
 
 /**
@@ -58,7 +59,7 @@
  */
 class CiviUnitTestCase extends PHPUnit\Framework\TestCase {
 
-  use \Civi\Test\Api3DocTrait;
+  use Api3DocTrait;
   use \Civi\Test\GenericAssertionsTrait;
   use \Civi\Test\DbTestTrait;
   use \Civi\Test\ContactTestTrait;
@@ -3122,7 +3123,7 @@ protected function enableTaxAndInvoicing($params = []) {
    *
    * @throws \CRM_Core_Exception
    */
-  protected function disableTaxAndInvoicing() {
+  protected function disableTaxAndInvoicing(): \Civi\Core\SettingsBag {
     $accounts = $this->callAPISuccess('EntityFinancialAccount', 'get', ['account_relationship' => 'Sales Tax Account is'])['values'];
     foreach ($accounts as $account) {
       $this->callAPISuccess('EntityFinancialAccount', 'delete', ['id' => $account['id']]);