Misc. tweaks

* `refreshImages` no longer needs an `emitChanges` arg
* prefer `array.some` to `array.find` to determine existence only

Signed-off-by: Eric Promislow <epromislow@suse.com>

background.ts

@@ -198,7 +198,7 @@ Electron.ipcMain.on('confirm-do-image-deletion', async(event, imageName, imageID
       for (i = 0; i < maxNumAttempts; i++) {
         await imageManager.deleteImage(imageID);
         await imageManager.refreshImages();
-        if (!imageManager.listImages().find(image => image.imageID === imageID)) {
+        if (!imageManager.listImages().some(image => image.imageID === imageID)) {
           break;
         }
         await util.promisify(setTimeout)(500);

src/k8s-engine/kim.ts

@@ -134,7 +134,7 @@ export default class Kim extends EventEmitter {
     return this.images;
   }
 
-  async refreshImages(emitChanges = true) {
+  async refreshImages() {
     try {
       const result: childResultType = await this.getImages();
 
@@ -149,9 +149,7 @@ export default class Kim extends EventEmitter {
       this.images = this.parse(result.stdout);
       // Start a new interval for image-list refreshing
       this.start();
-      if (emitChanges) {
-        this.emit('images-changed', this.images);
-      }
+      this.emit('images-changed', this.images);
     } catch (err) {
       if (!this.showedStderr) {
         if (err.stderr && !err.stdout && !err.signal) {