Skip to content

Conversation

ulin-evgeny
Copy link
Contributor

No description provided.

@ulin-evgeny ulin-evgeny requested a review from DenTray November 28, 2024 10:56
@ulin-evgeny ulin-evgeny self-assigned this Nov 28, 2024
@DenTray DenTray assigned AZabolotnikov and unassigned ulin-evgeny Dec 2, 2024
@AZabolotnikov AZabolotnikov changed the title #153: ModelTestState change fixtures directory #153: ModelTestState db_change fixtures directory Dec 2, 2024
@AZabolotnikov AZabolotnikov changed the title #153: ModelTestState db_change fixtures directory #153: ModelTestState db_changes fixtures directory Dec 2, 2024
$dir = Str::beforeLast($path, '/');

if (!is_dir($dir)) {
mkdir($dir, 0755, true);
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
mkdir($dir, 0755, true);
mkdir(
directory: $dir,
recursive: true
);


$fixtureName = 'export_json/some_directory/response.json';

$this->exportJson($fixtureName, new TestResponse(
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
$this->exportJson($fixtureName, new TestResponse(
$this->exportContent($fixtureName, $result);

@DenTray DenTray merged commit 098b4cc into master Dec 2, 2024
2 checks passed
@DenTray DenTray deleted the 153-modelteststate-change-fixtures-directory branch July 9, 2025 03:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants