Do not create scan_js workflow on --skip-js (rails#53553)

* Do not create scan_js workflow on --skip-js

* Update railties/lib/rails/generators/rails/app/templates/github/ci.yml.tt

Co-authored-by: Ryuta Kamizono <[email protected]>

---------

Co-authored-by: Ryuta Kamizono <[email protected]>

Author: skryukov

railties/lib/rails/generators/rails/app/templates/github/ci.yml.tt

@@ -24,7 +24,7 @@ jobs:
         run: bin/brakeman --no-pager
 
 <% end -%>
-<%- if options[:javascript] == "importmap"  && !options[:api] -%>
+<%- if options[:javascript] == "importmap" && !options[:api] && !options[:skip_javascript] -%>
   scan_js:
     runs-on: ubuntu-latest
 

railties/test/generators/api_app_generator_test.rb

@@ -61,6 +61,7 @@ def test_api_modified_files
     assert_file ".github/workflows/ci.yml" do |content|
       assert_no_match(/test:system/, content)
       assert_no_match(/screenshots/, content)
+      assert_no_match(/scan_js/, content)
     end
   end
 

railties/test/generators/app_generator_test.rb

@@ -548,6 +548,10 @@ def test_javascript_is_skipped_if_required
     assert_file "app/views/layouts/application.html.erb" do |contents|
       assert_match(/stylesheet_link_tag\s+:app %>/, contents)
     end
+
+    assert_file ".github/workflows/ci.yml" do |file|
+      assert_no_match("scan_js", file)
+    end
   end
 
   def test_inclusion_of_jbuilder