Skip to content

Commit

Permalink
Rename 'downloader_suffix' to 'suffix'
Browse files Browse the repository at this point in the history
his commit changes the variable name from 'downloader_suffix' to 'suffix' in the relevant test code. This helps avoid the implication that we are using more than one suffix in this test, which could lead to confusion. It also aligns the naming convention in this test with that in the production code, where 'suffix' is used in the same context.
  • Loading branch information
Glutexo committed May 20, 2023
1 parent 276b28d commit 56fed27
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions test/onigumo_downloader_test.exs
Original file line number Diff line number Diff line change
Expand Up @@ -117,8 +117,8 @@ defmodule OnigumoDownloaderTest do
created_file_name = Onigumo.Downloader.create_file_name(input_url)

input_url_hash = Onigumo.Utilities.Hash.md5(input_url, :hex)
downloaded_suffix = Application.get_env(:onigumo, :downloaded_suffix)
expected_file_name = input_url_hash <> downloaded_suffix
suffix = Application.get_env(:onigumo, :downloaded_suffix)
expected_file_name = input_url_hash <> suffix

assert(created_file_name == expected_file_name)
end
Expand Down

0 comments on commit 56fed27

Please sign in to comment.