Move config file class out of stack

Author: gururajsh

app/models/helpers/config_file.rb

@@ -0,0 +1,37 @@
+# frozen_string_literal: true
+module VCAP::CloudController
+    class ConfigFile
+      def initialize(file_path)
+        @hash = YAMLConfig.safe_load_file(file_path).tap do |h|
+          Schema.validate(h)
+        end
+      end
+
+      def stacks
+        @hash['stacks']
+      end
+
+      def deprecated_stacks
+        @hash['deprecated_stacks']
+      end
+
+      def default
+        @hash['default']
+      end
+
+      Schema = Membrane::SchemaParser.parse do
+        {
+          'default' => String,
+          'stacks' => [{
+                         'name' => String,
+                         'description' => String,
+                         optional('build_rootfs_image') => String,
+                         optional('run_rootfs_image') => String
+                       }],
+          optional('deprecated_stacks') => [
+            String
+          ]
+        }
+      end
+    end
+  end

app/models/runtime/stack.rb

@@ -1,4 +1,5 @@
 require 'models/helpers/process_types'
+require 'models/helpers/config_file'
 
 module VCAP::CloudController
   class Stack < Sequel::Model
@@ -97,40 +98,5 @@ def self.populate_from_hash(hash)
         create(hash.slice('name', 'description', 'build_rootfs_image', 'run_rootfs_image'))
       end
     end
-
-    class ConfigFile
-      def initialize(file_path)
-        @hash = YAMLConfig.safe_load_file(file_path).tap do |h|
-          Schema.validate(h)
-        end
-      end
-
-      def stacks
-        @hash['stacks']
-      end
-
-      def deprecated_stacks
-        @hash['deprecated_stacks']
-      end
-
-      def default
-        @hash['default']
-      end
-
-      Schema = Membrane::SchemaParser.parse do
-        {
-          'default' => String,
-          'stacks' => [{
-            'name' => String,
-            'description' => String,
-            optional('build_rootfs_image') => String,
-            optional('run_rootfs_image') => String
-          }],
-          optional('deprecated_stacks') => [
-            String
-          ]
-        }
-      end
-    end
   end
-end
+end

lib/cloud_controller/check_stacks.rb

@@ -4,7 +4,7 @@ class CheckStacks
 
     def initialize(config, db)
       @config = config
-      @stack_config = VCAP::CloudController::Stack::ConfigFile.new(config.get(:stacks_file))
+      @stack_config = VCAP::CloudController::ConfigFile.new(config.get(:stacks_file))
       @db = db
     end
 
@@ -30,4 +30,4 @@ def validate_stack(deprecated_stack)
       raise "rake task 'stack_check' failed, stack '#{deprecated_stack}' not supported" if deprecated_stack_in_db
     end
   end
-end
+end

lib/tasks/stack_check.rake

@@ -7,7 +7,8 @@ namespace :stacks do
     next unless db.table_exists?(:buildpack_lifecycle_data)
 
     RakeConfig.config.load_db_encryption_key
+    require 'models/helpers/config_file'
     require 'cloud_controller/check_stacks'
     VCAP::CloudController::CheckStacks.new(RakeConfig.config, db).validate_stacks
   end
-end
+end