test-recorder 0.1.3 → 0.1.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3b7a9be6d6179b7ea6275e1e9b9463817dd0062452ca6b45c95c85dd9a3d8840
-  data.tar.gz: af977f2a4990f75f107210dd50b3f4c755f5efdbc7779e9c070dc480a715e4d1
+  metadata.gz: db491c5f3e52392c234291c174bbc1fff1bfb67d05a0ba23e50560529ea21d25
+  data.tar.gz: ae3f23d20992e0ec4e9cc4597fa15e346e52e1d2c31e482eab750d64942f8df1
 SHA512:
-  metadata.gz: 9f0faa3366d09f18b142ed4f5c42fa9ae1a7be31c8359eccae8695004920a973188880e0599759fc8b480e4c3e080d3a6985931d5d34fea2e349d36ed4d8005e
-  data.tar.gz: 634c35482c87795803ed7509d5042c357c440e876f03e6969a80501ca7d6e80b8b71292f1d13e9199e1c37dcf72763026b05e4c07fd91851a50162dc6e309ed8
+  metadata.gz: d4964d2703dc893155e0d246843141221e75991c57eb93da21e3c1331c9d9d648f56cf32d88885d4f89fc6da07b3889215549c055c9986aeecd3ac9c43bde1d7
+  data.tar.gz: d4c944875e4ee763fa6b88c434e700afa63fd1c3380d392e2d18f589a3e6f15aae4c2c83a52ece118829b0b343581efbe8b4a9d982b291921715c4e72f553f31

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    test-recorder (0.1.3)
+    test-recorder (0.1.4)
       activesupport
       selenium-devtools
       selenium-webdriver (>= 4.0)
@@ -9,7 +9,7 @@ PATH
 GEM
   remote: https://rubygems.org/
   specs:
-    activesupport (7.0.1)
+    activesupport (7.0.3)
       concurrent-ruby (~> 1.0, >= 1.0.2)
       i18n (>= 1.6, < 2)
       minitest (>= 5.1)
@@ -17,22 +17,22 @@ GEM
     activesupport-testing-metadata (0.1.0)
       activesupport (>= 4.0)
     childprocess (4.1.0)
-    concurrent-ruby (1.1.9)
-    debug (1.4.0)
+    concurrent-ruby (1.1.10)
+    debug (1.5.0)
       irb (>= 1.3.6)
       reline (>= 0.2.7)
-    i18n (1.8.11)
+    i18n (1.10.0)
       concurrent-ruby (~> 1.0)
     io-console (0.5.11)
     irb (1.4.1)
       reline (>= 0.3.0)
-    minitest (5.14.2)
+    minitest (5.15.0)
     rake (12.3.3)
     reline (0.3.1)
       io-console (~> 0.5)
     rexml (3.2.5)
     rubyzip (2.3.2)
-    selenium-devtools (0.100.0)
+    selenium-devtools (0.101.0)
       websocket (~> 1.0)
     selenium-webdriver (4.1.0)
       childprocess (>= 0.5, < 5.0)

data/lib/test_recorder/cdp_recorder.rb

@@ -1,11 +1,10 @@
-require "open3"
 require "fileutils"
-require "tempfile"
 
 module TestRecorder
   class CdpRecorder
     def initialize(enabled:)
       @enabled = enabled
+      @tmpdir = nil
       setup
     end
 
@@ -18,20 +17,19 @@ module TestRecorder
       enabled = @enabled if enabled.nil?
       return unless enabled
 
-      @tmp_video = Tempfile.open(["testrecorder", ".mp4"])
-      cmd = "ffmpeg -loglevel quiet -f image2pipe -avioflags direct -fpsprobesize 0 -probesize 32 -analyzeduration 0 -c:v mjpeg -i - -y -an -r 25 -qmin 0 -qmax 50 -crf 8 -deadline realtime -speed 8 -b:v 1M -threads 1 #{@tmp_video.path}"
-      @stdin, @wait_thrs = *Open3.pipeline_w(cmd)
-      @stdin.set_encoding("ASCII-8BIT")
+      @tmpdir = Dir.mktmpdir("testrecorder")
+      counter = 1
 
       @page = page
       @page.driver.browser.devtools.page.enable
 
       @page.driver.browser.devtools.page.on(:screencast_frame) do |event|
-        unless @stdin.closed?
-          decoded_data = Base64.decode64(event["data"])
-          @stdin.print(decoded_data)
+        decoded_data = Base64.decode64(event["data"])
+        filename = "%010d.jpeg" %  counter
+        if Dir.exist?(@tmpdir)
+          IO.binwrite("#{File.join(@tmpdir, filename)}", decoded_data)
+          counter += 1
         end
-
         @page.driver.browser.devtools.page.screencast_frame_ack(session_id: event["sessionId"])
       end
 
@@ -39,19 +37,25 @@ module TestRecorder
     end
 
     def stop_and_discard
-      @stdin.close
+      FileUtils.rm_rf(@tmpdir) unless @tmpdir.nil?
     end
 
     def stop_and_save(filename)
-      return if @page.nil?
-
-      @stdin.close
-      @wait_thrs.each(&:join)
+      return if @tmpdir.nil? || @page.nil?
 
+      @page.driver.browser.devtools.page.stop_screencast
       video_path = File.join(@video_dir, filename)
-      FileUtils.copy(@tmp_video.path, video_path)
-      @tmp_video.close(true)
+
+      args = %W(-loglevel error -f image2 -avioflags direct -fpsprobesize 0
+        -probesize 32 -analyzeduration 0 -c:v mjpeg -i #{File.join(@tmpdir, "%010d.jpeg")}
+        -y -an -r 25 -qmin 0 -qmax 50 -crf 8 -deadline realtime -speed 8 -b:v 1M
+        -threads 1 #{video_path})
+      system("ffmpeg", *args, exception: true)
       video_path
+    rescue => e
+      $stderr.puts("[TestRecorder] ffmpeg failed: #{e.message}")
+    ensure
+      FileUtils.rm_rf(@tmpdir)
     end
   end
 end

data/lib/test_recorder/rails/setup_and_teardown.rb

@@ -1,10 +1,7 @@
-require "fileutils"
-
 module TestRecorder
   module Rails
     module SetupAndTeardown
       def before_setup
-
         @cdp_recorder = TestRecorder::CdpRecorder.new(enabled: TestRecorder.enabled?)
         enabled = respond_to?(:metadata) ? metadata[:test_recorder] : nil
         @cdp_recorder.start(page: page, enabled: enabled)

data/lib/test_recorder/version.rb

@@ -1,3 +1,3 @@
 module TestRecorder
-  VERSION = "0.1.3"
+  VERSION = "0.1.4"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: test-recorder
 version: !ruby/object:Gem::Version
-  version: 0.1.3
+  version: 0.1.4
 platform: ruby
 authors:
 - Yuji Yaginuma
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2022-04-10 00:00:00.000000000 Z
+date: 2022-05-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport