* Article query spec for scheduled articles * Added scheduled article badge on the user dashboard * Added published_at field to editor options * Accept and validate published_at from editor * Refactor published_at validation * Allow 1-minute difference in published_at * Notice on an unpublished article page * Added specs for 'Click to edit' link on scheduled article preview page * ContextNotification model * Articles::Publish worker * Added specs for articles publish worker * Schedule publish articles worker * Added tests to check for scheduled posts in feeds * Don't allow managing scheduled articles * Don't send notifications for scheduled articles * Set published_at in Articles::Updater when publishing * Published_at value in post options * Pass timezone and set published_at accordingly * Limit setting published_at to the future * Readonly published_at for articles that were already published * Chagning published_at format in editor v1 (start) * Changed published_at format in frontmatter, specs * Added specs for updating published_at from frontmatter * Fixed accepting past published_at for articles published_from_feed * Enabled published_at validation: don't allow updating published_at for already published articles * Validate published_at on create * Added a spec for updating published_at for exported articles * Fixed specs related to creating articles with past published_at * Fixed specs related to past published_at for articles * Added a hack so that admins would be able to update published_at * Switch button text schedule/publish when changin publishedAt * Fixed saving published_at with timezone * Added a feature flag for scheduling articles * Default text in markdown editor depends on feature flag * Enable article editor cache again * Fixed the default value in the markdown editor * Fix sitemaps spec * Removed tooltip * Fixed articles update specs * Added missing locales * Fixed article create specs * Fixed spec * Removed commented code * Returned enabling extensions in the schema * Returned accidentally deleted constraint * Make articles query spec more stable Co-authored-by: Jeremy Friesen <jeremy.n.friesen@gmail.com> * Removed commented code * Removed unused code * A clearer policy Co-authored-by: Jeremy Friesen <jeremy.n.friesen@gmail.com> * Use StringInquirer for article current state * Added a note and todo to articles factory past trait * Remove duplicated PropType Co-authored-by: Suzanne Aitchison <suzanne@forem.com> * Refactor query in the Articles::PublishWorker Co-authored-by: Mac Siri <krairit.siri@gmail.com> * Refactor articleForm.jsx Co-authored-by: Mac Siri <krairit.siri@gmail.com> * Removed specs that are no longer relevant * Removed useless onKeyUp on a hidden input * Refactored articleForm * Hide scheduling from post options when published_at is readonly * Run sends notifications worker every 5 minutes instead of every minute Co-authored-by: Jeremy Friesen <jeremy.n.friesen@gmail.com> Co-authored-by: Suzanne Aitchison <suzanne@forem.com> Co-authored-by: Mac Siri <krairit.siri@gmail.com>
161 lines
5.1 KiB
Ruby
161 lines
5.1 KiB
Ruby
require "rails_helper"
|
|
|
|
RSpec.describe Articles::Builder, type: :service do
|
|
let(:user) { create(:user) }
|
|
let(:tag) { nil }
|
|
let(:prefill) { nil }
|
|
|
|
context "when tag_user_editor_v2" do
|
|
let(:user) { create(:user) }
|
|
let(:tag) { create(:tag) }
|
|
let(:submission_template) { tag.submission_template_customized(user.name).to_s }
|
|
let(:correct_attributes) do
|
|
{
|
|
body_markdown: submission_template.split("---").last.to_s.strip,
|
|
cached_tag_list: tag.name,
|
|
processed_html: "",
|
|
user_id: user.id,
|
|
title: submission_template.split("title:")[1].to_s.split("\n")[0].to_s.strip
|
|
}
|
|
end
|
|
|
|
it "initializes an article with the correct attributes and needs authorization" do
|
|
subject, needs_authorization = described_class.call(user, tag, prefill)
|
|
|
|
expect(subject).to be_an_instance_of(Article)
|
|
expect(subject).to have_attributes(correct_attributes)
|
|
expect(needs_authorization).to be true
|
|
end
|
|
end
|
|
|
|
context "when tag_user" do
|
|
let(:tag) { create(:tag, submission_template: "submission_template") }
|
|
let(:correct_attributes) do
|
|
{
|
|
body_markdown: tag.submission_template_customized(user.name),
|
|
processed_html: "",
|
|
user_id: user.id
|
|
}
|
|
end
|
|
|
|
it "initializes an article with the correct attributes and needs authorization" do
|
|
subject, needs_authorization = described_class.call(user, tag, prefill)
|
|
|
|
expect(subject).to be_an_instance_of(Article)
|
|
expect(subject).to have_attributes(correct_attributes)
|
|
expect(needs_authorization).to be true
|
|
end
|
|
end
|
|
|
|
context "when prefill_user_editor_v2" do
|
|
let(:user) { create(:user) }
|
|
let(:prefill) { "dsdweewewew" }
|
|
let(:correct_attributes) do
|
|
{
|
|
body_markdown: prefill.split("---").last.to_s.strip,
|
|
cached_tag_list: prefill.split("tags:")[1].to_s.split("\n")[0].to_s.strip,
|
|
processed_html: "",
|
|
user_id: user.id,
|
|
title: prefill.split("title:")[1].to_s.split("\n")[0].to_s.strip
|
|
}
|
|
end
|
|
|
|
it "initializes an article with the correct attributes and needs authorization" do
|
|
subject, needs_authorization = described_class.call(user, tag, prefill)
|
|
|
|
expect(subject).to be_an_instance_of(Article)
|
|
expect(subject).to have_attributes(correct_attributes)
|
|
expect(needs_authorization).to be true
|
|
end
|
|
end
|
|
|
|
context "when prefill_user" do
|
|
let(:prefill) { "dsdweewewew" }
|
|
let(:correct_attributes) do
|
|
{
|
|
body_markdown: prefill,
|
|
processed_html: "",
|
|
user_id: user.id
|
|
}
|
|
end
|
|
|
|
it "initializes an article with the correct attributes and needs authorization" do
|
|
subject, needs_authorization = described_class.call(user, tag, prefill)
|
|
|
|
expect(subject).to be_an_instance_of(Article)
|
|
expect(subject).to have_attributes(correct_attributes)
|
|
expect(needs_authorization).to be true
|
|
end
|
|
end
|
|
|
|
context "when tag" do
|
|
let(:user) { create(:user) }
|
|
let(:tag) { create(:tag) }
|
|
let(:correct_attributes) do
|
|
{
|
|
body_markdown: "---\ntitle: \npublished: false\ndescription: \ntags: #{tag.name}\n---\n\n",
|
|
processed_html: "",
|
|
user_id: user.id
|
|
}
|
|
end
|
|
|
|
it "initializes an article with the correct attributes and does not need authorization" do
|
|
user.setting.update(editor_version: "v1")
|
|
subject, needs_authorization = described_class.call(user, tag, prefill)
|
|
|
|
expect(subject).to be_an_instance_of(Article)
|
|
expect(subject).to have_attributes(correct_attributes)
|
|
expect(needs_authorization).to be false
|
|
end
|
|
end
|
|
|
|
context "when user_editor_v2" do
|
|
let(:user) { create(:user) }
|
|
let(:correct_attributes) do
|
|
{
|
|
user_id: user.id
|
|
}
|
|
end
|
|
|
|
it "initializes an article with the correct attributes and does not need authorization" do
|
|
subject, needs_authorization = described_class.call(user, tag, prefill)
|
|
|
|
expect(subject).to be_an_instance_of(Article)
|
|
expect(subject).to have_attributes(correct_attributes)
|
|
expect(needs_authorization).to be false
|
|
end
|
|
end
|
|
|
|
context "when user_editor_v1" do
|
|
let(:user) { create(:user) }
|
|
|
|
let(:correct_attributes) do
|
|
{
|
|
processed_html: "",
|
|
user_id: user.id
|
|
}
|
|
end
|
|
|
|
it "initializes an article with the correct attributes and does not need authorization" do
|
|
user.setting.update(editor_version: "v1")
|
|
allow(FeatureFlag).to receive(:enabled?).with(:schedule_articles).and_return(true)
|
|
|
|
subject, needs_authorization = described_class.call(user, tag, prefill)
|
|
|
|
expect(subject).to be_an_instance_of(Article)
|
|
expect(subject).to have_attributes(correct_attributes)
|
|
|
|
date = Time.current.strftime("%Y-%m-%d")
|
|
zone = Time.current.strftime("%z")
|
|
|
|
body_start = "---\ntitle: \npublished: false\ndescription: " \
|
|
"\ntags: \n# cover_image: https://direct_url_to_image.jpg" \
|
|
"\n# Use a ratio of 100:42 for best results.\n# published_at: #{date} "
|
|
|
|
expect(subject.body_markdown).to start_with(body_start)
|
|
expect(subject.body_markdown).to end_with("#{zone}\n---\n\n")
|
|
|
|
expect(needs_authorization).to be false
|
|
end
|
|
end
|
|
end
|