From 6327a5a4a2cf4081b2be5efbdfcce849fe913d5b Mon Sep 17 00:00:00 2001 From: Samuel Attard Date: Mon, 19 Apr 2021 11:27:34 -0700 Subject: [PATCH] chore: clean up some spec things (#28680) --- .gitignore | 4 +++- spec-main/chromium-spec.ts | 8 +++++++- 2 files changed, 10 insertions(+), 2 deletions(-) diff --git a/.gitignore b/.gitignore index e92abcd0baa..47b067646d8 100644 --- a/.gitignore +++ b/.gitignore @@ -68,4 +68,6 @@ ts-gen .depshash-target # Used to accelerate builds after sync -patches/mtime-cache.json \ No newline at end of file +patches/mtime-cache.json + +spec/fixtures/logo.png \ No newline at end of file diff --git a/spec-main/chromium-spec.ts b/spec-main/chromium-spec.ts index 56de3f1bd59..83f40dfe978 100644 --- a/spec-main/chromium-spec.ts +++ b/spec-main/chromium-spec.ts @@ -1454,7 +1454,13 @@ describe('font fallback', () => { const fonts = await getRenderedFonts(html); expect(fonts).to.be.an('array'); expect(fonts).to.have.length(1); - if (process.platform === 'win32') { expect(fonts[0].familyName).to.equal('Arial'); } else if (process.platform === 'darwin') { expect(fonts[0].familyName).to.equal('Helvetica'); } else if (process.platform === 'linux') { expect(fonts[0].familyName).to.equal('DejaVu Sans'); } // I think this depends on the distro? We don't specify a default. + if (process.platform === 'win32') { + expect(fonts[0].familyName).to.equal('Arial'); + } else if (process.platform === 'darwin') { + expect(fonts[0].familyName).to.equal('Helvetica'); + } else if (process.platform === 'linux') { + expect(fonts[0].familyName).to.equal('DejaVu Sans'); + } // I think this depends on the distro? We don't specify a default. }); ifit(process.platform !== 'linux')('should fall back to Japanese font for sans-serif Japanese script', async function () {