megalabs_sms 0.1.1 → 0.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6e446dd9f5f499406d31feb11b439f9a2a70801655e93568a57377cedd143949
-  data.tar.gz: c0708973c1a366d0589f39ff6382ebf87489adfd7869200811fe708ca6ffa2a9
+  metadata.gz: d82e07153abbae70e6bd8cb9a967290b1ec0dc6f29f43a1d3e1f4335b7690883
+  data.tar.gz: f4532bc91e3b879133cb33c9929ceafecd3e42a325146429fc299b620cb72bc5
 SHA512:
-  metadata.gz: 0b4c585530127f78b2df5311ec6c7ae9bebc6111ea048a6e9b04606cd9204b2364853259f7d159f86d25964fd1196857c0e1769307799a4ce07b886ae95f662d
-  data.tar.gz: 2daf04602c196d85edbc7883a59f02ffb665da4ce0a8d0b900b7c4dae4ad8e96859d8a2ce1c861cc38fd0870d6ded9b445b2407ee36a19fb9d9d7f73b412df4d
+  metadata.gz: 6bab059a7777d377914b460e34249a04011a98abe3096cafe8a385a5097ad34e0807542e67b3dd1c80b6fc9ad14179f84a26ffeb3dfedbffcc9dcecb70e23782
+  data.tar.gz: 9e23e8a94f9018fef98f1046fc63f1f3f6ca6657adc9daa0c4323b7c32674516799dc8aebce6856254e271fc8d064709c25c79b03482d5842350bc953aace426

data/CHANGELOG.md

@@ -1,5 +1,13 @@
 # Changelog
 
+## [0.2.0] - 2025-03-14
+
+### Изменено
+
+- Изменён тип возвращаемого значения метода `send_sms` с String на Boolean для более чёткого определения успешности вызова API
+- Изменена документация для методов `send_sms`, `handle_stub_response`, `send_request`, `perform_http_request` и `process_http_response`
+- Обновлены тесты для работы с булевым возвращаемым значением вместо строкового
+
 ## [0.1.1] - 2024-03-13
 
 ### Добавлено

data/lib/megalabs_sms/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module MegalabsSms
-  VERSION = '0.1.1'
+  VERSION = '0.2.0'
 end

data/lib/megalabs_sms.rb

@@ -54,7 +54,7 @@ module MegalabsSms
     # @param to [String] номер телефона получателя
     # @param message [String] текст сообщения
     #
-    # @return [String] статус отправки или текст ошибки
+    # @return [Boolean] true если SMS отправлено успешно, false в случае ошибки
     #
     def send_sms(from, to, message)
       return handle_stub_response if stub_enabled?
@@ -77,12 +77,16 @@ module MegalabsSms
     #
     # Обрабатывает эмуляцию ответов
     #
-    # @return [String] сообщение об успехе или ошибке эмуляции
+    # @return [Boolean] true если эмулируется успех, false если эмулируется ошибка
     #
     def handle_stub_response
-      return log_message('Stubbed error: SMS not sent') if @error_stub
-
-      log_message('Stubbed success: SMS would be sent') if @success_stub
+      if @error_stub
+        log_message('Stubbed error: SMS not sent')
+        false
+      elsif @success_stub
+        log_message('Stubbed success: SMS would be sent')
+        true
+      end
     end
 
     #
@@ -125,13 +129,13 @@ module MegalabsSms
     #
     # @param request [Net::HTTP::Post] объект HTTP-запроса
     #
-    # @return [String] тело ответа или сообщение об ошибке
+    # @return [Boolean] true если запрос выполнен успешно, false в случае ошибки
     #
     def send_request(request)
       uri = URI('https://a2p-api.megalabs.ru/sms/v1/sms')
-      response_body = perform_http_request(uri, request)
+      success = perform_http_request(uri, request)
       sleep(@sleep_time) if @sleep_time.positive?
-      response_body
+      success
     end
 
     #
@@ -140,7 +144,7 @@ module MegalabsSms
     # @param uri [URI] URI-адрес для запроса
     # @param request [Net::HTTP::Post] объект HTTP-запроса
     #
-    # @return [String] тело ответа или сообщение об ошибке
+    # @return [Boolean] true если запрос выполнен успешно, false в случае ошибки
     #
     def perform_http_request(uri, request)
       response = Net::HTTP.start(uri.hostname, uri.port, use_ssl: uri.scheme == 'https') do |http|
@@ -150,10 +154,11 @@ module MegalabsSms
       if response.is_a?(Net::HTTPSuccess)
         process_http_response(response.body)
       else
-        log_message("Failed to send: #{response}")
+        log_message("Failed to send: #{response}") && false
       end
     rescue StandardError => e
       log_message("Exception occurred: #{e.message}")
+      false
     end
 
     #
@@ -162,17 +167,21 @@ module MegalabsSms
     #
     # @param raw_body [String] оригинальный ответ от HTTP-запроса
     #
-    # @return [String] либо оригинальное тело (если все хорошо), либо сообщение об ошибке
+    # @return [Boolean] true если сообщение отправлено успешно, false в случае ошибки
     #
     def process_http_response(raw_body)
       body = raw_body.dup.force_encoding('UTF-8')
       parsed = JSON.parse(body)
       result = parsed.dig('result', 'status')
-      return body if result&.fetch('code', nil)&.zero? && result&.fetch('description', '')&.downcase == 'ok'
-
-      log_message("Failed to send: #{body}")
+      success = result&.fetch('code', nil)&.zero? && result&.fetch('description', '')&.downcase == 'ok'
+      if success
+        log_message("Successfully sent: #{body}") && true
+      else
+        log_message("Failed to send: #{body}") && false
+      end
     rescue JSON::ParserError => e
       log_message("Failed to parse JSON: #{e.message}")
+      false
     end
   end
 end

data/spec/megalabs_sms/client_spec.rb

@@ -27,9 +27,11 @@ RSpec.describe MegalabsSms::Client do
       let(:success_stub) { true }
       let(:error_stub)   { false }
 
-      it 'returns a success message (stub)' do
+      it 'returns true indicating success' do
+        allow(client).to receive(:log_message).and_return(true)
         result = client.send_sms(from, to, message)
-        expect(result).to eq('[MegalabsSms] Stubbed success: SMS would be sent')
+        expect(result).to eq(true)
+        expect(client).to have_received(:log_message).with('Stubbed success: SMS would be sent')
       end
     end
 
@@ -37,9 +39,11 @@ RSpec.describe MegalabsSms::Client do
       let(:success_stub) { false }
       let(:error_stub)   { true }
 
-      it 'returns an error message (stub)' do
+      it 'returns false indicating failure' do
+        allow(client).to receive(:log_message).and_return(false)
         result = client.send_sms(from, to, message)
-        expect(result).to eq('[MegalabsSms] Stubbed error: SMS not sent')
+        expect(result).to eq(false)
+        expect(client).to have_received(:log_message).with('Stubbed error: SMS not sent')
       end
     end
   end
@@ -51,12 +55,14 @@ RSpec.describe MegalabsSms::Client do
       '{"result":{"msg_id":"id","status":{"code":0,"description":"ok"}}}'
     end
 
-    it 'makes an HTTP request' do
+    it 'makes an HTTP request and returns true on success' do
       stub_request(:post, 'https://a2p-api.megalabs.ru/sms/v1/sms')
         .to_return(status: 200, body: valid_response, headers: { 'Content-Type': 'application/json' })
 
+      allow(client).to receive(:log_message).and_return(true)
+
       result = client.send_sms(from, to, message)
-      expect(result).to eq(valid_response)
+      expect(result).to eq(true)
       expect(WebMock).to have_requested(:post, 'https://a2p-api.megalabs.ru/sms/v1/sms')
         .with(body: {
           from: from,
@@ -65,6 +71,20 @@ RSpec.describe MegalabsSms::Client do
           # rubocop:enable Style/NumericLiterals
           message: message
         }.to_json)
+      expect(client).to have_received(:log_message).with("Successfully sent: #{valid_response}")
+    end
+
+    it 'returns false when the API returns an error' do
+      error_response = '{"result":{"msg_id":"id","status":{"code":1,"description":"error"}}}'
+
+      stub_request(:post, 'https://a2p-api.megalabs.ru/sms/v1/sms')
+        .to_return(status: 200, body: error_response, headers: { 'Content-Type': 'application/json' })
+
+      allow(client).to receive(:log_message).and_return(false)
+
+      result = client.send_sms(from, to, message)
+      expect(result).to eq(false)
+      expect(client).to have_received(:log_message).with("Failed to send: #{error_response}")
     end
   end
 end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: megalabs_sms
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.2.0
 platform: ruby
 authors:
 - Vitalii Dementev
 bindir: bin
 cert_chain: []
-date: 2025-03-13 00:00:00.000000000 Z
+date: 2025-03-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: net-http