end

?

в самих тестах

let(:author) {FactoryGirl.create(:user) }

sequence email есть

если сделать question = create(:question) create_list :answers, 2 question: question то все работает и у question.answer[0].user есть пароль если create(:question, :with_answers), то пароля нет

сами тесты я уже по другому переписал, но интерес остался, почему же есть имейл и нет пароля

если сделать question = create(:question) create_list :answers, 2 question: question то все работает и у question.answer[0].user есть пароль если create(:question, :with_answers), то пароля нет

А пароля точно нет? Что происходит если .password у юзера вызвать?

Я имел ввиду то, что если я проверяю одни действия смысл городить обно и тоже как то не DRY получается и тут у меня сразу несколько мыслей, либо сделать хэлперы чтобы проще и короче, либо в одном it do проверять все их сразу, но это уже не приветствуется. Мб есть смысл просто проверить сам before action и его only except если можно

я ничо не понял ._.

)))

Вот смотри я щас прямо делаю тесты экшенов, там в каждом проверка идет может ли залогиненый с правами админа сделать и без, получается по 2 it do в одном describe. Я видел на стэке примеры например просто [actions].each и внутри блока проверять, пишем меньше же верно, но тут уже не особо читаемо становится. С помощью shared_examples не получается уменьшить код т.к. он пишет что внутри it do не работает. В итоге я щас сделал внутри просто обычный метод sign_in и в него все засунул и юзаю его в каждом describe. Такой подход помоему как то противочерит поди рспекам

Короче вот мой не законченный пример https://gist.github.com/EvgenyKungurov/cdc74ec567096426146487295aceffbb

и тебя смущает повторение expect(response).to redirect_to '/users/sign_in'?

Ну это только одна проверка. а их же однотипных может быть много. Ладно как сделаю так сделаю, подскажи вынос в метод sign_in норм практика или не стоит так делать?

нормальная

эти проверки нужно в shared examples выносить

с ними какие конкретно проблемы возникли?

(ну и require лучше вынести в .rspec, include делать в rails_helper для type: :controller, вместо '/users/sign_in` использовать new_user_session_path)

и в блоках it лучше не писать ничего кроме самой проверки, выносить всё в let/let!/before

с ними какие конкретно проблемы возникли?

Писал что то на подобии, что в it do не работает, только в блоке describe самом. Спс за помощь, отрефакторю все это дело

в блоках RSpec.shared_examples и иже it ок работает

ок

Научите писать тесты :) вот не могу полюбить писать тесты, а очень надо :(

Попробуй rspec

Я вот не любил писать тесты, а потом написал и за первый прогон поймал 6 багов

Научите писать тесты :) вот не могу полюбить писать тесты, а очень надо :(

Берешь рспек. И начинаешь сначала писать тест потом контроллер. Если не получается то пишешь их в паре.

Я так уже без теста разрабатывать не могу.

Логику удобно тестировать/проверять когда у тебя в базе только то что ты там создал, а не весь dev

Иногда получается накидать тест до реализации но не всегда.

Берешь рспек. И начинаешь сначала писать тест потом контроллер. Если не получается то пишешь их в паре.

А когда 3 года проекту и кода написано масса, что делать?

А когда 3 года проекту и кода написано масса, что делать?

Использовать систему тестов которая там есть. PS Спеку дофигамилион лет.

При этом не я этот код писал, а индусы, белорусы, россияне, Украина

Там до сих пор rails 4.0.13 :)

Иногда получается накидать тест до реализации но не всегда.

И когда не получается пишешь их в паре, думаешь что и как, юзаешь binding.pry (из гема pry-byebug) пишешь код прямо в консоли смотришь что как, если ок, то вставляешь код из консоли в код, продолжаешь.

Юзаешь guard + guard-rspec

либо если минитест то guard + guard-minitest соответственно.

Там до сих пор rails 4.0.13 :)

Ты так говоришь как будто это что то плохое.

Ты так говоришь как будто это что то плохое.

Ну как бы rails 4.0.x не поддерживают, минимум LTS 4.2, хотя бы на него перевести проект, чтобы получать фиксы безопасности

после неудачного, ошибочного мержа , и прочей тупости со сменой адреса репозитория возникла такая ошибка master -> master (non-fast-forward) другой рабочий бранч востановил, а вот это как исправить? надо привести в полную копию мастера из репозитория

git reset --hard origin/master?

(вместо origin — твой remote, вместо master — желаемый бранч, если они отличаются)

Ну как бы rails 4.0.x не поддерживают, минимум LTS 4.2, хотя бы на него перевести проект, чтобы получать фиксы безопасности

Не поддерживают, факт. Но морально ветка не сильно устарела

Не поддерживают, факт. Но морально ветка не сильно устарела

Проект такой, что если взломают, то будут потери финансовые, риски дырок надо убрать, поддержка важный аспект. На миграцию готовы потратить месяцы разработки и миграцию на поддерживаемое.

Проект такой, что если взломают, то будут потери финансовые, риски дырок надо убрать, поддержка важный аспект. На миграцию готовы потратить месяцы разработки и миграцию на поддерживаемое.

Ну обнови рельсы. Тестовое покрытие нормалтное? Биллинг весь протестирован?

Тебе помимо рельс, надо убедиться что нет дырок в гемах.

Ну обнови рельсы. Тестовое покрытие нормалтное? Биллинг весь протестирован?

Нету тестов вообще, обновить просто не получиться, что-то сломается, поэтому надо писать тесты, чтобы можно было проверить, после обновления, что ничего не упало.

Написать нормальные тесты на большое приложение имхо почти тоже что с нуля писать.

Финансовое приложение без тестов даже на биллинг ?

Уверен что там точно где то неправильно считается, или скоро начнет.

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

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

Всё возможно, но пока не начнёшь, не узнаешь

Всё возможно, но пока не начнёшь, не узнаешь

А в чем проблема то?) Я вот только тодже недавно начал, у меня пока только вопросы по организации всего этого, а с вопросами что именно тестировать вроде не возникает

Всё возможно, но пока не начнёшь, не узнаешь

Будет сложно если вообще возможно учесть все ньюансы бизнеслогики, если при этом и сам код под тесты не заточен

А в чем проблема то?) Я вот только тодже недавно начал, у меня пока только вопросы по организации всего этого, а с вопросами что именно тестировать вроде не возникает

Проблема в том что уже готовый проект содержит тонну ньюансов бизнеслогики, и его код под тесты не предназначен.

А он показывал его код? Как определили что он не подходит, все равно можно же тесты написать и единственное, что на мой взгляд может тормозить это рабочие руки и время, которые могли бы это все написать и где что отрефакторить

git reset --hard origin/master?

это чето не помогает

рабочую копию так востановил git reset —hard origin/work если сделать так git reset —hard origin/master то обе ветки выдают такую ошибку поэтому обратно делаю git reset —hard origin/work

походу понял, надо сделать так чтобы мастер указал на последний коммит, судя по всему но это надо провернуть на боевом сервере :))) там желательно не делать чекаут на другую ветку, т.к. будут глюки

на ремоут резетиться так себе затея

резетить лучше свою ветку сильно назад и подтягивать изменения из ремоута

в чём сакральный смысл этого действия?

и что плохого в ресете на remote?

пришлось чекаутит на мастер и там сделать резет оригин мастер вроде помогло

надеюсь никто не заметил, очень быстро сделал

да ничего плохого, все норм

пришлось аж вручную корректировать настройки гит в .git/config и там добавить страннным образом исчезнувщие бранчи

и все началось из того поменял адрес репозитория с https на git

ERROR: S client not available

да ничего плохого, все норм

ок, а что в таком случае имелось в виду под "так себе затея"?

просто твоё дерево и ремоут это две отдельные сущности, откатывать одного дерево до состояния другого для меня так себе

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

git pull делаешь он все тянет, мержит автотом, возникают куча конфликтов отменяешь все git merge —abort и тут видишь git status выдает кучу изменений изменения даже из другой ветки походу, все перемешгалось

уже не понимаю что происходит :)

видит новые файлы, а так же измененные

просто твоё дерево и ремоут это две отдельные сущности, откатывать одного дерево до состояния другого для меня так себе

wtf отдельные сущности? HEAD переключается на другой коммит, какая разница в каком он бранче?

git pull делаешь он все тянет, мержит автотом, возникают куча конфликтов отменяешь все git merge —abort и тут видишь git status выдает кучу изменений изменения даже из другой ветки походу, все перемешгалось

Автоматом не должно мержить, он должен предложить исправить ошибки, значит где то настроено так, что автоматом

Автоматом не должно мержить, он должен предложить исправить ошибки, значит где то настроено так, что автоматом

без никаких настроек , установка по умолчанию в параметрах только имя и почта

git fetch автоматом не мержим а просто тянет вроде

Я конечно не спец, но я точно помню, что когда последний раз делал пул на рабочем сайте, он выпендривался)

ща гляну доки уж забыл все)

Все верно сказал пул автоматом

А фетч тоже автоматом у теюя что ли сливает?

вроде нет

Тогда зачем делаете git pull ?

по ошибке :)

меня там озадачило когда после git merge —abort который по сути должен был отменить все мой проект превратился в кашу показывал кучу изменений из разных веток или что было потом я просто запутался

Наверное нужно еще освоить бранчи, чтобы в основну ветку не сливать и в случае чего удалить не боясь за свои данные

проще научиться пользоваться cp и scp, и делать бекапы

Вот в руководстве пишут так git checkout HEAD — файлы копирует файлы из текущего коммита и на сцену, и в рабочую директорию. какая еще сцена и директория

прозреваю staging

есть оказывается понятие сцены и рабочей директории

но вообще не стоит читать русскую документацию

На сцену и сплясать с бубном

А пароля точно нет? Что происходит если .password у юзера вызвать?

Nil

Nil

А покажи фабрику question?

FactoryGirl.define do sequence :title do |n| "Question N #{n} title" end factory :question do title body "MyText" user trait :with_answers do after(:create) do |question| create_list :answer, 2, question: question end end end factory :invalid_question, class: "question" do title nil body nil user end end

лучше в gist такое кидать

а в answer что?

FactoryGirl.define do factory :answer do body "MyText" question user end end

гм