added test to remove unneeded updates if a file is not found on the server

internal/app/images.go

@@ -275,9 +275,13 @@ func updatePiwigoIdIfAlreadyUploaded(provider ImageMetadataProvider, piwigoCtx p
 	}
 	for md5sum, piwigoId := range missingResults {
 		logrus.Debugf("Setting piwigo id of %s to %d", md5sum, piwigoId)
-		err = provider.SavePiwigoIdAndUpdateUploadFlag(md5sum, piwigoId)
+		if piwigoId > 0 {
-		if err != nil {
+			err = provider.SavePiwigoIdAndUpdateUploadFlag(md5sum, piwigoId)
-			logrus.Warnf("Could not save piwigo id %d for file %s", piwigoId, md5sum)
+			if err != nil {
+				logrus.Warnf("Could not save piwigo id %d for file %s", piwigoId, md5sum)
+			}
+		} else {
+			logrus.Tracef("Image %s not found on server", md5sum)
 		}
 	}
 

internal/app/images_test.go

@@ -407,6 +407,33 @@ func Test_updatePiwigoIdIfAlreadyUploaded_with_image_to_check(t *testing.T) {
 	}
 }
 
+func Test_updatePiwigoIdIfAlreadyUploaded_with_image_to_check_missing_on_server(t *testing.T) {
+	mockCtrl := gomock.NewController(t)
+	defer mockCtrl.Finish()
+
+	img := ImageMetaData{
+		ImageId:        1,
+		PiwigoId:       0,
+		UploadRequired: true,
+		Md5Sum:         "1234",
+	}
+	images := []ImageMetaData{img}
+
+	dbmock := NewMockImageMetadataProvider(mockCtrl)
+	dbmock.EXPECT().ImageMetadataToUpload().Return(images, nil)
+	dbmock.EXPECT().SavePiwigoIdAndUpdateUploadFlag("1234", 1).Times(0)
+
+	piwigoResponose := make(map[string]int)
+
+	piwigomock := NewMockPiwigoImageApi(mockCtrl)
+	piwigomock.EXPECT().ImagesExistOnPiwigo(gomock.Any()).Times(1).Return(piwigoResponose, nil)
+
+	err := updatePiwigoIdIfAlreadyUploaded(dbmock, piwigomock)
+	if err != nil {
+		t.Error(err)
+	}
+}
+
 func Test_uploadImages_saves_new_id_to_db(t *testing.T) {
 	mockCtrl := gomock.NewController(t)
 	defer mockCtrl.Finish()