94273: Invalidate object on successful patch

Author: ybnd

src/app/core/data/data.service.spec.ts

@@ -351,7 +351,12 @@ describe('DataService', () => {
 
   describe('patch', () => {
     const dso = {
-      uuid: 'dso-uuid'
+      uuid: 'dso-uuid',
+      _links: {
+        self: {
+          href: 'dso-href',
+        }
+      }
     };
     const operations = [
       Object.assign({
@@ -361,12 +366,23 @@ describe('DataService', () => {
       }) as Operation
     ];
 
-    beforeEach(() => {
+    it('should send a PatchRequest', () => {
       service.patch(dso, operations);
+      expect(requestService.send).toHaveBeenCalledWith(jasmine.any(PatchRequest));
     });
 
-    it('should send a PatchRequest', () => {
-      expect(requestService.send).toHaveBeenCalledWith(jasmine.any(PatchRequest));
+    it('should invalidate the cached object if successfully patched', () => {
+      spyOn(rdbService, 'buildFromRequestUUIDAndAwait');
+      spyOn(service, 'invalidateByHref');
+
+      service.patch(dso, operations);
+
+      expect(rdbService.buildFromRequestUUIDAndAwait).toHaveBeenCalled();
+      expect((rdbService.buildFromRequestUUIDAndAwait as jasmine.Spy).calls.argsFor(0)[0]).toBe(requestService.generateRequestId());
+      const callback = (rdbService.buildFromRequestUUIDAndAwait as jasmine.Spy).calls.argsFor(0)[1];
+      callback();
+
+      expect(service.invalidateByHref).toHaveBeenCalledWith('dso-href');
     });
   });
 

src/app/core/data/data.service.ts

@@ -484,7 +484,7 @@ export abstract class DataService<T extends CacheableObject> implements UpdateDa
       this.requestService.send(request);
     });
 
-    return this.rdbService.buildFromRequestUUID(requestId);
+    return this.rdbService.buildFromRequestUUIDAndAwait(requestId, () => this.invalidateByHref(object._links.self.href));
   }
 
   createPatchFromCache(object: T): Observable<Operation[]> {