Add migration for better_together_metrics_rich_text_links and update schema; include WebMock for testing

Author: rsmithlal

db/migrate/20241125190948_create_better_together_metrics_rich_text_links.rb

@@ -4,6 +4,8 @@
 # associations between ActionText content and discovered links.
 class CreateBetterTogetherMetricsRichTextLinks < ActiveRecord::Migration[7.1]
   def change
+    return if table_exists? :better_together_metrics_rich_text_links
+
     create_bt_table :rich_text_links, prefix: :better_together_metrics do |t|
       t.bt_references :link, foreign_key: { to_table: :better_together_content_links }
       t.bt_references :rich_text, foreign_key: { to_table: :action_text_rich_texts }

spec/dummy/db/schema.rb

@@ -348,6 +348,28 @@
     t.index ["privacy"], name: "by_better_together_content_blocks_privacy"
   end
 
+  create_table "better_together_content_links", id: :uuid, default: -> { "gen_random_uuid()" }, force: :cascade do |t|
+    t.integer "lock_version", default: 0, null: false
+    t.datetime "created_at", null: false
+    t.datetime "updated_at", null: false
+    t.string "link_type", null: false
+    t.string "url", null: false
+    t.string "scheme"
+    t.string "host"
+    t.boolean "external"
+    t.boolean "valid_link"
+    t.datetime "last_checked_at"
+    t.string "latest_status_code"
+    t.text "error_message"
+    t.index ["external"], name: "index_better_together_content_links_on_external"
+    t.index ["host"], name: "index_better_together_content_links_on_host"
+    t.index ["last_checked_at"], name: "index_better_together_content_links_on_last_checked_at"
+    t.index ["latest_status_code"], name: "index_better_together_content_links_on_latest_status_code"
+    t.index ["link_type"], name: "index_better_together_content_links_on_link_type"
+    t.index ["url"], name: "index_better_together_content_links_on_url"
+    t.index ["valid_link"], name: "index_better_together_content_links_on_valid_link"
+  end
+
   create_table "better_together_content_page_blocks", id: :uuid, default: -> { "gen_random_uuid()" }, force: :cascade do |t|
     t.integer "lock_version", default: 0, null: false
     t.datetime "created_at", null: false
@@ -871,6 +893,20 @@
     t.index ["pageable_type", "pageable_id"], name: "index_better_together_metrics_page_views_on_pageable"
   end
 
+  create_table "better_together_metrics_rich_text_links", id: :uuid, default: -> { "gen_random_uuid()" }, force: :cascade do |t|
+    t.integer "lock_version", default: 0, null: false
+    t.datetime "created_at", null: false
+    t.datetime "updated_at", null: false
+    t.uuid "rich_text_id", null: false
+    t.string "url", null: false
+    t.string "link_type", null: false
+    t.boolean "external", null: false
+    t.boolean "valid", default: false
+    t.string "host"
+    t.text "error_message"
+    t.index ["rich_text_id"], name: "index_better_together_metrics_rich_text_links_on_rich_text_id"
+  end
+
   create_table "better_together_metrics_search_queries", id: :uuid, default: -> { "gen_random_uuid()" }, force: :cascade do |t|
     t.integer "lock_version", default: 0, null: false
     t.datetime "created_at", null: false
@@ -1097,7 +1133,6 @@
     t.boolean "protected", default: false, null: false
     t.uuid "community_id", null: false
     t.string "privacy", limit: 50, default: "private", null: false
-    t.string "slug"
     t.string "url", null: false
     t.string "time_zone", null: false
     t.jsonb "settings", default: {}, null: false

spec/spec_helper.rb

@@ -21,6 +21,12 @@
 require 'simplecov'
 require 'coveralls'
 require 'rspec/rebound'
+require 'webmock/rspec'
+
+# Disable real external HTTP connections in tests but allow localhost so
+# Capybara drivers (cuprite/ferrum/selenium) can communicate with the app
+# server started by the test suite.
+WebMock.disable_net_connect!(allow_localhost: true)
 
 # Allow CI/local runs to override coverage output to avoid permission issues
 SimpleCov.coverage_dir ENV['SIMPLECOV_DIR'] if ENV['SIMPLECOV_DIR']