Merge pull request #4164 from alphagov/2664-investigate-adding-javasc…

…ript-test-coverage-and-implement-it-if-possible-l-2

Separate JavaScript files which should undergo test coverage checking from other files

app/assets/config/manifest.js

@@ -1,7 +1,9 @@
 //= link_tree ../images
-//= link all.js
 //= link es6-components.js
 //= link application.js
+//= link main.js
+//= link dependencies.js
 //= link test-dependencies.js
+//= link views/travel-advice.js
 
 //= link_tree ../builds

app/assets/javascripts/application.js

@@ -1,9 +1,3 @@
-//= require govuk_publishing_components/lib
-//= require govuk_publishing_components/components/govspeak
-//= require govuk_publishing_components/components/image-card
-//= require govuk_publishing_components/components/intervention
-//= require govuk_publishing_components/components/step-by-step-nav
-//= require govuk_publishing_components/components/table
-
-//= require support
-//= require_tree ./modules
+// This file is linked to in the application.
+//= require dependencies
+//= require main

app/assets/javascripts/dependencies.js

@@ -0,0 +1,7 @@
+// This file contains the dependencies required by the application.
+//= require govuk_publishing_components/lib
+//= require govuk_publishing_components/components/govspeak
+//= require govuk_publishing_components/components/image-card
+//= require govuk_publishing_components/components/intervention
+//= require govuk_publishing_components/components/step-by-step-nav
+//= require govuk_publishing_components/components/table

app/assets/javascripts/main.js

@@ -0,0 +1,5 @@
+// This file should only contain the code being a part
+// of this application, not its dependencies.
+// It will be checked with respect to test coverage.
+//= require support
+//= require_tree ./modules

config/application.rb

@@ -48,12 +48,6 @@ class Application < Rails::Application
     # Enable the asset pipeline
     config.assets.enabled = true
 
-    config.assets.precompile += %w[
-      views/travel-advice.js
-      frontend.js
-      application.css
-    ]
-
     # Path within public/ where assets are compiled to
     config.assets.prefix = "/assets/frontend"