Skip to content

Homework 1#1

Open
volukrem wants to merge 6 commits into
mainfrom
hw_1
Open

Homework 1#1
volukrem wants to merge 6 commits into
mainfrom
hw_1

Conversation

@volukrem

@volukrem volukrem commented Aug 4, 2024

Copy link
Copy Markdown
Owner

No description provided.

@volukrem volukrem self-assigned this Aug 4, 2024
@volukrem
volukrem requested a review from YlabCourse August 4, 2024 09:10
Comment thread carshop-service.iml

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Это не нужно в репо

* @return A string representing the user, excluding the password.
*/
@Override
public String toString() {

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

зачем переопределяешь, если это у тебя в аннотации

*/
@AllArgsConstructor
public class CarRepository {
private final Map<Integer, Car> carsMap;

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

коллекция - это отдельный слой, имитирующий БД, а репозиторий - слой, отвечающий за взаимодействие с БД

}

@Test
@DisplayName("Test toString method")

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

ну это слишком)


@BeforeEach
public void setUp() {
MockitoAnnotations.openMocks(this);

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

ты ведь не используешь аннотации, зачем это?

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.

2 participants