Skip to content

Commit

Permalink
Unify screenshot logs
Browse files Browse the repository at this point in the history
  • Loading branch information
kapoorlakshya authored Jul 20, 2024
1 parent 9b07ad7 commit 615d64b
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 3 deletions.
2 changes: 1 addition & 1 deletion lib/screen-recorder/screenshot.rb
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ module Screenshot
#
# Takes a screenshot in the current context (input) - desktop or current window
#
def screenshot(filename, resolution = nil, log = 'ffmpeg-screenshot.log')
def screenshot(filename, resolution = nil, log = 'ffmpeg.log')
ScreenRecorder.logger.debug "Screenshot filename: #{filename}, resolution: #{resolution}"
cmd = screenshot_cmd(filename: filename, resolution: resolution)
process = execute_command(cmd, log) # exits when done
Expand Down
2 changes: 1 addition & 1 deletion spec/screen-recorder/screenshot_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
context 'when recording a desktop' do
it 'can take a screenshot' do
recorder.start
file = recorder.screenshot(image_file, "ffmpeg-#{Time.now.to_i}.log")
file = recorder.screenshot(image_file, 'ffmpeg.log')
recorder.stop
sleep(3)
expect(file).to eq(image_file)
Expand Down
1 change: 0 additions & 1 deletion spec/spec_helper.rb
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,6 @@
c.syntax = :expect
end

# Set Selenium webdrivers install folder
config.before(:suite) do
# Establish clean state
delete_file '*.mkv'
Expand Down

0 comments on commit 615d64b

Please sign in to comment.