Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Statements & Testing modules added + little additions to .gitignore(.idea added) #3

Merged
merged 6 commits into from Dec 7, 2021

Conversation

tre3p
Copy link
Contributor

@tre3p tre3p commented Dec 7, 2021

No description provided.

@fey
Copy link
Contributor

fey commented Dec 7, 2021

Тестики упали

make[1]: Entering directory '/exercises-kotlin/modules/10-basics/40-testing'
error: source file or directory not found: /exercises-kotlin/modules/10-basics/40-testing/Test.kt
error: could not find or load main class TestKt
make[1]: *** [Makefile:2: test] Error 1

@tre3p
Copy link
Contributor Author

tre3p commented Dec 7, 2021

Сможешь помочь немного с тестами? Я так и не разобрался как их правильно написать в таком контексте. Скопировал с первого урока и поменял данные просто

@fey
Copy link
Contributor

fey commented Dec 7, 2021

В данном случае ругается на то, что файлика нет внутри урока.
Вот пример наличия файла из другого урока
https://github.com/k0damaDEV/exercises-kotlin/blob/main/modules/10-basics/30-statements/Test.kt

+ error
)
}
}
Copy link
Contributor

Choose a reason for hiding this comment

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

Настрой плз редактор, чтобы он вставлял перенос строки в конце файлов =)

@fey
Copy link
Contributor

fey commented Dec 7, 2021

Сейчас падают конкретно тесты. Нужно разобраться в чем дело и поправить это дело

@tre3p
Copy link
Contributor Author

tre3p commented Dec 7, 2021

Я вроде понял в чем проблема. Я почему-то локально тесты не могу поднять, поэтому щас дофикшу и залью. Должно работать

@fey
Copy link
Contributor

fey commented Dec 7, 2021

Еще такая шутка, мы стараемся использовать простые, Но достаточно подробные (в плане вывода) тесты.
Посмотри, можно ли заиспользовать библиотеку типа https://github.com/bnorm/kotlin-power-assert

@tre3p
Copy link
Contributor Author

tre3p commented Dec 7, 2021

Окей, щас я разберусь с тестами, пойму как это работает, а потом везде заменю на эту либу

@fey
Copy link
Contributor

fey commented Dec 7, 2021

Ок, можно сделать отдельным ПРом. Для текущего достаточно, чтобы тесты работали

@tre3p
Copy link
Contributor Author

tre3p commented Dec 7, 2021

Во, кажется я понял теперь как это работает все. Тесты пофикшены :)

@fey fey merged commit 22a011c into hexlet-basics:main Dec 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants