firespring_dev_commands 2.1.21.pre.alpha.3 → 2.1.21.pre.alpha.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 23ad8131b620682ed6d6e9b496934646615c3aca3c0656886a624e03c73f0041
-  data.tar.gz: 936887064f1e8e3c71193af00a105b3808ede716b5bf72cc959c7db63d459ac0
+  metadata.gz: d35a98b66dfe013f7552c2d5baf17b373ad71e3893945844a24768620755bf5f
+  data.tar.gz: faa627fca3af6b2c2ef417fd1442439b16d7acafa6a1a3dfa5da1fc01c26e327
 SHA512:
-  metadata.gz: 89cd73bbf3342e5ff66a9762311bfd07a5c78f18aa7dc92d3edaef96d55277054a887b011b672a3a9d2689870a98c1a1784ee5061f6539bba8a95856fe86d095
-  data.tar.gz: e0a8077e578697502912063cdf9004d21c993cb32b54cbfb46b158f59b7f7a37ac6362358606070451113ead000bf7759859999cc9818eddbbeba7cdda43965f
+  metadata.gz: 434c1348564f615f90438f4682080aa70a659693a264e2b47ac17f94e935fb0dfbace4875df3c76dcd5ac76d2ce44cdc7ded70ec10882e5fa4927f01e9e4fb0c
+  data.tar.gz: a11c82d457d76e6eeca78217db40a62d7cc9c6eade77c2f07cb9c68ed89b4b1d274b6e7cad5e64abefa0c1814c23b29a03f2e4b0d837d9f9faece9b24ca5821b

data/lib/firespring_dev_commands/coverage/cobertura.rb

@@ -3,11 +3,13 @@ module Dev
   module Coverage
     # Class for checking code coverage using cobertura
     class Cobertura
-      attr_reader :local_filename, :container_filename, :filename, :threshold
+      attr_reader :local_path, :container_path, :filename, :threshold, :local_path, :local_filename, :container_path, :container_filename
 
       def initialize(filename: 'cobertura.xml', threshold: nil, container_path: nil, local_path: nil)
         @filename = filename
+        @local_path = local_path
         @local_filename = File.join(local_path || '.', @filename)
+        @container_path = container_path
         @container_filename = File.join(container_path || '.', @filename)
         @threshold = threshold.to_f
       end
@@ -18,17 +20,25 @@ module Dev
         # Remove any previous coverage info
         FileUtils.rm_f(local_filename, verbose: true)
 
+        # Return the needed php commands to generate the cobertura report
         %W(--coverage-cobertura #{container_filename})
       end
 
       # Parse the cobertura file as a hash and check the total coverage against the desired threshold
-      def check
+      def check(application: nil)
+        # If an application has been specified and the file does not exist locally, attempt to copy it back from the docker container
+        if application && !File.exist?(local_filename)
+          container = Dev::Docker::Compose.new.container_by_name(application)
+          workdir = Dev::Docker.new.default_working_dir(container)
+          Dev::Docker.new.copy_from_container(container, File.join(workdir, filename), local_filename, required: true)
+        end
+
+        # Load the file from disk and parse with ox
         report = Ox.load(File.read(local_filename), mode: :hash)
         attrs, = report[:coverage]
         cov_pct = attrs[:'line-rate'].to_f * 100
-        raise format('Line coverage %.2f%% is less than the threshold %.2f%%', cov_pct, threshold) if cov_pct < threshold
-
-        puts format('Line coverage %.2f%% is above the threshold %.2f%%', cov_pct, threshold)
+        puts format('Line coverage was %.2f%%. Configured threshold was %.2f%%', cov_pct, threshold)
+        raise 'Code coverage not met' if cov_pct < threshold
       end
     end
   end

data/lib/firespring_dev_commands/docker.rb

@@ -138,6 +138,11 @@ module Dev
       Docker::Compose.new.mapped_public_port(name, private_port)
     end
 
+    # Gets the default working dir of the container
+    def default_working_dir(container)
+      container.json['Config']['WorkingDir']
+    end
+
     # Copies the source path on your local machine to the destination path on the container
     def copy_to_container(container, source_path, dest_path)
       LOG.info "Copying #{source_path} to #{dest_path}... "

data/lib/firespring_dev_commands/php.rb

@@ -101,8 +101,8 @@ module Dev
     end
 
     # Run the check to ensure code coverage meets the desired threshold
-    def check_test_coverage
-      coverage.check
+    def check_test_coverage(application:)
+      coverage.check(application:)
     end
 
     # Build the php fast test command

data/lib/firespring_dev_commands/templates/docker/php/application.rb

@@ -135,7 +135,7 @@ module Dev
                     options = []
                     options << '-T' if Dev::Common.new.running_codebuild?
                     Dev::Docker::Compose.new(services: application, options:).exec(*php.test_command)
-                    php.check_test_coverage
+                    php.check_test_coverage(application:)
                   end
                 end
               end

data/lib/firespring_dev_commands/version.rb

@@ -6,6 +6,6 @@ module Dev
     # Use 'v.v.v.pre.alpha.v' for pre-release vesions
     # Use 'v.v.v.beta.v for beta versions
     # Use semantic versioning for any releases (https://semver.org/)
-    VERSION = '2.1.21.pre.alpha.3'.freeze
+    VERSION = '2.1.21.pre.alpha.4'.freeze
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: firespring_dev_commands
 version: !ruby/object:Gem::Version
-  version: 2.1.21.pre.alpha.3
+  version: 2.1.21.pre.alpha.4
 platform: ruby
 authors:
 - Firespring