From 493d437e79f88ab9829a36fa000cc6107085e424 Mon Sep 17 00:00:00 2001 From: Phil Pirozhkov Date: Thu, 28 Jul 2022 05:27:38 +0300 Subject: [PATCH] Add RSpec 4 compatibility (#17652) * Remove outdated option https://github.com/rspec/rspec-core/commit/04078317ba6577699d06cf4dccf014254dcde7a6 * Use the non-globally exposed RSpec syntax https://github.com/rspec/rspec-core/pull/2803 * Use the non-globally exposed RSpec syntax, cont https://github.com/rspec/rspec-core/pull/2803 * Comply to strict predicate matchers See: - https://github.com/rspec/rspec-expectations/pull/1195 - https://github.com/rspec/rspec-expectations/pull/1196 - https://github.com/rspec/rspec-expectations/pull/1277 --- .rspec | 1 - .../spec/components/pretty_text_spec.rb | 2 +- .../spec/lib/pretty_text_spec.rb | 2 +- .../spec/models/post_spec.rb | 2 +- .../new_user_narrative_spec.rb | 2 +- .../discourse_narrative_bot/store_spec.rb | 2 +- .../track_selector_spec.rb | 2 +- .../jobs/onceoff/remap_old_bot_images_spec.rb | 2 +- .../requests/discobot_certificate_spec.rb | 2 +- .../requests/discobot_welcome_post_spec.rb | 2 +- .../discourse-narrative-bot/spec/user_spec.rb | 2 +- .../spec/integration/presence_spec.rb | 2 +- .../spec/controllers/polls_controller_spec.rb | 2 +- .../spec/controllers/posts_controller_spec.rb | 2 +- .../spec/integration/poll_endpoints_spec.rb | 2 +- .../poll/spec/jobs/regular/close_poll_spec.rb | 2 +- .../poll/spec/lib/new_post_manager_spec.rb | 2 +- plugins/poll/spec/lib/poll_spec.rb | 2 +- plugins/poll/spec/lib/polls_updater_spec.rb | 2 +- plugins/poll/spec/lib/polls_validator_spec.rb | 2 +- plugins/poll/spec/lib/pretty_text_spec.rb | 2 +- plugins/poll/spec/models/poll_spec.rb | 2 +- .../spec/requests/users_controller_spec.rb | 2 +- .../poll_option_serializer_spec.rb | 2 +- .../spec/integration/access_spec.rb | 4 +-- .../spec/integration/assets_spec.rb | 2 +- spec/helpers/application_helper_spec.rb | 2 +- spec/helpers/topics_helper_spec.rb | 2 +- .../helpers/user_notifications_helper_spec.rb | 2 +- spec/import_export/category_exporter_spec.rb | 2 +- .../category_structure_exporter_spec.rb | 2 +- spec/import_export/group_exporter_spec.rb | 2 +- spec/import_export/importer_spec.rb | 2 +- spec/import_export/topic_exporter_spec.rb | 2 +- .../track_setting_changes_spec.rb | 2 +- spec/integration/api_keys_spec.rb | 4 +-- .../auto_reject_reviewable_users_spec.rb | 2 +- .../blocked_hotlinked_media_spec.rb | 2 +- spec/integration/category_tag_spec.rb | 4 +-- .../content_security_policy_spec.rb | 2 +- spec/integration/email_outbound_spec.rb | 2 +- spec/integration/email_style_spec.rb | 2 +- spec/integration/flags_spec.rb | 2 +- spec/integration/group_spec.rb | 2 +- spec/integration/invalid_request_spec.rb | 2 +- .../invite_only_registration_spec.rb | 2 +- spec/integration/message_bus_spec.rb | 2 +- spec/integration/multisite_cookies_spec.rb | 2 +- spec/integration/multisite_spec.rb | 2 +- spec/integration/rate_limiting_spec.rb | 2 +- spec/integration/same_ip_spammers_spec.rb | 2 +- spec/integration/spam_rules_spec.rb | 2 +- spec/integration/topic_auto_close_spec.rb | 2 +- spec/integration/topic_thumbnail_spec.rb | 2 +- spec/integration/watched_words_spec.rb | 2 +- spec/integrity/coding_style_spec.rb | 2 +- spec/integrity/common_mark_spec.rb | 2 +- spec/integrity/i18n_spec.rb | 4 +-- spec/integrity/js_constants_spec.rb | 2 +- spec/integrity/oj_spec.rb | 2 +- spec/integrity/onceoff_integrity_spec.rb | 2 +- spec/integrity/site_setting_spec.rb | 2 +- spec/jobs/about_stats_spec.rb | 2 +- spec/jobs/activation_reminder_emails_spec.rb | 2 +- spec/jobs/auto_queue_handler_spec.rb | 2 +- spec/jobs/automatic_group_membership_spec.rb | 2 +- spec/jobs/bulk_grant_trust_level_spec.rb | 2 +- spec/jobs/bulk_invite_spec.rb | 2 +- spec/jobs/bump_topic_spec.rb | 2 +- spec/jobs/clean_dismissed_topic_users_spec.rb | 2 +- .../jobs/clean_up_associated_accounts_spec.rb | 2 +- spec/jobs/clean_up_crawler_stats_spec.rb | 2 +- .../clean_up_email_change_requests_spec.rb | 2 +- spec/jobs/clean_up_email_logs_spec.rb | 2 +- spec/jobs/clean_up_uploads_spec.rb | 2 +- spec/jobs/cleanup_imap_sync_log_spec.rb | 2 +- spec/jobs/close_topic_spec.rb | 2 +- .../correct_missing_dualstack_urls_spec.rb | 2 +- spec/jobs/crawl_topic_link_spec.rb | 2 +- spec/jobs/create_linked_topic_spec.rb | 2 +- spec/jobs/create_missing_avatars_spec.rb | 2 +- .../create_recent_post_search_indexes_spec.rb | 2 +- spec/jobs/create_user_reviewable_spec.rb | 2 +- spec/jobs/dashboard_stats_spec.rb | 2 +- spec/jobs/delete_replies_spec.rb | 2 +- spec/jobs/delete_topic_spec.rb | 2 +- spec/jobs/disable_bootstrap_mode_spec.rb | 2 +- spec/jobs/emit_web_hook_event_spec.rb | 2 +- spec/jobs/enable_bootstrap_mode_spec.rb | 2 +- spec/jobs/enqueue_digest_emails_spec.rb | 2 +- spec/jobs/enqueue_suspect_users_spec.rb | 2 +- spec/jobs/export_csv_file_spec.rb | 2 +- spec/jobs/export_user_archive_spec.rb | 2 +- spec/jobs/feature_topic_users_spec.rb | 2 +- spec/jobs/grant_anniversary_badges_spec.rb | 2 +- ...grant_new_user_of_the_month_badges_spec.rb | 2 +- spec/jobs/heartbeat_spec.rb | 2 +- spec/jobs/ignored_users_summary_spec.rb | 2 +- spec/jobs/invalidate_inactive_admins_spec.rb | 2 +- spec/jobs/invite_email_spec.rb | 2 +- spec/jobs/jobs_base_spec.rb | 2 +- spec/jobs/jobs_spec.rb | 2 +- spec/jobs/mass_award_badge_spec.rb | 2 +- .../migrate_badge_image_to_uploads_spec.rb | 2 +- .../notify_mailing_list_subscribers_spec.rb | 2 +- spec/jobs/notify_moved_posts_spec.rb | 2 +- spec/jobs/notify_reviewable_spec.rb | 2 +- spec/jobs/notify_tag_change_spec.rb | 2 +- spec/jobs/old_keys_reminder_spec.rb | 2 +- spec/jobs/open_topic_spec.rb | 2 +- .../pending_queued_posts_reminder_spec.rb | 2 +- .../jobs/pending_reviewables_reminder_spec.rb | 2 +- spec/jobs/pending_users_reminder_spec.rb | 2 +- spec/jobs/periodical_updates_spec.rb | 2 +- spec/jobs/poll_mailbox_spec.rb | 2 +- spec/jobs/problem_checks_spec.rb | 2 +- spec/jobs/process_bulk_invite_emails_spec.rb | 2 +- spec/jobs/process_email_spec.rb | 2 +- spec/jobs/process_post_spec.rb | 2 +- .../process_shelved_notifications_spec.rb | 2 +- spec/jobs/pull_hotlinked_images_spec.rb | 2 +- ...pull_user_profile_hotlinked_images_spec.rb | 2 +- spec/jobs/purge_expired_ignored_users_spec.rb | 2 +- .../regular/bulk_user_title_update_spec.rb | 2 +- .../update_post_uploads_secure_status_spec.rb | 2 +- spec/jobs/reindex_search_spec.rb | 2 +- spec/jobs/remove_banner_spec.rb | 2 +- spec/jobs/reviewable_priorities_spec.rb | 2 +- spec/jobs/send_system_message_spec.rb | 2 +- spec/jobs/suspicious_login_spec.rb | 2 +- spec/jobs/sync_acls_for_uploads_spec.rb | 2 +- spec/jobs/tl3_promotions_spec.rb | 2 +- spec/jobs/toggle_topic_closed_spec.rb | 2 +- spec/jobs/truncate_user_flag_stats_spec.rb | 2 +- spec/jobs/unsilence_users_spec.rb | 2 +- spec/jobs/update_animated_uploads_spec.rb | 2 +- spec/jobs/update_gravatar_spec.rb | 2 +- spec/jobs/update_s3_inventory_spec.rb | 2 +- spec/jobs/user_email_spec.rb | 2 +- spec/lib/admin_confirmation_spec.rb | 2 +- spec/lib/admin_user_index_query_spec.rb | 2 +- spec/lib/archetype_spec.rb | 2 +- .../default_current_user_provider_spec.rb | 2 +- spec/lib/auth/discord_authenticator_spec.rb | 2 +- spec/lib/auth/facebook_authenticator_spec.rb | 2 +- spec/lib/auth/github_authenticator_spec.rb | 2 +- .../auth/google_oauth2_authenticator_spec.rb | 2 +- spec/lib/auth/managed_authenticator_spec.rb | 2 +- .../discourse_google_oauth2_spec.rb | 2 +- spec/lib/auth/result_spec.rb | 2 +- spec/lib/auth/twitter_authenticator_spec.rb | 2 +- .../backup_file_handler_multisite_spec.rb | 2 +- .../backup_file_handler_spec.rb | 2 +- spec/lib/backup_restore/backuper_spec.rb | 2 +- .../database_restorer_multisite_spec.rb | 2 +- .../backup_restore/database_restorer_spec.rb | 2 +- .../backup_restore/local_backup_store_spec.rb | 2 +- .../backup_restore/meta_data_handler_spec.rb | 2 +- .../backup_restore/s3_backup_store_spec.rb | 2 +- .../shared_context_for_backup_restore.rb | 2 +- .../shared_examples_for_backup_store.rb | 6 ++-- .../system_interface_multisite_spec.rb | 2 +- .../backup_restore/system_interface_spec.rb | 2 +- .../backup_restore/uploads_restorer_spec.rb | 2 +- spec/lib/browser_detection_spec.rb | 2 +- spec/lib/cache_spec.rb | 2 +- spec/lib/category_badge_spec.rb | 2 +- .../common_passwords/common_passwords_spec.rb | 2 +- spec/lib/composer_messages_finder_spec.rb | 2 +- spec/lib/compression/engine_spec.rb | 2 +- spec/lib/concern/cached_counting_spec.rb | 2 +- spec/lib/concern/category_hashtag_spec.rb | 2 +- spec/lib/concern/has_custom_fields_spec.rb | 2 +- spec/lib/concern/has_search_data_spec.rb | 2 +- spec/lib/concern/positionable_spec.rb | 2 +- spec/lib/concern/searchable_spec.rb | 2 +- spec/lib/content_buffer_spec.rb | 2 +- .../content_security_policy/builder_spec.rb | 2 +- spec/lib/content_security_policy_spec.rb | 2 +- spec/lib/cooked_post_processor_spec.rb | 2 +- spec/lib/crawler_detection_spec.rb | 2 +- spec/lib/current_user_spec.rb | 2 +- spec/lib/directory_helper_spec.rb | 2 +- spec/lib/discourse_diff_spec.rb | 2 +- spec/lib/discourse_event_spec.rb | 2 +- spec/lib/discourse_hub_spec.rb | 2 +- spec/lib/discourse_js_processor_spec.rb | 2 +- spec/lib/discourse_plugin_registry_spec.rb | 2 +- spec/lib/discourse_redis_spec.rb | 2 +- ...course_sourcemapping_url_processor_spec.rb | 2 +- spec/lib/discourse_spec.rb | 2 +- spec/lib/discourse_tagging_spec.rb | 2 +- spec/lib/discourse_updates_spec.rb | 2 +- spec/lib/distributed_cache_spec.rb | 2 +- spec/lib/distributed_memoizer_spec.rb | 2 +- spec/lib/distributed_mutex_spec.rb | 2 +- spec/lib/email/authentication_results_spec.rb | 2 +- spec/lib/email/cleaner_spec.rb | 2 +- spec/lib/email/email_spec.rb | 2 +- spec/lib/email/message_builder_spec.rb | 2 +- spec/lib/email/processor_spec.rb | 2 +- spec/lib/email/receiver_spec.rb | 2 +- spec/lib/email/renderer_spec.rb | 2 +- spec/lib/email/sender_spec.rb | 2 +- spec/lib/email/styles_spec.rb | 2 +- spec/lib/email_cook_spec.rb | 2 +- spec/lib/email_updater_spec.rb | 2 +- spec/lib/encodings_spec.rb | 2 +- spec/lib/enum_spec.rb | 2 +- spec/lib/excerpt_parser_spec.rb | 2 +- spec/lib/feed_element_installer_spec.rb | 2 +- spec/lib/feed_item_accessor_spec.rb | 2 +- spec/lib/file_helper_spec.rb | 2 +- spec/lib/file_store/local_store_spec.rb | 2 +- spec/lib/file_store/s3_store_spec.rb | 2 +- spec/lib/filter_best_posts_spec.rb | 2 +- spec/lib/final_destination_spec.rb | 2 +- .../mail_disable_starttls_spec.rb | 6 ++-- spec/lib/freedom_patches/safe_buffer_spec.rb | 2 +- .../schema_migration_details_spec.rb | 2 +- spec/lib/freedom_patches/seed_fu_spec.rb | 2 +- .../translate_accelerator_spec.rb | 2 +- spec/lib/gaps_spec.rb | 2 +- spec/lib/global_path_spec.rb | 2 +- .../lib/group_email_credentials_check_spec.rb | 2 +- spec/lib/guardian/topic_guardian_spec.rb | 2 +- spec/lib/guardian/user_guardian_spec.rb | 2 +- spec/lib/guardian_spec.rb | 2 +- spec/lib/has_errors_spec.rb | 2 +- spec/lib/highlight_js/highlight_js_spec.rb | 2 +- spec/lib/hijack_spec.rb | 2 +- spec/lib/html_prettify_spec.rb | 2 +- spec/lib/html_to_markdown_spec.rb | 2 +- spec/lib/i18n/discourse_i18n_spec.rb | 2 +- spec/lib/i18n/fallback_locale_list_spec.rb | 2 +- spec/lib/image_sizer_spec.rb | 2 +- spec/lib/imap/sync_spec.rb | 2 +- spec/lib/import/normalize_spec.rb | 2 +- spec/lib/inline_oneboxer_spec.rb | 2 +- spec/lib/js_locale_helper_spec.rb | 2 +- spec/lib/json_error_spec.rb | 4 +-- spec/lib/letter_avatar_spec.rb | 2 +- spec/lib/message_id_service_spec.rb | 2 +- spec/lib/method_profiler_spec.rb | 2 +- spec/lib/middleware/anonymous_cache_spec.rb | 2 +- .../discourse_public_exceptions_spec.rb | 2 +- spec/lib/middleware/enforce_hostname_spec.rb | 2 +- spec/lib/middleware/request_tracker_spec.rb | 2 +- spec/lib/migration/safe_migrate_spec.rb | 2 +- spec/lib/migration/table_dropper_spec.rb | 2 +- .../lib/mini_sql_multisite_connection_spec.rb | 2 +- spec/lib/mobile_detection_spec.rb | 2 +- spec/lib/new_post_manager_spec.rb | 2 +- spec/lib/new_post_result_spec.rb | 2 +- spec/lib/onebox/domain_checker_spec.rb | 2 +- .../engine/allowlisted_generic_onebox_spec.rb | 2 +- spec/lib/onebox/engine/amazon_onebox_spec.rb | 2 +- .../engine/animated_image_onebox_spec.rb | 2 +- spec/lib/onebox/engine/audio_onebox_spec.rb | 2 +- .../lib/onebox/engine/cloudapp_onebox_spec.rb | 2 +- spec/lib/onebox/engine/gfycat_onebox_spec.rb | 2 +- .../engine/github_actions_onebox_spec.rb | 2 +- .../onebox/engine/github_blob_onebox_spec.rb | 2 +- .../engine/github_commit_onebox_spec.rb | 2 +- .../engine/github_folder_onebox_spec.rb | 2 +- .../onebox/engine/github_gist_onebox_spec.rb | 2 +- .../engine/github_pullrequest_onebox_spec.rb | 2 +- .../onebox/engine/gitlab_blob_onebox_spec.rb | 2 +- .../onebox/engine/google_docs_onebox_spec.rb | 2 +- .../onebox/engine/google_drive_onebox_spec.rb | 2 +- .../onebox/engine/google_maps_onebox_spec.rb | 2 +- .../engine/google_photos_onebox_spec.rb | 2 +- .../engine/google_play_app_onebox_spec.rb | 2 +- spec/lib/onebox/engine/hackernews_spec.rb | 2 +- spec/lib/onebox/engine/html_spec.rb | 2 +- spec/lib/onebox/engine/image_onebox_spec.rb | 2 +- spec/lib/onebox/engine/imgur_onebox_spec.rb | 2 +- .../onebox/engine/instagram_onebox_spec.rb | 2 +- spec/lib/onebox/engine/json_spec.rb | 2 +- spec/lib/onebox/engine/kaltura_onebox_spec.rb | 2 +- spec/lib/onebox/engine/pdf_onebox_spec.rb | 2 +- spec/lib/onebox/engine/pubmed_onebox_spec.rb | 2 +- spec/lib/onebox/engine/reddit_media_onebox.rb | 2 +- spec/lib/onebox/engine/slides_onebox_spec.rb | 2 +- .../engine/stack_exchange_onebox_spec.rb | 2 +- spec/lib/onebox/engine/standard_embed_spec.rb | 2 +- spec/lib/onebox/engine/trello_onebox_spec.rb | 2 +- .../onebox/engine/twitch_clips_onebox_spec.rb | 2 +- .../engine/twitch_stream_onebox_spec.rb | 2 +- .../onebox/engine/twitch_video_onebox_spec.rb | 2 +- .../engine/twitter_status_onebox_spec.rb | 2 +- .../lib/onebox/engine/typeform_onebox_spec.rb | 2 +- spec/lib/onebox/engine/video_onebox_spec.rb | 2 +- .../onebox/engine/wikimedia_onebox_spec.rb | 2 +- .../onebox/engine/wikipedia_onebox_spec.rb | 2 +- spec/lib/onebox/engine/wistia_onebox_spec.rb | 2 +- spec/lib/onebox/engine/xkcd_spec.rb | 2 +- spec/lib/onebox/engine/youku_onebox_spec.rb | 2 +- spec/lib/onebox/engine/youtube_onebox_spec.rb | 2 +- spec/lib/onebox/engine_spec.rb | 4 +-- spec/lib/onebox/json_ld_spec.rb | 2 +- spec/lib/onebox/layout_spec.rb | 2 +- spec/lib/onebox/matcher_spec.rb | 2 +- spec/lib/onebox/movie_spec.rb | 2 +- spec/lib/onebox/oembed_spec.rb | 2 +- spec/lib/onebox/open_graph_spec.rb | 2 +- spec/lib/onebox/preview_spec.rb | 2 +- spec/lib/onebox/status_check_spec.rb | 2 +- spec/lib/onebox_spec.rb | 2 +- spec/lib/oneboxer_spec.rb | 2 +- spec/lib/onpdiff_spec.rb | 2 +- spec/lib/pbkdf2_spec.rb | 2 +- spec/lib/pinned_check_spec.rb | 2 +- spec/lib/plain_text_to_markdown_spec.rb | 2 +- spec/lib/plugin/filter_manager_spec.rb | 2 +- spec/lib/plugin/instance_spec.rb | 2 +- spec/lib/plugin/metadata_spec.rb | 2 +- spec/lib/post_action_creator_spec.rb | 2 +- spec/lib/post_action_destroyer_spec.rb | 2 +- spec/lib/post_creator_spec.rb | 2 +- spec/lib/post_destroyer_spec.rb | 2 +- spec/lib/post_locker_spec.rb | 2 +- spec/lib/post_merger_spec.rb | 2 +- spec/lib/post_revisor_spec.rb | 2 +- spec/lib/presence_channel_spec.rb | 2 +- spec/lib/pretty_text/helpers_spec.rb | 2 +- spec/lib/pretty_text_spec.rb | 2 +- spec/lib/promotion_spec.rb | 2 +- spec/lib/quote_comparer_spec.rb | 10 +++--- spec/lib/rate_limiter_spec.rb | 2 +- spec/lib/redis_store_spec.rb | 2 +- spec/lib/retrieve_title_spec.rb | 2 +- spec/lib/rtl_spec.rb | 2 +- spec/lib/s3_helper_spec.rb | 2 +- spec/lib/s3_inventory_multisite_spec.rb | 2 +- spec/lib/s3_inventory_spec.rb | 2 +- spec/lib/scheduler/defer_spec.rb | 2 +- spec/lib/score_calculator_spec.rb | 2 +- spec/lib/scss_checker_spec.rb | 2 +- spec/lib/search_spec.rb | 2 +- .../discourse_connect_provider_spec.rb | 2 +- .../second_factor/actions/grant_admin_spec.rb | 2 +- spec/lib/second_factor/auth_manager_spec.rb | 2 +- spec/lib/secure_session_spec.rb | 2 +- spec/lib/seed_data/categories_spec.rb | 2 +- spec/lib/seed_data/topics_spec.rb | 2 +- spec/lib/shrink_uploaded_image_spec.rb | 2 +- spec/lib/site_icon_manager_spec.rb | 2 +- .../site_setting_extension_multisite_spec.rb | 2 +- spec/lib/site_setting_extension_spec.rb | 2 +- spec/lib/site_settings/db_provider_spec.rb | 2 +- .../site_settings/defaults_provider_spec.rb | 2 +- .../local_process_provider_multisite_spec.rb | 2 +- .../local_process_provider_spec.rb | 2 +- .../lib/site_settings/type_supervisor_spec.rb | 2 +- spec/lib/site_settings/validations_spec.rb | 2 +- spec/lib/site_settings/yaml_loader_spec.rb | 2 +- spec/lib/slug_spec.rb | 2 +- spec/lib/spam_handler_spec.rb | 2 +- spec/lib/stylesheet/compiler_spec.rb | 2 +- spec/lib/stylesheet/importer_spec.rb | 2 +- spec/lib/stylesheet/manager_spec.rb | 2 +- spec/lib/suggested_topics_builder_spec.rb | 2 +- spec/lib/svg_sprite/svg_sprite_spec.rb | 8 ++--- spec/lib/system_message_spec.rb | 2 +- spec/lib/text_cleaner_spec.rb | 2 +- spec/lib/text_sentinel_spec.rb | 2 +- spec/lib/theme_flag_modifier_spec.rb | 2 +- spec/lib/theme_javascript_compiler_spec.rb | 2 +- spec/lib/theme_settings_manager_spec.rb | 2 +- spec/lib/theme_settings_parser_spec.rb | 2 +- spec/lib/theme_store/git_importer_spec.rb | 2 +- spec/lib/theme_store/zip_exporter_spec.rb | 2 +- spec/lib/theme_store/zip_importer_spec.rb | 2 +- spec/lib/timeline_lookup_spec.rb | 2 +- spec/lib/tiny_japanese_segmenter_spec.rb | 2 +- spec/lib/topic_creator_spec.rb | 2 +- spec/lib/topic_publisher_spec.rb | 2 +- .../topic_query/private_message_lists_spec.rb | 2 +- spec/lib/topic_query_spec.rb | 2 +- spec/lib/topic_retriever_spec.rb | 2 +- .../lib/topic_upload_security_manager_spec.rb | 2 +- spec/lib/topic_view_spec.rb | 36 +++++++++---------- spec/lib/topics_bulk_action_spec.rb | 2 +- spec/lib/trashable_spec.rb | 2 +- spec/lib/trust_level_spec.rb | 2 +- spec/lib/twitter_api_spec.rb | 2 +- spec/lib/unread_spec.rb | 2 +- spec/lib/upload_markdown_spec.rb | 2 +- spec/lib/url_helper_spec.rb | 2 +- spec/lib/user_comm_screener_spec.rb | 2 +- spec/lib/user_lookup_spec.rb | 2 +- spec/lib/user_name_suggester_spec.rb | 2 +- .../allowed_ip_address_validator_spec.rb | 2 +- .../censored_words_validator_spec.rb | 2 +- .../validators/css_color_validator_spec.rb | 2 +- .../email_address_validator_spec.rb | 2 +- .../email_setting_validator_spec.rb | 2 +- spec/lib/validators/email_validator_spec.rb | 2 +- .../external_system_avatars_validator_spec.rb | 2 +- .../group_setting_validator_spec.rb | 2 +- .../host_list_setting_validator_spec.rb | 2 +- .../integer_setting_validator_spec.rb | 2 +- .../ip_address_format_validator_spec.rb | 2 +- .../validators/max_emojis_validator_spec.rb | 2 +- .../max_username_length_validator_spec.rb | 2 +- .../min_username_length_validator_spec.rb | 2 +- .../lib/validators/password_validator_spec.rb | 2 +- spec/lib/validators/post_validator_spec.rb | 2 +- .../quality_title_validator_spec.rb | 2 +- .../regex_setting_validator_spec.rb | 2 +- .../validators/regexp_list_validator_spec.rb | 2 +- .../reply_by_email_address_validator_spec.rb | 2 +- .../reply_by_email_enabled_validator_spec.rb | 2 +- .../search_tokenize_chinese_validator_spec.rb | 2 +- ...search_tokenize_japanese_validator_spec.rb | 2 +- .../selectable_avatars_mode_validator_spec.rb | 2 +- .../string_setting_validator_spec.rb | 2 +- .../lib/validators/timezone_validator_spec.rb | 2 +- .../topic_title_length_validator_spec.rb | 2 +- ...icode_username_allowlist_validator_spec.rb | 2 +- .../unicode_username_validator_spec.rb | 2 +- spec/lib/validators/upload_validator_spec.rb | 2 +- .../user_full_name_validator_spec.rb | 2 +- .../username_setting_validator_spec.rb | 2 +- spec/lib/version_spec.rb | 2 +- spec/lib/webauthn/challenge_generator_spec.rb | 2 +- ...ecurity_key_authentication_service_spec.rb | 2 +- .../security_key_registration_service_spec.rb | 2 +- spec/lib/wizard/step_updater_spec.rb | 2 +- spec/lib/wizard/wizard_builder_spec.rb | 2 +- spec/lib/wizard/wizard_spec.rb | 2 +- spec/lib/wizard/wizard_step_spec.rb | 2 +- spec/mailers/group_smtp_mailer_spec.rb | 2 +- spec/mailers/invite_mailer_spec.rb | 2 +- spec/mailers/rejection_mailer_spec.rb | 2 +- spec/mailers/subscription_mailer_spec.rb | 2 +- spec/mailers/test_mailer_spec.rb | 2 +- spec/mailers/user_notifications_spec.rb | 2 +- spec/mailers/version_mailer_spec.rb | 2 +- spec/models/about_spec.rb | 2 +- spec/models/admin_dashboard_data_spec.rb | 2 +- spec/models/api_key_scope_spec.rb | 2 +- spec/models/api_key_spec.rb | 2 +- spec/models/application_request_spec.rb | 2 +- spec/models/associated_group_spec.rb | 2 +- spec/models/badge_spec.rb | 2 +- spec/models/badge_type_spec.rb | 2 +- spec/models/bookmark_spec.rb | 2 +- spec/models/category_featured_topic_spec.rb | 2 +- spec/models/category_group_spec.rb | 2 +- spec/models/category_list_spec.rb | 2 +- spec/models/category_spec.rb | 2 +- spec/models/category_user_spec.rb | 2 +- spec/models/child_theme_spec.rb | 2 +- spec/models/color_scheme_color_spec.rb | 2 +- spec/models/color_scheme_spec.rb | 2 +- spec/models/developer_spec.rb | 2 +- spec/models/digest_email_site_setting_spec.rb | 2 +- spec/models/directory_item_spec.rb | 2 +- spec/models/discourse_connect_spec.rb | 2 +- spec/models/do_not_disturb_timing_spec.rb | 2 +- spec/models/draft_sequence_spec.rb | 2 +- spec/models/draft_spec.rb | 2 +- spec/models/email_log_spec.rb | 2 +- spec/models/email_token_spec.rb | 2 +- spec/models/embeddable_host_spec.rb | 2 +- spec/models/emoji_spec.rb | 2 +- spec/models/given_daily_like_spec.rb | 2 +- spec/models/global_setting_spec.rb | 6 ++-- spec/models/group_archived_message_spec.rb | 2 +- spec/models/group_associated_group_spec.rb | 2 +- spec/models/group_spec.rb | 2 +- spec/models/group_user_spec.rb | 2 +- spec/models/incoming_link_spec.rb | 2 +- spec/models/incoming_links_report_spec.rb | 2 +- spec/models/invite_redeemer_spec.rb | 2 +- spec/models/invite_spec.rb | 2 +- spec/models/locale_site_setting_spec.rb | 2 +- .../mailing_list_mode_site_setting_spec.rb | 2 +- spec/models/notification_spec.rb | 4 +-- spec/models/optimized_image_spec.rb | 2 +- spec/models/permalink_spec.rb | 2 +- spec/models/plugin_store_spec.rb | 2 +- spec/models/post_action_spec.rb | 2 +- spec/models/post_action_type_spec.rb | 2 +- spec/models/post_analyzer_spec.rb | 2 +- spec/models/post_detail_spec.rb | 2 +- spec/models/post_mover_spec.rb | 2 +- spec/models/post_reply_spec.rb | 2 +- spec/models/post_revision_spec.rb | 2 +- spec/models/post_spec.rb | 2 +- spec/models/post_timing_spec.rb | 2 +- ...ivate_message_topic_tracking_state_spec.rb | 2 +- spec/models/quoted_post_spec.rb | 2 +- spec/models/remote_theme_spec.rb | 2 +- spec/models/report_spec.rb | 2 +- spec/models/reviewable_post_spec.rb | 2 +- spec/models/s3_region_site_setting_spec.rb | 2 +- spec/models/screened_email_spec.rb | 2 +- spec/models/screened_ip_address_spec.rb | 2 +- spec/models/screened_url_spec.rb | 2 +- spec/models/site_setting_spec.rb | 2 +- spec/models/site_spec.rb | 2 +- spec/models/sitemap_spec.rb | 2 +- spec/models/stylesheet_cache_spec.rb | 2 +- spec/models/tag_group_spec.rb | 2 +- spec/models/tag_spec.rb | 2 +- spec/models/tag_user_spec.rb | 2 +- spec/models/theme_field_spec.rb | 2 +- spec/models/theme_modifier_set_spec.rb | 2 +- spec/models/theme_spec.rb | 4 +-- spec/models/top_menu_item_spec.rb | 2 +- spec/models/top_topic_spec.rb | 2 +- spec/models/topic_allowed_user_spec.rb | 2 +- spec/models/topic_converter_spec.rb | 2 +- spec/models/topic_embed_spec.rb | 2 +- spec/models/topic_featured_users_spec.rb | 2 +- spec/models/topic_group_spec.rb | 2 +- spec/models/topic_invite_spec.rb | 2 +- spec/models/topic_link_click_spec.rb | 2 +- spec/models/topic_link_spec.rb | 2 +- spec/models/topic_list_spec.rb | 2 +- .../models/topic_participants_summary_spec.rb | 2 +- spec/models/topic_posters_summary_spec.rb | 2 +- spec/models/topic_spec.rb | 10 +++--- spec/models/topic_tag_spec.rb | 2 +- spec/models/topic_thumbnail_spec.rb | 2 +- spec/models/topic_tracking_state_spec.rb | 2 +- spec/models/topic_user_spec.rb | 2 +- spec/models/topic_view_item_spec.rb | 2 +- spec/models/translation_override_spec.rb | 2 +- spec/models/trust_level3_requirements_spec.rb | 2 +- .../trust_level_and_staff_setting_spec.rb | 2 +- spec/models/trust_level_setting_spec.rb | 2 +- spec/models/unsubscribe_key_spec.rb | 2 +- spec/models/upload_reference_spec.rb | 2 +- spec/models/upload_spec.rb | 2 +- spec/models/user_action_spec.rb | 2 +- spec/models/user_api_key_spec.rb | 2 +- spec/models/user_archived_message_spec.rb | 2 +- spec/models/user_associated_group_spec.rb | 2 +- spec/models/user_auth_token_spec.rb | 2 +- spec/models/user_avatar_spec.rb | 2 +- spec/models/user_badge_spec.rb | 2 +- spec/models/user_email_spec.rb | 2 +- spec/models/user_field_spec.rb | 2 +- spec/models/user_history_spec.rb | 2 +- .../models/user_notification_schedule_spec.rb | 2 +- spec/models/user_option_spec.rb | 2 +- spec/models/user_search_spec.rb | 2 +- spec/models/user_stat_spec.rb | 2 +- spec/models/user_status_spec.rb | 2 +- spec/models/user_summary_spec.rb | 2 +- spec/models/user_visit_spec.rb | 2 +- spec/models/username_validator_spec.rb | 2 +- spec/models/watched_word_spec.rb | 2 +- spec/models/web_crawler_request_spec.rb | 2 +- spec/models/web_hook_event_spec.rb | 2 +- spec/models/web_hook_spec.rb | 2 +- spec/multisite/request_tracker_spec.rb | 2 +- spec/requests/about_controller_spec.rb | 2 +- spec/requests/admin/api_controller_spec.rb | 2 +- spec/requests/admin/badges_controller_spec.rb | 2 +- .../admin/color_schemes_controller_spec.rb | 2 +- .../admin/dashboard_controller_spec.rb | 2 +- spec/requests/admin/email_controller_spec.rb | 2 +- .../admin/email_styles_controller_spec.rb | 2 +- .../admin/embeddable_hosts_controller_spec.rb | 2 +- .../admin/embedding_controller_spec.rb | 2 +- .../admin/impersonate_controller_spec.rb | 2 +- .../admin/permalinks_controller_spec.rb | 2 +- .../requests/admin/plugins_controller_spec.rb | 2 +- .../requests/admin/reports_controller_spec.rb | 2 +- .../admin/robots_txt_controller_spec.rb | 2 +- .../admin/screened_emails_controller_spec.rb | 2 +- .../screened_ip_addresses_controller_spec.rb | 2 +- .../admin/screened_urls_controller_spec.rb | 2 +- .../admin/site_settings_controller_spec.rb | 2 +- .../staff_action_logs_controller_spec.rb | 2 +- spec/requests/admin/themes_controller_spec.rb | 2 +- .../admin/user_fields_controller_spec.rb | 2 +- .../admin/versions_controller_spec.rb | 2 +- .../admin/web_hooks_controller_spec.rb | 2 +- spec/requests/api/backups_spec.rb | 2 +- spec/requests/api/badges_spec.rb | 2 +- spec/requests/api/categories_spec.rb | 2 +- spec/requests/api/groups_spec.rb | 2 +- spec/requests/api/invites_spec.rb | 2 +- spec/requests/api/notifications_spec.rb | 2 +- spec/requests/api/posts_spec.rb | 2 +- spec/requests/api/private_messages_spec.rb | 2 +- spec/requests/api/search_spec.rb | 2 +- spec/requests/api/site_spec.rb | 2 +- spec/requests/api/tags_spec.rb | 2 +- spec/requests/api/topics_spec.rb | 2 +- spec/requests/api/uploads_spec.rb | 2 +- spec/requests/api/user_badges_spec.rb | 2 +- spec/requests/api/users_spec.rb | 2 +- spec/requests/badges_controller_spec.rb | 2 +- spec/requests/bookmarks_controller_spec.rb | 2 +- spec/requests/bootstrap_controller_spec.rb | 2 +- spec/requests/categories_controller_spec.rb | 2 +- spec/requests/clicks_controller_spec.rb | 2 +- .../composer_messages_controller_spec.rb | 2 +- spec/requests/csp_reports_controller_spec.rb | 2 +- .../directory_columns_controller_spec.rb | 2 +- .../directory_items_controller_spec.rb | 2 +- .../do_not_disturb_controller_spec.rb | 2 +- spec/requests/drafts_controller_spec.rb | 2 +- spec/requests/email_controller_spec.rb | 2 +- spec/requests/embed_controller_spec.rb | 2 +- spec/requests/export_csv_controller_spec.rb | 2 +- .../requests/extra_locales_controller_spec.rb | 2 +- .../finish_installation_controller_spec.rb | 2 +- spec/requests/groups_controller_spec.rb | 2 +- spec/requests/hashtags_controller_spec.rb | 2 +- .../requests/inline_onebox_controller_spec.rb | 2 +- spec/requests/invites_controller_spec.rb | 2 +- .../requests/notifications_controller_spec.rb | 2 +- spec/requests/offline_controller_spec.rb | 2 +- spec/requests/onebox_controller_spec.rb | 2 +- spec/requests/permalinks_controller_spec.rb | 2 +- .../post_action_users_controller_spec.rb | 2 +- spec/requests/post_readers_controller_spec.rb | 2 +- spec/requests/posts_controller_spec.rb | 6 ++-- spec/requests/presence_controller_spec.rb | 2 +- .../push_notification_controller_spec.rb | 2 +- spec/requests/qunit_controller_spec.rb | 2 +- ...viewable_claimed_topics_controller_spec.rb | 2 +- spec/requests/reviewables_controller_spec.rb | 2 +- spec/requests/search_controller_spec.rb | 2 +- spec/requests/session_controller_spec.rb | 2 +- .../similar_topics_controller_spec.rb | 2 +- spec/requests/site_controller_spec.rb | 2 +- spec/requests/sitemap_controller_spec.rb | 2 +- spec/requests/static_controller_spec.rb | 2 +- spec/requests/steps_controller_spec.rb | 2 +- spec/requests/stylesheets_controller_spec.rb | 2 +- spec/requests/svg_sprite_controller_spec.rb | 2 +- spec/requests/tags_controller_spec.rb | 2 +- .../theme_javascripts_controller_spec.rb | 2 +- .../uploads_controller_multisite_spec.rb | 2 +- spec/requests/uploads_controller_spec.rb | 2 +- spec/requests/user_actions_controller_spec.rb | 2 +- .../requests/user_api_keys_controller_spec.rb | 2 +- spec/requests/user_avatars_controller_spec.rb | 2 +- spec/requests/user_badges_controller_spec.rb | 2 +- spec/requests/user_status_controller_spec.rb | 2 +- spec/requests/users_controller_spec.rb | 2 +- spec/requests/users_email_controller_spec.rb | 2 +- spec/requests/webhooks_controller_spec.rb | 2 +- spec/requests/wizard_controller_spec.rb | 2 +- spec/script/import_scripts/base_spec.rb | 2 +- .../vanilla_body_parser_spec.rb | 2 +- spec/serializers/about_serializer_spec.rb | 2 +- .../admin_user_list_serializer_spec.rb | 2 +- .../basic_group_serializer_spec.rb | 2 +- .../basic_group_user_serializer_spec.rb | 2 +- .../serializers/basic_post_serializer_spec.rb | 2 +- .../serializers/basic_user_serializer_spec.rb | 2 +- .../category_detailed_serializer_spec.rb | 2 +- spec/serializers/category_serializer_spec.rb | 2 +- .../category_upload_serializer_spec.rb | 2 +- spec/serializers/emoji_serializer_spec.rb | 2 +- .../serializers/group_show_serializer_spec.rb | 2 +- .../listable_topic_serializer_spec.rb | 2 +- .../new_post_result_serializer_spec.rb | 2 +- .../notification_serializer_spec.rb | 2 +- .../post_revision_serializer_spec.rb | 2 +- spec/serializers/post_serializer_spec.rb | 2 +- ...reviewable_flagged_post_serializer_spec.rb | 2 +- .../reviewable_queued_post_serializer_spec.rb | 2 +- .../reviewable_score_serializer_spec.rb | 2 +- .../serializers/reviewable_serializer_spec.rb | 2 +- .../reviewable_user_serializer_spec.rb | 2 +- spec/serializers/site_serializer_spec.rb | 2 +- .../suggested_topic_serializer_spec.rb | 2 +- spec/serializers/tag_group_serializer_spec.rb | 2 +- .../serializers/topic_link_serializer_spec.rb | 2 +- .../topic_list_item_serializer_spec.rb | 2 +- .../topic_tracking_state_serializer_spec.rb | 2 +- .../topic_view_details_serializer_spec.rb | 2 +- .../topic_view_posts_serializer_spec.rb | 2 +- .../serializers/topic_view_serializer_spec.rb | 2 +- .../user_auth_token_serializer_spec.rb | 2 +- spec/serializers/user_card_serializer_spec.rb | 2 +- spec/serializers/user_serializer_spec.rb | 2 +- .../user_summary_serializer_spec.rb | 2 +- spec/serializers/wizard_serializer_spec.rb | 2 +- .../services/anonymous_shadow_creator_spec.rb | 2 +- spec/services/auto_silence_spec.rb | 2 +- spec/services/badge_granter_spec.rb | 2 +- spec/services/base_bookmarkable_spec.rb | 2 +- spec/services/color_scheme_revisor_spec.rb | 2 +- spec/services/destroy_task_spec.rb | 2 +- spec/services/email_style_updater_spec.rb | 2 +- spec/services/flag_sockpuppets_spec.rb | 2 +- spec/services/group_message_spec.rb | 2 +- spec/services/heat_settings_updater_spec.rb | 2 +- spec/services/notification_emailer_spec.rb | 2 +- .../consolidate_notifications_spec.rb | 2 +- .../consolidation_planner_spec.rb | 2 +- spec/services/post_action_notifier_spec.rb | 2 +- spec/services/post_alerter_spec.rb | 2 +- spec/services/post_bookmarkable_spec.rb | 2 +- spec/services/post_owner_changer_spec.rb | 2 +- spec/services/random_topic_selector_spec.rb | 2 +- spec/services/search_indexer_spec.rb | 2 +- spec/services/site_settings_spec.rb | 2 +- spec/services/staff_action_logger_spec.rb | 2 +- spec/services/themes_spec.rb | 2 +- spec/services/topic_bookmarkable_spec.rb | 2 +- spec/services/topic_status_updater_spec.rb | 2 +- spec/services/topic_timestamp_changer_spec.rb | 2 +- spec/services/trust_level_granter_spec.rb | 2 +- spec/services/user_activator_spec.rb | 2 +- spec/services/user_anonymizer_spec.rb | 2 +- spec/services/user_authenticator_spec.rb | 2 +- spec/services/user_destroyer_spec.rb | 2 +- spec/services/user_merger_spec.rb | 2 +- ...er_notification_schedule_processor_spec.rb | 2 +- spec/services/user_silencer_spec.rb | 2 +- spec/services/user_stat_count_updater_spec.rb | 2 +- spec/services/user_updater_spec.rb | 2 +- spec/services/username_changer_spec.rb | 2 +- .../services/username_checker_service_spec.rb | 2 +- spec/services/wildcard_domain_checker_spec.rb | 2 +- spec/services/wildcard_url_checker_spec.rb | 2 +- spec/services/word_watcher_spec.rb | 2 +- spec/support/onebox_helpers.rb | 6 ++-- .../shared_examples_for_stats_cacheable.rb | 2 +- .../shared_examples_for_versioned_model.rb | 2 +- spec/views/list/list.erb_spec.rb | 2 +- .../failure.html.erb_spec.rb | 2 +- spec/views/topics/show.html.erb_spec.rb | 2 +- 736 files changed, 781 insertions(+), 782 deletions(-) diff --git a/.rspec b/.rspec index 4d520043cf..91185b8938 100644 --- a/.rspec +++ b/.rspec @@ -1,2 +1 @@ ---colour --require 'rails_helper' diff --git a/plugins/discourse-details/spec/components/pretty_text_spec.rb b/plugins/discourse-details/spec/components/pretty_text_spec.rb index 7c74a53c28..bbbcb6ba27 100644 --- a/plugins/discourse-details/spec/components/pretty_text_spec.rb +++ b/plugins/discourse-details/spec/components/pretty_text_spec.rb @@ -3,7 +3,7 @@ require 'rails_helper' require 'pretty_text' -describe PrettyText do +RSpec.describe PrettyText do let(:post) { Fabricate(:post) } diff --git a/plugins/discourse-local-dates/spec/lib/pretty_text_spec.rb b/plugins/discourse-local-dates/spec/lib/pretty_text_spec.rb index c365ffaf5e..326c9dabb3 100644 --- a/plugins/discourse-local-dates/spec/lib/pretty_text_spec.rb +++ b/plugins/discourse-local-dates/spec/lib/pretty_text_spec.rb @@ -14,7 +14,7 @@ def generate_html(text, opts = {}) output + "

" end -describe PrettyText do +RSpec.describe PrettyText do before do freeze_time end diff --git a/plugins/discourse-local-dates/spec/models/post_spec.rb b/plugins/discourse-local-dates/spec/models/post_spec.rb index 9cdaa72495..98d5e6e3d2 100644 --- a/plugins/discourse-local-dates/spec/models/post_spec.rb +++ b/plugins/discourse-local-dates/spec/models/post_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Post do +RSpec.describe Post do before do Jobs.run_immediately! diff --git a/plugins/discourse-narrative-bot/spec/discourse_narrative_bot/new_user_narrative_spec.rb b/plugins/discourse-narrative-bot/spec/discourse_narrative_bot/new_user_narrative_spec.rb index f2aea1e2e1..59fd663099 100644 --- a/plugins/discourse-narrative-bot/spec/discourse_narrative_bot/new_user_narrative_spec.rb +++ b/plugins/discourse-narrative-bot/spec/discourse_narrative_bot/new_user_narrative_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe DiscourseNarrativeBot::NewUserNarrative do +RSpec.describe DiscourseNarrativeBot::NewUserNarrative do fab!(:welcome_topic) { Fabricate(:topic, title: 'Welcome to Discourse') } fab!(:narrative_bot) { ::DiscourseNarrativeBot::Base.new } fab!(:discobot_user) { narrative_bot.discobot_user } diff --git a/plugins/discourse-narrative-bot/spec/discourse_narrative_bot/store_spec.rb b/plugins/discourse-narrative-bot/spec/discourse_narrative_bot/store_spec.rb index 0f8af31f15..2f42a2ee3c 100644 --- a/plugins/discourse-narrative-bot/spec/discourse_narrative_bot/store_spec.rb +++ b/plugins/discourse-narrative-bot/spec/discourse_narrative_bot/store_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe DiscourseNarrativeBot::Store do +RSpec.describe DiscourseNarrativeBot::Store do describe '.set' do it 'should set the right value in the plugin store' do key = 'somekey' diff --git a/plugins/discourse-narrative-bot/spec/discourse_narrative_bot/track_selector_spec.rb b/plugins/discourse-narrative-bot/spec/discourse_narrative_bot/track_selector_spec.rb index a82c3eb9b2..f1b68d154d 100644 --- a/plugins/discourse-narrative-bot/spec/discourse_narrative_bot/track_selector_spec.rb +++ b/plugins/discourse-narrative-bot/spec/discourse_narrative_bot/track_selector_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe DiscourseNarrativeBot::TrackSelector do +RSpec.describe DiscourseNarrativeBot::TrackSelector do let(:user) { Fabricate(:user) } let(:narrative_bot) { ::DiscourseNarrativeBot::Base.new } let(:discobot_user) { narrative_bot.discobot_user } diff --git a/plugins/discourse-narrative-bot/spec/jobs/onceoff/remap_old_bot_images_spec.rb b/plugins/discourse-narrative-bot/spec/jobs/onceoff/remap_old_bot_images_spec.rb index 5179edbeff..6b0326b7f5 100644 --- a/plugins/discourse-narrative-bot/spec/jobs/onceoff/remap_old_bot_images_spec.rb +++ b/plugins/discourse-narrative-bot/spec/jobs/onceoff/remap_old_bot_images_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::DiscourseNarrativeBot::RemapOldBotImages do +RSpec.describe Jobs::DiscourseNarrativeBot::RemapOldBotImages do context "when bot's post contains an old link" do let!(:post) do Fabricate(:post, diff --git a/plugins/discourse-narrative-bot/spec/requests/discobot_certificate_spec.rb b/plugins/discourse-narrative-bot/spec/requests/discobot_certificate_spec.rb index 4fcc04666d..803b4887a4 100644 --- a/plugins/discourse-narrative-bot/spec/requests/discobot_certificate_spec.rb +++ b/plugins/discourse-narrative-bot/spec/requests/discobot_certificate_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe "Discobot Certificate" do +RSpec.describe "Discobot Certificate" do let(:user) { Fabricate(:user, name: 'Jeff Atwood') } let(:params) { diff --git a/plugins/discourse-narrative-bot/spec/requests/discobot_welcome_post_spec.rb b/plugins/discourse-narrative-bot/spec/requests/discobot_welcome_post_spec.rb index b9ed97d944..6fa4be2b9d 100644 --- a/plugins/discourse-narrative-bot/spec/requests/discobot_welcome_post_spec.rb +++ b/plugins/discourse-narrative-bot/spec/requests/discobot_welcome_post_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe "Discobot welcome post" do +RSpec.describe "Discobot welcome post" do let(:user) { Fabricate(:user) } before do diff --git a/plugins/discourse-narrative-bot/spec/user_spec.rb b/plugins/discourse-narrative-bot/spec/user_spec.rb index 4018ea4acf..7b3d51b6f0 100644 --- a/plugins/discourse-narrative-bot/spec/user_spec.rb +++ b/plugins/discourse-narrative-bot/spec/user_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe User do +RSpec.describe User do let(:user) { Fabricate(:user) } let(:profile_page_url) { "#{Discourse.base_url}/users/#{user.username}" } diff --git a/plugins/discourse-presence/spec/integration/presence_spec.rb b/plugins/discourse-presence/spec/integration/presence_spec.rb index 6c6c480f65..b884a88ada 100644 --- a/plugins/discourse-presence/spec/integration/presence_spec.rb +++ b/plugins/discourse-presence/spec/integration/presence_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe "discourse-presence" do +RSpec.describe "discourse-presence" do describe 'PresenceChannel configuration' do fab!(:user) { Fabricate(:user) } fab!(:user2) { Fabricate(:user) } diff --git a/plugins/poll/spec/controllers/polls_controller_spec.rb b/plugins/poll/spec/controllers/polls_controller_spec.rb index 6eb5520515..a37fc7413f 100644 --- a/plugins/poll/spec/controllers/polls_controller_spec.rb +++ b/plugins/poll/spec/controllers/polls_controller_spec.rb @@ -2,7 +2,7 @@ require "rails_helper" -describe ::DiscoursePoll::PollsController do +RSpec.describe ::DiscoursePoll::PollsController do routes { ::DiscoursePoll::Engine.routes } let!(:user) { log_in } diff --git a/plugins/poll/spec/controllers/posts_controller_spec.rb b/plugins/poll/spec/controllers/posts_controller_spec.rb index 5cc316a9b4..dec4cf0190 100644 --- a/plugins/poll/spec/controllers/posts_controller_spec.rb +++ b/plugins/poll/spec/controllers/posts_controller_spec.rb @@ -2,7 +2,7 @@ require "rails_helper" -describe PostsController do +RSpec.describe PostsController do let!(:user) { log_in } let!(:title) { "Testing Poll Plugin" } diff --git a/plugins/poll/spec/integration/poll_endpoints_spec.rb b/plugins/poll/spec/integration/poll_endpoints_spec.rb index 68eb963c9e..0fd3723646 100644 --- a/plugins/poll/spec/integration/poll_endpoints_spec.rb +++ b/plugins/poll/spec/integration/poll_endpoints_spec.rb @@ -2,7 +2,7 @@ require "rails_helper" -describe "DiscoursePoll endpoints" do +RSpec.describe "DiscoursePoll endpoints" do describe "fetch voters for a poll" do fab!(:user) { Fabricate(:user) } fab!(:post) { Fabricate(:post, raw: "[poll public=true]\n- A\n- B\n[/poll]") } diff --git a/plugins/poll/spec/jobs/regular/close_poll_spec.rb b/plugins/poll/spec/jobs/regular/close_poll_spec.rb index 8bd8004c78..891b35c9c1 100644 --- a/plugins/poll/spec/jobs/regular/close_poll_spec.rb +++ b/plugins/poll/spec/jobs/regular/close_poll_spec.rb @@ -2,7 +2,7 @@ require "rails_helper" -describe Jobs::ClosePoll do +RSpec.describe Jobs::ClosePoll do let(:post) { Fabricate(:post, raw: "[poll]\n- A\n- B\n[/poll]") } describe 'missing arguments' do diff --git a/plugins/poll/spec/lib/new_post_manager_spec.rb b/plugins/poll/spec/lib/new_post_manager_spec.rb index 5f4c975cfc..fb10d9cf2d 100644 --- a/plugins/poll/spec/lib/new_post_manager_spec.rb +++ b/plugins/poll/spec/lib/new_post_manager_spec.rb @@ -2,7 +2,7 @@ require "rails_helper" -describe NewPostManager do +RSpec.describe NewPostManager do let(:user) { Fabricate(:newuser) } let(:admin) { Fabricate(:admin) } diff --git a/plugins/poll/spec/lib/poll_spec.rb b/plugins/poll/spec/lib/poll_spec.rb index 289cccf19d..1b885947f7 100644 --- a/plugins/poll/spec/lib/poll_spec.rb +++ b/plugins/poll/spec/lib/poll_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe DiscoursePoll::Poll do +RSpec.describe DiscoursePoll::Poll do fab!(:user) { Fabricate(:user) } fab!(:user_2) { Fabricate(:user) } diff --git a/plugins/poll/spec/lib/polls_updater_spec.rb b/plugins/poll/spec/lib/polls_updater_spec.rb index 371eb57a18..b6dde6817a 100644 --- a/plugins/poll/spec/lib/polls_updater_spec.rb +++ b/plugins/poll/spec/lib/polls_updater_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe DiscoursePoll::PollsUpdater do +RSpec.describe DiscoursePoll::PollsUpdater do def update(post, polls) DiscoursePoll::PollsUpdater.update(post, polls) diff --git a/plugins/poll/spec/lib/polls_validator_spec.rb b/plugins/poll/spec/lib/polls_validator_spec.rb index 761fc359f1..2ad690c81c 100644 --- a/plugins/poll/spec/lib/polls_validator_spec.rb +++ b/plugins/poll/spec/lib/polls_validator_spec.rb @@ -2,7 +2,7 @@ require "rails_helper" -describe ::DiscoursePoll::PollsValidator do +RSpec.describe ::DiscoursePoll::PollsValidator do let(:post) { Fabricate(:post) } subject { described_class.new(post) } diff --git a/plugins/poll/spec/lib/pretty_text_spec.rb b/plugins/poll/spec/lib/pretty_text_spec.rb index e427db0dc7..6c4c48c188 100644 --- a/plugins/poll/spec/lib/pretty_text_spec.rb +++ b/plugins/poll/spec/lib/pretty_text_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe PrettyText do +RSpec.describe PrettyText do def n(html) html.strip diff --git a/plugins/poll/spec/models/poll_spec.rb b/plugins/poll/spec/models/poll_spec.rb index 7ab5cab0d6..9b72a4a9ec 100644 --- a/plugins/poll/spec/models/poll_spec.rb +++ b/plugins/poll/spec/models/poll_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe ::DiscoursePoll::Poll do +RSpec.describe ::DiscoursePoll::Poll do describe ".transform_for_user_field_override" do it "Transforms UserField name if a matching CustomUserField is present" do user_field_name = "Something Cool" diff --git a/plugins/poll/spec/requests/users_controller_spec.rb b/plugins/poll/spec/requests/users_controller_spec.rb index fa18d6f6d3..87b354ea48 100644 --- a/plugins/poll/spec/requests/users_controller_spec.rb +++ b/plugins/poll/spec/requests/users_controller_spec.rb @@ -2,7 +2,7 @@ require "rails_helper" -describe Admin::UsersController do +RSpec.describe Admin::UsersController do let(:admin) { Fabricate(:admin) } diff --git a/plugins/poll/spec/serializers/poll_option_serializer_spec.rb b/plugins/poll/spec/serializers/poll_option_serializer_spec.rb index 90b53536f8..1a31df9838 100644 --- a/plugins/poll/spec/serializers/poll_option_serializer_spec.rb +++ b/plugins/poll/spec/serializers/poll_option_serializer_spec.rb @@ -8,7 +8,7 @@ def serialize_option(option, user) ) end -describe PollOptionSerializer do +RSpec.describe PollOptionSerializer do let(:voter) { Fabricate(:user) } let(:poll) { post.polls.first } diff --git a/plugins/styleguide/spec/integration/access_spec.rb b/plugins/styleguide/spec/integration/access_spec.rb index bd35e1ee1d..cf5081f299 100644 --- a/plugins/styleguide/spec/integration/access_spec.rb +++ b/plugins/styleguide/spec/integration/access_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe 'SiteSetting.styleguide_admin_only' do +RSpec.describe 'SiteSetting.styleguide_admin_only' do before do SiteSetting.styleguide_enabled = true end @@ -34,7 +34,7 @@ describe 'SiteSetting.styleguide_admin_only' do end end -describe 'SiteSetting.styleguide_enabled' do +RSpec.describe 'SiteSetting.styleguide_enabled' do before do sign_in(Fabricate(:admin)) end diff --git a/plugins/styleguide/spec/integration/assets_spec.rb b/plugins/styleguide/spec/integration/assets_spec.rb index eb056a0fbb..ad555711a6 100644 --- a/plugins/styleguide/spec/integration/assets_spec.rb +++ b/plugins/styleguide/spec/integration/assets_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe 'Styleguide assets' do +RSpec.describe 'Styleguide assets' do before do SiteSetting.styleguide_enabled = true sign_in(Fabricate(:admin)) diff --git a/spec/helpers/application_helper_spec.rb b/spec/helpers/application_helper_spec.rb index 84138e8ec1..a972e62cdc 100644 --- a/spec/helpers/application_helper_spec.rb +++ b/spec/helpers/application_helper_spec.rb @@ -1,7 +1,7 @@ # coding: utf-8 # frozen_string_literal: true -describe ApplicationHelper do +RSpec.describe ApplicationHelper do describe "preload_script" do def preload_link(url) diff --git a/spec/helpers/topics_helper_spec.rb b/spec/helpers/topics_helper_spec.rb index 2d1780bda1..5f0462fd97 100644 --- a/spec/helpers/topics_helper_spec.rb +++ b/spec/helpers/topics_helper_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe TopicsHelper do +RSpec.describe TopicsHelper do describe "#categories_breadcrumb" do let(:user) { Fabricate(:user) } diff --git a/spec/helpers/user_notifications_helper_spec.rb b/spec/helpers/user_notifications_helper_spec.rb index 4f356d813b..175c7bbb64 100644 --- a/spec/helpers/user_notifications_helper_spec.rb +++ b/spec/helpers/user_notifications_helper_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe UserNotificationsHelper do +RSpec.describe UserNotificationsHelper do let(:upload_path) { Discourse.store.upload_path } describe '#email_excerpt' do diff --git a/spec/import_export/category_exporter_spec.rb b/spec/import_export/category_exporter_spec.rb index 6f943d6937..5fda50252b 100644 --- a/spec/import_export/category_exporter_spec.rb +++ b/spec/import_export/category_exporter_spec.rb @@ -2,7 +2,7 @@ require "import_export" -describe ImportExport::CategoryExporter do +RSpec.describe ImportExport::CategoryExporter do fab!(:category) { Fabricate(:category) } fab!(:group) { Fabricate(:group) } diff --git a/spec/import_export/category_structure_exporter_spec.rb b/spec/import_export/category_structure_exporter_spec.rb index cd06e2dc2b..b5ee22fbde 100644 --- a/spec/import_export/category_structure_exporter_spec.rb +++ b/spec/import_export/category_structure_exporter_spec.rb @@ -2,7 +2,7 @@ require "import_export/category_structure_exporter" -describe ImportExport::CategoryStructureExporter do +RSpec.describe ImportExport::CategoryStructureExporter do before do STDOUT.stubs(:write) diff --git a/spec/import_export/group_exporter_spec.rb b/spec/import_export/group_exporter_spec.rb index e70e549db7..8fd5c0f908 100644 --- a/spec/import_export/group_exporter_spec.rb +++ b/spec/import_export/group_exporter_spec.rb @@ -2,7 +2,7 @@ require "import_export/group_exporter" -describe ImportExport::GroupExporter do +RSpec.describe ImportExport::GroupExporter do before do STDOUT.stubs(:write) diff --git a/spec/import_export/importer_spec.rb b/spec/import_export/importer_spec.rb index 217c68da57..ad75f2ade7 100644 --- a/spec/import_export/importer_spec.rb +++ b/spec/import_export/importer_spec.rb @@ -2,7 +2,7 @@ require "import_export" -describe ImportExport::Importer do +RSpec.describe ImportExport::Importer do before do STDOUT.stubs(:write) diff --git a/spec/import_export/topic_exporter_spec.rb b/spec/import_export/topic_exporter_spec.rb index bf10d59d7f..809189158c 100644 --- a/spec/import_export/topic_exporter_spec.rb +++ b/spec/import_export/topic_exporter_spec.rb @@ -2,7 +2,7 @@ require "import_export" -describe ImportExport::TopicExporter do +RSpec.describe ImportExport::TopicExporter do before do STDOUT.stubs(:write) diff --git a/spec/initializers/track_setting_changes_spec.rb b/spec/initializers/track_setting_changes_spec.rb index 14704cc147..bab78a1d33 100644 --- a/spec/initializers/track_setting_changes_spec.rb +++ b/spec/initializers/track_setting_changes_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe 'Setting changes' do +RSpec.describe 'Setting changes' do describe '#must_approve_users' do before { SiteSetting.must_approve_users = false } diff --git a/spec/integration/api_keys_spec.rb b/spec/integration/api_keys_spec.rb index 4998f847eb..be9e790a68 100644 --- a/spec/integration/api_keys_spec.rb +++ b/spec/integration/api_keys_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe 'api keys' do +RSpec.describe 'api keys' do let(:user) { Fabricate(:user) } let(:api_key) { ApiKey.create!(user_id: user.id, created_by_id: Discourse.system_user) } @@ -61,7 +61,7 @@ describe 'api keys' do end end -describe 'user api keys' do +RSpec.describe 'user api keys' do let(:user) { Fabricate(:user) } let(:user_api_key) { Fabricate(:readonly_user_api_key, user: user) } diff --git a/spec/integration/auto_reject_reviewable_users_spec.rb b/spec/integration/auto_reject_reviewable_users_spec.rb index 25029eb987..f51d41705c 100644 --- a/spec/integration/auto_reject_reviewable_users_spec.rb +++ b/spec/integration/auto_reject_reviewable_users_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe "auto reject reviewable users" do +RSpec.describe "auto reject reviewable users" do context "reviewable users" do fab!(:old_user) { Fabricate(:reviewable, created_at: 80.days.ago) } diff --git a/spec/integration/blocked_hotlinked_media_spec.rb b/spec/integration/blocked_hotlinked_media_spec.rb index 5ea328c4f5..c727210c48 100644 --- a/spec/integration/blocked_hotlinked_media_spec.rb +++ b/spec/integration/blocked_hotlinked_media_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe "hotlinked media blocking" do +RSpec.describe "hotlinked media blocking" do let(:hotlinked_url) { "http://example.com/images/2/2e/Longcat1.png" } let(:onebox_url) { "http://example.com/onebox" } let(:png) { Base64.decode64("R0lGODlhAQABALMAAAAAAIAAAACAAICAAAAAgIAAgACAgMDAwICAgP8AAAD/AP//AAAA//8A/wD//wBiZCH5BAEAAA8ALAAAAAABAAEAAAQC8EUAOw==") } diff --git a/spec/integration/category_tag_spec.rb b/spec/integration/category_tag_spec.rb index 6e3399846e..4650f02cd3 100644 --- a/spec/integration/category_tag_spec.rb +++ b/spec/integration/category_tag_spec.rb @@ -1,7 +1,7 @@ # encoding: UTF-8 # frozen_string_literal: true -describe "category tag restrictions" do +RSpec.describe "category tag restrictions" do def filter_allowed_tags(opts = {}) DiscourseTagging.filter_allowed_tags(Guardian.new(user), opts) @@ -427,7 +427,7 @@ describe "category tag restrictions" do end end -describe "tag topic counts per category" do +RSpec.describe "tag topic counts per category" do fab!(:admin) { Fabricate(:admin) } fab!(:category) { Fabricate(:category) } fab!(:category2) { Fabricate(:category) } diff --git a/spec/integration/content_security_policy_spec.rb b/spec/integration/content_security_policy_spec.rb index 0b6dcea346..69a56acc65 100644 --- a/spec/integration/content_security_policy_spec.rb +++ b/spec/integration/content_security_policy_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe 'content security policy integration' do +RSpec.describe 'content security policy integration' do it "adds the csp headers correctly" do SiteSetting.content_security_policy = false diff --git a/spec/integration/email_outbound_spec.rb b/spec/integration/email_outbound_spec.rb index 25d441d517..eaeebbe3ea 100644 --- a/spec/integration/email_outbound_spec.rb +++ b/spec/integration/email_outbound_spec.rb @@ -4,7 +4,7 @@ # with mailer specific mailer specs like UserEmail, but sometimes we need # to test things along the whole outbound flow including the MessageBuilder # and the Sender. -describe "Outbound Email" do +RSpec.describe "Outbound Email" do def send_email(opts = {}) message = TestMailer.send_test("test@test.com", opts) result = Email::Sender.new(message, :test_message).send diff --git a/spec/integration/email_style_spec.rb b/spec/integration/email_style_spec.rb index d00ab661cf..b23cbe35e5 100644 --- a/spec/integration/email_style_spec.rb +++ b/spec/integration/email_style_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe EmailStyle do +RSpec.describe EmailStyle do context "ERB evaluation" do it "does not evaluate ERB outside of the email itself" do diff --git a/spec/integration/flags_spec.rb b/spec/integration/flags_spec.rb index bdc0ed8c9a..7459174757 100644 --- a/spec/integration/flags_spec.rb +++ b/spec/integration/flags_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe PostAction do +RSpec.describe PostAction do it "triggers the 'flag_reviewed' event when there was at least one flag" do admin = Fabricate(:admin) diff --git a/spec/integration/group_spec.rb b/spec/integration/group_spec.rb index b32abb9fcf..0573053b68 100644 --- a/spec/integration/group_spec.rb +++ b/spec/integration/group_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Group do +RSpec.describe Group do let(:group) do Fabricate( :group, diff --git a/spec/integration/invalid_request_spec.rb b/spec/integration/invalid_request_spec.rb index 4f1bde6d9a..e3e4e0bad7 100644 --- a/spec/integration/invalid_request_spec.rb +++ b/spec/integration/invalid_request_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe 'invalid requests', type: :request do +RSpec.describe 'invalid requests', type: :request do before do @orig_logger = Rails.logger Rails.logger = @fake_logger = FakeLogger.new diff --git a/spec/integration/invite_only_registration_spec.rb b/spec/integration/invite_only_registration_spec.rb index 5dd2aedfa8..a30869094e 100644 --- a/spec/integration/invite_only_registration_spec.rb +++ b/spec/integration/invite_only_registration_spec.rb @@ -1,7 +1,7 @@ # encoding: UTF-8 # frozen_string_literal: true -describe 'invite only' do +RSpec.describe 'invite only' do describe '#create invite only' do it 'can create user via API' do diff --git a/spec/integration/message_bus_spec.rb b/spec/integration/message_bus_spec.rb index 571eabf008..a2f961ddc5 100644 --- a/spec/integration/message_bus_spec.rb +++ b/spec/integration/message_bus_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe 'message bus integration' do +RSpec.describe 'message bus integration' do it "allows anonymous requests to the messagebus" do post "/message-bus/poll" diff --git a/spec/integration/multisite_cookies_spec.rb b/spec/integration/multisite_cookies_spec.rb index c5d2a3b2ef..15bad1c74b 100644 --- a/spec/integration/multisite_cookies_spec.rb +++ b/spec/integration/multisite_cookies_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe 'multisite', type: [:multisite, :request] do +RSpec.describe 'multisite', type: [:multisite, :request] do it "works" do get "http://test.localhost/session/csrf.json" expect(response.status).to eq(200) diff --git a/spec/integration/multisite_spec.rb b/spec/integration/multisite_spec.rb index 46859666e2..d53c443560 100644 --- a/spec/integration/multisite_spec.rb +++ b/spec/integration/multisite_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe 'multisite', type: [:multisite, :request] do +RSpec.describe 'multisite', type: [:multisite, :request] do it "should always allow /srv/status through" do get "http://unknown.com/srv/status" expect(response.status).to eq(200) diff --git a/spec/integration/rate_limiting_spec.rb b/spec/integration/rate_limiting_spec.rb index 8bf499aee8..ebf24e909b 100644 --- a/spec/integration/rate_limiting_spec.rb +++ b/spec/integration/rate_limiting_spec.rb @@ -1,7 +1,7 @@ # encoding: UTF-8 # frozen_string_literal: true -describe 'rate limiter integration' do +RSpec.describe 'rate limiter integration' do before do RateLimiter.enable diff --git a/spec/integration/same_ip_spammers_spec.rb b/spec/integration/same_ip_spammers_spec.rb index 1ce26b8cbe..b7cde6e3f6 100644 --- a/spec/integration/same_ip_spammers_spec.rb +++ b/spec/integration/same_ip_spammers_spec.rb @@ -1,7 +1,7 @@ # encoding: UTF-8 # frozen_string_literal: true -describe "spammers on same IP" do +RSpec.describe "spammers on same IP" do let(:ip_address) { '182.189.119.174' } let!(:spammer1) { Fabricate(:user, ip_address: ip_address) } diff --git a/spec/integration/spam_rules_spec.rb b/spec/integration/spam_rules_spec.rb index cbd95df080..cfc22442d8 100644 --- a/spec/integration/spam_rules_spec.rb +++ b/spec/integration/spam_rules_spec.rb @@ -1,7 +1,7 @@ # encoding: UTF-8 # frozen_string_literal: true -describe "spam rules for users" do +RSpec.describe "spam rules for users" do describe 'auto-silence users based on flagging' do fab!(:admin) { Fabricate(:admin) } # needed to send a system message diff --git a/spec/integration/topic_auto_close_spec.rb b/spec/integration/topic_auto_close_spec.rb index 3ec395c075..11d851678e 100644 --- a/spec/integration/topic_auto_close_spec.rb +++ b/spec/integration/topic_auto_close_spec.rb @@ -1,7 +1,7 @@ # encoding: UTF-8 # frozen_string_literal: true -describe Topic do +RSpec.describe Topic do let(:job_klass) { Jobs::CloseTopic } context 'creating a topic without auto-close' do diff --git a/spec/integration/topic_thumbnail_spec.rb b/spec/integration/topic_thumbnail_spec.rb index db67e603b1..9245716b14 100644 --- a/spec/integration/topic_thumbnail_spec.rb +++ b/spec/integration/topic_thumbnail_spec.rb @@ -1,5 +1,5 @@ # frozen_string_literal: true -describe "Topic Thumbnails" do +RSpec.describe "Topic Thumbnails" do before do SiteSetting.create_thumbnails = true ImageSizer.stubs(:resize).returns([9, 9]) diff --git a/spec/integration/watched_words_spec.rb b/spec/integration/watched_words_spec.rb index 5d4052e328..0aec8298d1 100644 --- a/spec/integration/watched_words_spec.rb +++ b/spec/integration/watched_words_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe WatchedWord do +RSpec.describe WatchedWord do fab!(:tl2_user) { Fabricate(:user, trust_level: TrustLevel[2]) } fab!(:admin) { Fabricate(:admin) } fab!(:moderator) { Fabricate(:moderator) } diff --git a/spec/integrity/coding_style_spec.rb b/spec/integrity/coding_style_spec.rb index 123b03ba28..b51e6da61f 100644 --- a/spec/integrity/coding_style_spec.rb +++ b/spec/integrity/coding_style_spec.rb @@ -17,7 +17,7 @@ def grep_file(file, regex) lines.count > 0 ? file : nil end -describe 'Coding style' do +RSpec.describe 'Coding style' do describe 'Javascript' do it 'prevents this.get("foo") pattern' do js_files = list_js_files('app/assets/javascripts') diff --git a/spec/integrity/common_mark_spec.rb b/spec/integrity/common_mark_spec.rb index 07ebdd0768..01cc77683f 100644 --- a/spec/integrity/common_mark_spec.rb +++ b/spec/integrity/common_mark_spec.rb @@ -1,5 +1,5 @@ # frozen_string_literal: true -describe "CommonMark" do +RSpec.describe "CommonMark" do it 'passes spec' do SiteSetting.traditional_markdown_linebreaks = true diff --git a/spec/integrity/i18n_spec.rb b/spec/integrity/i18n_spec.rb index 2ab21f5069..0205584c24 100644 --- a/spec/integrity/i18n_spec.rb +++ b/spec/integrity/i18n_spec.rb @@ -20,7 +20,7 @@ def is_yaml_compatible?(english, translated) true end -describe "i18n integrity checks" do +RSpec.describe "i18n integrity checks" do it "has an i18n key for each Site Setting" do SiteSetting.all_settings.each do |s| next if s[:setting][/^test_/] @@ -97,7 +97,7 @@ describe "i18n integrity checks" do end end -describe "fallbacks" do +RSpec.describe "fallbacks" do before do I18n.backend = I18n::Backend::DiscourseI18n.new I18n.fallbacks = I18n::Backend::FallbackLocaleList.new diff --git a/spec/integrity/js_constants_spec.rb b/spec/integrity/js_constants_spec.rb index 50b81d3565..fe13669d54 100644 --- a/spec/integrity/js_constants_spec.rb +++ b/spec/integrity/js_constants_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe "constants match ruby" do +RSpec.describe "constants match ruby" do let(:ctx) { MiniRacer::Context.new } diff --git a/spec/integrity/oj_spec.rb b/spec/integrity/oj_spec.rb index c0ccdecf2f..799f7578f2 100644 --- a/spec/integrity/oj_spec.rb +++ b/spec/integrity/oj_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe 'Oj' do +RSpec.describe 'Oj' do it "is enabled" do classes = Set.new tracer = TracePoint.new(:c_call) { |tp| classes << tp.defined_class } diff --git a/spec/integrity/onceoff_integrity_spec.rb b/spec/integrity/onceoff_integrity_spec.rb index ebc0e685f6..e913d2c816 100644 --- a/spec/integrity/onceoff_integrity_spec.rb +++ b/spec/integrity/onceoff_integrity_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe ::Jobs::Onceoff do +RSpec.describe ::Jobs::Onceoff do it "can run all once off jobs without errors" do # Load all once offs Dir[Rails.root + 'app/jobs/onceoff/*.rb'].each do |f| diff --git a/spec/integrity/site_setting_spec.rb b/spec/integrity/site_setting_spec.rb index 51d1755183..8779af4576 100644 --- a/spec/integrity/site_setting_spec.rb +++ b/spec/integrity/site_setting_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe "site setting integrity checks" do +RSpec.describe "site setting integrity checks" do let(:site_setting_file) { File.join(Rails.root, 'config', 'site_settings.yml') } let(:yaml) { YAML.load_file(site_setting_file) } diff --git a/spec/jobs/about_stats_spec.rb b/spec/jobs/about_stats_spec.rb index fd30464d68..3a2e5d6aa9 100644 --- a/spec/jobs/about_stats_spec.rb +++ b/spec/jobs/about_stats_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::AboutStats do +RSpec.describe Jobs::AboutStats do it 'caches the stats' do begin stats = About.fetch_stats.to_json diff --git a/spec/jobs/activation_reminder_emails_spec.rb b/spec/jobs/activation_reminder_emails_spec.rb index 32a59ff9f4..d7956d3dcd 100644 --- a/spec/jobs/activation_reminder_emails_spec.rb +++ b/spec/jobs/activation_reminder_emails_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::ActivationReminderEmails do +RSpec.describe Jobs::ActivationReminderEmails do before { Jobs.run_immediately! } # should be between 2 and 3 days diff --git a/spec/jobs/auto_queue_handler_spec.rb b/spec/jobs/auto_queue_handler_spec.rb index 39c98700c7..ece2af8473 100644 --- a/spec/jobs/auto_queue_handler_spec.rb +++ b/spec/jobs/auto_queue_handler_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::AutoQueueHandler do +RSpec.describe Jobs::AutoQueueHandler do subject { Jobs::AutoQueueHandler.new.execute({}) } diff --git a/spec/jobs/automatic_group_membership_spec.rb b/spec/jobs/automatic_group_membership_spec.rb index dac188f4ee..5ad6140895 100644 --- a/spec/jobs/automatic_group_membership_spec.rb +++ b/spec/jobs/automatic_group_membership_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::AutomaticGroupMembership do +RSpec.describe Jobs::AutomaticGroupMembership do it "raises an error when the group id is missing" do expect { Jobs::AutomaticGroupMembership.new.execute({}) }.to raise_error(Discourse::InvalidParameters) diff --git a/spec/jobs/bulk_grant_trust_level_spec.rb b/spec/jobs/bulk_grant_trust_level_spec.rb index b91dd14282..bfda953eba 100644 --- a/spec/jobs/bulk_grant_trust_level_spec.rb +++ b/spec/jobs/bulk_grant_trust_level_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::BulkGrantTrustLevel do +RSpec.describe Jobs::BulkGrantTrustLevel do it "raises an error when trust_level is missing" do expect { Jobs::BulkGrantTrustLevel.new.execute(user_ids: [1, 2]) }.to raise_error(Discourse::InvalidParameters) diff --git a/spec/jobs/bulk_invite_spec.rb b/spec/jobs/bulk_invite_spec.rb index 66c1cf0377..0bbddc17b4 100644 --- a/spec/jobs/bulk_invite_spec.rb +++ b/spec/jobs/bulk_invite_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::BulkInvite do +RSpec.describe Jobs::BulkInvite do describe '#execute' do fab!(:user) { Fabricate(:user) } fab!(:admin) { Fabricate(:admin) } diff --git a/spec/jobs/bump_topic_spec.rb b/spec/jobs/bump_topic_spec.rb index 0f1359eb08..ed7668d9c5 100644 --- a/spec/jobs/bump_topic_spec.rb +++ b/spec/jobs/bump_topic_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::BumpTopic do +RSpec.describe Jobs::BumpTopic do fab!(:admin) { Fabricate(:admin) } fab!(:user) { Fabricate(:user) } diff --git a/spec/jobs/clean_dismissed_topic_users_spec.rb b/spec/jobs/clean_dismissed_topic_users_spec.rb index b74a534043..187ce7c79d 100644 --- a/spec/jobs/clean_dismissed_topic_users_spec.rb +++ b/spec/jobs/clean_dismissed_topic_users_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::CleanDismissedTopicUsers do +RSpec.describe Jobs::CleanDismissedTopicUsers do fab!(:user) { Fabricate(:user, created_at: 1.days.ago, previous_visit_at: 1.days.ago) } fab!(:topic) { Fabricate(:topic, created_at: 5.hours.ago) } fab!(:dismissed_topic_user) { Fabricate(:dismissed_topic_user, user: user, topic: topic) } diff --git a/spec/jobs/clean_up_associated_accounts_spec.rb b/spec/jobs/clean_up_associated_accounts_spec.rb index 18a81d3439..433c5fcaac 100644 --- a/spec/jobs/clean_up_associated_accounts_spec.rb +++ b/spec/jobs/clean_up_associated_accounts_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::CleanUpAssociatedAccounts do +RSpec.describe Jobs::CleanUpAssociatedAccounts do subject { Jobs::CleanUpAssociatedAccounts.new.execute({}) } it "deletes the correct records" do diff --git a/spec/jobs/clean_up_crawler_stats_spec.rb b/spec/jobs/clean_up_crawler_stats_spec.rb index 48d0d80d94..8f954b4334 100644 --- a/spec/jobs/clean_up_crawler_stats_spec.rb +++ b/spec/jobs/clean_up_crawler_stats_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::CleanUpCrawlerStats do +RSpec.describe Jobs::CleanUpCrawlerStats do subject { Jobs::CleanUpCrawlerStats.new.execute({}) } it "deletes records older than 30 days old" do diff --git a/spec/jobs/clean_up_email_change_requests_spec.rb b/spec/jobs/clean_up_email_change_requests_spec.rb index f8e6de63fe..f7432990c7 100644 --- a/spec/jobs/clean_up_email_change_requests_spec.rb +++ b/spec/jobs/clean_up_email_change_requests_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::CleanUpEmailChangeRequests do +RSpec.describe Jobs::CleanUpEmailChangeRequests do it "deletes records older than 1 month" do very_old = Fabricate(:email_change_request, updated_at: 32.days.ago) yesterday = Fabricate(:email_change_request, updated_at: 1.day.ago) diff --git a/spec/jobs/clean_up_email_logs_spec.rb b/spec/jobs/clean_up_email_logs_spec.rb index 78fcca1228..3d0ec29b09 100644 --- a/spec/jobs/clean_up_email_logs_spec.rb +++ b/spec/jobs/clean_up_email_logs_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::CleanUpEmailLogs do +RSpec.describe Jobs::CleanUpEmailLogs do fab!(:email_log) { Fabricate(:email_log, created_at: 2.years.ago) } fab!(:email_log2) { Fabricate(:email_log, created_at: 2.weeks.ago) } fab!(:email_log3) { Fabricate(:email_log, created_at: 2.days.ago) } diff --git a/spec/jobs/clean_up_uploads_spec.rb b/spec/jobs/clean_up_uploads_spec.rb index d6448995d6..842f3e0d1b 100644 --- a/spec/jobs/clean_up_uploads_spec.rb +++ b/spec/jobs/clean_up_uploads_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::CleanUpUploads do +RSpec.describe Jobs::CleanUpUploads do def fabricate_upload(attributes = {}) Fabricate(:upload, { created_at: 2.hours.ago }.merge(attributes)) diff --git a/spec/jobs/cleanup_imap_sync_log_spec.rb b/spec/jobs/cleanup_imap_sync_log_spec.rb index ca906dc929..2ca22f17f9 100644 --- a/spec/jobs/cleanup_imap_sync_log_spec.rb +++ b/spec/jobs/cleanup_imap_sync_log_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::CleanupImapSyncLog do +RSpec.describe Jobs::CleanupImapSyncLog do let(:job_class) { Jobs::CleanupImapSyncLog.new } it "deletes logs older than RETAIN_LOGS_DAYS" do diff --git a/spec/jobs/close_topic_spec.rb b/spec/jobs/close_topic_spec.rb index 248e74a329..a3b24a82fd 100644 --- a/spec/jobs/close_topic_spec.rb +++ b/spec/jobs/close_topic_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::CloseTopic do +RSpec.describe Jobs::CloseTopic do fab!(:admin) { Fabricate(:admin) } fab!(:topic) do diff --git a/spec/jobs/correct_missing_dualstack_urls_spec.rb b/spec/jobs/correct_missing_dualstack_urls_spec.rb index fe900a9b6a..09c35a3abd 100644 --- a/spec/jobs/correct_missing_dualstack_urls_spec.rb +++ b/spec/jobs/correct_missing_dualstack_urls_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::CorrectMissingDualstackUrls do +RSpec.describe Jobs::CorrectMissingDualstackUrls do it 'corrects the urls' do setup_s3 SiteSetting.s3_region = "us-east-1" diff --git a/spec/jobs/crawl_topic_link_spec.rb b/spec/jobs/crawl_topic_link_spec.rb index bef14cee11..2ced470cff 100644 --- a/spec/jobs/crawl_topic_link_spec.rb +++ b/spec/jobs/crawl_topic_link_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::CrawlTopicLink do +RSpec.describe Jobs::CrawlTopicLink do let(:job) { Jobs::CrawlTopicLink.new } diff --git a/spec/jobs/create_linked_topic_spec.rb b/spec/jobs/create_linked_topic_spec.rb index e8ff674554..9583116469 100644 --- a/spec/jobs/create_linked_topic_spec.rb +++ b/spec/jobs/create_linked_topic_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::CreateLinkedTopic do +RSpec.describe Jobs::CreateLinkedTopic do it "returns when the post cannot be found" do expect { Jobs::CreateLinkedTopic.new.perform(post_id: 1, sync_exec: true) }.not_to raise_error end diff --git a/spec/jobs/create_missing_avatars_spec.rb b/spec/jobs/create_missing_avatars_spec.rb index d4995a331d..cd74e9c034 100644 --- a/spec/jobs/create_missing_avatars_spec.rb +++ b/spec/jobs/create_missing_avatars_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::CreateMissingAvatars do +RSpec.describe Jobs::CreateMissingAvatars do it "runs correctly without crashing" do Jobs::CreateMissingAvatars.new.execute(nil) end diff --git a/spec/jobs/create_recent_post_search_indexes_spec.rb b/spec/jobs/create_recent_post_search_indexes_spec.rb index aea66a761b..fa7e9a931e 100644 --- a/spec/jobs/create_recent_post_search_indexes_spec.rb +++ b/spec/jobs/create_recent_post_search_indexes_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::CreateRecentPostSearchIndexes do +RSpec.describe Jobs::CreateRecentPostSearchIndexes do subject { described_class.new } fab!(:post) do diff --git a/spec/jobs/create_user_reviewable_spec.rb b/spec/jobs/create_user_reviewable_spec.rb index c052987488..1a9634af94 100644 --- a/spec/jobs/create_user_reviewable_spec.rb +++ b/spec/jobs/create_user_reviewable_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::CreateUserReviewable do +RSpec.describe Jobs::CreateUserReviewable do let(:user) { Fabricate(:user) } diff --git a/spec/jobs/dashboard_stats_spec.rb b/spec/jobs/dashboard_stats_spec.rb index 87485406be..02682958d1 100644 --- a/spec/jobs/dashboard_stats_spec.rb +++ b/spec/jobs/dashboard_stats_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe ::Jobs::DashboardStats do +RSpec.describe ::Jobs::DashboardStats do let(:group_message) { GroupMessage.new(Group[:admins].name, :dashboard_problems, limit_once_per: 7.days.to_i) } def clear_recently_sent! diff --git a/spec/jobs/delete_replies_spec.rb b/spec/jobs/delete_replies_spec.rb index 70e1704f53..7bd9e219db 100644 --- a/spec/jobs/delete_replies_spec.rb +++ b/spec/jobs/delete_replies_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::DeleteReplies do +RSpec.describe Jobs::DeleteReplies do fab!(:admin) { Fabricate(:admin) } fab!(:topic) { Fabricate(:topic) } diff --git a/spec/jobs/delete_topic_spec.rb b/spec/jobs/delete_topic_spec.rb index a70323dcf6..a65fb3052b 100644 --- a/spec/jobs/delete_topic_spec.rb +++ b/spec/jobs/delete_topic_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::DeleteTopic do +RSpec.describe Jobs::DeleteTopic do fab!(:admin) { Fabricate(:admin) } fab!(:topic) do diff --git a/spec/jobs/disable_bootstrap_mode_spec.rb b/spec/jobs/disable_bootstrap_mode_spec.rb index 1b9e1cfb7d..3d5b8d4e13 100644 --- a/spec/jobs/disable_bootstrap_mode_spec.rb +++ b/spec/jobs/disable_bootstrap_mode_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::DisableBootstrapMode do +RSpec.describe Jobs::DisableBootstrapMode do describe '.execute' do fab!(:admin) { Fabricate(:admin) } diff --git a/spec/jobs/emit_web_hook_event_spec.rb b/spec/jobs/emit_web_hook_event_spec.rb index 9410c2e2c6..031c98c524 100644 --- a/spec/jobs/emit_web_hook_event_spec.rb +++ b/spec/jobs/emit_web_hook_event_spec.rb @@ -2,7 +2,7 @@ require 'excon' -describe Jobs::EmitWebHookEvent do +RSpec.describe Jobs::EmitWebHookEvent do fab!(:post_hook) { Fabricate(:web_hook) } fab!(:inactive_hook) { Fabricate(:inactive_web_hook) } fab!(:post) { Fabricate(:post) } diff --git a/spec/jobs/enable_bootstrap_mode_spec.rb b/spec/jobs/enable_bootstrap_mode_spec.rb index 29a2284862..583caa041e 100644 --- a/spec/jobs/enable_bootstrap_mode_spec.rb +++ b/spec/jobs/enable_bootstrap_mode_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::EnableBootstrapMode do +RSpec.describe Jobs::EnableBootstrapMode do describe '.execute' do fab!(:admin) { Fabricate(:admin) } diff --git a/spec/jobs/enqueue_digest_emails_spec.rb b/spec/jobs/enqueue_digest_emails_spec.rb index e7ce802680..958c7a5064 100644 --- a/spec/jobs/enqueue_digest_emails_spec.rb +++ b/spec/jobs/enqueue_digest_emails_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::EnqueueDigestEmails do +RSpec.describe Jobs::EnqueueDigestEmails do describe '#target_users' do diff --git a/spec/jobs/enqueue_suspect_users_spec.rb b/spec/jobs/enqueue_suspect_users_spec.rb index 98ad4fc849..d1c52285ba 100644 --- a/spec/jobs/enqueue_suspect_users_spec.rb +++ b/spec/jobs/enqueue_suspect_users_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::EnqueueSuspectUsers do +RSpec.describe Jobs::EnqueueSuspectUsers do before { SiteSetting.approve_suspect_users = true } it 'does nothing when there are no suspect users' do diff --git a/spec/jobs/export_csv_file_spec.rb b/spec/jobs/export_csv_file_spec.rb index 9b6fcb8b03..c126cbc9a8 100644 --- a/spec/jobs/export_csv_file_spec.rb +++ b/spec/jobs/export_csv_file_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::ExportCsvFile do +RSpec.describe Jobs::ExportCsvFile do describe '#execute' do let(:other_user) { Fabricate(:user) } diff --git a/spec/jobs/export_user_archive_spec.rb b/spec/jobs/export_user_archive_spec.rb index 234d26697d..15eb36313a 100644 --- a/spec/jobs/export_user_archive_spec.rb +++ b/spec/jobs/export_user_archive_spec.rb @@ -2,7 +2,7 @@ require 'csv' -describe Jobs::ExportUserArchive do +RSpec.describe Jobs::ExportUserArchive do fab!(:user) { Fabricate(:user, username: "john_doe") } fab!(:user2) { Fabricate(:user) } let(:extra) { {} } diff --git a/spec/jobs/feature_topic_users_spec.rb b/spec/jobs/feature_topic_users_spec.rb index 3d4aa0fe3d..bae3f19a6d 100644 --- a/spec/jobs/feature_topic_users_spec.rb +++ b/spec/jobs/feature_topic_users_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::FeatureTopicUsers do +RSpec.describe Jobs::FeatureTopicUsers do it "raises an error without a topic_id" do expect { Jobs::FeatureTopicUsers.new.execute({}) }.to raise_error(Discourse::InvalidParameters) end diff --git a/spec/jobs/grant_anniversary_badges_spec.rb b/spec/jobs/grant_anniversary_badges_spec.rb index 702e77cb01..e2e2b70c84 100644 --- a/spec/jobs/grant_anniversary_badges_spec.rb +++ b/spec/jobs/grant_anniversary_badges_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::GrantAnniversaryBadges do +RSpec.describe Jobs::GrantAnniversaryBadges do let(:granter) { described_class.new } diff --git a/spec/jobs/grant_new_user_of_the_month_badges_spec.rb b/spec/jobs/grant_new_user_of_the_month_badges_spec.rb index 8eb19a15b0..419f709cad 100644 --- a/spec/jobs/grant_new_user_of_the_month_badges_spec.rb +++ b/spec/jobs/grant_new_user_of_the_month_badges_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::GrantNewUserOfTheMonthBadges do +RSpec.describe Jobs::GrantNewUserOfTheMonthBadges do let(:granter) { described_class.new } diff --git a/spec/jobs/heartbeat_spec.rb b/spec/jobs/heartbeat_spec.rb index b1988bcfe1..f62043e287 100644 --- a/spec/jobs/heartbeat_spec.rb +++ b/spec/jobs/heartbeat_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe ::Jobs::Heartbeat do +RSpec.describe ::Jobs::Heartbeat do after do Discourse.disable_readonly_mode end diff --git a/spec/jobs/ignored_users_summary_spec.rb b/spec/jobs/ignored_users_summary_spec.rb index d111652ecf..1768161a7c 100644 --- a/spec/jobs/ignored_users_summary_spec.rb +++ b/spec/jobs/ignored_users_summary_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::IgnoredUsersSummary do +RSpec.describe Jobs::IgnoredUsersSummary do before do SiteSetting.ignored_users_count_message_threshold = 1 SiteSetting.ignored_users_message_gap_days = 365 diff --git a/spec/jobs/invalidate_inactive_admins_spec.rb b/spec/jobs/invalidate_inactive_admins_spec.rb index 1ec5b7367f..ca84c2e857 100644 --- a/spec/jobs/invalidate_inactive_admins_spec.rb +++ b/spec/jobs/invalidate_inactive_admins_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::InvalidateInactiveAdmins do +RSpec.describe Jobs::InvalidateInactiveAdmins do fab!(:active_admin) { Fabricate(:admin, last_seen_at: 1.hour.ago) } before { active_admin.email_tokens.update_all(confirmed: true) } diff --git a/spec/jobs/invite_email_spec.rb b/spec/jobs/invite_email_spec.rb index e494f327ee..ec4bae6c51 100644 --- a/spec/jobs/invite_email_spec.rb +++ b/spec/jobs/invite_email_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::InviteEmail do +RSpec.describe Jobs::InviteEmail do describe '.execute' do diff --git a/spec/jobs/jobs_base_spec.rb b/spec/jobs/jobs_base_spec.rb index a8030719d6..5225019b03 100644 --- a/spec/jobs/jobs_base_spec.rb +++ b/spec/jobs/jobs_base_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe ::Jobs::Base do +RSpec.describe ::Jobs::Base do class GoodJob < ::Jobs::Base attr_accessor :count def execute(args) diff --git a/spec/jobs/jobs_spec.rb b/spec/jobs/jobs_spec.rb index 09a83d905e..f5ce9e2ccd 100644 --- a/spec/jobs/jobs_spec.rb +++ b/spec/jobs/jobs_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs do +RSpec.describe Jobs do describe 'enqueue' do diff --git a/spec/jobs/mass_award_badge_spec.rb b/spec/jobs/mass_award_badge_spec.rb index 435d4af59d..024014a84a 100644 --- a/spec/jobs/mass_award_badge_spec.rb +++ b/spec/jobs/mass_award_badge_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::MassAwardBadge do +RSpec.describe Jobs::MassAwardBadge do describe '#execute' do fab!(:badge) { Fabricate(:badge) } fab!(:user) { Fabricate(:user) } diff --git a/spec/jobs/migrate_badge_image_to_uploads_spec.rb b/spec/jobs/migrate_badge_image_to_uploads_spec.rb index 43794ee17d..21dfabe780 100644 --- a/spec/jobs/migrate_badge_image_to_uploads_spec.rb +++ b/spec/jobs/migrate_badge_image_to_uploads_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::MigrateBadgeImageToUploads do +RSpec.describe Jobs::MigrateBadgeImageToUploads do let(:image_url) { "https://omg.aws.somestack/test.png" } let(:badge) { Fabricate(:badge) } diff --git a/spec/jobs/notify_mailing_list_subscribers_spec.rb b/spec/jobs/notify_mailing_list_subscribers_spec.rb index 271bc26a45..ceed63d598 100644 --- a/spec/jobs/notify_mailing_list_subscribers_spec.rb +++ b/spec/jobs/notify_mailing_list_subscribers_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::NotifyMailingListSubscribers do +RSpec.describe Jobs::NotifyMailingListSubscribers do fab!(:mailing_list_user) { Fabricate(:user) } diff --git a/spec/jobs/notify_moved_posts_spec.rb b/spec/jobs/notify_moved_posts_spec.rb index 341a261f65..dda5fdbbe6 100644 --- a/spec/jobs/notify_moved_posts_spec.rb +++ b/spec/jobs/notify_moved_posts_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::NotifyMovedPosts do +RSpec.describe Jobs::NotifyMovedPosts do it "raises an error without post_ids" do expect { Jobs::NotifyMovedPosts.new.execute(moved_by_id: 1234) }.to raise_error(Discourse::InvalidParameters) diff --git a/spec/jobs/notify_reviewable_spec.rb b/spec/jobs/notify_reviewable_spec.rb index aff1b32de9..d54c08f704 100644 --- a/spec/jobs/notify_reviewable_spec.rb +++ b/spec/jobs/notify_reviewable_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::NotifyReviewable do +RSpec.describe Jobs::NotifyReviewable do describe '.execute' do fab!(:admin) { Fabricate(:admin, moderator: true) } fab!(:moderator) { Fabricate(:moderator) } diff --git a/spec/jobs/notify_tag_change_spec.rb b/spec/jobs/notify_tag_change_spec.rb index f5bb388319..efe15c959f 100644 --- a/spec/jobs/notify_tag_change_spec.rb +++ b/spec/jobs/notify_tag_change_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe ::Jobs::NotifyTagChange do +RSpec.describe ::Jobs::NotifyTagChange do fab!(:user) { Fabricate(:user) } fab!(:regular_user) { Fabricate(:trust_level_4) } diff --git a/spec/jobs/old_keys_reminder_spec.rb b/spec/jobs/old_keys_reminder_spec.rb index 8459ca1b06..a6d4fb89db 100644 --- a/spec/jobs/old_keys_reminder_spec.rb +++ b/spec/jobs/old_keys_reminder_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::OldKeysReminder do +RSpec.describe Jobs::OldKeysReminder do let!(:google_secret) { SiteSetting.create!(name: 'google_oauth2_client_secret', value: '123', data_type: 1) } let!(:github_secret) { SiteSetting.create!(name: 'github_client_secret', value: '123', data_type: 1) } let!(:api_key) { Fabricate(:api_key, description: 'api key description') } diff --git a/spec/jobs/open_topic_spec.rb b/spec/jobs/open_topic_spec.rb index 27d52cb3fe..9a401db727 100644 --- a/spec/jobs/open_topic_spec.rb +++ b/spec/jobs/open_topic_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::OpenTopic do +RSpec.describe Jobs::OpenTopic do fab!(:admin) { Fabricate(:admin) } fab!(:topic) do diff --git a/spec/jobs/pending_queued_posts_reminder_spec.rb b/spec/jobs/pending_queued_posts_reminder_spec.rb index ccc6961be2..fde2b66e0b 100644 --- a/spec/jobs/pending_queued_posts_reminder_spec.rb +++ b/spec/jobs/pending_queued_posts_reminder_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::PendingQueuedPostsReminder do +RSpec.describe Jobs::PendingQueuedPostsReminder do let(:job) { described_class.new } context "notify_about_queued_posts_after is 0" do diff --git a/spec/jobs/pending_reviewables_reminder_spec.rb b/spec/jobs/pending_reviewables_reminder_spec.rb index 6658e3d6ff..c93928c35d 100644 --- a/spec/jobs/pending_reviewables_reminder_spec.rb +++ b/spec/jobs/pending_reviewables_reminder_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::PendingReviewablesReminder do +RSpec.describe Jobs::PendingReviewablesReminder do let(:job) { described_class.new } def create_flag(created_at) diff --git a/spec/jobs/pending_users_reminder_spec.rb b/spec/jobs/pending_users_reminder_spec.rb index 75f2bd6f04..d3a9c4d011 100644 --- a/spec/jobs/pending_users_reminder_spec.rb +++ b/spec/jobs/pending_users_reminder_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::PendingUsersReminder do +RSpec.describe Jobs::PendingUsersReminder do context 'must_approve_users is true' do before do diff --git a/spec/jobs/periodical_updates_spec.rb b/spec/jobs/periodical_updates_spec.rb index 5541cc181b..f6282fb956 100644 --- a/spec/jobs/periodical_updates_spec.rb +++ b/spec/jobs/periodical_updates_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::PeriodicalUpdates do +RSpec.describe Jobs::PeriodicalUpdates do it "works" do diff --git a/spec/jobs/poll_mailbox_spec.rb b/spec/jobs/poll_mailbox_spec.rb index 791a070bee..1f5e59e764 100644 --- a/spec/jobs/poll_mailbox_spec.rb +++ b/spec/jobs/poll_mailbox_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::PollMailbox do +RSpec.describe Jobs::PollMailbox do let(:poller) { Jobs::PollMailbox.new } diff --git a/spec/jobs/problem_checks_spec.rb b/spec/jobs/problem_checks_spec.rb index a0ccfb43dc..7f254f001d 100644 --- a/spec/jobs/problem_checks_spec.rb +++ b/spec/jobs/problem_checks_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::ProblemChecks do +RSpec.describe Jobs::ProblemChecks do after do Discourse.redis.flushdb AdminDashboardData.reset_problem_checks diff --git a/spec/jobs/process_bulk_invite_emails_spec.rb b/spec/jobs/process_bulk_invite_emails_spec.rb index 7cc1ffc605..a177e244b8 100644 --- a/spec/jobs/process_bulk_invite_emails_spec.rb +++ b/spec/jobs/process_bulk_invite_emails_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::ProcessBulkInviteEmails do +RSpec.describe Jobs::ProcessBulkInviteEmails do describe '#execute' do it 'processes pending invites' do invite = Fabricate(:invite, emailed_status: Invite.emailed_status_types[:bulk_pending]) diff --git a/spec/jobs/process_email_spec.rb b/spec/jobs/process_email_spec.rb index 69ca615b26..eeedaf8ace 100644 --- a/spec/jobs/process_email_spec.rb +++ b/spec/jobs/process_email_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::ProcessEmail do +RSpec.describe Jobs::ProcessEmail do let(:mail) { "From: foo@bar.com\nTo: bar@foo.com\nSubject: FOO BAR\n\nFoo foo bar bar?" } diff --git a/spec/jobs/process_post_spec.rb b/spec/jobs/process_post_spec.rb index ddcfe66355..98d843fcdc 100644 --- a/spec/jobs/process_post_spec.rb +++ b/spec/jobs/process_post_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::ProcessPost do +RSpec.describe Jobs::ProcessPost do it "returns when the post cannot be found" do expect { Jobs::ProcessPost.new.perform(post_id: 1, sync_exec: true) }.not_to raise_error end diff --git a/spec/jobs/process_shelved_notifications_spec.rb b/spec/jobs/process_shelved_notifications_spec.rb index 4f406e46ee..fb100d36da 100644 --- a/spec/jobs/process_shelved_notifications_spec.rb +++ b/spec/jobs/process_shelved_notifications_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::ProcessShelvedNotifications do +RSpec.describe Jobs::ProcessShelvedNotifications do fab!(:user) { Fabricate(:user) } let(:post) { Fabricate(:post) } diff --git a/spec/jobs/pull_hotlinked_images_spec.rb b/spec/jobs/pull_hotlinked_images_spec.rb index a9cfeeec45..c30f5a714b 100644 --- a/spec/jobs/pull_hotlinked_images_spec.rb +++ b/spec/jobs/pull_hotlinked_images_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::PullHotlinkedImages do +RSpec.describe Jobs::PullHotlinkedImages do let(:image_url) { "http://wiki.mozilla.org/images/2/2e/Longcat1.png" } let(:broken_image_url) { "http://wiki.mozilla.org/images/2/2e/Longcat2.png" } let(:large_image_url) { "http://wiki.mozilla.org/images/2/2e/Longcat3.png" } diff --git a/spec/jobs/pull_user_profile_hotlinked_images_spec.rb b/spec/jobs/pull_user_profile_hotlinked_images_spec.rb index c6b7743905..2dbce8437f 100644 --- a/spec/jobs/pull_user_profile_hotlinked_images_spec.rb +++ b/spec/jobs/pull_user_profile_hotlinked_images_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::PullUserProfileHotlinkedImages do +RSpec.describe Jobs::PullUserProfileHotlinkedImages do fab!(:user) { Fabricate(:user) } let(:image_url) { "http://wiki.mozilla.org/images/2/2e/Longcat1.png" } diff --git a/spec/jobs/purge_expired_ignored_users_spec.rb b/spec/jobs/purge_expired_ignored_users_spec.rb index 78ac68bc62..67f74e5331 100644 --- a/spec/jobs/purge_expired_ignored_users_spec.rb +++ b/spec/jobs/purge_expired_ignored_users_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::PurgeExpiredIgnoredUsers do +RSpec.describe Jobs::PurgeExpiredIgnoredUsers do subject { Jobs::PurgeExpiredIgnoredUsers.new.execute({}) } context "with no ignored users" do diff --git a/spec/jobs/regular/bulk_user_title_update_spec.rb b/spec/jobs/regular/bulk_user_title_update_spec.rb index 2f29f500b7..fa6c9b8da1 100644 --- a/spec/jobs/regular/bulk_user_title_update_spec.rb +++ b/spec/jobs/regular/bulk_user_title_update_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::BulkUserTitleUpdate do +RSpec.describe Jobs::BulkUserTitleUpdate do fab!(:badge) { Fabricate(:badge, name: 'Protector of the Realm', allow_title: true) } fab!(:user) { Fabricate(:user) } fab!(:other_user) { Fabricate(:user) } diff --git a/spec/jobs/regular/update_post_uploads_secure_status_spec.rb b/spec/jobs/regular/update_post_uploads_secure_status_spec.rb index 34913bd7a2..f8e2b7be53 100644 --- a/spec/jobs/regular/update_post_uploads_secure_status_spec.rb +++ b/spec/jobs/regular/update_post_uploads_secure_status_spec.rb @@ -2,7 +2,7 @@ require 'rails_helper' -describe Jobs::UpdatePostUploadsSecureStatus do +RSpec.describe Jobs::UpdatePostUploadsSecureStatus do fab!(:post) { Fabricate(:post) } before do diff --git a/spec/jobs/reindex_search_spec.rb b/spec/jobs/reindex_search_spec.rb index 414de794ca..e61f9e71a8 100644 --- a/spec/jobs/reindex_search_spec.rb +++ b/spec/jobs/reindex_search_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::ReindexSearch do +RSpec.describe Jobs::ReindexSearch do before do SearchIndexer.enable Jobs.run_immediately! diff --git a/spec/jobs/remove_banner_spec.rb b/spec/jobs/remove_banner_spec.rb index b9026f1afc..f218d801fc 100644 --- a/spec/jobs/remove_banner_spec.rb +++ b/spec/jobs/remove_banner_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::RemoveBanner do +RSpec.describe Jobs::RemoveBanner do fab!(:topic) { Fabricate(:topic) } fab!(:user) { topic.user } diff --git a/spec/jobs/reviewable_priorities_spec.rb b/spec/jobs/reviewable_priorities_spec.rb index d456322994..9b3ea9f030 100644 --- a/spec/jobs/reviewable_priorities_spec.rb +++ b/spec/jobs/reviewable_priorities_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::ReviewablePriorities do +RSpec.describe Jobs::ReviewablePriorities do it "needs returns 0s with no existing reviewables" do Jobs::ReviewablePriorities.new.execute({}) diff --git a/spec/jobs/send_system_message_spec.rb b/spec/jobs/send_system_message_spec.rb index 5497e78dbf..a21942d01f 100644 --- a/spec/jobs/send_system_message_spec.rb +++ b/spec/jobs/send_system_message_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::SendSystemMessage do +RSpec.describe Jobs::SendSystemMessage do it "raises an error without a user_id" do expect { Jobs::SendSystemMessage.new.execute(message_type: 'welcome_invite') }.to raise_error(Discourse::InvalidParameters) end diff --git a/spec/jobs/suspicious_login_spec.rb b/spec/jobs/suspicious_login_spec.rb index 418374400c..5d0ac1aa0a 100644 --- a/spec/jobs/suspicious_login_spec.rb +++ b/spec/jobs/suspicious_login_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::SuspiciousLogin do +RSpec.describe Jobs::SuspiciousLogin do fab!(:user) { Fabricate(:moderator) } diff --git a/spec/jobs/sync_acls_for_uploads_spec.rb b/spec/jobs/sync_acls_for_uploads_spec.rb index c0c07b3f5d..87d5e42d26 100644 --- a/spec/jobs/sync_acls_for_uploads_spec.rb +++ b/spec/jobs/sync_acls_for_uploads_spec.rb @@ -2,7 +2,7 @@ require 'rails_helper' -describe Jobs::SyncAclsForUploads do +RSpec.describe Jobs::SyncAclsForUploads do let(:upload1) { Fabricate(:upload) } let(:upload2) { Fabricate(:upload) } let(:upload3) { Fabricate(:secure_upload) } diff --git a/spec/jobs/tl3_promotions_spec.rb b/spec/jobs/tl3_promotions_spec.rb index fadb6da3ce..5c43b378e9 100644 --- a/spec/jobs/tl3_promotions_spec.rb +++ b/spec/jobs/tl3_promotions_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::Tl3Promotions do +RSpec.describe Jobs::Tl3Promotions do def create_qualifying_stats(user) user.create_user_stat if user.user_stat.nil? diff --git a/spec/jobs/toggle_topic_closed_spec.rb b/spec/jobs/toggle_topic_closed_spec.rb index 65bc171394..b7c8eacbc4 100644 --- a/spec/jobs/toggle_topic_closed_spec.rb +++ b/spec/jobs/toggle_topic_closed_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::ToggleTopicClosed do +RSpec.describe Jobs::ToggleTopicClosed do fab!(:admin) { Fabricate(:admin) } fab!(:topic) do diff --git a/spec/jobs/truncate_user_flag_stats_spec.rb b/spec/jobs/truncate_user_flag_stats_spec.rb index 7682ff3bfc..3c64369ff2 100644 --- a/spec/jobs/truncate_user_flag_stats_spec.rb +++ b/spec/jobs/truncate_user_flag_stats_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::TruncateUserFlagStats do +RSpec.describe Jobs::TruncateUserFlagStats do fab!(:user) { Fabricate(:user) } fab!(:other_user) { Fabricate(:user) } diff --git a/spec/jobs/unsilence_users_spec.rb b/spec/jobs/unsilence_users_spec.rb index c9af68975b..03e1ca4896 100644 --- a/spec/jobs/unsilence_users_spec.rb +++ b/spec/jobs/unsilence_users_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::UnsilenceUsers do +RSpec.describe Jobs::UnsilenceUsers do it "unsilence users" do user = Fabricate(:user) UserSilencer.silence(user, Discourse.system_user, silenced_till: 2.days.ago) diff --git a/spec/jobs/update_animated_uploads_spec.rb b/spec/jobs/update_animated_uploads_spec.rb index 9541bf9fef..00fc83ef78 100644 --- a/spec/jobs/update_animated_uploads_spec.rb +++ b/spec/jobs/update_animated_uploads_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::UpdateAnimatedUploads do +RSpec.describe Jobs::UpdateAnimatedUploads do let!(:upload) { Fabricate(:upload) } let!(:gif_upload) { Fabricate(:upload, extension: "gif") } diff --git a/spec/jobs/update_gravatar_spec.rb b/spec/jobs/update_gravatar_spec.rb index 98addd9806..4b72fe9527 100644 --- a/spec/jobs/update_gravatar_spec.rb +++ b/spec/jobs/update_gravatar_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::UpdateGravatar do +RSpec.describe Jobs::UpdateGravatar do fab!(:user) { Fabricate(:user) } let(:temp) { Tempfile.new('test') } fab!(:upload) { Fabricate(:upload, user: user) } diff --git a/spec/jobs/update_s3_inventory_spec.rb b/spec/jobs/update_s3_inventory_spec.rb index 457c24d644..ff8e032f04 100644 --- a/spec/jobs/update_s3_inventory_spec.rb +++ b/spec/jobs/update_s3_inventory_spec.rb @@ -2,7 +2,7 @@ require "file_store/s3_store" -describe Jobs::UpdateS3Inventory do +RSpec.describe Jobs::UpdateS3Inventory do before do setup_s3 SiteSetting.s3_upload_bucket = "special-bucket" diff --git a/spec/jobs/user_email_spec.rb b/spec/jobs/user_email_spec.rb index f15b02a7d7..3025fbe474 100644 --- a/spec/jobs/user_email_spec.rb +++ b/spec/jobs/user_email_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Jobs::UserEmail do +RSpec.describe Jobs::UserEmail do before do SiteSetting.email_time_window_mins = 10 diff --git a/spec/lib/admin_confirmation_spec.rb b/spec/lib/admin_confirmation_spec.rb index e604289b27..10e065c603 100644 --- a/spec/lib/admin_confirmation_spec.rb +++ b/spec/lib/admin_confirmation_spec.rb @@ -1,7 +1,7 @@ # frozen_string_literal: true require 'admin_confirmation' -describe AdminConfirmation do +RSpec.describe AdminConfirmation do fab!(:admin) { Fabricate(:admin) } fab!(:user) { Fabricate(:user) } diff --git a/spec/lib/admin_user_index_query_spec.rb b/spec/lib/admin_user_index_query_spec.rb index f9fb86b8d5..d9519d33ea 100644 --- a/spec/lib/admin_user_index_query_spec.rb +++ b/spec/lib/admin_user_index_query_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe AdminUserIndexQuery do +RSpec.describe AdminUserIndexQuery do def real_users(query) query.find_users_query.where('users.id > 0') end diff --git a/spec/lib/archetype_spec.rb b/spec/lib/archetype_spec.rb index d20f1105f2..1f662c7b89 100644 --- a/spec/lib/archetype_spec.rb +++ b/spec/lib/archetype_spec.rb @@ -3,7 +3,7 @@ require 'archetype' -describe Archetype do +RSpec.describe Archetype do context 'default archetype' do diff --git a/spec/lib/auth/default_current_user_provider_spec.rb b/spec/lib/auth/default_current_user_provider_spec.rb index e3e93ee748..b26d91afa9 100644 --- a/spec/lib/auth/default_current_user_provider_spec.rb +++ b/spec/lib/auth/default_current_user_provider_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Auth::DefaultCurrentUserProvider do +RSpec.describe Auth::DefaultCurrentUserProvider do # careful using fab! here is can lead to an erratic test # we want a distinct user object per test so last_seen_at is # handled correctly diff --git a/spec/lib/auth/discord_authenticator_spec.rb b/spec/lib/auth/discord_authenticator_spec.rb index 13ea3a44e0..ab925bedc7 100644 --- a/spec/lib/auth/discord_authenticator_spec.rb +++ b/spec/lib/auth/discord_authenticator_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Auth::DiscordAuthenticator do +RSpec.describe Auth::DiscordAuthenticator do let(:hash) { OmniAuth::AuthHash.new( provider: "facebook", diff --git a/spec/lib/auth/facebook_authenticator_spec.rb b/spec/lib/auth/facebook_authenticator_spec.rb index 93a8778f39..8237d3a67d 100644 --- a/spec/lib/auth/facebook_authenticator_spec.rb +++ b/spec/lib/auth/facebook_authenticator_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Auth::FacebookAuthenticator do +RSpec.describe Auth::FacebookAuthenticator do let(:hash) { { provider: "facebook", diff --git a/spec/lib/auth/github_authenticator_spec.rb b/spec/lib/auth/github_authenticator_spec.rb index 33a9a7ac54..ce2dc26492 100644 --- a/spec/lib/auth/github_authenticator_spec.rb +++ b/spec/lib/auth/github_authenticator_spec.rb @@ -20,7 +20,7 @@ def auth_token_for(user) } end -describe Auth::GithubAuthenticator do +RSpec.describe Auth::GithubAuthenticator do let(:authenticator) { described_class.new } fab!(:user) { Fabricate(:user) } diff --git a/spec/lib/auth/google_oauth2_authenticator_spec.rb b/spec/lib/auth/google_oauth2_authenticator_spec.rb index 76ec7e9d76..e00b51d40c 100644 --- a/spec/lib/auth/google_oauth2_authenticator_spec.rb +++ b/spec/lib/auth/google_oauth2_authenticator_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Auth::GoogleOAuth2Authenticator do +RSpec.describe Auth::GoogleOAuth2Authenticator do it 'does not look up user unless email is verified' do # note, emails that come back from google via omniauth are always valid # this protects against future regressions diff --git a/spec/lib/auth/managed_authenticator_spec.rb b/spec/lib/auth/managed_authenticator_spec.rb index 8a7fbf1844..e279ebf048 100644 --- a/spec/lib/auth/managed_authenticator_spec.rb +++ b/spec/lib/auth/managed_authenticator_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Auth::ManagedAuthenticator do +RSpec.describe Auth::ManagedAuthenticator do let(:authenticator) { Class.new(described_class) do def name diff --git a/spec/lib/auth/omniauth_strategies/discourse_google_oauth2_spec.rb b/spec/lib/auth/omniauth_strategies/discourse_google_oauth2_spec.rb index d2ef65ceb4..db4adc1c26 100644 --- a/spec/lib/auth/omniauth_strategies/discourse_google_oauth2_spec.rb +++ b/spec/lib/auth/omniauth_strategies/discourse_google_oauth2_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Auth::OmniAuthStrategies::DiscourseGoogleOauth2 do +RSpec.describe Auth::OmniAuthStrategies::DiscourseGoogleOauth2 do let(:response_hash) do { email: 'user@domain.com', diff --git a/spec/lib/auth/result_spec.rb b/spec/lib/auth/result_spec.rb index f63e57b2d5..2c9a83e057 100644 --- a/spec/lib/auth/result_spec.rb +++ b/spec/lib/auth/result_spec.rb @@ -1,5 +1,5 @@ # frozen_string_literal: true -describe Auth::Result do +RSpec.describe Auth::Result do fab!(:initial_email) { "initialemail@example.org" } fab!(:initial_username) { "initialusername" } fab!(:initial_name) { "Initial Name" } diff --git a/spec/lib/auth/twitter_authenticator_spec.rb b/spec/lib/auth/twitter_authenticator_spec.rb index 3cf3fe861d..0e2863c3f4 100644 --- a/spec/lib/auth/twitter_authenticator_spec.rb +++ b/spec/lib/auth/twitter_authenticator_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Auth::TwitterAuthenticator do +RSpec.describe Auth::TwitterAuthenticator do it "takes over account if email is supplied" do auth = Auth::TwitterAuthenticator.new diff --git a/spec/lib/backup_restore/backup_file_handler_multisite_spec.rb b/spec/lib/backup_restore/backup_file_handler_multisite_spec.rb index f5c6ac4717..c7d4417093 100644 --- a/spec/lib/backup_restore/backup_file_handler_multisite_spec.rb +++ b/spec/lib/backup_restore/backup_file_handler_multisite_spec.rb @@ -2,7 +2,7 @@ require_relative "shared_context_for_backup_restore" -describe BackupRestore::BackupFileHandler, type: :multisite do +RSpec.describe BackupRestore::BackupFileHandler, type: :multisite do include_context "shared stuff" it "works with old backup file format" do diff --git a/spec/lib/backup_restore/backup_file_handler_spec.rb b/spec/lib/backup_restore/backup_file_handler_spec.rb index 1e0eaeeb58..5b35a6c4ff 100644 --- a/spec/lib/backup_restore/backup_file_handler_spec.rb +++ b/spec/lib/backup_restore/backup_file_handler_spec.rb @@ -2,7 +2,7 @@ require_relative 'shared_context_for_backup_restore' -describe BackupRestore::BackupFileHandler do +RSpec.describe BackupRestore::BackupFileHandler do include_context "shared stuff" it "works with current backup file format" do diff --git a/spec/lib/backup_restore/backuper_spec.rb b/spec/lib/backup_restore/backuper_spec.rb index affb41081a..c66006d631 100644 --- a/spec/lib/backup_restore/backuper_spec.rb +++ b/spec/lib/backup_restore/backuper_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe BackupRestore::Backuper do +RSpec.describe BackupRestore::Backuper do it 'returns a non-empty parameterized title when site title contains unicode' do SiteSetting.title = 'Ɣ' backuper = BackupRestore::Backuper.new(Discourse.system_user.id) diff --git a/spec/lib/backup_restore/database_restorer_multisite_spec.rb b/spec/lib/backup_restore/database_restorer_multisite_spec.rb index dea3029418..ec3fc81954 100644 --- a/spec/lib/backup_restore/database_restorer_multisite_spec.rb +++ b/spec/lib/backup_restore/database_restorer_multisite_spec.rb @@ -2,7 +2,7 @@ require_relative "shared_context_for_backup_restore" -describe BackupRestore::DatabaseRestorer, type: :multisite do +RSpec.describe BackupRestore::DatabaseRestorer, type: :multisite do include_context "shared stuff" let(:current_db) { RailsMultisite::ConnectionManagement.current_db } diff --git a/spec/lib/backup_restore/database_restorer_spec.rb b/spec/lib/backup_restore/database_restorer_spec.rb index 1655ec68ee..1f2de1a8ef 100644 --- a/spec/lib/backup_restore/database_restorer_spec.rb +++ b/spec/lib/backup_restore/database_restorer_spec.rb @@ -2,7 +2,7 @@ require_relative 'shared_context_for_backup_restore' -describe BackupRestore::DatabaseRestorer do +RSpec.describe BackupRestore::DatabaseRestorer do include_context "shared stuff" let(:current_db) { RailsMultisite::ConnectionManagement.current_db } diff --git a/spec/lib/backup_restore/local_backup_store_spec.rb b/spec/lib/backup_restore/local_backup_store_spec.rb index 3d829ceab4..faa01c4f16 100644 --- a/spec/lib/backup_restore/local_backup_store_spec.rb +++ b/spec/lib/backup_restore/local_backup_store_spec.rb @@ -3,7 +3,7 @@ require 'backup_restore/local_backup_store' require_relative 'shared_examples_for_backup_store' -describe BackupRestore::LocalBackupStore do +RSpec.describe BackupRestore::LocalBackupStore do before do @root_directory = Dir.mktmpdir @paths = [] diff --git a/spec/lib/backup_restore/meta_data_handler_spec.rb b/spec/lib/backup_restore/meta_data_handler_spec.rb index fb3229a2ab..6ea18c4837 100644 --- a/spec/lib/backup_restore/meta_data_handler_spec.rb +++ b/spec/lib/backup_restore/meta_data_handler_spec.rb @@ -2,7 +2,7 @@ require_relative 'shared_context_for_backup_restore' -describe BackupRestore::MetaDataHandler do +RSpec.describe BackupRestore::MetaDataHandler do include_context "shared stuff" let!(:backup_filename) { 'discourse-2019-11-18-143242-v20191108000414.tar.gz' } diff --git a/spec/lib/backup_restore/s3_backup_store_spec.rb b/spec/lib/backup_restore/s3_backup_store_spec.rb index 4247001a0a..dbe62f6c46 100644 --- a/spec/lib/backup_restore/s3_backup_store_spec.rb +++ b/spec/lib/backup_restore/s3_backup_store_spec.rb @@ -4,7 +4,7 @@ require 's3_helper' require 'backup_restore/s3_backup_store' require_relative 'shared_examples_for_backup_store' -describe BackupRestore::S3BackupStore do +RSpec.describe BackupRestore::S3BackupStore do before do @s3_client = Aws::S3::Client.new(stub_responses: true) @s3_options = { client: @s3_client } diff --git a/spec/lib/backup_restore/shared_context_for_backup_restore.rb b/spec/lib/backup_restore/shared_context_for_backup_restore.rb index 0fbc8ade15..66047d1bc3 100644 --- a/spec/lib/backup_restore/shared_context_for_backup_restore.rb +++ b/spec/lib/backup_restore/shared_context_for_backup_restore.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -shared_context "shared stuff" do +RSpec.shared_context "shared stuff" do let!(:logger) do Class.new do def log(message, ex = nil); end diff --git a/spec/lib/backup_restore/shared_examples_for_backup_store.rb b/spec/lib/backup_restore/shared_examples_for_backup_store.rb index 21f0a515ba..b65dadd9d8 100644 --- a/spec/lib/backup_restore/shared_examples_for_backup_store.rb +++ b/spec/lib/backup_restore/shared_examples_for_backup_store.rb @@ -1,7 +1,7 @@ # frozen_string_literal: true # rubocop:disable Discourse/OnlyTopLevelMultisiteSpecs -shared_context "backups" do +RSpec.shared_context "backups" do before { create_backups } after { remove_backups } @@ -15,7 +15,7 @@ shared_context "backups" do let(:backup5) { BackupFile.new(filename: "multi-2.tar.gz", size: 19, last_modified: Time.parse("2018-11-27T03:16:54Z")) } end -shared_examples "backup store" do +RSpec.shared_examples "backup store" do it "creates the correct backup store" do expect(store).to be_a(expected_type) end @@ -208,7 +208,7 @@ shared_examples "backup store" do end end -shared_examples "remote backup store" do +RSpec.shared_examples "remote backup store" do it "is a remote store" do expect(store.remote?).to eq(true) end diff --git a/spec/lib/backup_restore/system_interface_multisite_spec.rb b/spec/lib/backup_restore/system_interface_multisite_spec.rb index eeea97336f..6587a71669 100644 --- a/spec/lib/backup_restore/system_interface_multisite_spec.rb +++ b/spec/lib/backup_restore/system_interface_multisite_spec.rb @@ -2,7 +2,7 @@ require_relative "shared_context_for_backup_restore" -describe BackupRestore::SystemInterface, type: :multisite do +RSpec.describe BackupRestore::SystemInterface, type: :multisite do include_context "shared stuff" subject { BackupRestore::SystemInterface.new(logger) } diff --git a/spec/lib/backup_restore/system_interface_spec.rb b/spec/lib/backup_restore/system_interface_spec.rb index edc7405c3c..8c5c295c2f 100644 --- a/spec/lib/backup_restore/system_interface_spec.rb +++ b/spec/lib/backup_restore/system_interface_spec.rb @@ -2,7 +2,7 @@ require_relative 'shared_context_for_backup_restore' -describe BackupRestore::SystemInterface do +RSpec.describe BackupRestore::SystemInterface do include_context "shared stuff" subject { BackupRestore::SystemInterface.new(logger) } diff --git a/spec/lib/backup_restore/uploads_restorer_spec.rb b/spec/lib/backup_restore/uploads_restorer_spec.rb index 02db3c5090..0b55fbeeba 100644 --- a/spec/lib/backup_restore/uploads_restorer_spec.rb +++ b/spec/lib/backup_restore/uploads_restorer_spec.rb @@ -3,7 +3,7 @@ require_relative 'shared_context_for_backup_restore' -describe BackupRestore::UploadsRestorer do +RSpec.describe BackupRestore::UploadsRestorer do include_context "shared stuff" subject { BackupRestore::UploadsRestorer.new(logger) } diff --git a/spec/lib/browser_detection_spec.rb b/spec/lib/browser_detection_spec.rb index e0e5f69521..d0820bc511 100644 --- a/spec/lib/browser_detection_spec.rb +++ b/spec/lib/browser_detection_spec.rb @@ -2,7 +2,7 @@ require 'browser_detection' -describe BrowserDetection do +RSpec.describe BrowserDetection do it "detects browser, device and operating system" do [ diff --git a/spec/lib/cache_spec.rb b/spec/lib/cache_spec.rb index 2a8d77a5ed..19d79d81ba 100644 --- a/spec/lib/cache_spec.rb +++ b/spec/lib/cache_spec.rb @@ -2,7 +2,7 @@ require 'cache' -describe Cache do +RSpec.describe Cache do let :cache do Cache.new diff --git a/spec/lib/category_badge_spec.rb b/spec/lib/category_badge_spec.rb index 46d45c9291..c6d04cead2 100644 --- a/spec/lib/category_badge_spec.rb +++ b/spec/lib/category_badge_spec.rb @@ -2,7 +2,7 @@ require 'category_badge' -describe CategoryBadge do +RSpec.describe CategoryBadge do it "escapes HTML in category names / descriptions" do c = Fabricate(:category, name: 'name', description: 'title') diff --git a/spec/lib/common_passwords/common_passwords_spec.rb b/spec/lib/common_passwords/common_passwords_spec.rb index cd0f9ab7b1..8d3ebf2499 100644 --- a/spec/lib/common_passwords/common_passwords_spec.rb +++ b/spec/lib/common_passwords/common_passwords_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe CommonPasswords do +RSpec.describe CommonPasswords do it "the passwords file should exist" do expect(File.exist?(described_class::PASSWORD_FILE)).to eq(true) end diff --git a/spec/lib/composer_messages_finder_spec.rb b/spec/lib/composer_messages_finder_spec.rb index 735614bf74..e52ff3e73f 100644 --- a/spec/lib/composer_messages_finder_spec.rb +++ b/spec/lib/composer_messages_finder_spec.rb @@ -3,7 +3,7 @@ require 'composer_messages_finder' -describe ComposerMessagesFinder do +RSpec.describe ComposerMessagesFinder do context "delegates work" do let(:user) { Fabricate.build(:user) } diff --git a/spec/lib/compression/engine_spec.rb b/spec/lib/compression/engine_spec.rb index a025eaf043..913c5da072 100644 --- a/spec/lib/compression/engine_spec.rb +++ b/spec/lib/compression/engine_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Compression::Engine do +RSpec.describe Compression::Engine do let(:available_size) { SiteSetting.decompressed_theme_max_file_size_mb } before do diff --git a/spec/lib/concern/cached_counting_spec.rb b/spec/lib/concern/cached_counting_spec.rb index 190fe105b0..44976a3737 100644 --- a/spec/lib/concern/cached_counting_spec.rb +++ b/spec/lib/concern/cached_counting_spec.rb @@ -14,7 +14,7 @@ class TestCachedCounting end end -describe CachedCounting do +RSpec.describe CachedCounting do it "should be default disabled in test" do expect(CachedCounting.enabled?).to eq(false) diff --git a/spec/lib/concern/category_hashtag_spec.rb b/spec/lib/concern/category_hashtag_spec.rb index 957c3c92f0..94bcb3f7d0 100644 --- a/spec/lib/concern/category_hashtag_spec.rb +++ b/spec/lib/concern/category_hashtag_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe CategoryHashtag do +RSpec.describe CategoryHashtag do describe '#query_from_hashtag_slug' do fab!(:parent_category) { Fabricate(:category) } fab!(:child_category) { Fabricate(:category, parent_category: parent_category) } diff --git a/spec/lib/concern/has_custom_fields_spec.rb b/spec/lib/concern/has_custom_fields_spec.rb index 0a8616bd3c..698d82604c 100644 --- a/spec/lib/concern/has_custom_fields_spec.rb +++ b/spec/lib/concern/has_custom_fields_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe HasCustomFields do +RSpec.describe HasCustomFields do context "custom_fields" do before do DB.exec("create temporary table custom_fields_test_items(id SERIAL primary key)") diff --git a/spec/lib/concern/has_search_data_spec.rb b/spec/lib/concern/has_search_data_spec.rb index 72795c3f6f..e0a0583af1 100644 --- a/spec/lib/concern/has_search_data_spec.rb +++ b/spec/lib/concern/has_search_data_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe HasSearchData do +RSpec.describe HasSearchData do context "belongs to its model" do before do DB.exec("create temporary table model_items(id SERIAL primary key)") diff --git a/spec/lib/concern/positionable_spec.rb b/spec/lib/concern/positionable_spec.rb index 76ca109ffa..b491ab1090 100644 --- a/spec/lib/concern/positionable_spec.rb +++ b/spec/lib/concern/positionable_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Positionable do +RSpec.describe Positionable do def positions TestItem.order('position asc, id asc').pluck(:id) diff --git a/spec/lib/concern/searchable_spec.rb b/spec/lib/concern/searchable_spec.rb index 830ea311e4..75e3cafa08 100644 --- a/spec/lib/concern/searchable_spec.rb +++ b/spec/lib/concern/searchable_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Searchable do +RSpec.describe Searchable do context "has search data" do before do DB.exec("create temporary table searchable_records(id SERIAL primary key)") diff --git a/spec/lib/content_buffer_spec.rb b/spec/lib/content_buffer_spec.rb index a604421038..efc8ffe005 100644 --- a/spec/lib/content_buffer_spec.rb +++ b/spec/lib/content_buffer_spec.rb @@ -2,7 +2,7 @@ require 'content_buffer' -describe ContentBuffer do +RSpec.describe ContentBuffer do it "handles deletion across lines properly" do c = ContentBuffer.new("a\nbc\nc") diff --git a/spec/lib/content_security_policy/builder_spec.rb b/spec/lib/content_security_policy/builder_spec.rb index 59e4bb505a..fd0ad5ba10 100644 --- a/spec/lib/content_security_policy/builder_spec.rb +++ b/spec/lib/content_security_policy/builder_spec.rb @@ -1,5 +1,5 @@ # frozen_string_literal: true -describe ContentSecurityPolicy::Builder do +RSpec.describe ContentSecurityPolicy::Builder do let(:builder) { described_class.new(base_url: Discourse.base_url) } describe '#<<' do diff --git a/spec/lib/content_security_policy_spec.rb b/spec/lib/content_security_policy_spec.rb index bba9d45dc3..504b2cdf3d 100644 --- a/spec/lib/content_security_policy_spec.rb +++ b/spec/lib/content_security_policy_spec.rb @@ -1,5 +1,5 @@ # frozen_string_literal: true -describe ContentSecurityPolicy do +RSpec.describe ContentSecurityPolicy do after do DiscoursePluginRegistry.reset! end diff --git a/spec/lib/cooked_post_processor_spec.rb b/spec/lib/cooked_post_processor_spec.rb index 9f327dd986..2dc678159c 100644 --- a/spec/lib/cooked_post_processor_spec.rb +++ b/spec/lib/cooked_post_processor_spec.rb @@ -3,7 +3,7 @@ require "cooked_post_processor" require "file_store/s3_store" -describe CookedPostProcessor do +RSpec.describe CookedPostProcessor do fab!(:upload) { Fabricate(:upload) } let(:upload_path) { Discourse.store.upload_path } diff --git a/spec/lib/crawler_detection_spec.rb b/spec/lib/crawler_detection_spec.rb index 98a31fa8a5..905950a1e6 100644 --- a/spec/lib/crawler_detection_spec.rb +++ b/spec/lib/crawler_detection_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe CrawlerDetection do +RSpec.describe CrawlerDetection do def crawler!(user_agent, via = nil) if (!CrawlerDetection.crawler?(user_agent, via)) diff --git a/spec/lib/current_user_spec.rb b/spec/lib/current_user_spec.rb index 41beb1f0d6..73c27385cd 100644 --- a/spec/lib/current_user_spec.rb +++ b/spec/lib/current_user_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe CurrentUser do +RSpec.describe CurrentUser do it "allows us to lookup a user from our environment" do user = Fabricate(:user, active: true) token = UserAuthToken.generate!(user_id: user.id) diff --git a/spec/lib/directory_helper_spec.rb b/spec/lib/directory_helper_spec.rb index cbbc79ec42..dc5f1f64d4 100644 --- a/spec/lib/directory_helper_spec.rb +++ b/spec/lib/directory_helper_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe DirectoryHelper do +RSpec.describe DirectoryHelper do class DummyClass include DirectoryHelper end diff --git a/spec/lib/discourse_diff_spec.rb b/spec/lib/discourse_diff_spec.rb index 27596393aa..3413dab3e3 100644 --- a/spec/lib/discourse_diff_spec.rb +++ b/spec/lib/discourse_diff_spec.rb @@ -2,7 +2,7 @@ require 'discourse_diff' -describe DiscourseDiff do +RSpec.describe DiscourseDiff do describe "inline_html" do diff --git a/spec/lib/discourse_event_spec.rb b/spec/lib/discourse_event_spec.rb index ee7a0c73e9..b3e49e5f96 100644 --- a/spec/lib/discourse_event_spec.rb +++ b/spec/lib/discourse_event_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe DiscourseEvent do +RSpec.describe DiscourseEvent do describe "#events" do it "defaults to {}" do diff --git a/spec/lib/discourse_hub_spec.rb b/spec/lib/discourse_hub_spec.rb index e42cd4202c..b80edb397b 100644 --- a/spec/lib/discourse_hub_spec.rb +++ b/spec/lib/discourse_hub_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe DiscourseHub do +RSpec.describe DiscourseHub do describe '.discourse_version_check' do it 'should return just return the json that the hub returns' do hub_response = { 'success' => 'OK', 'latest_version' => '0.8.1', 'critical_updates' => false } diff --git a/spec/lib/discourse_js_processor_spec.rb b/spec/lib/discourse_js_processor_spec.rb index dd04584a3b..4d48dc7a1c 100644 --- a/spec/lib/discourse_js_processor_spec.rb +++ b/spec/lib/discourse_js_processor_spec.rb @@ -2,7 +2,7 @@ require 'discourse_js_processor' -describe DiscourseJsProcessor do +RSpec.describe DiscourseJsProcessor do describe 'should_transpile?' do it "returns false for empty strings" do diff --git a/spec/lib/discourse_plugin_registry_spec.rb b/spec/lib/discourse_plugin_registry_spec.rb index a212f9bc02..78c8852670 100644 --- a/spec/lib/discourse_plugin_registry_spec.rb +++ b/spec/lib/discourse_plugin_registry_spec.rb @@ -2,7 +2,7 @@ require 'discourse_plugin_registry' -describe DiscoursePluginRegistry do +RSpec.describe DiscoursePluginRegistry do class TestRegistry < DiscoursePluginRegistry; end diff --git a/spec/lib/discourse_redis_spec.rb b/spec/lib/discourse_redis_spec.rb index 215ba35738..ba037580e7 100644 --- a/spec/lib/discourse_redis_spec.rb +++ b/spec/lib/discourse_redis_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe DiscourseRedis do +RSpec.describe DiscourseRedis do it "ignore_readonly returns nil from a pure exception" do result = DiscourseRedis.ignore_readonly { raise Redis::CommandError.new("READONLY") } expect(result).to eq(nil) diff --git a/spec/lib/discourse_sourcemapping_url_processor_spec.rb b/spec/lib/discourse_sourcemapping_url_processor_spec.rb index 1c348c42ea..6aad3809a8 100644 --- a/spec/lib/discourse_sourcemapping_url_processor_spec.rb +++ b/spec/lib/discourse_sourcemapping_url_processor_spec.rb @@ -2,7 +2,7 @@ require 'discourse_sourcemapping_url_processor' -describe DiscourseSourcemappingUrlProcessor do +RSpec.describe DiscourseSourcemappingUrlProcessor do def process(input) env = Sprockets::Environment.new env.context_class.class_eval do diff --git a/spec/lib/discourse_spec.rb b/spec/lib/discourse_spec.rb index 1438789a3e..3d1401d044 100644 --- a/spec/lib/discourse_spec.rb +++ b/spec/lib/discourse_spec.rb @@ -2,7 +2,7 @@ require 'discourse' -describe Discourse do +RSpec.describe Discourse do before do RailsMultisite::ConnectionManagement.stubs(:current_hostname).returns('foo.com') diff --git a/spec/lib/discourse_tagging_spec.rb b/spec/lib/discourse_tagging_spec.rb index 11b97e8ba1..acf7f821fd 100644 --- a/spec/lib/discourse_tagging_spec.rb +++ b/spec/lib/discourse_tagging_spec.rb @@ -5,7 +5,7 @@ require 'discourse_tagging' # More tests are found in the category_tag_spec integration specs -describe DiscourseTagging do +RSpec.describe DiscourseTagging do fab!(:admin) { Fabricate(:admin) } fab!(:user) { Fabricate(:user) } diff --git a/spec/lib/discourse_updates_spec.rb b/spec/lib/discourse_updates_spec.rb index 59fb8b51bb..140d7e33ca 100644 --- a/spec/lib/discourse_updates_spec.rb +++ b/spec/lib/discourse_updates_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe DiscourseUpdates do +RSpec.describe DiscourseUpdates do def stub_data(latest, missing, critical, updated_at) DiscourseUpdates.latest_version = latest DiscourseUpdates.missing_versions_count = missing diff --git a/spec/lib/distributed_cache_spec.rb b/spec/lib/distributed_cache_spec.rb index 540dee32ce..21ef2a3f51 100644 --- a/spec/lib/distributed_cache_spec.rb +++ b/spec/lib/distributed_cache_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe "DistributedCache extensions" do +RSpec.describe "DistributedCache extensions" do let(:cache) { DistributedCache.new('mytest') } it "can defer_get_set" do diff --git a/spec/lib/distributed_memoizer_spec.rb b/spec/lib/distributed_memoizer_spec.rb index 256716aecc..980a3a6678 100644 --- a/spec/lib/distributed_memoizer_spec.rb +++ b/spec/lib/distributed_memoizer_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe DistributedMemoizer do +RSpec.describe DistributedMemoizer do after do Discourse.redis.del(DistributedMemoizer.redis_key("hello")) Discourse.redis.del(DistributedMemoizer.redis_lock_key("hello")) diff --git a/spec/lib/distributed_mutex_spec.rb b/spec/lib/distributed_mutex_spec.rb index ec6db7f212..aaa25161c7 100644 --- a/spec/lib/distributed_mutex_spec.rb +++ b/spec/lib/distributed_mutex_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe DistributedMutex do +RSpec.describe DistributedMutex do let(:key) { "test_mutex_key" } after do diff --git a/spec/lib/email/authentication_results_spec.rb b/spec/lib/email/authentication_results_spec.rb index 4a7b257a28..3c32eb4dc2 100644 --- a/spec/lib/email/authentication_results_spec.rb +++ b/spec/lib/email/authentication_results_spec.rb @@ -2,7 +2,7 @@ require "email/authentication_results" -describe Email::AuthenticationResults do +RSpec.describe Email::AuthenticationResults do describe "#results" do it "parses 'Nearly Trivial Case: Service Provided, but No Authentication Done' correctly" do # https://tools.ietf.org/html/rfc8601#appendix-B.2 diff --git a/spec/lib/email/cleaner_spec.rb b/spec/lib/email/cleaner_spec.rb index ad9e1a71d2..b4ae09403d 100644 --- a/spec/lib/email/cleaner_spec.rb +++ b/spec/lib/email/cleaner_spec.rb @@ -2,7 +2,7 @@ require "email/receiver" -describe Email::Cleaner do +RSpec.describe Email::Cleaner do it 'removes attachments from raw message' do email = email(:attached_txt_file) diff --git a/spec/lib/email/email_spec.rb b/spec/lib/email/email_spec.rb index 084337a2a3..46f8cd77f9 100644 --- a/spec/lib/email/email_spec.rb +++ b/spec/lib/email/email_spec.rb @@ -2,7 +2,7 @@ require 'email' -describe Email do +RSpec.describe Email do describe "is_valid?" do diff --git a/spec/lib/email/message_builder_spec.rb b/spec/lib/email/message_builder_spec.rb index 27a16d3f88..0f0c795a19 100644 --- a/spec/lib/email/message_builder_spec.rb +++ b/spec/lib/email/message_builder_spec.rb @@ -2,7 +2,7 @@ require 'email/message_builder' -describe Email::MessageBuilder do +RSpec.describe Email::MessageBuilder do let(:to_address) { "jake@adventuretime.ooo" } let(:subject) { "Tree Trunks has made some apple pie!" } diff --git a/spec/lib/email/processor_spec.rb b/spec/lib/email/processor_spec.rb index 931aff5b11..c0a52ffd1b 100644 --- a/spec/lib/email/processor_spec.rb +++ b/spec/lib/email/processor_spec.rb @@ -2,7 +2,7 @@ require "email/processor" -describe Email::Processor do +RSpec.describe Email::Processor do after do Discourse.redis.flushdb end diff --git a/spec/lib/email/receiver_spec.rb b/spec/lib/email/receiver_spec.rb index 2a12c0d1a1..de77d2acd1 100644 --- a/spec/lib/email/receiver_spec.rb +++ b/spec/lib/email/receiver_spec.rb @@ -2,7 +2,7 @@ require "email/receiver" -describe Email::Receiver do +RSpec.describe Email::Receiver do before do SiteSetting.email_in = true diff --git a/spec/lib/email/renderer_spec.rb b/spec/lib/email/renderer_spec.rb index 78df067afd..8e74115486 100644 --- a/spec/lib/email/renderer_spec.rb +++ b/spec/lib/email/renderer_spec.rb @@ -2,7 +2,7 @@ require 'email/renderer' -describe Email::Renderer do +RSpec.describe Email::Renderer do let(:message) do mail = Mail.new diff --git a/spec/lib/email/sender_spec.rb b/spec/lib/email/sender_spec.rb index 1dea67595f..92c29a9423 100644 --- a/spec/lib/email/sender_spec.rb +++ b/spec/lib/email/sender_spec.rb @@ -2,7 +2,7 @@ require 'email/sender' -describe Email::Sender do +RSpec.describe Email::Sender do before do SiteSetting.secure_media_allow_embed_images_in_emails = false end diff --git a/spec/lib/email/styles_spec.rb b/spec/lib/email/styles_spec.rb index 115ca404ef..a150ea8176 100644 --- a/spec/lib/email/styles_spec.rb +++ b/spec/lib/email/styles_spec.rb @@ -2,7 +2,7 @@ require 'email' -describe Email::Styles do +RSpec.describe Email::Styles do let(:attachments) { {} } def basic_fragment(html) diff --git a/spec/lib/email_cook_spec.rb b/spec/lib/email_cook_spec.rb index 128a23502b..19ac9c4f38 100644 --- a/spec/lib/email_cook_spec.rb +++ b/spec/lib/email_cook_spec.rb @@ -3,7 +3,7 @@ require 'email_cook' require 'pretty_text' -describe EmailCook do +RSpec.describe EmailCook do it "uses to PrettyText when there is no [plaintext] in raw" do raw = "**Hello world!**" expect(cook(raw)).to eq(PrettyText.cook(raw)) diff --git a/spec/lib/email_updater_spec.rb b/spec/lib/email_updater_spec.rb index a407383802..7416fd9127 100644 --- a/spec/lib/email_updater_spec.rb +++ b/spec/lib/email_updater_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe EmailUpdater do +RSpec.describe EmailUpdater do let(:old_email) { 'old.email@example.com' } let(:new_email) { 'new.email@example.com' } diff --git a/spec/lib/encodings_spec.rb b/spec/lib/encodings_spec.rb index 5853e33a0c..be94d5a8e1 100644 --- a/spec/lib/encodings_spec.rb +++ b/spec/lib/encodings_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Encodings do +RSpec.describe Encodings do def to_utf8(filename) string = File.read("#{Rails.root}/spec/fixtures/encodings/#{filename}").chomp Encodings.to_utf8(string) diff --git a/spec/lib/enum_spec.rb b/spec/lib/enum_spec.rb index c99450daea..99db4f1703 100644 --- a/spec/lib/enum_spec.rb +++ b/spec/lib/enum_spec.rb @@ -2,7 +2,7 @@ require 'email' -describe Enum do +RSpec.describe Enum do let(:array_enum) { Enum.new(:jake, :finn, :princess_bubblegum, :peppermint_butler) } let(:hash_enum) { Enum.new(jake: 1, finn: 2, princess_bubblegum: 3, peppermint_butler: 4) } diff --git a/spec/lib/excerpt_parser_spec.rb b/spec/lib/excerpt_parser_spec.rb index d1b032c7f5..5e4d57c293 100644 --- a/spec/lib/excerpt_parser_spec.rb +++ b/spec/lib/excerpt_parser_spec.rb @@ -2,7 +2,7 @@ require "excerpt_parser" -describe ExcerptParser do +RSpec.describe ExcerptParser do it "handles nested
blocks" do html = <<~HTML.strip diff --git a/spec/lib/feed_element_installer_spec.rb b/spec/lib/feed_element_installer_spec.rb index 4ca30dbeb8..4e327d41d0 100644 --- a/spec/lib/feed_element_installer_spec.rb +++ b/spec/lib/feed_element_installer_spec.rb @@ -1,7 +1,7 @@ # frozen_string_literal: true require 'feed_element_installer' -describe FeedElementInstaller do +RSpec.describe FeedElementInstaller do describe '#install_rss_element' do let(:raw_feed) { file_from_fixtures('feed.rss', 'feed').read } diff --git a/spec/lib/feed_item_accessor_spec.rb b/spec/lib/feed_item_accessor_spec.rb index 30293f0e83..c86baacecc 100644 --- a/spec/lib/feed_item_accessor_spec.rb +++ b/spec/lib/feed_item_accessor_spec.rb @@ -2,7 +2,7 @@ require 'rss' require 'feed_item_accessor' -describe FeedItemAccessor do +RSpec.describe FeedItemAccessor do context 'for ATOM feed' do let(:atom_feed) { RSS::Parser.parse(file_from_fixtures('feed.atom', 'feed'), false) } let(:atom_feed_item) { atom_feed.items.first } diff --git a/spec/lib/file_helper_spec.rb b/spec/lib/file_helper_spec.rb index 54c2233a3a..e8bc0c3ef5 100644 --- a/spec/lib/file_helper_spec.rb +++ b/spec/lib/file_helper_spec.rb @@ -2,7 +2,7 @@ require 'file_helper' -describe FileHelper do +RSpec.describe FileHelper do let(:url) { "https://eviltrout.com/trout.png" } let(:png) { File.read("#{Rails.root}/spec/fixtures/images/cropped.png") } diff --git a/spec/lib/file_store/local_store_spec.rb b/spec/lib/file_store/local_store_spec.rb index 7a0fb17518..0ed98ef01e 100644 --- a/spec/lib/file_store/local_store_spec.rb +++ b/spec/lib/file_store/local_store_spec.rb @@ -2,7 +2,7 @@ require 'file_store/local_store' -describe FileStore::LocalStore do +RSpec.describe FileStore::LocalStore do let(:store) { FileStore::LocalStore.new } diff --git a/spec/lib/file_store/s3_store_spec.rb b/spec/lib/file_store/s3_store_spec.rb index 346ad44000..bfebbe7531 100644 --- a/spec/lib/file_store/s3_store_spec.rb +++ b/spec/lib/file_store/s3_store_spec.rb @@ -3,7 +3,7 @@ require 'file_store/s3_store' require 'file_store/local_store' -describe FileStore::S3Store do +RSpec.describe FileStore::S3Store do let(:store) { FileStore::S3Store.new } let(:s3_helper) { store.s3_helper } let(:client) { Aws::S3::Client.new(stub_responses: true) } diff --git a/spec/lib/filter_best_posts_spec.rb b/spec/lib/filter_best_posts_spec.rb index a536b6cce0..91aeddd55a 100644 --- a/spec/lib/filter_best_posts_spec.rb +++ b/spec/lib/filter_best_posts_spec.rb @@ -3,7 +3,7 @@ require 'filter_best_posts' require 'topic_view' -describe FilterBestPosts do +RSpec.describe FilterBestPosts do fab!(:topic) { Fabricate(:topic) } fab!(:coding_horror) { Fabricate(:coding_horror) } diff --git a/spec/lib/final_destination_spec.rb b/spec/lib/final_destination_spec.rb index 1e44823709..2782f7b894 100644 --- a/spec/lib/final_destination_spec.rb +++ b/spec/lib/final_destination_spec.rb @@ -2,7 +2,7 @@ require 'final_destination' -describe FinalDestination do +RSpec.describe FinalDestination do let(:opts) do { diff --git a/spec/lib/freedom_patches/mail_disable_starttls_spec.rb b/spec/lib/freedom_patches/mail_disable_starttls_spec.rb index 454c037255..86aa6b87bd 100644 --- a/spec/lib/freedom_patches/mail_disable_starttls_spec.rb +++ b/spec/lib/freedom_patches/mail_disable_starttls_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe FreedomPatches::MailDisableStarttls do +RSpec.describe FreedomPatches::MailDisableStarttls do subject(:smtp_session) { smtp.build_smtp_session } let(:smtp) { Mail::SMTP.new(options) } @@ -9,7 +9,7 @@ describe FreedomPatches::MailDisableStarttls do let(:options) { {} } it "doesn't disable starttls" do - expect(smtp_session).to be_starttls + expect(smtp_session.starttls?).to eq(:auto) end end @@ -17,7 +17,7 @@ describe FreedomPatches::MailDisableStarttls do let(:options) { { enable_starttls_auto: false } } it "properly disables starttls" do - expect(smtp_session).not_to be_starttls + expect(smtp_session.starttls?).to eq(false) end end end diff --git a/spec/lib/freedom_patches/safe_buffer_spec.rb b/spec/lib/freedom_patches/safe_buffer_spec.rb index 0ddc769268..6e6542889b 100644 --- a/spec/lib/freedom_patches/safe_buffer_spec.rb +++ b/spec/lib/freedom_patches/safe_buffer_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe ActiveSupport::SafeBuffer do +RSpec.describe ActiveSupport::SafeBuffer do it "can cope with encoding weirdness" do buffer = ActiveSupport::SafeBuffer.new buffer << (+"\330").force_encoding("ASCII-8BIT") diff --git a/spec/lib/freedom_patches/schema_migration_details_spec.rb b/spec/lib/freedom_patches/schema_migration_details_spec.rb index 5812e86551..8a8317d703 100644 --- a/spec/lib/freedom_patches/schema_migration_details_spec.rb +++ b/spec/lib/freedom_patches/schema_migration_details_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe FreedomPatches::SchemaMigrationDetails do +RSpec.describe FreedomPatches::SchemaMigrationDetails do # we usually don't really need this model so lets not clutter up with it class SchemaMigrationDetail < ActiveRecord::Base diff --git a/spec/lib/freedom_patches/seed_fu_spec.rb b/spec/lib/freedom_patches/seed_fu_spec.rb index 80fd3e79f6..12ba1dd745 100644 --- a/spec/lib/freedom_patches/seed_fu_spec.rb +++ b/spec/lib/freedom_patches/seed_fu_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe "seed-fu patch" do +RSpec.describe "seed-fu patch" do it "does not modify a sequence on an existing table" do u = User.create!(username: "test1", email: "test1@example.com") uid1 = u.id diff --git a/spec/lib/freedom_patches/translate_accelerator_spec.rb b/spec/lib/freedom_patches/translate_accelerator_spec.rb index f6c67c7878..4903b066af 100644 --- a/spec/lib/freedom_patches/translate_accelerator_spec.rb +++ b/spec/lib/freedom_patches/translate_accelerator_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe "translate accelerator" do +RSpec.describe "translate accelerator" do before do @original_i18n_load_path = I18n.load_path.dup I18n.load_path += Dir["#{Rails.root}/spec/fixtures/i18n/translate_accelerator.*.yml"] diff --git a/spec/lib/gaps_spec.rb b/spec/lib/gaps_spec.rb index cf270d7f47..9cd2cbfb7f 100644 --- a/spec/lib/gaps_spec.rb +++ b/spec/lib/gaps_spec.rb @@ -2,7 +2,7 @@ require 'cache' -describe Gaps do +RSpec.describe Gaps do it 'returns no gaps for empty data' do expect(Gaps.new(nil, nil)).to be_blank diff --git a/spec/lib/global_path_spec.rb b/spec/lib/global_path_spec.rb index 9d3f24461f..40c063817e 100644 --- a/spec/lib/global_path_spec.rb +++ b/spec/lib/global_path_spec.rb @@ -6,7 +6,7 @@ class GlobalPathInstance extend GlobalPath end -describe GlobalPath do +RSpec.describe GlobalPath do context 'cdn_relative_path' do def cdn_relative_path(p) diff --git a/spec/lib/group_email_credentials_check_spec.rb b/spec/lib/group_email_credentials_check_spec.rb index 4952a94c1c..074b540285 100644 --- a/spec/lib/group_email_credentials_check_spec.rb +++ b/spec/lib/group_email_credentials_check_spec.rb @@ -3,7 +3,7 @@ require 'net/smtp' require 'net/imap' -describe GroupEmailCredentialsCheck do +RSpec.describe GroupEmailCredentialsCheck do fab!(:group1) { Fabricate(:group) } fab!(:group2) { Fabricate(:smtp_group) } fab!(:group3) { Fabricate(:imap_group) } diff --git a/spec/lib/guardian/topic_guardian_spec.rb b/spec/lib/guardian/topic_guardian_spec.rb index 538928ea46..24ef2e0332 100644 --- a/spec/lib/guardian/topic_guardian_spec.rb +++ b/spec/lib/guardian/topic_guardian_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe TopicGuardian do +RSpec.describe TopicGuardian do fab!(:admin) { Fabricate(:admin) } fab!(:tl3_user) { Fabricate(:leader) } fab!(:moderator) { Fabricate(:moderator) } diff --git a/spec/lib/guardian/user_guardian_spec.rb b/spec/lib/guardian/user_guardian_spec.rb index 70ef35f289..9015e8d554 100644 --- a/spec/lib/guardian/user_guardian_spec.rb +++ b/spec/lib/guardian/user_guardian_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe UserGuardian do +RSpec.describe UserGuardian do let :user do Fabricate(:user) diff --git a/spec/lib/guardian_spec.rb b/spec/lib/guardian_spec.rb index 2d0a3ef627..833a19e80c 100644 --- a/spec/lib/guardian_spec.rb +++ b/spec/lib/guardian_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Guardian do +RSpec.describe Guardian do fab!(:user) { Fabricate(:user) } fab!(:another_user) { Fabricate(:user) } diff --git a/spec/lib/has_errors_spec.rb b/spec/lib/has_errors_spec.rb index 1e0b63b813..436039706c 100644 --- a/spec/lib/has_errors_spec.rb +++ b/spec/lib/has_errors_spec.rb @@ -2,7 +2,7 @@ require 'has_errors' -describe HasErrors do +RSpec.describe HasErrors do class ErrorTestClass include HasErrors diff --git a/spec/lib/highlight_js/highlight_js_spec.rb b/spec/lib/highlight_js/highlight_js_spec.rb index 8c8da79a7f..d3fd705886 100644 --- a/spec/lib/highlight_js/highlight_js_spec.rb +++ b/spec/lib/highlight_js/highlight_js_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe HighlightJs do +RSpec.describe HighlightJs do it 'can list languages' do expect(HighlightJs.languages).to include('thrift') end diff --git a/spec/lib/hijack_spec.rb b/spec/lib/hijack_spec.rb index beb8b8622a..c05985f105 100644 --- a/spec/lib/hijack_spec.rb +++ b/spec/lib/hijack_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Hijack do +RSpec.describe Hijack do class Hijack::Tester < ApplicationController attr_reader :io diff --git a/spec/lib/html_prettify_spec.rb b/spec/lib/html_prettify_spec.rb index bea24b271b..e9bd6c5138 100644 --- a/spec/lib/html_prettify_spec.rb +++ b/spec/lib/html_prettify_spec.rb @@ -2,7 +2,7 @@ require 'html_prettify' -describe HtmlPrettify do +RSpec.describe HtmlPrettify do def t(source, expected) expect(HtmlPrettify.render(source)).to eq(expected) diff --git a/spec/lib/html_to_markdown_spec.rb b/spec/lib/html_to_markdown_spec.rb index 23ac121c88..0666688077 100644 --- a/spec/lib/html_to_markdown_spec.rb +++ b/spec/lib/html_to_markdown_spec.rb @@ -2,7 +2,7 @@ require 'html_to_markdown' -describe HtmlToMarkdown do +RSpec.describe HtmlToMarkdown do def html_to_markdown(html, opts = {}) HtmlToMarkdown.new(html, opts).to_markdown diff --git a/spec/lib/i18n/discourse_i18n_spec.rb b/spec/lib/i18n/discourse_i18n_spec.rb index 001c4aadb9..3015edce55 100644 --- a/spec/lib/i18n/discourse_i18n_spec.rb +++ b/spec/lib/i18n/discourse_i18n_spec.rb @@ -3,7 +3,7 @@ require 'i18n/backend/discourse_i18n' require 'translation_override' -describe I18n::Backend::DiscourseI18n do +RSpec.describe I18n::Backend::DiscourseI18n do let(:backend) { I18n::Backend::DiscourseI18n.new } diff --git a/spec/lib/i18n/fallback_locale_list_spec.rb b/spec/lib/i18n/fallback_locale_list_spec.rb index 4951d7761b..900bf37025 100644 --- a/spec/lib/i18n/fallback_locale_list_spec.rb +++ b/spec/lib/i18n/fallback_locale_list_spec.rb @@ -2,7 +2,7 @@ require 'i18n/backend/fallback_locale_list' -describe I18n::Backend::FallbackLocaleList do +RSpec.describe I18n::Backend::FallbackLocaleList do let(:list) { I18n::Backend::FallbackLocaleList.new } it "works when default_locale is English" do diff --git a/spec/lib/image_sizer_spec.rb b/spec/lib/image_sizer_spec.rb index cd378a2531..483177d32f 100644 --- a/spec/lib/image_sizer_spec.rb +++ b/spec/lib/image_sizer_spec.rb @@ -2,7 +2,7 @@ require 'image_sizer' -describe ImageSizer do +RSpec.describe ImageSizer do before do SiteSetting.max_image_width = 500 diff --git a/spec/lib/imap/sync_spec.rb b/spec/lib/imap/sync_spec.rb index d5ff2d0245..d7f66fd052 100644 --- a/spec/lib/imap/sync_spec.rb +++ b/spec/lib/imap/sync_spec.rb @@ -2,7 +2,7 @@ require 'imap/sync' -describe Imap::Sync do +RSpec.describe Imap::Sync do before do SiteSetting.tagging_enabled = true diff --git a/spec/lib/import/normalize_spec.rb b/spec/lib/import/normalize_spec.rb index 724e7ae317..77b483fa58 100644 --- a/spec/lib/import/normalize_spec.rb +++ b/spec/lib/import/normalize_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Import::Normalize do +RSpec.describe Import::Normalize do describe "#normalize_code_blocks" do it "normalizes 2 code blocks correctly" do markdown = < "http://test.com", diff --git a/spec/lib/migration/safe_migrate_spec.rb b/spec/lib/migration/safe_migrate_spec.rb index 5ab63616c5..a0c8e736cd 100644 --- a/spec/lib/migration/safe_migrate_spec.rb +++ b/spec/lib/migration/safe_migrate_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Migration::SafeMigrate do +RSpec.describe Migration::SafeMigrate do before do Migration::SafeMigrate::SafeMigration.disable_safe! end diff --git a/spec/lib/migration/table_dropper_spec.rb b/spec/lib/migration/table_dropper_spec.rb index dc5397e141..535e67388a 100644 --- a/spec/lib/migration/table_dropper_spec.rb +++ b/spec/lib/migration/table_dropper_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Migration::TableDropper do +RSpec.describe Migration::TableDropper do def table_exists?(table_name) DB.exec(<<~SQL) > 0 diff --git a/spec/lib/mini_sql_multisite_connection_spec.rb b/spec/lib/mini_sql_multisite_connection_spec.rb index 6072593ded..2a30ac161b 100644 --- a/spec/lib/mini_sql_multisite_connection_spec.rb +++ b/spec/lib/mini_sql_multisite_connection_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe MiniSqlMultisiteConnection do +RSpec.describe MiniSqlMultisiteConnection do describe "after_commit" do it "works for 'fake' (joinable) transactions" do diff --git a/spec/lib/mobile_detection_spec.rb b/spec/lib/mobile_detection_spec.rb index e1b8ac2ce1..5267383150 100644 --- a/spec/lib/mobile_detection_spec.rb +++ b/spec/lib/mobile_detection_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe MobileDetection do +RSpec.describe MobileDetection do let :old_user_agents do (<<~STR).split("\n") Mozilla/5.0 (Linux; U; Android 4.0.3; en-us; LG-L160L Build/IML74K) AppleWebkit/534.30 (KHTML, like Gecko) Version/4.0 Mobile Safari/534.30 diff --git a/spec/lib/new_post_manager_spec.rb b/spec/lib/new_post_manager_spec.rb index 1fd2785cce..4241ab72b5 100644 --- a/spec/lib/new_post_manager_spec.rb +++ b/spec/lib/new_post_manager_spec.rb @@ -2,7 +2,7 @@ require 'new_post_manager' -describe NewPostManager do +RSpec.describe NewPostManager do fab!(:user) { Fabricate(:user) } fab!(:topic) { Fabricate(:topic) } diff --git a/spec/lib/new_post_result_spec.rb b/spec/lib/new_post_result_spec.rb index ad1f94f918..9e8b2335b5 100644 --- a/spec/lib/new_post_result_spec.rb +++ b/spec/lib/new_post_result_spec.rb @@ -2,7 +2,7 @@ require 'new_post_result' -describe NewPostResult do +RSpec.describe NewPostResult do it "fails by default" do result = NewPostResult.new(:eviltrout) diff --git a/spec/lib/onebox/domain_checker_spec.rb b/spec/lib/onebox/domain_checker_spec.rb index aa4bd04338..de3d71050a 100644 --- a/spec/lib/onebox/domain_checker_spec.rb +++ b/spec/lib/onebox/domain_checker_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Onebox::DomainChecker do +RSpec.describe Onebox::DomainChecker do describe '.is_blocked?' do before do SiteSetting.blocked_onebox_domains = "api.cat.org|kitten.cloud" diff --git a/spec/lib/onebox/engine/allowlisted_generic_onebox_spec.rb b/spec/lib/onebox/engine/allowlisted_generic_onebox_spec.rb index e4dd0e905d..23c1733491 100644 --- a/spec/lib/onebox/engine/allowlisted_generic_onebox_spec.rb +++ b/spec/lib/onebox/engine/allowlisted_generic_onebox_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Onebox::Engine::AllowlistedGenericOnebox do +RSpec.describe Onebox::Engine::AllowlistedGenericOnebox do describe ".===" do it "matches any domain" do expect(described_class === URI('http://foo.bar/resource')).to be(true) diff --git a/spec/lib/onebox/engine/amazon_onebox_spec.rb b/spec/lib/onebox/engine/amazon_onebox_spec.rb index a4a7800334..f9be9bc696 100644 --- a/spec/lib/onebox/engine/amazon_onebox_spec.rb +++ b/spec/lib/onebox/engine/amazon_onebox_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Onebox::Engine::AmazonOnebox do +RSpec.describe Onebox::Engine::AmazonOnebox do context "regular amazon page" do before do @link = "https://www.amazon.com/Knit-Noro-Accessories-Colorful-Little/dp/193609620X" diff --git a/spec/lib/onebox/engine/animated_image_onebox_spec.rb b/spec/lib/onebox/engine/animated_image_onebox_spec.rb index 49796b91fe..049ce7e793 100644 --- a/spec/lib/onebox/engine/animated_image_onebox_spec.rb +++ b/spec/lib/onebox/engine/animated_image_onebox_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Onebox::Engine::AnimatedImageOnebox do +RSpec.describe Onebox::Engine::AnimatedImageOnebox do let(:giphy) { "http://gph.is/15bRbWf" } let(:direct_gif) { "https://media4.giphy.com/media/Zatyu5LBO2zCyhiAAs/giphy.gif" } let(:tenor) { "https://tenor.com/bb3fQ.gif" } diff --git a/spec/lib/onebox/engine/audio_onebox_spec.rb b/spec/lib/onebox/engine/audio_onebox_spec.rb index a7054f908d..cf3a3a9257 100644 --- a/spec/lib/onebox/engine/audio_onebox_spec.rb +++ b/spec/lib/onebox/engine/audio_onebox_spec.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -describe Onebox::Engine::AudioOnebox do +RSpec.describe Onebox::Engine::AudioOnebox do it "supports ogg" do expect(Onebox.preview('http://upload.wikimedia.org/wikipedia/commons/c/c8/Example.ogg').to_s).to match(/