2 files changed, 126 insertions, 33 deletions

diff --git a/src/data/things/thing.js b/src/data/things/thing.js
index 1186c389..decde6f4 100644
--- a/src/data/things/thing.js
+++ b/src/data/things/thing.js
@@ -419,7 +419,7 @@ export default class Thing extends CacheableObject {
     return (
       contribsByRef
         .map(({who, what}) => ({
-          who: find.artist(who, artistData),
+          who: find.artist(who, artistData, {mode: 'quiet'}),
           what,
         }))
         .filter(({who}) => who));
diff --git a/test/unit/data/things/track.js b/test/unit/data/things/track.js
index d0e50c7f..37aa7b9f 100644
--- a/test/unit/data/things/track.js
+++ b/test/unit/data/things/track.js
@@ -20,66 +20,159 @@ function stubAlbum(tracks) {
   return album;
 }
 
-t.test(`Track.coverArtDate`, t => {
-  t.plan(6);
-
-  const albumTrackArtDate = new Date('2012-12-12');
-  const trackCoverArtDate = new Date('2009-09-09');
-  const dummyContribs = [{who: 'Test Artist', what: null}]
-
+function stubTrack() {
   const track = new Track();
   track.directory = 'foo';
-  track.coverArtistContribsByRef = dummyContribs;
+  return track;
+}
 
+function stubTrackAndAlbum() {
+  const track = stubTrack();
   const album = stubAlbum([track]);
+  track.albumData = [album];
+  return {track, album};
+}
 
+function stubArtistAndContribs() {
   const artist = new Artist();
-  artist.name = 'Test Artist';
+  artist.name = `Test Artist`;
+
+  const contribs = [{who: `Test Artist`, what: null}];
+  const badContribs = [{who: `Figment of Your Imagination`, what: null}];
+  return {artist, contribs, badContribs};
+}
 
+function XXX_CLEAR_TRACK_ALBUM_CACHE(track, album) {
+  // XXX clear cache so change in album's property is reflected
+  track.albumData = [];
   track.albumData = [album];
-  track.artistData = [artist];
+}
 
-  const XXX_CLEAR_TRACK_ALBUM_CACHE = () => {
-    // XXX clear cache so change in album's property is reflected
-    track.albumData = [];
-    track.albumData = [album];
-  };
+t.test(`Track.color`, t => {
+  t.plan(3);
 
-  // 1. coverArtDate defaults to null
+  const {track, album} = stubTrackAndAlbum();
 
-  t.equal(track.coverArtDate, null);
+  t.equal(track.color, null,
+    `color #1: defaults to null`);
 
-  // 2. coverArtDate inherits album trackArtDate
+  album.color = '#abcdef';
+  XXX_CLEAR_TRACK_ALBUM_CACHE(track, album);
 
-  album.trackArtDate = albumTrackArtDate;
+  t.equal(track.color, '#abcdef',
+    `color #2: inherits from album`);
 
-  XXX_CLEAR_TRACK_ALBUM_CACHE();
+  track.color = '#123456';
 
-  t.equal(track.coverArtDate, albumTrackArtDate);
+  t.equal(track.color, '#123456',
+    `color #3: is own value`);
+});
 
-  // 3. coverArtDate is own value
+t.test(`Track.coverArtDate`, t => {
+  t.plan(6);
+
+  const {track, album} = stubTrackAndAlbum();
+  const {artist, contribs} = stubArtistAndContribs();
+
+  track.coverArtistContribsByRef = contribs;
+  track.artistData = [artist];
+
+  t.equal(track.coverArtDate, null,
+    `coverArtDate #1: defaults to null`);
 
-  track.coverArtDate = trackCoverArtDate;
+  album.trackArtDate = new Date('2012-12-12');
+  XXX_CLEAR_TRACK_ALBUM_CACHE(track, album);
 
-  t.equal(track.coverArtDate, trackCoverArtDate);
+  t.same(track.coverArtDate, new Date('2012-12-12'),
+    `coverArtDate #2: inherits album trackArtDate`);
 
-  // 4. coverArtDate is null if track is missing coverArtists
+  track.coverArtDate = new Date('2009-09-09');
+
+  t.same(track.coverArtDate, new Date('2009-09-09'),
+    `coverArtDate #3: is own value`);
 
   track.coverArtistContribsByRef = [];
 
-  t.equal(track.coverArtDate, null);
+  t.equal(track.coverArtDate, null,
+    `coverArtDate #4: is null if track is missing coverArtists`);
+
+  album.trackCoverArtistContribsByRef = contribs;
+  XXX_CLEAR_TRACK_ALBUM_CACHE(track, album);
+
+  t.same(track.coverArtDate, new Date('2009-09-09'),
+    `coverArtDate #5: is not null if album specifies trackCoverArtistContribs`);
+
+  track.disableUniqueCoverArt = true;
+
+  t.equal(track.coverArtDate, null,
+    `coverArtDate #6: is null if track disables unique cover artwork`);
+});
+
+t.test(`Track.date`, t => {
+  t.plan(3);
+
+  const {track, album} = stubTrackAndAlbum();
+
+  t.equal(track.date, null,
+    `date #1: defaults to null`);
+
+  album.date = new Date('2012-12-12');
+  XXX_CLEAR_TRACK_ALBUM_CACHE(track, album);
 
-  // 5. coverArtDate is not null if album specifies trackCoverArtistContribs
+  t.same(track.date, album.date,
+    `date #2: inherits from album`);
 
-  album.trackCoverArtistContribsByRef = dummyContribs;
+  track.dateFirstReleased = new Date('2009-09-09');
+
+  t.same(track.date, new Date('2009-09-09'),
+    `date #3: is own dateFirstReleased`);
+});
+
+t.test(`Track.hasUniqueCoverArt`, t => {
+  t.plan(7);
+
+  const {track, album} = stubTrackAndAlbum();
+  const {artist, contribs, badContribs} = stubArtistAndContribs();
+
+  track.artistData = [artist];
+  album.artistData = [artist];
 
-  XXX_CLEAR_TRACK_ALBUM_CACHE();
+  t.equal(track.hasUniqueCoverArt, false,
+    `hasUniqueCoverArt #1: defaults to false`);
 
-  t.equal(track.coverArtDate, trackCoverArtDate);
+  album.trackCoverArtistContribsByRef = contribs;
+  XXX_CLEAR_TRACK_ALBUM_CACHE(track, album);
 
-  // 6. coverArtDate is null if track disables unique cover artwork
+  t.equal(track.hasUniqueCoverArt, true,
+    `hasUniqueCoverArt #2: is true if album specifies trackCoverArtistContribs`);
 
   track.disableUniqueCoverArt = true;
 
-  t.equal(track.coverArtDate, null);
+  t.equal(track.hasUniqueCoverArt, false,
+    `hasUniqueCoverArt #3: is false if disableUniqueCoverArt is true (1/2)`);
+
+  track.disableUniqueCoverArt = false;
+
+  album.trackCoverArtistContribsByRef = badContribs;
+  XXX_CLEAR_TRACK_ALBUM_CACHE(track, album);
+
+  t.equal(track.hasUniqueCoverArt, false,
+    `hasUniqueCoverArt #4: is false if album's trackCoverArtistContribsByRef resolve empty`);
+
+  track.coverArtistContribsByRef = contribs;
+
+  t.equal(track.hasUniqueCoverArt, true,
+    `hasUniqueCoverArt #5: is true if track specifies coverArtistContribs`);
+
+  track.disableUniqueCoverArt = true;
+
+  t.equal(track.hasUniqueCoverArt, false,
+    `hasUniqueCoverArt #6: is false if disableUniqueCoverArt is true (2/2)`);
+
+  track.disableUniqueCoverArt = false;
+
+  track.coverArtistContribsByRef = badContribs;
+
+  t.equal(track.hasUniqueCoverArt, false,
+    `hasUniqueCoverArt #7: is false if track's coverArtistContribsByRef resolve empty`);
 });