Make sure DelayType function gets consistent values of n

[haaawk]

Without this change DelayType function gets n either from 0,1,2,... or from 1,2,3,... depending on the value of Attempts config value.

This change unifies the behaviour and makes n always take values from 1,2,3,...

Fixes #124

[haaawk]

It seems that there's some timing issue with one of the tests:

--- FAIL: TestMaxDelay (0.25s)
    retry_test.go:281: 
        	Error Trace:	D:/a/retry-go/retry-go/retry_test.go:281
        	Error:      	"251.57ms" is not less than "250ms"
        	Test:       	TestMaxDelay
        	Messages:   	5 times with maximum delay retry is longer than [25](https://github.com/avast/retry-go/actions/runs/12891476984/job/35943505687?pr=125#step:6:26)0ms

[haaawk]

It seems that there's some timing issue with one of the tests:

--- FAIL: TestMaxDelay (0.25s)
    retry_test.go:281: 
        	Error Trace:	D:/a/retry-go/retry-go/retry_test.go:281
        	Error:      	"251.57ms" is not less than "250ms"
        	Test:       	TestMaxDelay
        	Messages:   	5 times with maximum delay retry is longer than [25](https://github.com/avast/retry-go/actions/runs/12891476984/job/35943505687?pr=125#step:6:26)0ms

It seems that this test is already disabled for mac:

         if os.Getenv("OS") == "macos-latest" {
		t.Skip("Skipping testing in MacOS GitHub actions - too slow, duration is wrong")
	}

[JaSei]

Hi, I sincerely apologize for the delayed response. I've fixed the tests in #99. Please rebase your PR against the main branch so we can proceed with the review. Thank you for your patience and contribution!

[haaawk]

Hi, I sincerely apologize for the delayed response. I've fixed the tests in #99. Please rebase your PR against the main branch so we can proceed with the review. Thank you for your patience and contribution!

Thanks @JaSei! I pushed the rebased version.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.86%. Comparing base (79a35c4) to head (8d7cef7).
Report is 2 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master     #125   +/-   ##
=======================================
  Coverage   90.86%   90.86%           
=======================================
  Files           2        2           
  Lines         230      230           
=======================================
  Hits          209      209           
  Misses         17       17           
  Partials        4        4           

Flag	Coverage Δ
unittest	90.86% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[JaSei]

Thank you, for your PR

[haaawk]

Thanks @JaSei !