Merge branch 'hchouraria-ci-log-section-duration-format-tz' into 'master'

Force use of UTC in formatting seconds into MM:SS

See merge request gitlab-org/gitlab!62873

lib/gitlab/ci/ansi2json/line.rb

@@ -77,7 +77,7 @@ module Gitlab
         end
 
         def set_section_duration(duration)
-          @section_duration = Time.at(duration.to_i).strftime('%M:%S')
+          @section_duration = Time.at(duration.to_i).utc.strftime('%M:%S')
         end
 
         def flush_current_segment!

spec/lib/gitlab/ci/ansi2json/line_spec.rb

@@ -76,6 +76,7 @@ RSpec.describe Gitlab::Ci::Ansi2json::Line do
   end
 
   describe '#set_section_duration' do
+    shared_examples 'set_section_duration' do
       it 'sets and formats the section_duration' do
         subject.set_section_duration(75)
 
@@ -83,6 +84,25 @@ RSpec.describe Gitlab::Ci::Ansi2json::Line do
       end
     end
 
+    context 'with default timezone' do
+      it_behaves_like 'set_section_duration'
+    end
+
+    context 'with a timezone carrying minutes offset' do
+      before do
+        # The actual call by does use Time.at(...).utc that the following
+        # rubocop rule (Rails/TimeZone) suggests, but for this specific
+        # test's purposes we needed to mock at the Time.at call point.
+
+        # rubocop:disable Rails/TimeZone
+        allow(Time).to receive(:at).with(75).and_return(Time.at(75, in: '+05:30'))
+        # rubocop:enable Rails/TimeZone
+      end
+
+      it_behaves_like 'set_section_duration'
+    end
+  end
+
   describe '#flush_current_segment!' do
     context 'when current segment is not empty' do
       before do