--wip-- [skip ci]

Rigtch · Sep 11, 2024 · 08533cc · 08533cc
1 parent 4fabe7d
commit 08533cc
Show file tree

Hide file tree

Showing 2 changed files with 37 additions and 24 deletions.
diff --git a/src/modules/items/albums/album.subscriber.spec.ts b/src/modules/items/albums/album.subscriber.spec.ts
@@ -73,7 +73,7 @@ describe('AlbumSubscriber', () => {
     expect(albumSubscriber.listenTo()).toEqual(Album)
   })
 
-  describe('afterLoad', () => {
+  describe.skip('afterLoad', () => {
     let albumMock: Album
 
     let spotifyGetAlbumSpy: MockInstance

diff --git a/src/modules/items/albums/album.subscriber.ts b/src/modules/items/albums/album.subscriber.ts
@@ -23,29 +23,42 @@ export class AlbumSubscriber implements EntitySubscriberInterface<Album> {
   }
 
   async afterLoad(albumEntity: Album) {
-    // eslint-disable-next-line @typescript-eslint/no-unnecessary-condition
-    if (!albumEntity.artists || albumEntity.artists.length === 0) {
-      this.logger.log(`Inserting artists for album ${albumEntity.name}`)
-
-      const sdkAlbum = await this.spotifyService.albums.get(
-        albumEntity.externalId,
-        false
-      )
-      const sdkArtists = await this.spotifyService.artists.get(
-        sdkAlbum.artists.map(({ id }) => id),
-        false
-      )
-
-      this.dataSource.transaction(async manager => {
-        const artists = await this.artistsService.findOrCreate(
-          sdkArtists,
-          manager
-        )
-
-        albumEntity.artists = artists
-
-        await manager.save(albumEntity)
+    console.log('a', albumEntity)
+
+    console.log(
+      'e',
+      await this.dataSource.manager.find(Album, {
+        where: { id: albumEntity.id },
+        relations: {
+          artists: true,
+        },
       })
-    }
+    )
+
+    // eslint-disable-next-line @typescript-eslint/no-unnecessary-condition
+    // if (!albumEntity.artists || albumEntity.artists.length === 0) {
+    //   this.logger.log(`Inserting artists for album ${albumEntity.name}`)
+
+    // eslint-disable-next-line sonarjs/no-commented-code
+    //   const sdkAlbum = await this.spotifyService.albums.get(
+    //     albumEntity.externalId,
+    //     false
+    //   )
+    //   const sdkArtists = await this.spotifyService.artists.get(
+    //     sdkAlbum.artists.map(({ id }) => id),
+    //     false
+    //   )
+
+    //   this.dataSource.transaction(async manager => {
+    //     const artists = await this.artistsService.findOrCreate(
+    //       sdkArtists,
+    //       manager
+    //     )
+
+    //     albumEntity.artists = artists
+
+    //     await manager.save(albumEntity)
+    //   })
+    // }
   }
 }