Merge pull request #205 from increments/add-figma-script-host

Add new figma embed script host embed.figma.com

Author: acronhub

lib/qiita/markdown/embed/figma.rb

@@ -4,7 +4,10 @@ module Qiita
   module Markdown
     module Embed
       module Figma
-        SCRIPT_HOST = "www.figma.com"
+        SCRIPT_HOSTS = [
+          "www.figma.com",
+          "embed.figma.com",
+        ].freeze
       end
     end
   end

lib/qiita/markdown/transformers/filter_iframe.rb

@@ -9,7 +9,7 @@ class FilterIframe
           Embed::SlideShare::SCRIPT_HOST,
           Embed::GoogleSlide::SCRIPT_HOST,
           Embed::Docswell::SCRIPT_HOSTS,
-          Embed::Figma::SCRIPT_HOST,
+          Embed::Figma::SCRIPT_HOSTS,
           Embed::GoogleDrive::SCRIPT_HOST,
           Embed::StackBlitz::SCRIPT_HOST,
           Embed::BlueprintUe::SCRIPT_HOST,

spec/qiita/markdown/processor_spec.rb

@@ -1632,25 +1632,33 @@
 
       context "with HTML embed code for Figma" do
         shared_examples "embed code figma example" do
-          let(:markdown) do
-            <<-MARKDOWN.strip_heredoc
-              <iframe style="border: 1px solid rgba(0, 0, 0, 0.1);" width="100" height="100" src="#{url}"></iframe>
-            MARKDOWN
-          end
-          let(:url) { "#{scheme}//www.figma.com/embed?embed_host=share&url=https%3A%2F%2Fwww.figma.com" }
-          let(:encoded_url) { CGI.escapeHTML(url) }
+          [
+            "www.figma.com/embed?embed_host=share&url=https%3A%2F%2Fwww.figma.com",
+            "embed.figma.com/design/nrPSsILSYjesyc5UHjYYa4?embed_host=share",
+          ].each do |script_url|
+            context "with HTML embed code for Figma using script url `#{script_url}`" do
+              let(:markdown) do
+                <<-MARKDOWN.strip_heredoc
+                  <iframe style="border: 1px solid rgba(0, 0, 0, 0.1);" width="100" height="100" src="#{url}"></iframe>
+                MARKDOWN
+              end
 
-          if allowed
-            it "does not sanitize embed code" do
-              should eq <<-HTML.strip_heredoc
-                <iframe style="border: 1px solid rgba(0, 0, 0, 0.1);" width="100" height="100" src="#{encoded_url}"></iframe>
-              HTML
-            end
-          else
-            it "forces width attribute on iframe" do
-              should eq <<-HTML.strip_heredoc
-                <iframe style="border: 1px solid rgba(0, 0, 0, 0.1);" width="100%" height="100" src="#{encoded_url}"></iframe>
-              HTML
+              let(:url) { "#{scheme}//#{script_url}" }
+              let(:encoded_url) { CGI.escapeHTML(url) }
+
+              if allowed
+                it "does not sanitize embed code" do
+                  should eq <<-HTML.strip_heredoc
+                    <iframe style="border: 1px solid rgba(0, 0, 0, 0.1);" width="100" height="100" src="#{encoded_url}"></iframe>
+                  HTML
+                end
+              else
+                it "forces width attribute on iframe" do
+                  should eq <<-HTML.strip_heredoc
+                    <iframe style="border: 1px solid rgba(0, 0, 0, 0.1);" width="100%" height="100" src="#{encoded_url}"></iframe>
+                  HTML
+                end
+              end
             end
           end
         end