Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(34)

Unified Diff: ui/file_manager/gallery/js/gallery_item_unittest.js

Issue 976713004: Add thumbnailMetadataItem to GalleryItem. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebased. Created 5 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: ui/file_manager/gallery/js/gallery_item_unittest.js
diff --git a/ui/file_manager/gallery/js/gallery_item_unittest.js b/ui/file_manager/gallery/js/gallery_item_unittest.js
index cd933300341f8e2f588ab7f662e57faea72c71ba..613bbb2be38808dd401ec1f57610026cd0145495 100644
--- a/ui/file_manager/gallery/js/gallery_item_unittest.js
+++ b/ui/file_manager/gallery/js/gallery_item_unittest.js
@@ -49,39 +49,34 @@ function testSaveToFile(callback) {
}
});
};
- var fetchedMediaCleared = false;
- var metadataCache = {
- getLatest: function(entries, type, callback) {
- callback([{name: 'newMetadata'}]);
+ var entryChanged = false;
+ var metadataModel = {
+ get: function(entries, names) {
+ return Promise.resolve([
+ {size: 200}
+ ]);
},
- clear: function(entries, type) {
- fetchedMediaCleared = true;
+ notifyEntriesChanged: function() {
+ entryChanged = true;
}
};
var item = new Gallery.Item(
entry,
{isReadOnly: false},
- {name: 'oldMetadata'},
+ {size: 100},
{},
- metadataCache,
- // Mock of MetadataModel.
- {
- get: function() {
- return Promise.resolve([{}]);
- },
- notifyEntriesChanged: function() {}
- },
/* original */ true);
- assertEquals('oldMetadata', item.getMetadata().name);
- assertFalse(fetchedMediaCleared);
+ assertEquals(100, item.getMetadataItem().size);
+ assertFalse(entryChanged);
reportPromise(
new Promise(item.saveToFile.bind(
item,
{getLocationInfo: function() { return {}; }},
- null,
- true,
+ metadataModel,
+ /* fallbackDir */ null,
+ /* overwrite */ true,
document.createElement('canvas'))).then(function() {
- assertEquals('newMetadata', item.getMetadata().name);
- assertTrue(fetchedMediaCleared);
+ assertEquals(200, item.getMetadataItem().size);
+ assertTrue(entryChanged);
}), callback);
}

Powered by Google App Engine
This is Rietveld 408576698