rename enforce_layer -> enforce_layers (#45)

* rename enforce_layer -> enforce_layers

* bump version to 0.25.0

Author: schoblaska

Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    parse_packwerk (0.24.0)
+    parse_packwerk (0.25.0)
       bigdecimal
       sorbet-runtime
 

lib/parse_packwerk.rb

@@ -73,7 +73,7 @@ def self.write_package_yml!(package)
       end
 
       if Extensions.layer_extension_installed?
-        merged_config.merge!('enforce_layer' => package.enforce_layer)
+        merged_config.merge!('enforce_layers' => package.enforce_layers)
       end
 
       unless package.public_path == DEFAULT_PUBLIC_PATH
@@ -107,7 +107,7 @@ def self.key_sort_order
       enforce_dependencies
       enforce_privacy
       enforce_visibility
-      enforce_layer
+      enforce_layers
       public_path
       owner
       layer

lib/parse_packwerk/constants.rb

@@ -7,7 +7,7 @@ module ParsePackwerk
   PACKAGE_TODO_YML_NAME = T.let('package_todo.yml'.freeze, String)
   ENFORCE_DEPENDENCIES = T.let('enforce_dependencies'.freeze, String)
   ENFORCE_PRIVACY = T.let('enforce_privacy'.freeze, String)
-  ENFORCE_LAYER = T.let('enforce_layer'.freeze, String)
+  ENFORCE_LAYERS = T.let('enforce_layers'.freeze, String)
   DEPENDENCY_VIOLATION_TYPE = T.let('dependency'.freeze, String)
   PRIVACY_VIOLATION_TYPE = T.let('privacy'.freeze, String)
   PUBLIC_PATH = T.let('public_path'.freeze, String)

lib/parse_packwerk/package.rb

@@ -7,7 +7,7 @@ class Package < T::Struct
     const :name, String
     const :enforce_dependencies, T.nilable(T.any(T::Boolean, String))
     const :enforce_privacy, T.any(T::Boolean, String), default: false
-    const :enforce_layer, T.any(T::Boolean, String), default: false
+    const :enforce_layers, T.any(T::Boolean, String), default: false
     const :public_path, String, default: DEFAULT_PUBLIC_PATH
     const :metadata, MetadataYmlType
     const :dependencies, T::Array[String]
@@ -23,7 +23,7 @@ def self.from(pathname)
         name: package_name,
         enforce_dependencies: package_loaded_yml[ENFORCE_DEPENDENCIES],
         enforce_privacy: package_loaded_yml[ENFORCE_PRIVACY] || false,
-        enforce_layer: package_loaded_yml[ENFORCE_LAYER] || false,
+        enforce_layers: package_loaded_yml[ENFORCE_LAYERS] || false,
         public_path: package_loaded_yml[PUBLIC_PATH] || DEFAULT_PUBLIC_PATH,
         metadata: package_loaded_yml[METADATA] || {},
         dependencies: package_loaded_yml[DEPENDENCIES] || [],
@@ -63,8 +63,8 @@ def enforces_privacy?
     end
 
     sig { returns(T.any(T::Boolean, String)) }
-    def enforces_layer?
-      enforce_layer
+    def enforces_layers?
+      enforce_layers
     end
   end
 end

parse_packwerk.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |spec|
   spec.name          = 'parse_packwerk'
-  spec.version       = '0.24.0'
+  spec.version       = '0.25.0'
   spec.authors       = ['Gusto Engineers']
   spec.email         = ['dev@gusto.com']
   spec.summary       = 'A low-dependency gem for parsing and writing packwerk YML files'

spec/parse_packwerk_spec.rb

@@ -983,12 +983,12 @@
     let(:package_yml) { package_dir.join('package.yml') }
     let(:package_todo_yml) { package_dir.join('package_todo.yml') }
 
-    def build_pack(public_path: 'app/public', enforce_privacy: true, enforce_layer: true, dependencies: [], metadata: {}, config: {})
+    def build_pack(public_path: 'app/public', enforce_privacy: true, enforce_layers: true, dependencies: [], metadata: {}, config: {})
       ParsePackwerk::Package.new(
         name: package_dir.to_s,
         enforce_dependencies: true,
         enforce_privacy: enforce_privacy,
-        enforce_layer: enforce_layer,
+        enforce_layers: enforce_layers,
         public_path: public_path,
         dependencies: dependencies,
         metadata: metadata,
@@ -1002,7 +1002,7 @@ def pack_as_hash(package)
         name: package.name,
         enforce_dependencies: package.enforce_dependencies,
         enforce_privacy: package.enforce_privacy,
-        enforce_layer: package.enforce_layer,
+        enforce_layers: package.enforce_layers,
         dependencies: package.dependencies,
         metadata: package.metadata
       }
@@ -1016,7 +1016,7 @@ def pack_as_hash(package)
         expect(package_yml.read).to eq <<~PACKAGEYML
           enforce_dependencies: true
           enforce_privacy: true
-          enforce_layer: true
+          enforce_layers: true
         PACKAGEYML
 
         expect(all_packages.count).to eq 1
@@ -1034,7 +1034,7 @@ def pack_as_hash(package)
         expect(package_yml.read).to eq <<~PACKAGEYML
           enforce_dependencies: true
           enforce_privacy: true
-          enforce_layer: true
+          enforce_layers: true
           public_path: other/path
         PACKAGEYML
 
@@ -1053,7 +1053,7 @@ def pack_as_hash(package)
         expect(package_yml.read).to eq <<~PACKAGEYML
           enforce_dependencies: true
           enforce_privacy: strict
-          enforce_layer: true
+          enforce_layers: true
         PACKAGEYML
 
         expect(all_packages.count).to eq 1
@@ -1071,7 +1071,7 @@ def pack_as_hash(package)
         expect(package_yml.read).to eq <<~PACKAGEYML
           enforce_dependencies: true
           enforce_privacy: true
-          enforce_layer: true
+          enforce_layers: true
           dependencies:
             - my_other_pack1
             - my_other_pack2
@@ -1096,7 +1096,7 @@ def pack_as_hash(package)
         expect(package_yml.read).to eq <<~PACKAGEYML
           enforce_dependencies: true
           enforce_privacy: true
-          enforce_layer: true
+          enforce_layers: true
           metadata:
             owner: Mission > Team
             protections:
@@ -1113,7 +1113,7 @@ def pack_as_hash(package)
           write_file(package_yml, <<~CONTENTS)
             enforce_dependencies: true
             enforce_privacy: true
-            enforce_layer: true
+            enforce_layers: true
             public_path: other/path
             dependencies:
             - packs/package2
@@ -1132,7 +1132,7 @@ def pack_as_hash(package)
           expect(package_yml.read).to eq <<~PACKAGEYML
             enforce_dependencies: true
             enforce_privacy: true
-            enforce_layer: true
+            enforce_layers: true
             public_path: other/path
           PACKAGEYML
         end
@@ -1153,7 +1153,7 @@ def pack_as_hash(package)
         expect(package_yml.read).to eq <<~PACKAGEYML
           enforce_dependencies: true
           enforce_privacy: true
-          enforce_layer: true
+          enforce_layers: true
           my_special_key:
             blah: 1
           my_other_special_key: true
@@ -1169,7 +1169,7 @@ def pack_as_hash(package)
         write_file('packwerk.yml', '{}')
       end
 
-      let(:package) { build_pack(enforce_privacy: false, enforce_layer: false) }
+      let(:package) { build_pack(enforce_privacy: false, enforce_layers: false) }
 
       it 'writes the right package' do
         ParsePackwerk.write_package_yml!(package)