Conversation
jskonst
left a comment
jskonst
left a comment
There was a problem hiding this comment.
У вас в комит просочились файлы каталогов obj и build - их надо убрать, они в gitignore отмечены, но вы их закоммитили еще до
| [Fact] | ||
| public void TestMyFunctionZeros() | ||
| { | ||
| var res = Program.MyFunction(0.0, 0.0); |
There was a problem hiding this comment.
Надо еще пару тестов добавить именно к вычислению функции
There was a problem hiding this comment.
Нужен еще тест на функцию, где параметры не равны 0
There was a problem hiding this comment.
Это все еще важно, может я конечно слепой :(
# Conflicts: # CourseApp/.vscode/launch.json
jskonst
left a comment
jskonst
left a comment
There was a problem hiding this comment.
Много лишних файлов - надо сделать зачистку
| [Fact] | ||
| public void TestMyFunctionZeros() | ||
| { | ||
| var res = Program.MyFunction(0.0, 0.0); |
| var expy = new double[] { 2.54483385166692, 2.5615887682505, 11.2873629585646, 363.108837198182, 10971.2863691829 }; | ||
| for (int i = 0; i < 5; i++) | ||
| { | ||
| Xunit.Assert.Equal(expy[i], res[i], 3); |
|
Слишком много файлов, которые должны быть в игноре |
|
Надо убрать лишние файлы! |
…gramming_42_2019 # Conflicts: # CourseApp/.vscode/tasks.json
|
@jskonst |
jskonst
left a comment
jskonst
left a comment
There was a problem hiding this comment.
🔧 Файлы остались - это легко проверить посмотрев список файлов в PR - 41. Файлы надо не только заигнорить, но и дополнительно убрать их (на Stackoverflow первая ссылка - https://stackoverflow.com/questions/13541615/how-to-remove-files-that-are-listed-in-the-gitignore-but-still-on-the-repositor
| @@ -0,0 +1,9 @@ | |||
| { | |||
| [Fact] | ||
| public void TestMyFunctionZeros() | ||
| { | ||
| var res = Program.MyFunction(0.0, 0.0); |
There was a problem hiding this comment.
Нужен еще тест на функцию, где параметры не равны 0
| [Fact] | ||
| public void TestTaskA() | ||
| { | ||
| double a = 1.6; |
There was a problem hiding this comment.
На задачу А нужно покрыть еще 2 случая xn > xk, и dx > xk-xn
CourseApp.Tests/PersonTest.cs
Outdated
| public void TestView() | ||
| { | ||
| var item = new Person(); | ||
| var view = @"hello"; |
There was a problem hiding this comment.
@ - нужен только для многострочного текста, тут можно убрать 🔕
CourseApp.Tests/PersonTest.cs
Outdated
| { | ||
| var item = new Person(); | ||
| item.Age = -5; | ||
| Assert.Equal(0, item.Age); |
There was a problem hiding this comment.
Тут надо проверять на Exception
| @@ -0,0 +1,13 @@ | |||
| <?xml version="1.0" encoding="utf-8" standalone="no"?> | |||
CourseApp/Person.cs
Outdated
|
|
||
| public string View() | ||
| { | ||
| return @"hello"; |
There was a problem hiding this comment.
Замените метод view на перекрытый toString
There was a problem hiding this comment.
В описании пусть он себя опишет - выведет имя например
|
🔧 Файлы остались - это легко проверить посмотрев список файлов в PR - 41. Файлы надо не только заигнорить, но и дополнительно убрать их (на Stackoverflow первая ссылка - https://stackoverflow.com/questions/13541615/how-to-remove-files-that-are-listed-in-the-gitignore-but-still-on-the-repositor |
|
пофиксил с классами всё, осталось тесты слегка переписать и будет готово |
| [Fact] | ||
| public void TestMyFunctionZeros() | ||
| { | ||
| var res = Program.MyFunction(0.0, 0.0); |
There was a problem hiding this comment.
Это все еще важно, может я конечно слепой :(
CourseApp/Person.cs
Outdated
| { | ||
| if (value == string.Empty) | ||
| { | ||
| Console.WriteLine("This field can't be empty."); |
There was a problem hiding this comment.
тут мы уже exception бросаем
CourseApp/Program.cs
Outdated
| return y; | ||
| } | ||
|
|
||
| public static double[] TaskB(double a, double[] x) |
There was a problem hiding this comment.
можно уже переезжать на использование List
|
@jskonst я исправил ошибки, добавил тест не на нули (там вопрос закомментирован ещё), добавил exception'ы и тесты на них, переехал на List. |
|
superclass done |
3 participants
DONE TEST