Follow-up from "Update error message for bursting push limit"
The following discussion from !132943 (merged) should be addressed:
-
@Andysoiron started a discussion: Suggestion (non-blocking):
- We can use
let
to assign the variables to consistant. - I think it would be nicer to use the output of the operation for
size_msg
. This makes it easier to see what the expected value is.
let(:mock_blob_id) { "88acbfafb1b8fdb7c51db870babce21bd861ac4f" } let(:mock_blob_size) { 300 * 1024 * 1024 } # 300 MiB let(:size_msg) { '300.0' }
- We can use