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

Fix message reply #230

Merged
merged 9 commits into from
Dec 6, 2023
Merged

Fix message reply #230

merged 9 commits into from
Dec 6, 2023

Conversation

stage-rl
Copy link
Collaborator

@stage-rl stage-rl commented Dec 5, 2023

No description provided.

@stage-rl stage-rl requested a review from jsuchal December 5, 2023 13:01
@jsuchal
Copy link
Member

jsuchal commented Dec 5, 2023

Kym nebude test tak to nezoberiem :)

Copy link
Member

@jsuchal jsuchal left a comment

Choose a reason for hiding this comment

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

Sprav test!

Copy link
Collaborator Author

@stage-rl stage-rl left a comment

Choose a reason for hiding this comment

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

Ja som s tym v pohode. Aspon sa zladime, ako sa robia testy ;-)

@stage-rl stage-rl requested a review from jsuchal December 5, 2023 14:15
@stage-rl stage-rl requested a review from jsuchal December 6, 2023 08:03
end

within_message_in_thread(message_one) do
find_button("Odpovedať").click
Copy link
Member

Choose a reason for hiding this comment

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

click_on "Odpovedať" by malo ist

find_button("Odoslať").click
end

assert GoodJob::Job.count, job_count_previous + 1
Copy link
Member

Choose a reason for hiding this comment

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

Tu over ci sa tam nastavil nejaky status "Sprava sa odosiela" alebo nieco take by tam malo byt vidiet.

Comment on lines 181 to 183
within_thread_in_listing(thread_issue) do
click_link
end
Copy link
Member

@jsuchal jsuchal Dec 6, 2023

Choose a reason for hiding this comment

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

Namiesto tohto by som asi zvazil proste visit message_thread_path(thread_issue) nech ten test zacne rovno na tej subpage

@stage-rl stage-rl requested a review from jsuchal December 6, 2023 12:19
end

GoodJob.perform_inline
assert GoodJob::Job.where(job_class: 'Govbox::SubmitMessageDraftJob').order(scheduled_at: :desc).first
Copy link
Member

Choose a reason for hiding this comment

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

Asi nerozumiem naco tu si tie order a first. AK chces vediet ci existuje tak sa spytaj exists?

Copy link
Member

Choose a reason for hiding this comment

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

Taktiez si myslim, ze toto bude true aj ked nezavolas to nad tym.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Ordre boli prezitok z inej verzie, ktoru som skusal. perform_inline tam musi byt, inak ten job nenajde. On sa pusta ako perform_later, a zrejme sa aj do tabulky uz dostava nejako asynchronne.

within '#new_drafts' do
fill_in("Predmet", with: "Testovaci predmet")
fill_in("Text", with: "Testovacie telo")
unfocus_input
Copy link
Member

Choose a reason for hiding this comment

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

Ak robime taketo kuzla tak si napisme do komentu ze preco.

fill_in("Predmet", with: "Testovaci predmet")
fill_in("Text", with: "Testovacie telo")
unfocus_input
find("turbo-frame", id: /submittable/, visible: false)
Copy link
Member

Choose a reason for hiding this comment

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

Toto tu musi byt? nestaci to pod tym?

end

within '#new_drafts' do
fill_in("Predmet", with: "Testovaci predmet")
Copy link
Member

Choose a reason for hiding this comment

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

V system testoch sa nezvyknu pouzivat na toto zatvorky ale je to vec stylu

@stage-rl stage-rl requested a review from jsuchal December 6, 2023 13:14
thread_issue = message_threads(:ssd_main_issue)
message_one = messages(:ssd_main_issue_one)

assert_not GoodJob::Job.where(job_class: 'Govbox::SubmitMessageDraftJob').exists?
Copy link
Member

Choose a reason for hiding this comment

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

Toto tu tiez asi uz netreba nie?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Netreba, sorry. Tolkokrat som tam pridaval/odoberal, ze som to tam zabudol

@stage-rl stage-rl merged commit fcd5008 into main Dec 6, 2023
3 checks passed
@jsuchal jsuchal deleted the GO-419/fix_message_reply branch December 24, 2023 09:25
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.

2 participants