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

SPT 1998 Правки после попытки миграции (5) #135

Merged
merged 6 commits into from
May 24, 2024

Conversation

NullIsOne
Copy link
Contributor

@NullIsOne NullIsOne commented May 7, 2024

Что сделано?

Решены небольшие проблемки, возникшие во время миграции модуля с сетевым слоем на новую версию.

  • Убрана привязка AccessSafeNode.Output к Json
  • Добавлен публичный init для URLCacheWriterNode
  • Удалены файлы устаревших тестов
  • Поправлен надоедливый warning от Xcode в etag
  • Скомбинированы моки chain Builder-а

@NullIsOne NullIsOne self-assigned this May 7, 2024
@NullIsOne NullIsOne changed the title SPT 1998 fix migration problems SPT 1998 Правки после попытки миграции May 7, 2024
@codecov-commenter
Copy link

codecov-commenter commented May 7, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 91.41%. Comparing base (04b4066) to head (09fdbc8).

Additional details and impacted files
@@                      Coverage Diff                      @@
##           SPT-1998-fix-branch-build     #135      +/-   ##
=============================================================
+ Coverage                      91.33%   91.41%   +0.07%     
=============================================================
  Files                             87       87              
  Lines                           1351     1351              
=============================================================
+ Hits                            1234     1235       +1     
+ Misses                           117      116       -1     
Flag Coverage Δ
tests 91.41% <100.00%> (+0.07%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Эти старые тесты видимо потерялись просто при ребейзах.
Они исключены из pbxproj, но сами файлы остались и не компилировались.
Если открывать проект как package, то Xcode включал их в список тестов и пытался собрать. Безуспешно.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Та же история.
У этого теста точно есть дублер, который Андрей уже написал на новом NodeKit

@mrandrewsmith mrandrewsmith force-pushed the SPT-1998-fix-branch-build branch from 6417752 to 04b4066 Compare May 13, 2024 12:10
@mrandrewsmith mrandrewsmith changed the title SPT 1998 Правки после попытки миграции SPT 1998 Правки после попытки миграции (5) May 13, 2024
@NullIsOne NullIsOne force-pushed the SPT-1998-fix-migration-problems branch from 4b94036 to 09fdbc8 Compare May 13, 2024 12:31
@mrandrewsmith mrandrewsmith changed the base branch from SPT-1998-fix-branch-build to 5.0.0 May 24, 2024 11:06
@mrandrewsmith mrandrewsmith merged commit eecafb0 into 5.0.0 May 24, 2024
2 checks passed
@mrandrewsmith mrandrewsmith deleted the SPT-1998-fix-migration-problems branch June 13, 2024 08:51
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.

4 participants