more error cleanup

This commit is contained in:
Dan Anglin 2024-08-17 11:01:52 +01:00
parent 87d8f0ad04
commit 684b4426eb
Signed by: dananglin
GPG key ID: 0C1D44CFBEE68638
2 changed files with 19 additions and 12 deletions

View file

@ -234,6 +234,7 @@ func (c *CreateExecutor) createStatus(gtsClient *client.Client) error {
func (c *CreateExecutor) createMediaAttachment(gtsClient *client.Client) error {
expectedNumValues := 1
if !c.mediaFiles.ExpectedLength(expectedNumValues) {
return UnexpectedNumValuesError{
name: "media files",

View file

@ -69,10 +69,11 @@ func (e *EditExecutor) editMediaAttachment(gtsClient *client.Client) error {
expectedNumValues := 1
if !e.attachmentIDs.ExpectedLength(expectedNumValues) {
return fmt.Errorf(
"received an unexpected number of media attachment IDs: want %d",
expectedNumValues,
)
return UnexpectedNumValuesError{
name: "media attachment IDs",
expected: expectedNumValues,
actual: len(e.attachmentIDs),
}
}
attachment, err := gtsClient.GetMediaAttachment(e.attachmentIDs[0])
@ -83,18 +84,21 @@ func (e *EditExecutor) editMediaAttachment(gtsClient *client.Client) error {
description := attachment.Description
if !e.mediaDescriptions.Empty() {
if !e.mediaDescriptions.ExpectedLength(expectedNumValues) {
return fmt.Errorf(
"received an unexpected number of media descriptions: want %d",
expectedNumValues,
)
return UnexpectedNumValuesError{
name: "media description",
expected: expectedNumValues,
actual: len(e.mediaDescriptions),
}
}
var err error
description, err = utilities.ReadContents(e.mediaDescriptions[0])
if err != nil {
return fmt.Errorf(
"unable to read the contents from %s: %w",
e.mediaDescriptions[0],
err,
)
}
}
@ -102,11 +106,13 @@ func (e *EditExecutor) editMediaAttachment(gtsClient *client.Client) error {
focus := fmt.Sprintf("%f,%f", attachment.Meta.Focus.X, attachment.Meta.Focus.Y)
if !e.mediaFocusValues.Empty() {
if !e.mediaFocusValues.ExpectedLength(expectedNumValues) {
return fmt.Errorf(
"received an unexpected number of media focus values: want %d",
expectedNumValues,
)
return UnexpectedNumValuesError{
name: "media focus values",
expected: expectedNumValues,
actual: len(e.mediaFocusValues),
}
}
focus = e.mediaFocusValues[0]
}