Repository: Azure/azure-sdk-for-net
File: sdk/batch/Azure.Compute.Batch/src/Custom/LongRunningOperations/DeleteJobOperation.cs (C#)

Code before:
        /// </param>
        internal DeleteJobOperation(
            BatchClient client,
            string deleteId,
            Response initialResponse)
        {
            Id = deleteId;
            _jobId = deleteId;
            _value = false;
            _rawResponse = initialResponse;
            _client = client;
        }

        /// <summary>
        /// Initializes a new <see cref="DeleteJobOperation"/> instance
        /// </summary>
        /// <param name="client">
        /// The client used to check for completion.
        /// </param>
        /// <param name="id">The ID of this operation.</param>
        public DeleteJobOperation(
            BatchClient client,
            string id)
        {
            if (String.IsNullOrEmpty(id))
                throw new ArgumentNullException("id is not formated correctly");
            _jobId = id;
            Id = id;
            _value = false;
            _rawResponse = null;
            _client = client;
        }

        /// <summary>
        /// Initializes a new <see cref="DeleteJobOperation"/> instance for
        /// mocking.
        /// </summary>
        protected DeleteJobOperation()
        {
        }

        /// <summary>
        /// Get the sucess state of the deletion operation
        /// </summary>
        public override bool Value => OperationHelpers.GetValue(ref _value);

        /// <summary>
        /// Gets a value indicating whether the operation completed and
        /// successfully produced a value.  The <see cref="Operation{Bool}.Value"/>
        /// property is the success of the operation.
        /// </summary>

Review comment (suggestion):
```suggestion
                throw new ArgumentNullException("id is not formatted correctly", nameof(id));
```

Code after:
        /// </param>
        internal DeleteJobOperation(
            BatchClient client,
            string deleteId,
            Response initialResponse)
        {
            Id = deleteId;
            _jobId = deleteId;
            _value = false;
            _rawResponse = initialResponse;
            _client = client;
        }

        /// <summary>
        /// Initializes a new <see cref="DeleteJobOperation"/> instance
        /// </summary>
        /// <param name="client">
        /// The client used to check for completion.
        /// </param>
        /// <param name="id">The ID of this operation.</param>
        public DeleteJobOperation(
            BatchClient client,
            string id)
        {
            if (String.IsNullOrEmpty(id))
                throw new ArgumentNullException("id is not formatted correctly");
            _jobId = id;
            Id = id;
            _value = false;
            _rawResponse = null;
            _client = client;
        }

        /// <summary>
        /// Initializes a new <see cref="DeleteJobOperation"/> instance for
        /// mocking.
        /// </summary>
        protected DeleteJobOperation()
        {
        }

        /// <summary>
        /// Get the sucess state of the deletion operation
        /// </summary>
        public override bool Value => OperationHelpers.GetValue(ref _value);

        /// <summary>
        /// Gets a value indicating whether the operation completed and
        /// successfully produced a value.  The <see cref="Operation{Bool}.Value"/>
        /// property is the success of the operation.
        /// </summary>