Improve resume skills description #744
Conversation
|
Задеплойте пожалуйста демку, чтобы посмотреть визуально |
Мне доступен только деплой на рендер, но там надо добавлять файлы в репозиторий |
|
@eiKulshan а что за файлы нужно добавлять? По идее деплой на рендер должен работать обычным способом, там есть поддержка рельсы. Посмотрите в ридми инструкцию по деплою https://github.com/Hexlet/hexlet-cv#deploy-to-rendercom |
|
Ну и тест добавить, что если строка длинная резюме не сохранится |
|
Спасибо за ревью! |
| # frozen_string_literal: true | ||
|
|
||
| module ResumePresenter | ||
| def skills |
There was a problem hiding this comment.
У модели резюме уже есть метод skills, через который работает гем ActsAsTaggableOn, тут ты его переопределил, нужно выбрать другое имя метода
| resume = Resume.find_by(name: attrs[:name]) | ||
| assert { !resume } | ||
|
|
||
| attrs[:skills_description] = 'Invalid_skill_lenght' * 2 |
There was a problem hiding this comment.
Если тестируешь два разных кейса то лучше разделить на два теста
| resume: attrs | ||
| } | ||
|
|
||
| post(account_resumes_path, params:) |
There was a problem hiding this comment.
в параметры добавь пожалуйста locale, что бы тест прогонялся для двух локалей
post account_resumes_path, params:, locale: I18n.locale| @@ -233,6 +233,7 @@ | |||
| t.string "evaluated_ai_state" | |||
| t.text "projects_description" | |||
| t.text "about_myself" | |||
| t.text "skills_description", limit: 250 | |||
There was a problem hiding this comment.
прогони плиз еще раз миграцию, в схеме осталась валидация на длину на уровне базы
| @@ -128,6 +131,12 @@ def initialize(attribute = nil) | |||
| super(attrs_with_defaults) | |||
| end | |||
|
|
|||
| def skills_description_valid_format | |||
| return unless skills.size > 10 || skills.any? { |element| element.size > 25 } | |||
There was a problem hiding this comment.
почему или, должно же быть и - длинна скила не больше 25 и не больше 10 скилов
| @@ -84,6 +84,7 @@ group :development, :test do | |||
| # Call 'byebug' anywhere in the code to stop execution and get a debugger console | |||
| gem 'byebug', platforms: %i[mri mingw x64_mingw] | |||
| # Use sqlite3 as the database for Active Record | |||
| gem 'debug' | |||
There was a problem hiding this comment.
Вроде в задачу не входило добавление гемов)
|
@eKulshan как успехи? Продолжите работу? |
|
взял в работу |
resolves #711