uffizzi-cli 0.11.0 → 0.11.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6a77480fa0ba3272d145f85b529890db26cbe746a1c24e4ab06bdd5722440a97
-  data.tar.gz: 81490020eded539424fc962df0513fab070948bf8ed11a4e4cb5e11eb9be48ea
+  metadata.gz: 692eb8af782d78cc92d06d6a0eda07e6d481019a81f44144791bf211ab6438c0
+  data.tar.gz: a0db18ffc332806c013647c0706294c3db9486b42777a8f622fea9ca46552062
 SHA512:
-  metadata.gz: 1deeb99a694ad314a86980bbd2d320234f893824cad9c325077145e05d586bff507e2b881ccd1990926dc7bbf94c60babe8008ad58c06b4f8da82df66a89737b
-  data.tar.gz: 3bbe1123c922b6117a1794e81fa93d4059581c15b5de3f8d15bc76b5ec52df43d4f0f86c662ef196bb98ce0b80a4434114533a23f51a7cdb95213204666ba1c1
+  metadata.gz: 97e7dd627b0edfbfb500003480dc0e30e3497f36c263f6ebb2e3a79dced658444432e13182a5b2d3149e41982543283190255118e4eeec393f6c01aaed4dc9d5
+  data.tar.gz: 942031954c139f313f20ae22f47b894131101e32f41b9b5a3001e35fde31b2f45b5b08a6e8791bf8584b8982234274c6295a71c9aa13fceb369495498810cdfb

data/lib/uffizzi/cli/login.rb

@@ -15,7 +15,7 @@ module Uffizzi
     end
 
     def run
-      Uffizzi.ui.say('Login to Uffizzi to view and manage your previews.')
+      Uffizzi.ui.say('Login to Uffizzi server.')
       server = set_server
 
       username = set_username
@@ -68,6 +68,8 @@ module Uffizzi
       ConfigFile.write_option(:cookie, response[:headers])
       ConfigFile.write_option(:account_id, account[:id])
 
+      Uffizzi.ui.say('Login successfull')
+
       default_project = ConfigFile.read_option(:project)
       return unless default_project
 

data/lib/uffizzi/cli/preview/service.rb

@@ -68,7 +68,7 @@ module Uffizzi
 
     def handle_succeed_logs_response(response, container_name)
       logs = response[:body][:logs] || []
-      return Uffizzi.ui.say("The service #{container_name} has no logs") if logs.empty?
+      return Uffizzi.ui.say("The service '#{container_name}' has no logs") if logs.empty?
 
       logs.each do |log|
         Uffizzi.ui.say(log)

data/lib/uffizzi/config_file.rb

@@ -92,10 +92,9 @@ module Uffizzi
       end
 
       def write(data)
-        file = create_file
         prepared_data = prepare_data(data)
-        file.write(prepared_data)
-        file.close
+
+        lock(config_path) { atomic_write(config_path, "#{config_path}.tmp", prepared_data) }
       end
 
       def prepare_data(data)
@@ -105,12 +104,18 @@ module Uffizzi
         end
       end
 
-      def create_file
-        dir = File.dirname(config_path)
+      def atomic_write(path, temp_path, content)
+        File.open(temp_path, 'w') { |f| f.write(content) }
+        FileUtils.mv(temp_path, path)
+      end
 
+      def lock(path)
+        dir = File.dirname(path)
         FileUtils.mkdir_p(dir) unless File.directory?(dir)
 
-        File.new(config_path, 'w')
+        File.open(path).flock(File::LOCK_EX) if File.exist?(path)
+        yield
+        File.open(path).flock(File::LOCK_UN)
       end
     end
   end

data/lib/uffizzi/response_helper.rb

@@ -42,14 +42,18 @@ module Uffizzi
       def prepare_errors(errors)
         errors.values.reduce('') do |acc, error_messages|
           if error_messages.is_a?(Array)
-            error_messages.each { |error_message| acc = "#{acc}#{error_message}\n" }
+            error_messages.each { |error_message| acc = "#{acc}#{prepare_error_message(error_message)}\n" }
           else
-            acc = "#{acc}#{error_messages}\n"
+            acc = "#{acc}#{prepare_error_message(error_message)}\n"
           end
 
           acc
         end
       end
+
+      def prepare_error_message(error_message)
+        error_message.split('::').last
+      end
     end
   end
 end

data/lib/uffizzi/services/preview_service.rb

@@ -20,7 +20,7 @@ class PreviewService
       deployment_id = deployment[:id]
       params = { id: deployment_id }
 
-      response = deploy_containers(Uffizzi::ConfigFile.read_option(:server), project_slug, deployment_id, params)
+      response = deploy_containers(server_url, project_slug, deployment_id, params)
 
       if !Uffizzi::ResponseHelper.no_content?(response)
         Uffizzi::ResponseHelper.handle_failed_response(response)
@@ -32,13 +32,17 @@ class PreviewService
 
     private
 
+    def server_url
+      @server_url ||= Uffizzi::ConfigFile.read_option(:server)
+    end
+
     def wait_containers_creation(deployment, project_slug)
       spinner = TTY::Spinner.new('[:spinner] Creating containers...', format: :dots)
       spinner.auto_spin
 
       activity_items = []
       loop do
-        response = get_activity_items(Uffizzi::ConfigFile.read_option(:server), project_slug, deployment[:id])
+        response = get_activity_items(server_url, project_slug, deployment[:id])
         handle_activity_items_response(response, spinner)
         activity_items = response[:body][:activity_items]
         break if activity_items.count == deployment[:containers].count
@@ -62,7 +66,7 @@ class PreviewService
       containers_spinners = create_containers_spinners(activity_items, spinner)
 
       loop do
-        response = get_activity_items(Uffizzi::ConfigFile.read_option(:server), project_slug, deployment[:id])
+        response = get_activity_items(server_url, project_slug, deployment[:id])
         handle_activity_items_response(response, spinner)
         activity_items = response[:body][:activity_items]
         update_containers_spinners!(activity_items, containers_spinners)

data/lib/uffizzi/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Uffizzi
-  VERSION = '0.11.0'
+  VERSION = '0.11.1'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: uffizzi-cli
 version: !ruby/object:Gem::Version
-  version: 0.11.0
+  version: 0.11.1
 platform: ruby
 authors:
 - Josh Thurman
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2022-06-15 00:00:00.000000000 Z
+date: 2022-06-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: awesome_print